No test exercises the table-format instanceNames.set, so dropping it would silently reopen #1976 for table-format sinks only

Aperta Adatta ai principianti
  • Qualcosa non va?
#2,067 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
76/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
javascript
Ambito
testing

Direzione di ricerca

Leggi il fixture del formato tabella in test/core/sink-registry.test.js:280 e i test dei nomi di istanza ostili in test/core/sink-hostile-instance-name.test.js. Aggiungi un caso di formato tabella con un accessor instanceName che genera un’eccezione e verifica che sinkInstanceName restituisca la chiave convalidata; il test dovrebbe fallire se instanceNames.set viene rimosso in src/core/registry/sinks.js:534.

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

Descrizione

Source: #2057 at fb6035d0448e2e44b05deabbe3387fb404265070
Location: src/core/registry/sinks.js:534
Severity: minor

Evidence

Verified at head fb6035d0: instanceNames.set(handle, instanceName) is written at src/core/registry/sinks.js:431 (blob/request) and :534 (table-format), and every fixture in test/core/sink-hostile-instance-name.test.js instantiates kind 'request', so only :431 is pinned (round 1's mutation run killed the :431 drop 0/8; nothing kills a :534 drop). An adjudicator probe instantiating a real table-format sink through createSinkRegistry and installing a throwing instanceName accessor confirms sinkInstanceName answers the validated key 'table-inst', so the line is behaviorally correct today; the gap is only that no test would catch its removal, and a removal is invisible for honest handles because the guarded fallback returns the same name.

Why deferral is safe

No incorrect behavior exists at the head, confirmed by direct execution, so this is a coverage guard against a future regression, not a defect. The line is a verbatim one-line mirror sitting directly after its handles.set, exactly like the covered call. A table-format hostile fixture is a larger lift than #1976's smallest change warrants in this PR, and the existing table-format instantiate fixture in test/core/sink-registry.test.js:280 makes the follow-up cheap. Backlog only; not admitted for automatic repair.

Acceptance condition

Add one table-format case to test/core/sink-hostile-instance-name.test.js (the fixture shape at test/core/sink-registry.test.js:280 suffices): with a throwing instanceName accessor on the table-format handle, sinkInstanceName answers the validated key, and deleting the set at sinks.js:534 makes the case fail.

Review preference; backlog only. A human may explicitly delegate it with neutral:fix.

Lingua principale
JavaScript
Stelle
11
Fork
0
Merge medio
2h 49m
PR unite (30g)
297

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

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 hyparam/hypaware

Tutte le issue di hyparam/hypaware

Issue simili

Altre issue su JavaScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.