DataView instances with different contents compare as equal
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- javascript, node.js
- Área
- backend, testing-qa
Línea de trabajo
Lee el manejo de deepEqual para ArrayBuffer y el fallback que compara las claves enumerables propias; el issue indica que actualmente DataView llega a ese fallback. Añade una reproducción basada en el ejemplo de DataView proporcionado y ejecuta las pruebas pertinentes tanto en modo flexible como estricto. Se considera terminado cuando los contenidos de bytes diferentes se comparan como desiguales, de acuerdo con el comportamiento de assert.deepEqual de Node.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
deepEqual returns true for any two DataView instances regardless of their byte length, offset, or contents, in both loose and strict mode. Node's assert.deepEqual (which this package mirrors) compares DataView contents and fails when they differ. ArrayBuffer had the same issue in #94 before explicit support was added, while DataView still falls through to comparing own enumerable keys (of which it has none).
Reproduction (node repro.js):
const deepEqual = require("deep-equal");
const assert = require("assert");
const a = new DataView(new Uint8Array([1, 2, 3]).buffer);
const b = new DataView(new Uint8Array([4, 5, 6]).buffer);
console.log(deepEqual(a, b), deepEqual(a, b, { strict: true }));
try { assert.deepEqual(a, b); console.log("assert: equal"); } catch { console.log("assert: not equal"); }
Actual:
true true
assert: not equal
Expected:
false false
assert: not equal
Tested with deep-equal 2.2.3 (npm) and main (83701ada), Node v26.13.0.
Credit: https://ucsbplse.github.io/
- Lenguaje dominante
- JavaScript
- Estrellas
- 792
- Forks
- 107
- Merge medio
- 9 d 20 h
- PR fusionados (30 d)
- 1
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin 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 inspect-js/node-deep-equal
-
breaking change in node 23Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
inspect-js/node-deep-equal#110 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
inspect-js/node-deep-equal#109 · 1 comentario ·
-
Some unit tests don't make senseAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
inspect-js/node-deep-equal#108 · 5 comentarios ·
-
Dependency DashboardAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 15/100
Todos los issues de inspect-js/node-deep-equal
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
good first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
cli: `studio --request <missing file>` answers with a raw ENOENT/EISDIR instead of a clear errorAbiertobug good first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
laaielichter/Ai-edit-studio#81 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
jasonacox/pypowerwall#472 ·
Los mantenedores suelen responder en 1 día