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

new(tests): initiated unit tests #5138

Merged
merged 7 commits into from
Aug 16, 2024
Merged

Conversation

omercier
Copy link
Contributor

@omercier omercier commented Aug 9, 2024

Description

Created a unit tests workflow to run test on our code functions.
It runs in the CI before building the fatpacks.

REFS: CTOR-805

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Functionality enhancement or optimization (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software

How this pull request can be tested ?

It's self-tested :)

- moved the robot tests directly under tests/
- first unit test file on perfdata.pm's trim()
- created the docker images builders
- created the actions
- added it to the plugins workflow

REFS: CTOR-805
@omercier omercier force-pushed the CTOR-805-Initiate-Unit-Tests branch from 8104595 to 7bc4b3a Compare August 13, 2024 13:13
@omercier omercier marked this pull request as ready for review August 13, 2024 14:30
@omercier omercier requested review from a team as code owners August 13, 2024 14:30
Evan-Adam
Evan-Adam previously approved these changes Aug 13, 2024
Evan-Adam
Evan-Adam previously approved these changes Aug 13, 2024
Evan-Adam
Evan-Adam previously approved these changes Aug 13, 2024
@mushroomempires mushroomempires dismissed stale reviews from Evan-Adam and themself via b4450d0 August 13, 2024 15:15
@omercier omercier merged commit fe90a52 into develop Aug 16, 2024
17 checks passed
@omercier omercier deleted the CTOR-805-Initiate-Unit-Tests branch August 16, 2024 07:34
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.

5 participants