vlang/vsl

test: deduplicate identical fft_test.v and fft64_test.v

Ouverte

#320 ouverte le 14 juil. 2026

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)V (49 forks)auto 404
Hacktoberfestenhancementgood first issue

Métriques du dépôt

Stars
 (400 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

fft/fft_test.v and fft/fft64_test.v are byte-identical (2365B each) — a copy-paste duplication with no f32 vs f64 cross-validation.

Task

  1. Extract shared test logic into fft/fft_test_common.v
  2. Have fft_test.v and fft64_test.v import and parameterize it
  3. Add a cross-validation test verifying f32 and f64 produce consistent results

Verification

  • v test vsl/fft passes

Guide contributeur

Direction de recherche
Étape 1 : Comparez fft/fft test.v et fft/fft64 test.v pour identifier la logique de test identique. Étape 2 : Créez fft/fft test common.v avec des helpers de test partagés. Étape 3 : Modifiez fft test.v et fft64 test.v pour importer et utiliser le module commun, en paramétrant pour f32 et f64. Étape 4 : Ajoutez un test de validation croisée dans fft/fft test.v ou un nouveau fichier pour vérifier que f32 et f64 produisent des résultats cohérents. Étape 5 : Exécutez `v test vsl/fft` pour confirmer que tous les tests réussissent.
Stack technique
Aucun
Domaine
testing
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.
1-3 heures
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
VGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75