githubnext / githubnext/gh-aw-test

Suggested test: copilot/submit-pull-request-review-self-authored-fallback

Cerrado
#8,653 0 comentarios 0 reacciones 0 asignados Ver en GitHub
suggested new test
Lenguaje dominante
Shell
Estrellas
8
Forks
5
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

## Motivation

Link to the gh-aw PR: github/gh-aw#47400 — fix: downgrade REQUEST_CHANGES→COMMENT on self-authored PRs in safe-outputs PR review

When the reviewing identity matches the PR author, GitHub rejects `REQUEST_CHANGES` with 422 "Can not request changes on your own pull request". The fix ensures reviewer workflows fall back to `COMMENT` in this case. Previously the fallback was bypassed when another 422 ("Line could not be resolved") occurred first. There is no regression test for this scenario.

## Proposed test

- **Workflow file**: `test-copilot-submit-pull-request-review-self-authored-fallback.md`
- **Trigger**: `workflow_dispatch`
- **Engine**: copilot
- **Safe output**: `submit-pull-request-review`
- **Variant**: standard

## Minimal test prompt sketch

Dispatch a review against a pull request authored by the same bot identity running the workflow. Ask the agent to request changes. The test passes if the review is successfully submitted (as a COMMENT) rather than failing with a 422 error.

## New fixtures or secrets needed

Requires a PR in `githubnext/gh-aw-test` authored by the same bot/app identity that submits the review. This may require a dedicated fixture PR or a test-setup step that creates such a PR.

## Notes

The existing `test-copilot-submit-pull-request-review.md` and `test-copilot-submit-pull-request-review-locked.md` both use PRs authored by a different identity. A self-authored PR fixture is the key missing element.

> [!WARNING]
>
> Firewall blocked 1 domain
>
> The following domain was blocked by the firewall during workflow execution:
>
> - `awmgmcpg`
>> To allow these domains, add them to the `network.allowed` list in your workflow frontmatter:
>
> ```yaml
> network:
> allowed:
> - defaults
> - "awmgmcpg"
> ```
>
> See [Network Configuration](https://github.github.com/gh-aw/reference/network/) for more information.
>
>

> Generated by [🔍 Suggest New E2E Tests](https://github.com/githubnext/gh-aw-test/actions/runs/29994494559) · sonnet46 22.2 AIC · ⌖ 5.72 AIC · ⊞ 5.7K · [◷](https://github.com/search?q=repo%3Agithubnext%2Fgh-aw-test+is%3Aissue+%22gh-aw-workflow-call-id%3A+githubnext%2Fgh-aw-test%2Fsuggest-new-e2e-tests%22&type=issues)

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Start with test-copilot-submit-pull-request-review.md and test-copilot-submit-pull-request-review-locked.md, then create test-copilot-submit-pull-request-review-self-authored-fallback.md using the proposed workflow settings. Investigate how githubnext/gh-aw-test supplies PR fixtures and bot identity credentials. Done means a workflow-dispatched review against a self-authored PR succeeds as COMMENT instead of failing with a 422.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
github-actions
Área
ci-cd, testing-qa
Tipo de issue
Nueva funcionalidad
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Tranquilo
Claridad
Bastante claro
Aptitud para principiantes
55/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.