vlang/vsl

test: deduplicate identical fft_test.v and fft64_test.v

Aberta

#320 aberto em 14 de jul. de 2026

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)V (49 forks)auto 404
Hacktoberfestenhancementgood first issue

Métricas do repositório

Stars
 (400 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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

Guia do colaborador

Direção de pesquisa
Passo 1: Compare fft/fft test.v e fft/fft64 test.v para identificar a lógica de teste idêntica. Passo 2: Crie fft/fft test common.v com helpers de teste compartilhados. Passo 3: Modifique fft test.v e fft64 test.v para importar e usar o módulo comum, parametrizando para f32 e f64. Passo 4: Adicione um teste de validação cruzada em fft/fft test.v ou em um novo arquivo para verificar se f32 e f64 produzem resultados consistentes. Passo 5: Execute `v test vsl/fft` para confirmar que todos os testes passam.
Pilha de tecnologia
None
Domain
testing
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
VGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75