Skip to content

Test/drop unused imports - #1710

Merged
saba8814 merged 2 commits into
mainfrom
test/drop-unused-imports
Oct 16, 2026
Merged

saba8814 merged 2 commits into
mainfrom
test/drop-unused-imports

Conversation

@saba8814

Copy link
Copy Markdown
Collaborator

Description

Ten test cases import modules they never use, mostly time and until,
and use_case/ospf_container assigns R1data and R3data without ever
reading them. No test behaviour changes.

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):

Remove imports that are never referenced in ten test cases.

Signed-off-by: Ejub Sabic <ejub1946@outlook.com>
Signed-off-by: Ejub Sabic <ejub1946@outlook.com>

@troglobit troglobit left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice housekeeping!

@saba8814
saba8814 merged commit bef1b65 into main Oct 16, 2026
8 of 9 checks passed
@saba8814
saba8814 deleted the test/drop-unused-imports branch October 16, 2026 06:53
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