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

Acceptance tests summary comment not added to the PR from forks #1960

Open
davidgamez opened this issue Feb 4, 2025 · 0 comments
Open

Acceptance tests summary comment not added to the PR from forks #1960

davidgamez opened this issue Feb 4, 2025 · 0 comments
Labels
bug Something isn't working (crash, a rule has a problem) status: Needs triage Applied to all new issues tech debt tests Anything related to our tests.

Comments

@davidgamez
Copy link
Member

Describe the bug

The acceptance tests summary is not added to the PR when the action is triggered in forks.
The following error is visible in the logs:

(node:1) UnhandledPromiseRejectionWarning: HttpError: Resource not accessible by integration
    at /node_modules/@octokit/request/dist-node/index.js:66:23
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:1) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:1) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Example:

Steps/Code to Reproduce

  • Create a fork with a non-contributor user
  • Create a PR that triggers the acceptance tests

Expected Results

The acceptance tests summary comment is added to the PR.

Actual Results

The acceptance tests summary comment is not added and an error is visible in the logs of the compare-output action.

Screenshots

No response

Files used

No response

Validator version

N/A

Operating system

N/A

Java version

No response

Additional notes

No response

@davidgamez davidgamez added bug Something isn't working (crash, a rule has a problem) status: Needs triage Applied to all new issues tech debt labels Feb 4, 2025
@github-project-automation github-project-automation bot moved this to Requires investigation in Bug triage Feb 4, 2025
@emmambd emmambd added the tests Anything related to our tests. label Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working (crash, a rule has a problem) status: Needs triage Applied to all new issues tech debt tests Anything related to our tests.
Projects
Status: Requires investigation
Development

No branches or pull requests

2 participants