valkey-io/valkey-glide

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

Ouverte

#3 947 ouverte le 22 mai 2025

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)Java (166 forks)auto 404
Task 🔧good first issuenode 🐢testing 🧪

Métriques du dépôt

Stars
 (762 étoiles)
Métriques de merge PR
 (Merge moyen 7j 22h) (94 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez les fichiers de test actuels dans node/tests/GlideClient.test.ts et identifiez les tests internes de protobuf. Déplacez ces tests vers un nouveau fichier de test ou un fichier existant dans le répertoire des tests internes, en vous assurant que toutes les importations sont correctement mises à jour et qu'aucune fonctionnalité existante n'est cassée.
Stack technique
typescriptnodejs
Domaine
backend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitNode.js
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85