alteryx / alteryx/evalml

[SPIKE] Update test coverage for all_components() method

Open
#2,555 1 comment 0 reactions 1 assignee Claimed by @asniyaz View on GitHub
spike testing
Dominant language
Python
Stars
850
Forks
96
PR merge metrics
No merged PRs in 30d

Description

This issue covers a couple of points:

1. Due to the various tests we have for the components we leverage in different environments and the codecoverage on tests we perform, we end up not covering `is_using_conda` and `is_running_py_39_or_above`. Therefore they fail to ensure complete code coverage when we edit these values.
```
def test_all_components(
has_minimal_dependencies, is_running_py_39_or_above, is_using_conda
):
if has_minimal_dependencies:
n_components = 37
elif is_using_conda:
n_components = 48
elif is_running_py_39_or_above:
n_components = 47
else:
n_components = 49
assert len(all_components()) == n_components
```

2. Ensure that merging this to main doesn't break the `build_conda_package` process, especially regarding the [passing](https://github.com/conda-forge/evalml-core-feedstock/blob/latest_release_changes/recipe/meta.yaml#L57) of `--has-minimal-dependencies` to `evalml-core` and `--is-using-conda` to `evalml`

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.