Skip to content

test: operational_all: Harden to catch more#1550

Merged
mattiaswal merged 2 commits into
mainfrom
refactor-misc-operational
Jun 27, 2026
Merged

test: operational_all: Harden to catch more#1550
mattiaswal merged 2 commits into
mainfrom
refactor-misc-operational

Conversation

@mattiaswal

Copy link
Copy Markdown
Contributor
  • Test so the value in operational actually is correct
  • Move test to last in suite to see that there are no stray data in operational (yangerd)

Description

Checklist

Tick relevant boxes, this PR is-a or has-a:

  • Bugfix
    • Regression tests
    • ChangeLog updates (for next release)
  • Feature
    • YANG model change => revision updated?
    • Regression tests added?
    • ChangeLog updates (for next release)
    • Documentation added?
  • Test changes
    • Checked in changed Readme.adoc (make test-spec)
    • Added new test to group Readme.adoc and yaml file
  • Code style update (formatting, renaming)
  • Refactoring (please detail in commit messages)
  • Build related changes
  • Documentation content changes
    • ChangeLog updated (for major changes)
  • Other (please describe):

Comment thread test/case/misc/operational_all/test.py
- Test so the value in operational actually is correct
- Move test to last in suite to see that there are no stray
  data in operational (yangerd)
@mattiaswal mattiaswal force-pushed the refactor-misc-operational branch from ea80aa6 to b8ba1a1 Compare June 27, 2026 08:57
@mattiaswal mattiaswal requested a review from troglobit June 27, 2026 09:09
@mattiaswal

Copy link
Copy Markdown
Contributor Author

The failure @ alder, may be fixed at yangerd branch, if there is a race issue. Has nothing to do with this PR, that only modify operation_all

@mattiaswal mattiaswal merged commit 7a9cece into main Jun 27, 2026
8 of 9 checks passed
@mattiaswal mattiaswal deleted the refactor-misc-operational branch June 27, 2026 10:58
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.

2 participants