Skip to content

Commit

Permalink
fixed test_build.py and skipped test_file_outputs() due to BLAST vers…
Browse files Browse the repository at this point in the history
…ion inconsistency
  • Loading branch information
kbessonov1984 committed Apr 9, 2024
1 parent 1458802 commit 76991fc
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/locidex-ci-pytest-workflow.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ jobs:
python-version: "3.10"
- name: Install dependencies
run: |
sudo apt-get install ncbi-blast+=2.15.0 mafft -y
sudo apt-get install ncbi-blast+ mafft -y
python -m pip install --upgrade pip
pip install flake8 pytest
if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
Expand Down
7 changes: 4 additions & 3 deletions tests/test_build.py
Original file line number Diff line number Diff line change
Expand Up @@ -118,17 +118,18 @@ def test_validate_field_continuity(output_directory):



@pytest.mark.skip(reason="BLAST versions provide different database structure and output results. Need to fix version to 2.15.0 and try to find better testing logic")
def test_file_outputs(output_directory, cmd_args):
#@pytest.mark.skip(reason="BLAST versions provide different database structure and output results. Need to fix version to 2.15.0 and try to find better testing logic")
def test_file_outputs(output_directory):
"""
Make sure all required blast outputs are created
Fails as blast version needs to be set
"""
test_build(cmd_args)
#test_build(cmd_args)
output_data = get_all_file_paths(output_directory)
expected_data = get_all_file_paths(EXPECTED_DATA_OUT)
output_files = [i[1] for i in output_data]
expected_files = [i[1] for i in expected_data]
print(output_files)
assert set(output_files) == set(expected_files)

outputs_sorted = sorted(output_data, key=lambda x: x[1])
Expand Down

0 comments on commit 76991fc

Please sign in to comment.