alteryx / alteryx/evalml

EPIC Merge Unit Test Dataframes

Ouverte
#2,715 0 commentaires 0 réactions 1 personne assignée Réclamée par @asniyaz Voir sur GitHub
epic refactor testing
Langage dominant
Python
Étoiles
850
Forks
96
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

This epic encompasses a refactoring of unit tests to merge together dataframes within unit test modules to minimize lines of code as well as minimize impact of Woodwork changes. It would be nice to merge all test dataframes into a single dataframe within the unit test module with column names that are descriptive of the data within and what behavior they're trying to invoke.

Code chunks like:
```
def test_first():
df = pd.DataFrame({"a":[1,2,3], "b":["a", "b", "c"], "c": ["cats", "are", "great"]})
df.ww.init(logical_types = {"a": "Double"})
...

def test_second():
df = pd.DataFrame({"a":[1,2,np.nan], "b":["a", pd.NA, "c"], "c": ["cats", pd.NA, "great"]})
df.ww.init(logical_types = {"a": "Double"})
...
```

Merge to something like:
```
@pytest.mark.fixture
def df():
df = pd.DataFrame({"integers":[1,2,3],
"categories":["a", "b", "c"],
"strings": ["cats", "are", "great"],
"integers_with_nan":[1,2,np.nan],
"categories_with_nan":["a", pd.NA, "c"],
"strings_with_nan": ["cats", pd.NA, "great"]})
df.ww.init(logical_types={"a":"Double"})

def test_first(df):
...

def test_second(df):
...
```

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.