Skip to content

Commit

Permalink
updated linting things that came up and formatting
Browse files Browse the repository at this point in the history
  • Loading branch information
SeriousHorncat committed Sep 16, 2024
1 parent 8b45c30 commit 2200aac
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 20 deletions.
5 changes: 0 additions & 5 deletions backend/src/core/dataset_manifest.py

This file was deleted.

9 changes: 4 additions & 5 deletions backend/tests/unit/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -137,11 +137,10 @@ def _create_dataset_manifest(analysis_name, dataset_name):
return _create_dataset_manifest



@pytest.fixture(name="cpam0046_analysis")
def fixture_cpam0046_analysis(cpam0046_analysis_json):
"""Returns the Analysis for CPAM0046 to verify creating annotation tasks"""
return Analysis(**cpam0046_analysis_json)
# @pytest.fixture(name="cpam0046_analysis")
# def fixture_cpam0046_analysis(cpam0046_analysis_json):
# """Returns the Analysis for CPAM0046 to verify creating annotation tasks"""
# return Analysis(**cpam0046_analysis_json)


@pytest.fixture(name="genomic_units_with_types")
Expand Down
17 changes: 7 additions & 10 deletions backend/tests/unit/core/test_annotation_task.py
Original file line number Diff line number Diff line change
@@ -1,15 +1,20 @@
"""Tests Annotation Tasks and the creation of them"""
import pytest

from src.core.annotation_task import AnnotationTaskFactory, ForgeAnnotationTask, HttpAnnotationTask, VersionAnnotationTask
from src.core.annotation_task import AnnotationTaskFactory, ForgeAnnotationTask, \
HttpAnnotationTask, VersionAnnotationTask
from src.enums import GenomicUnitType
from src.core.annotation_unit import AnnotationUnit


def test_http_annotation_task_build_url(http_annotation_transcript_id):
"""Verifies that the HTTP annotation task creates the base url using the 'url' and the genomic unit"""
actual = http_annotation_transcript_id.build_url()
assert actual == "http://grch37.rest.ensembl.org/vep/human/hgvs/NM_170707.3:c.745C>T?content-type=application/json;refseq=1;" # pylint: disable=line-too-long

assert (
actual ==
"http://grch37.rest.ensembl.org/vep/human/hgvs/NM_170707.3:c.745C>T?content-type=application/json;refseq=1;"
)
# This link cannot be shortened, will just disable for this one due to the nature of the long URL dependency


Expand Down Expand Up @@ -160,14 +165,6 @@ def _create_version_task(genomic_unit, dataset_name):
return _create_version_task


# @pytest.fixture(name="get_patch")
# def fixture_requests_patch():
# with(
# patch("requests.get") as requests_get
# ):
# yield: requests_get


@pytest.fixture(name="gene_ncbi_linkout_dataset")
def fixture_ncbi_linkout_dataset():
"""
Expand Down

0 comments on commit 2200aac

Please sign in to comment.