valkey-io/valkey-glide

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

Offen

#3.947 geöffnet am 22.05.2025

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Java (166 Forks)auto 404
Task 🔧good first issuenode 🐢testing 🧪

Repository-Metriken

Stars
 (762 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 7T 22h) (94 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Überprüfen Sie die aktuellen Testdateien in node/tests/GlideClient.test.ts und identifizieren Sie die internen Protobuf Tests. Verschieben Sie diese Tests in eine neue oder vorhandene Testdatei im internen Testverzeichnis und stellen Sie sicher, dass alle Importe korrekt aktualisiert werden und keine vorhandene Funktionalität beeinträchtigt wird.
Tech Stack
typescriptnodejs
Domain
backend
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitNode.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85