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

feat: make use of the toolkit validator #247

Merged
merged 1 commit into from
Sep 1, 2023
Merged

feat: make use of the toolkit validator #247

merged 1 commit into from
Sep 1, 2023

Conversation

davidmogar
Copy link
Collaborator

@davidmogar davidmogar commented Sep 1, 2023

The Operator Toolkit validator allow us to define the validation functions and helps on testing the EnsureReleaseIsValid operation as now those validation functions can be mocked.

Signed-off-by: David Moreno García [email protected]

@codecov
Copy link

codecov bot commented Sep 1, 2023

Codecov Report

Patch coverage: 80.00% and project coverage change: -0.41% ⚠️

Comparison is base (aa68f18) 85.21% compared to head (1039526) 84.80%.

❗ Current head 1039526 differs from pull request most recent head b5117a8. Consider uploading reports for the commit b5117a8 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #247      +/-   ##
==========================================
- Coverage   85.21%   84.80%   -0.41%     
==========================================
  Files          23       23              
  Lines        1474     1474              
==========================================
- Hits         1256     1250       -6     
- Misses        156      161       +5     
- Partials       62       63       +1     
Files Changed Coverage Δ
controllers/release/adapter.go 83.51% <80.00%> (-1.68%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@davidmogar davidmogar force-pushed the validator branch 2 times, most recently from ee594f6 to 1039526 Compare September 1, 2023 14:27
The Operator Toolkit validator allow us to define the validation
functions and helps on testing the EnsureReleaseIsValid operation
as now those validation functions can be mocked.

Signed-off-by: David Moreno García <[email protected]>
@davidmogar davidmogar changed the title Add validator feat: make use of the toolkit validator Sep 1, 2023
@davidmogar davidmogar marked this pull request as ready for review September 1, 2023 15:00
@johnbieren
Copy link
Collaborator

/test release-service-e2e

@johnbieren johnbieren merged commit ab5d3b1 into main Sep 1, 2023
6 checks passed
@johnbieren johnbieren deleted the validator branch September 1, 2023 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants