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

Deleting the human view of `daemon status` leaves the whole suite green

Cerrado Apto para principiantes
  • ¿Algo no está bien?
#429 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
68/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
typescript
Área
cli, testing-qa

Línea de trabajo

La rama sin probar es la salida legible para humanos en src/cli/index.ts, alrededor de las líneas 1667-1668, donde daemon status escribe formatStatus(...) a menos que se use --json. Lee las aserciones existentes de daemon start y status en src/cli/index.test.ts para seguir su estilo, y luego añade un test unitario que ejecute daemon status sin --json y espere Daemon: running (worker). Estará listo cuando ese test pase con el código actual y falle si la rama para humanos se reemplaza por el writeResult incondicional. Revisa #422 para ver el it corregido que propone, porque ese trabajo podría solaparse.

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

Descripción

bug:new

Testing rule 3: a code path that can be deleted with a green suite is untested.

I deleted the human branch of simlock daemon status in src/cli/index.ts:1667-1668 — turning

if (values.json) writeResult(environment, status);
else environment.stdout.write(`${formatStatus(status, environment.clock.now())}\n`);

into an unconditional writeResult(environment, status) — and every check stayed green:

  • pnpm exec vitest run --project unit src/cli/index.test.ts: 219 passed. That suite asserts Daemon: running (worker) for simlock status and Daemon running for simlock daemon start, so only daemon status's own human view has no unit assertion.
  • pnpm test:e2e e2e/daemon-lifecycle.test.ts: 3 passed, 1 expected fail. The one test that names daemon status's default view is an it.fails, which passes whether the view exists or not.
  • pnpm test:changed: green apart from the repro test from #422.

So the CLI publishes a human view of daemon status that no test can see. The corrected it proposed in #422 (asserting Daemon: running (worker) on daemon status with no --json) does cover it: run against this same deletion it fails with expected '{"devices":[]…' to contain 'Daemon: running (worker)'.

Found while triaging #422.

Written by an agent.

Lenguaje dominante
TypeScript
Estrellas
19
Forks
1
Merge medio
7 h 57 min
PR fusionados (30 d)
213

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 callstackincubator/simlock

Todos los issues de callstackincubator/simlock

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.