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

✅ Tests overhaul #116

Merged
merged 21 commits into from
Feb 18, 2025
Merged

✅ Tests overhaul #116

merged 21 commits into from
Feb 18, 2025

Conversation

wouterdebruijn
Copy link
Collaborator

@wouterdebruijn wouterdebruijn commented Feb 13, 2025

Improved functional tests in for better coverage of new image, and icon-map synchronization. In preparation of adding Zabbix 6.0 support as mentioned in #115

  • Added tests for images

  • Added tests for icon-maps

  • Added tests for cli-parser

  • Added missing cli arguments as reported by new test

  • Added new ZabbixCI exception for missing icons while handling icon-maps

  • Added warning and skip on synchronization of icon-maps without icons

@wouterdebruijn wouterdebruijn changed the title ✅ Unittest overhaul ✅ Tests overhaul Feb 18, 2025
@wouterdebruijn wouterdebruijn marked this pull request as ready for review February 18, 2025 12:30
@wouterdebruijn wouterdebruijn merged commit 89a2442 into develop Feb 18, 2025
2 checks passed
@wouterdebruijn wouterdebruijn deleted the unittest-overhaul branch February 18, 2025 12:31
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