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 tests for TF cleanup logic #290

Merged
merged 14 commits into from
Feb 20, 2025
Merged

test: add tests for TF cleanup logic #290

merged 14 commits into from
Feb 20, 2025

Conversation

lechnerc77
Copy link
Member

Purpose

Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[X] Other... Please describe: Test Setup

How to Test

  • Test the code via automated test
make test

What to Check

Verify that the following are valid:

  • Automated tests are executed successfully

Other Information

n/a

Checklist for reviewer

The following organizational tasks must be completed before merging this PR:

  • The PR status on the Project board is set (typically "in review").
  • The PR has the matching labels assigned to it.
  • If the PR closes an issue, the issue is referenced.
  • Possible follow-up issues are created and linked.

@github-actions github-actions bot added this to the 1.0.0-rc1 milestone Feb 20, 2025
@github-actions github-actions bot added ignore-for-release All things not to be mentioned in release notes internal Internal tasks test setup Setup of tests labels Feb 20, 2025
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
59.5% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@lechnerc77 lechnerc77 merged commit 3d89657 into main Feb 20, 2025
11 of 12 checks passed
@lechnerc77 lechnerc77 deleted the cleanup-unit-test branch February 20, 2025 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ignore-for-release All things not to be mentioned in release notes internal Internal tasks test setup Setup of tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TESTING] Reusability of generated code - Add test fixtures for core component of cleanup flow
1 participant