Assert the verb.retract warn payloads and the tolerance branch's silence in landed tests

Abierto Apto para principiantes
  • ¿Algo no está bien?
#2,018 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
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
67/100
Tipo de issue
Nueva funcionalidad
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
javascript
Área
testing

Línea de trabajo

Empieza con las ramas retract en src/core/registry/verbs.js:424-478 y las pruebas existentes en test/core/verb-registry.test.js. Añade aserciones que capturen el logger para las tres ramas de advertencia y para comprobar que no se registra nada en la ruta de tolerancia de la registry sin corchetes; ejecuta las pruebas de verb-registry para verificar los payloads esperados y que no haya advertencias en esa ruta.

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

Descripción

Source: #1995 at 13bea152c66247d6d683c808e0999ac4fd529205
Location: src/core/registry/verbs.js:424-478, test/core/verb-registry.test.js, test/core/command-context-plugin-ownership.test.js
Severity: low

Evidence

No landed test captures logger output for verb.retract.unsupported, verb.retract.not_a_projection, or verb.retract.registrar_mismatch: grep -rn "verb.retract" test/ returns nothing at head 13bea152. Round 2 proved the refusal warn's payload at the emitter (hyp_plugin, hyp_owner_plugin, verb_name) only via ad-hoc probes that were discarded, not landed.

Why deferral is safe

Safe outside this PR. The branch the round-1 deferral named (a registry with ownerOf but no registeringAs bracket) emits no warn by design and its retraction behavior is pinned by the landed test at test/core/verb-registry.test.js:162; the two sibling tolerance warns predate this PR and were likewise unasserted; the refusal's user-visible behavior (neighbour's and core's command survive, dispatch still runs the owner's body) is pinned by two landed dispatch-level regression tests. What remains is observability test hardening with no observed incorrect behavior, so it is a preference, not a defect.

Acceptance condition

Add tests that capture the verb-registry logger and assert (a) a forged release emits verb.retract.registrar_mismatch carrying both registrars (hyp_plugin, hyp_owner_plugin) and verb_name, (b) the registry-without-unregister and not-a-projection branches emit their warns, and (c) the bracketless-registry tolerance path retracts with no verb.retract warn at all.

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

Lenguaje dominante
JavaScript
Estrellas
11
Forks
0
Merge medio
2 h 55 min
PR fusionados (30 d)
294

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

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

Todos los issues de hyparam/hypaware

Issues similares

Más issues de JavaScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.