Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

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

Aperta
#178 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
78/100
Tipo di issue
Funzionalità
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
bash, go
Ambito
cli, testing-qa

Direzione di ricerca

Inizia con e2e/testdata/postgres/ob.yml.tmpl:37-42 e con il flusso esistente in e2e/server_test.go:397-400. Leggi e2e/apps/one-app-one-host.sh:54-63 e e2e/server_harness_test.go:385-400 per comprendere gli helper attuali di strong-approval. Aggiungi la copertura del fixture distruttivo ed esegui la suite e2e; il lavoro è completo quando il token errato viene rifiutato, il nome del job viene accettato e il riepilogo nomina il job.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
Go
Stelle
3
Fork
0
Merge medio
2h 40m
PR unite (30g)
63

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di labstack/onebox

Tutte le issue di labstack/onebox

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.