Title: Add tests for config-utils by robertbrignull · Pull Request #36 · github/codeql-action · GitHub
Open Graph Title: Add tests for config-utils by robertbrignull · Pull Request #36 · github/codeql-action
X Title: Add tests for config-utils by robertbrignull · Pull Request #36 · github/codeql-action
Description: Adds tests for the config-utils file. This does change the behaviour in a couple of minor cases, but largely this PR is trying to just assert the current behaviour. I'm not a huge fan of the fact that we silently ignore invalid yaml fields in certain cases. I'd prefer it if we always threw an error, but I didn't want to make that change here. If we decide to do that in the future then I'd feel much happier having tests of this functionality. A side note, I had to add the --serial flag when running the tests to stop running them in parallel. My understanding (according to the docs) is that tests from separate files are run in separate nodejs processes, but tests within a file are run in one process. Therefore since we are messing with process environment variables in order to control our code we need to run tests serially and avoid parallelism. Merge / deployment checklist Run test builds as necessary. Can be on this repository or elsewhere as needed in order to test the change - please include links to tests in other repos! CodeQL using init/analyze actions 3rd party tool using upload action Confirm this change is backwards compatible with existing workflows. Confirm the readme has been updated if necessary.
Open Graph Description: Adds tests for the config-utils file. This does change the behaviour in a couple of minor cases, but largely this PR is trying to just assert the current behaviour. I'm not a huge fan of the fa...
X Description: Adds tests for the config-utils file. This does change the behaviour in a couple of minor cases, but largely this PR is trying to just assert the current behaviour. I'm not a huge fan of th...
Opengraph URL: https://github.com/github/codeql-action/pull/36
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:ed878deb-a661-1ee0-af75-3c55c49cc1e4 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | C1AE:2893C0:239F0:31372:6A5583EA |
| html-safe-nonce | 50d81df2db00065940b412e91922a9eaa51128e5036788d31618aa344b80af9f |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDMUFFOjI4OTNDMDoyMzlGMDozMTM3Mjo2QTU1ODNFQSIsInZpc2l0b3JfaWQiOiI5OTM4MTY2NjY1Nzk2MDkzOCIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | d77d49e1c12b3d43721eceb0b3e3743f0a9a859494e444bfb89cb2abfd0e5ce4 |
| hovercard-subject-tag | pull_request:418087007 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/github/codeql-action/pull/36/files |
| twitter:image | https://avatars.githubusercontent.com/u/3749000?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/3749000?s=400&v=4 |
| og:image:alt | Adds tests for the config-utils file. This does change the behaviour in a couple of minor cases, but largely this PR is trying to just assert the current behaviour. I'm not a huge fan of the fa... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | b5665c84831ed9ac4fb79519c16c9c5580ba8092fb8bb6e3e72972ec7197348e |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/github/codeql-action git https://github.com/github/codeql-action.git |
| octolytics-dimension-user_id | 9919 |
| octolytics-dimension-user_login | github |
| octolytics-dimension-repository_id | 259445878 |
| octolytics-dimension-repository_nwo | github/codeql-action |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 259445878 |
| octolytics-dimension-repository_network_root_nwo | github/codeql-action |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 3215163789b3dc58dba97bd78a30c20c949f477f |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width