Skip to content
This repository was archived by the owner on Dec 16, 2025. It is now read-only.

✨ Add missing unit tests for openstackclusterstackrelease controller#61

Merged
michal-gubricky merged 1 commit into
mainfrom
feat/add-more-tests
Jan 31, 2024
Merged

✨ Add missing unit tests for openstackclusterstackrelease controller#61
michal-gubricky merged 1 commit into
mainfrom
feat/add-more-tests

Conversation

@michal-gubricky

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:
Related to #49

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

TODOs:

  • squash commits
  • include documentation
  • add unit tests

@michal-gubricky michal-gubricky self-assigned this Jan 31, 2024
@michal-gubricky michal-gubricky changed the title :sparkles Add missing unit tests for openstackclusterstackrelease controller ✨ Add missing unit tests for openstackclusterstackrelease controller Jan 31, 2024
Signed-off-by: michal.gubricky <michal.gubricky@dnation.cloud>

@matofeder matofeder left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@michal-gubricky
michal-gubricky merged commit da71737 into main Jan 31, 2024
@michal-gubricky
michal-gubricky deleted the feat/add-more-tests branch January 31, 2024 15:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants