Module "mock-socket" can cause false test positives and negatives
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Error
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- testing
Línea de trabajo
Empieza revisando el mock del servidor mock-socket como reemplazo de ws Server y los problemas de compatibilidad enumerados: ping, los valores del evento connect, los objetos de request y la identidad del socket. Después inspecciona la prueba de slow-reconnect bloqueada en el cuerpo del issue y determina qué comportamiento debe admitirse para que esa prueba sea fiable. Se considera terminado cuando se hayan resuelto los riesgos de falsos positivos y falsos negativos o se haya ajustado claramente la estrategia de pruebas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
There are multiple issues with the library, specifically with the server mock used - in our case - as a drop-in replacement for the ws Server. It may be the case that that is not its intended use case.
Issues:
- Misses
Server.prototype.ping - Gives server instead of client in 'connect' event, which is fixed in 8.0...
- But if we upgrade this causes issues: https://github.com/thoov/mock-socket/issues/215
- Does not give request object in 'connect' event (we use that)
- Uses same object as server socket and client socket (impact unknown)
This also prevents us from implementing this test:
it('prints connect on slow reconnect', async function() {
const rc = createRemoteConsoleClient(debugServer, webSocketFactory);
const logPromise = waitForCalls(terminal.log, 2, 8000);
rc._webSocket.close(1000);
rc._disposeSocket();
await new Promise(resolve => setTimeout(resolve, 4000));
rc._connect();
const log = await logPromise;
expect(log).to.contain(' connected');
}).timeout(10000);
- Lenguaje dominante
- JavaScript
- Estrellas
- 11
- Forks
- 4
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
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 eclipsesource/tabris-js-cli
-
Document TABRIS_BUILD_KEYAbiertodoc
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
-
feature
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
eclipsesource/tabris-js-cli#47 · 1 comentario ·
-
Notify client of file changesAbiertofeature
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
-
build
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
-
cli help should adjust to terminal width (or assume less space available)Quizá libre de nuevo @cpetrov la tomó hace 2337 días y no hay ningún pull request abierto. Abierto
eclipsesource/tabris-js-cli#32 · 1 asignado ·
Todos los issues de eclipsesource/tabris-js-cli
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
Design only Leadership Survey SLFS
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
bcgov/digital-journeys#2293 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
tursodatabase/turso#9405 ·
Los mantenedores suelen responder en 1 día
-
Toolkit
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
API Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
ProjectSidewalk/SidewalkWebpage#5556 ·
Los mantenedores suelen responder en 1 día