Skip to content

Add End-to-End Smoke tests#90

Merged
vojtapolasek merged 2 commits into
ComplianceAsCode:mainfrom
Mab879:add_intgergation_tests
Apr 10, 2026
Merged

Add End-to-End Smoke tests#90
vojtapolasek merged 2 commits into
ComplianceAsCode:mainfrom
Mab879:add_intgergation_tests

Conversation

@Mab879
Copy link
Copy Markdown
Member

@Mab879 Mab879 commented Apr 3, 2026

This PR will add a smoke test for cvetool. So that errors can be caught in CI.

This PR also adds a quick git-ignore update as well. Making it easier to do testing locally.

@Mab879 Mab879 force-pushed the add_intgergation_tests branch 3 times, most recently from aa9b170 to 36841f5 Compare April 6, 2026 19:43
@Mab879
Copy link
Copy Markdown
Member Author

Mab879 commented Apr 6, 2026

Failure will be fixed #89

Comment thread tests/e2e.sh Outdated
Comment thread tests/e2e.sh Outdated
Comment thread tests/e2e.sh Outdated
Copy link
Copy Markdown
Collaborator

@vojtapolasek vojtapolasek left a comment

Choose a reason for hiding this comment

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

See my comments. And please rebase, the fix which blocks tests is merged.

@Mab879 Mab879 force-pushed the add_intgergation_tests branch 3 times, most recently from 1b1093e to 99f6f8c Compare April 8, 2026 19:12
Mab879 added 2 commits April 9, 2026 14:06
Makes testing easier
This commit was created in part with Claude Code.
@Mab879 Mab879 force-pushed the add_intgergation_tests branch from 99f6f8c to 10a53e8 Compare April 9, 2026 19:07
Copy link
Copy Markdown
Collaborator

@vojtapolasek vojtapolasek left a comment

Choose a reason for hiding this comment

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

Looks good now.

@vojtapolasek vojtapolasek merged commit 27ef5ee into ComplianceAsCode:main Apr 10, 2026
7 checks passed
@Mab879 Mab879 deleted the add_intgergation_tests branch April 10, 2026 11:56
@vojtapolasek vojtapolasek self-assigned this Apr 14, 2026
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