Message ID | 1424822552-4366-21-git-send-email-joe.hershberger@ni.com |
---|---|
State | RFC |
Delegated to: | Simon Glass |
Headers | show |
Hi Joe, On 24 February 2015 at 17:02, Joe Hershberger <joe.hershberger@ni.com> wrote: > The effect of the "netretry" env var was recently changed. This test > checks that behavior. > > Signed-off-by: Joe Hershberger <joe.hershberger@ni.com> > > --- > > Changes in v4: > -Updated expected behavior based on changes to the NetLoop > > Changes in v3: > -Added testing for netretry > > Changes in v2: None > > test/dm/eth.c | 23 +++++++++++++++++++++++ > 1 file changed, 23 insertions(+) > > diff --git a/test/dm/eth.c b/test/dm/eth.c > index a3cbd3f..4581c8e 100644 > --- a/test/dm/eth.c > +++ b/test/dm/eth.c > @@ -104,3 +104,26 @@ static int dm_test_eth_rotate(struct dm_test_state *dms) > return 0; > } > DM_TEST(dm_test_eth_rotate, DM_TESTF_SCAN_FDT); > + > +static int dm_test_net_retry(struct dm_test_state *dms) > +{ > + NetPingIP = string_to_ip("1.1.2.2"); > + > + setenv("eth_sandbox_disable_1", "yes"); As mentioned in the other patch, something like: sandbox_eth_set_disable(1, true); > + setenv("ethact", "eth@10004000"); > + setenv("netretry", "yes"); > + ut_assertok(NetLoop(PING)); > + ut_asserteq_str("eth@10002000", getenv("ethact")); > + A comment against each test case as to why it works or doesn't would help. > + setenv("ethact", "eth@10004000"); > + setenv("netretry", "no"); > + ut_asserteq(-1, NetLoop(PING)); One day it would be nice if this could return an error code from errno.h. > + ut_asserteq_str("eth@10004000", getenv("ethact")); > + > + /* Restore the env */ > + setenv("netretry", NULL); > + setenv("eth_sandbox_disable_1", NULL); sandbox_eth_set_disable(1, false); > + > + return 0; > +} > +DM_TEST(dm_test_net_retry, DM_TESTF_SCAN_FDT); > -- > 1.7.11.5 > Regards, Simon
diff --git a/test/dm/eth.c b/test/dm/eth.c index a3cbd3f..4581c8e 100644 --- a/test/dm/eth.c +++ b/test/dm/eth.c @@ -104,3 +104,26 @@ static int dm_test_eth_rotate(struct dm_test_state *dms) return 0; } DM_TEST(dm_test_eth_rotate, DM_TESTF_SCAN_FDT); + +static int dm_test_net_retry(struct dm_test_state *dms) +{ + NetPingIP = string_to_ip("1.1.2.2"); + + setenv("eth_sandbox_disable_1", "yes"); + setenv("ethact", "eth@10004000"); + setenv("netretry", "yes"); + ut_assertok(NetLoop(PING)); + ut_asserteq_str("eth@10002000", getenv("ethact")); + + setenv("ethact", "eth@10004000"); + setenv("netretry", "no"); + ut_asserteq(-1, NetLoop(PING)); + ut_asserteq_str("eth@10004000", getenv("ethact")); + + /* Restore the env */ + setenv("netretry", NULL); + setenv("eth_sandbox_disable_1", NULL); + + return 0; +} +DM_TEST(dm_test_net_retry, DM_TESTF_SCAN_FDT);
The effect of the "netretry" env var was recently changed. This test checks that behavior. Signed-off-by: Joe Hershberger <joe.hershberger@ni.com> --- Changes in v4: -Updated expected behavior based on changes to the NetLoop Changes in v3: -Added testing for netretry Changes in v2: None test/dm/eth.c | 23 +++++++++++++++++++++++ 1 file changed, 23 insertions(+)