Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Increase constrains in link state testcases #909

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dfetti
Copy link
Contributor

@dfetti dfetti commented Oct 29, 2024

Previously the link state testcase scenarios were structured like setup, do some interaction between peers, sleep for a while and check the reported link states. This approach did not consider the timings of the events.

Proposed solution consists in checking the state constantly during the scenario, not only at the end of it.

☑️ Definition of Done checklist

  • Commit history is clean (requirements)
  • README.md is updated
  • Functionality is covered by unit or integration tests

@dfetti dfetti requested a review from a team as a code owner October 29, 2024 07:53
@dfetti dfetti force-pushed the LLT-5683-increase-constranins-in-link-state-testcases branch from e02a96d to 4b5ef12 Compare October 29, 2024 07:54
Previously the link state testcase scenarios were structured like setup,
do some interaction between peers, sleep for a while and check the reported link states.
This approach did not consider the timings of the events.

Proposed solution consists in checking the state constantly during the
scenario, not only at the end of it.

Signed-off-by: Daniel Fetti <[email protected]>
@dfetti dfetti force-pushed the LLT-5683-increase-constranins-in-link-state-testcases branch from 4b5ef12 to c1193b5 Compare December 18, 2024 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant