alteryx/evalml

Refactor Additional Tests to Use get_test_data_from_configuration()

Aberta

#3.666 aberto em 15 de ago. de 2022

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)Python (93 forks)auto 404
good first issuerefactortesting

Métricas do repositório

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

Description

As an EvalML dev, I would like all of my tests that are able to to use get_test_data_from_configuration() so that I can reduce total lines of code and confine changes to a single function.

Please see the attached PR, check out what lines of code were replaced in the suggestion with the call to get_test_data_from_configuration() and identify all the other places in our testing base that use those lines. Make a PR to replace those lines with calls to this function.

Originally posted by @eccabay in https://github.com/alteryx/evalml/pull/3650#discussion_r944699505

Guia do colaborador

Direção de pesquisa
Identifique todos os arquivos de teste que contêm as linhas de código específicas mencionadas na sugestão do PR (PR #3650) e substitua as por chamadas a `get test data from configuration()`.
Pilha de tecnologia
python
Domain
backend
Tipo Issue
Refatoração
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.
Meio dia
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
PythonTesting
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75