Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

No e2e coverage for the strong approval ceremony on a destructive job

Abierto
  • ¿Algo no está bien?
#178 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
78/100
Tipo de issue
Nueva funcionalidad
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
bash, go
Área
cli, testing-qa

Línea de trabajo

Empieza con e2e/testdata/postgres/ob.yml.tmpl:37-42 y el flujo existente en e2e/server_test.go:397-400. Lee e2e/apps/one-app-one-host.sh:54-63 y e2e/server_harness_test.go:385-400 para entender los helpers actuales de strong-approval. Añade la cobertura del fixture destructivo y ejecuta la suite de e2e; estará terminado cuando se rechace el token incorrecto, se acepte el nombre del job y el resumen mencione el job.

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

Descripción

enhancement

The problem

The strong approval ceremony — the typed confirmation guarding a destructive
or migration job — has no end-to-end coverage.

The only job run in the e2e suite is chore, which declares
data_effect: none (e2e/testdata/postgres/ob.yml.tmpl:37-42), driven with
y\n (e2e/server_test.go:397-400). That is the one-time class, not the
strong one. The unit test added in #171 covers ob approve --plan, and nothing
covers ob job run --plan or the inline ob job run <id> prompt against a
real binary.

Two e2e helpers still encode the pre-#171 rule and describe it as general:

  • e2e/apps/one-app-one-host.sh:54-63 — approval_answer supplies the release
    ID when the class is strong.
  • e2e/server_harness_test.go:385-400 — releaseIDRe extracts a release ID to
    feed the prompt.

Both are reached only from deploy helpers today, so nothing is broken. But a
destructive job added to the fixture later would fail confusingly against a
helper that looks like it handles the strong class in general.

Proposed change

Add a destructive manual job to an e2e fixture and drive the strong ceremony
against the real binary: the wrong token refused, the job name accepted, and
the summary naming the job. Then make the two helpers say which class they
handle, or teach them the job case.

Scope and safety

Test-only. No product change.

Found while auditing #171.

Lenguaje dominante
Go
Estrellas
3
Forks
0
Merge medio
3 h 10 min
PR fusionados (30 d)
39

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de labstack/onebox

Todos los issues de labstack/onebox

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.