alteryx / alteryx/evalml

EPIC Merge Unit Test Dataframes

Aperta
#2,715 0 commenti 0 reazioni 1 assegnatario Rivendicata da @asniyaz Vedi su GitHub
epic refactor testing
Lingua principale
Python
Stelle
850
Fork
96
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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):
...
```

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.