Refactor tests to use `checkPackage` directly
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- dart
- Área
- testing-qa
Línea de trabajo
Comienza con executable_test.dart y compáralo con las pruebas del workspace referenciadas en #138. Rastrea cómo las pruebas invocan actualmente la herramienta y comprueban stdout, stderr y el código de salida; después, revisa checkPackage() y los matchers de streams de package:test. Se considera completado cuando las pruebas del ejecutable usan el punto de entrada directo, conservan esas assertions y consiguen un tiempo de ejecución más cercano al de las pruebas del workspace.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The workspace tests (see #138) run 11 tests in just 1 second, or about 0.1 sec/test. The existing executable_test.dart tests run 25 tests in 1:40, or about 4 sec/test. That's not prohibitively long, but maybe it can be improved.
Both tests generate directories and files to disk in the same way and run the tool on those files. The main difference is that instead of calling dart run dependency_validator and checking the output, the workspace tests call checkPackage() directly and don't spawn any new processes.
The original tests do more than just check the return value though. They also check the stdout, stderr, and exit code, so they use Process.start. They can probably be rewritten to use checkPackage() directly with a custom logger, and use stream matchers from package:test instead of checking the entire process output.
If this results in tests that run similarly to the workspace tests, that could mean the 25 tests will be complete in about 2.5 seconds -- an over 95% speedup.
- Lenguaje dominante
- Dart
- Estrellas
- 66
- Forks
- 26
- Merge medio
- 37 min
- PR fusionados (30 d)
- 1
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Workiva/dependency_validator
-
Glob workspaces supportAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
Workiva/dependency_validator#176 · 2 reacciones ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
Workiva/dependency_validator#167 · 2 comentarios ·
-
Support Doc ImportsAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
Workiva/dependency_validator#153 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
Workiva/dependency_validator#141 · 1 reacción ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
Workiva/dependency_validator#134 · 1 comentario ·
Todos los issues de Workiva/dependency_validator
Issues similares
-
Standalone dash flashes as its own RSVP step (e.g. Russian dialogue)Posiblemente ocupada @bezig01 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
daniel-pimenta-cc/ledor#53 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
Iteration CounterAbiertoenhancement firebase
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
brysonddelozier/scribbles#24 ·
-
bug good first issue P2-Medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
basictech01/uniun#305 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
daohoangson/flutter_widget_from_html#1657 ·
Los mantenedores suelen responder en 3 días