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

feat(anta)!: Add support for path to custom tests #119

Merged
merged 4 commits into from
Oct 21, 2022
Merged

feat(anta)!: Add support for path to custom tests #119

merged 4 commits into from
Oct 21, 2022

Conversation

titom73
Copy link
Collaborator

@titom73 titom73 commented Sep 16, 2022

Add ability to load user's defined tests using ANTA catalog loader (Issue #118 )

Now, catalog uses python module name to target correct tests:

---
anta.tests.software:
  - verify_eos_version:
      versions: 
        - 4.25.4M
        - 4.26.1F
anta.tests.routing:
  generic:
    - verify_routing_protocol_model:
        model: multi-agent

First level has to use full module name and then, loader do recursive search within this module to find correct submodules and tests

Breaking changes

  • Catalog format must use anta.tests.<test module> and not just <test module>

Todo

  • Validate behaviour with external test library
  • Update documentation for catalog
  • Document how to create custom tests lib

@titom73 titom73 added the framework-enhancement New feature or request label Sep 16, 2022
@titom73 titom73 self-assigned this Sep 16, 2022
@titom73 titom73 marked this pull request as ready for review October 7, 2022 12:36
Copy link
Contributor

@ksator ksator left a comment

Choose a reason for hiding this comment

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

LGTM

@ksator ksator merged commit cbc6091 into aristanetworks:master Oct 21, 2022
@titom73 titom73 changed the title feat(anta.loader)!: Add support for path to custom tests feat(anta)!: Add support for path to custom tests Mar 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants