Skip to content

Commit

Permalink
Update test_isatools_utils.py
Browse files Browse the repository at this point in the history
Looks like the API changed a little bit, so it's 'ols4' instead of 'ols'. It also returns a '?lang=en' at the end, so I changed it to an assertIn instead of assertEqual. The description is also not empty now. I'm not sure why it was once empty. Perhaps 'efo' wasn't actually being found and now it is? I ran the test with the base 'develop' branch and the description is not empty their either.
  • Loading branch information
ptth222 committed Feb 29, 2024
1 parent 552f8c3 commit 27ba1ad
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions tests/utils/test_isatools_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -82,12 +82,13 @@ def test_get_ontology(self):
ontology_source = ols.get_ols_ontology('efo')
self.assertIsInstance(ontology_source, OntologySource)
self.assertEqual(ontology_source.name, 'efo')
self.assertEqual(
ontology_source.file,
'https://www.ebi.ac.uk/ols/api/ontologies/efo')
self.assertIn(
'https://www.ebi.ac.uk/ols4/api/ontologies/efo',
ontology_source.file
)
self.assertIsInstance(ontology_source.version, str)
self.assertEqual(
ontology_source.description, '')
ontology_source.description, 'Experimental Factor Ontology')

def test_search_for_term(self):
ontology_source = ols.get_ols_ontology('efo')
Expand Down

0 comments on commit 27ba1ad

Please sign in to comment.