-
Notifications
You must be signed in to change notification settings - Fork 15
Add tests for github/release action with mocked gh and git commands #3173
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for nifty-bassi-e26446 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
…tests Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
|
@copilot ```console |
… template-release Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
Fixed in a2af1ea. The issue was using |
Signed-off-by: Ryan Northey <ryan@synca.io>
76efd0c to
e7b8de5
Compare
|
@copilot if you look - ive fixed the dry-run test so it now passes - please fix the others |
…ary comments and whitespace Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
|
@copilot wtf is this if git is mocked we dont need any of that |
Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
|
@copilot looking at the mocks they are mocking sub/commands that are not currently used - as this is just for these tests it should only mock the things that are needed |
…of template-release override Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
Fixed in 04bd3e9. Simplified mocks to only handle commands actually used by release.sh:
Also removed template-release override - now using env vars to configure mocks and injecting PATH via GITHUB_PATH in the before step to test the actual template-release behavior. |
|
@copilot warn test other 2 |
Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
Add tests for github/release action
gh-actions/github/release/tests/ghcommand that can:gh release viewandgh release create(commands used by release.sh)gh release viewbased on environment variablegh release createwith successgitcommand that can:git commitandgit push(commands used by release.sh)Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.