It should be an comparation operation rather than an assignment
operation.
Fixes: 5e41ab250dfa("app/test: unit tests for bonding mode 4")
cc: Wodkowski, PawelX <[email protected]>
cc: Kulasek, TomaszX <[email protected]>
cc: Declan Doherty <[email protected]>
Signed-off-by: Zhiyong Yang <[email protected]>
---
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
