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

test: add E2E for coverage plugin, fix onlyPlugins logic #505

Merged
merged 5 commits into from
Feb 15, 2024
Merged

Conversation

Tlacenka
Copy link
Collaborator

@Tlacenka Tlacenka commented Feb 15, 2024

Closes #430
The final PR for the first version of coverage plugin.
In this PR:

  • I test the main functionality E2E, including running the coverage tool on an example repo with a few simple component tests.
  • Additionally, I fix an onlyPlugins issue where the filtering happed based on audit/group slug instead of plugin slug.

@Tlacenka Tlacenka requested a review from matejchalk February 15, 2024 12:51
@Tlacenka Tlacenka changed the title Coverage e2e test: Add E2E for coverage plugin, fix onlyPlugins logic Feb 15, 2024
@Tlacenka Tlacenka changed the title test: Add E2E for coverage plugin, fix onlyPlugins logic test: add E2E for coverage plugin, fix onlyPlugins logic Feb 15, 2024
@Tlacenka Tlacenka marked this pull request as ready for review February 15, 2024 15:40
@Tlacenka Tlacenka merged commit 27bd801 into main Feb 15, 2024
@Tlacenka Tlacenka deleted the coverage-e2e branch February 15, 2024 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Code coverage plugin
3 participants