valkey-io/valkey-glide

Node: Test for protobuf internals should be part of internals tests and not client tests

Aperta

#3947 aperta il 22 mag 2025

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)Java (166 fork)auto 404
Task 🔧good first issuenode 🐢testing 🧪

Metriche repository

Star
 (762 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Client tests are integration tests and not unit tests, testing of protobuf encoding should not be part of client tests, but part of the internal test, and should be moved to the right component of testing.

https://github.com/valkey-io/valkey-glide/blob/main/node/tests/GlideClient.test.ts#L95

Testing them inside the integration tests introduce importing of the glide code from specific files, in areas where the glide code should be tested as one piece, and introduces a door for errors of import export.

Guida contributor

Direzione di ricerca
Esamina i file di test correnti in node/tests/GlideClient.test.ts e identifica i test interni di protobuf. Sposta quei test in un file di test nuovo o esistente nella directory dei test interni, assicurandoti che tutte le importazioni siano aggiornate correttamente e che nessuna funzionalità esistente venga interrotta.
Tech stack
typescriptnodejs
Dominio
backend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85