Skip to content

Commit

Permalink
Shifted the CASE eviction test around to ensure it didn't cause fabri…
Browse files Browse the repository at this point in the history
…c2 interactions later to fail
  • Loading branch information
mrjerryjohns committed Jun 16, 2022
1 parent 612f158 commit a469c31
Showing 1 changed file with 13 additions and 2 deletions.
15 changes: 13 additions & 2 deletions src/controller/python/test/test_scripts/mobile-device-test.py
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,19 @@ def ethernet_commissioning(test: BaseTestHelper, discriminator: int, setup_pin:
nodeid=device_nodeid),
"Failed to finish key exchange")

#
# Run this before the MultiFabric test, since it will result in the resultant CASE session
# on fabric2 to be evicted (due to the stressful nature of that test) on the target.
#
# It doesn't actually evict the CASE session for fabric2 on the client, since we prioritize
# defunct sessions for eviction first, which means our CASE session on fabric2 remains preserved
# throughout the stress test. This results in a mis-match later.
#
# TODO: Once we implement fabric-adjusted LRU, we should see if this issue remains (it shouldn't)
#
logger.info("Testing CASE Eviction")
FailIfNot(asyncio.run(test.TestCaseEviction(device_nodeid)), "Failed TestCaseEviction")

ok = asyncio.run(test.TestMultiFabric(ip=address,
setuppin=20202021,
nodeid=1))
Expand Down Expand Up @@ -157,8 +170,6 @@ def do_tests(controller_nodeid, device_nodeid, address, timeout, discriminator,
ethernet_commissioning(test, discriminator, setup_pin, address,
device_nodeid)

FailIfNot(asyncio.run(test.TestCaseEviction(device_nodeid)), "Failed TestCaseEviction")

logger.info("Testing resolve")
FailIfNot(test.TestResolve(nodeid=device_nodeid),
"Failed to resolve nodeid")
Expand Down

0 comments on commit a469c31

Please sign in to comment.