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

Overhaul how/when integration tests are ran against the code base #3575

Open
2 tasks done
gep13 opened this issue Nov 26, 2024 · 0 comments
Open
2 tasks done

Overhaul how/when integration tests are ran against the code base #3575

gep13 opened this issue Nov 26, 2024 · 0 comments
Labels
0 - Backlog BuildAutomation NO RELEASE NOTES Should not be included in the release notes - not enhancing or fixing end product.

Comments

@gep13
Copy link
Member

gep13 commented Nov 26, 2024

Checklist

  • I have verified this is the correct repository for opening this issue.
  • I have verified no other issues exist related to my request.

Is Your Feature Request Related To A Problem? Please describe.

Currently, the Chocolatey CLI Integration Tests are not run as part of a typical CI build. Only the Unit Tests are executed during the CI builds. As a result, it is possible that issues are only found some time later, once the scheduled integration builds are ran.

The reason that this is done is simply because the Integration Tests are known to take longer to complete, so rather than run them on each build, they are only ran on a scheduled basis. 9 times out of 10, this isn't a problem, but on that one occassion where something would have been found earlier, we are having to wait until the results of the scheduled build completes.

In addition, it is not currently possible to manually trigger the scheduled integration build, as it is not possible to select all the possible branches that we could want.

Describe The Solution. Why is it needed?

There are a couple of ways that this could get better...

  1. Change all CI builds to always do both unit and integration tests
  2. Change all CI builds to run on each PR build
  3. Change all CI builds to allow manual triggering, including the ability to select PR builds

Of the presented solution, the last one makes the most sense, as this would allow fine-grained control over when a build is ran that includes integration tests. That way, if there are any concerns, for example, changes that are coming in with a PR, all the tests can selectively be ran, and if they all pass, the merge can happen. This eliminates the need to wait for the scheduled CI builds to run.

Additional Context

N/A

Related Issues

  • Internal 8689nmppu
@gep13 gep13 added 0 - Backlog BuildAutomation NO RELEASE NOTES Should not be included in the release notes - not enhancing or fixing end product. labels Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 - Backlog BuildAutomation NO RELEASE NOTES Should not be included in the release notes - not enhancing or fixing end product.
Projects
None yet
Development

No branches or pull requests

1 participant