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

Add tests for event rule processing #280

Closed
peteeckel opened this issue Jun 2, 2024 · 0 comments · Fixed by #281
Closed

Add tests for event rule processing #280

peteeckel opened this issue Jun 2, 2024 · 0 comments · Fixed by #281
Assignees
Labels
enhancement New feature or request

Comments

@peteeckel
Copy link
Owner

Add some tests to validate that the event rule processing in NetBox works properly.

  • Only one event per object is triggered in case of multiple save() and M2M field change operations
  • The pre- and postchange snapshots reflect the object status before and after the operation

This is mainly a validation that the issues reported in #89, #137, and #277 are fixed and that the NetBox fix is effective for the affected models in NetBox DNS.

Some of the new tests are skipped for all NetBox versions < 4.0.4 (assuming PR 16366 will be included in that release) as they are bound to fail as long as that PR is not merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant