actions icon indicating copy to clipboard operation
actions copied to clipboard

Actions lacking tests

Open Bo98 opened this issue 8 months ago • 1 comments

(Extracted from #542)

The following actions are lacking tests and thus we are currently blindly merging changes to:

Action Status Recommendation Status
bump-formulae Deprecated but stable.
Any bump-package test coverage would cover this one
Add tests to bump-packages
bump-packages Composite action. No tests Figure out how to test this
create-gcloud-instance Shell scripts. No tests @Bo98 will deprecate this soon
failures-summary-and-bottle-result Composite action. No tests Figure out how to test this
find-related-workflow-run-id Uses gh. No tests Migrate this to use octokit.js :white_check_mark: #590
limit-pull-requests Uses gh. No tests Migrate this to use octokit.js :white_check_mark: #589
post-build
pre-build
Composite action. No tests.
Probably the most actively maintained in this list
Move this to homebrew-core,
which already has CI coverage
remove-disabled-packages Uses Ruby, executes commands. No tests Figure out how to test this
wait-for-idle-runner Can easily be added to the Node test suite
but isn't expected to stay around for long
@Bo98 will deprecate this soon

Node.js tests are preferred (see *.test.mjs files) as it links with Codecov. Dedicated test workflows (like setup-homebrew) are acceptable if Node.js tests are demonstrated to not be possible/feasible (e.g. things written in pure shell that don't benefit from porting to JS, such as bump-packages). Any such test workflows however should not require any write tokens or secrets.

Bo98 avatar Jun 02 '24 02:06 Bo98