alteryx/evalml

Refactor Additional Tests to Use get_test_data_from_configuration()

Open

#3,666 opened on Aug 15, 2022

View on GitHubContributor guide
 (0 comments) (0 reactions) (0 assignees)Python (93 forks)auto 404
good first issuerefactortesting

Repository metrics

Stars
 (852 stars)
PR merge metrics
 (PR metrics pending)

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

Contributor guide

Research direction
Identify all test files that contain the specific lines of code mentioned in the PR suggestion (PR #3650) and replace them with calls to `get test data from configuration()`.
Tech stack
python
Domain
backend
Issue type
Refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
PythonTesting
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75