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

Fail if the build causes empty OCIL #13148

Merged
merged 1 commit into from
Mar 7, 2025

Conversation

jan-cerny
Copy link
Collaborator

If building a single rule data stream we need to check if the OCIL in that rule isn't empty otherwise we would build an invalid single rule data stream because it would have empty questionnaires element.

Review Hints:

  1. Remove ocil and ocil_clause from your favorite rule (or create a new rule that doesn't have these keys).
  2. Build a single rule thin data stream: ./build_product -r your_favorite_rule product_id

If building a single rule data stream we need to check if the
OCIL in that rule isn't empty otherwise we would build an invalid
single rule data stream because it would have empty questionnaires
element.
Copy link

codeclimate bot commented Mar 6, 2025

Code Climate has analyzed commit 2cdd8ec and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 0.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 62.0% (-0.1% change).

View more on Code Climate.

@Mab879 Mab879 self-assigned this Mar 6, 2025
@Mab879 Mab879 added this to the 0.1.77 milestone Mar 6, 2025
@Mab879 Mab879 added the Infrastructure Our content build system label Mar 6, 2025
@Mab879
Copy link
Member

Mab879 commented Mar 6, 2025

/packit retest-failed

@Mab879
Copy link
Member

Mab879 commented Mar 7, 2025

Waving the coverage tests as this code is run during the build process.

@Mab879 Mab879 merged commit 222b573 into ComplianceAsCode:master Mar 7, 2025
109 of 111 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Infrastructure Our content build system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants