It should be an comparation operation rather than an assignment
operation.
Fixes: 5e41ab250dfa("app/test: unit tests for bonding mode 4")
cc: Wodkowski, PawelX <pawelx.wodkow...@intel.com>
cc: Kulasek, TomaszX <tomaszx.kula...@intel.com>
cc: Declan Doherty <declan.dohe...@intel.com>
Signed-off-by: Zhiyong Yang <zhiyong.y...@intel.com>
---
test/test/test_link_bonding_mode4.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test/test/test_link_bonding_mode4.c
b/test/test/test_link_bonding_mode4.c
index 8e9e23db5..e246f0148 100644
--- a/test/test/test_link_bonding_mode4.c
+++ b/test/test/test_link_bonding_mode4.c
@@ -661,7 +661,7 @@ bond_handshake(void)
TEST_ASSERT_EQUAL(all_slaves_done, 1, "Bond handshake failed\n");
/* If flags doesn't match - report failure */
- return all_slaves_done = 1 ? TEST_SUCCESS : TEST_FAILED;
+ return all_slaves_done == 1 ? TEST_SUCCESS : TEST_FAILED;
}
#define TEST_LACP_SLAVE_COUT RTE_DIM(test_params.slave_ports)
--
2.13.3
