search

bio2bel / hmdb

A Bio2BEL package for converting the Human Metabolite Database (HMDB) to BEL
http://bio2bel-hmdb.rtfd.io/
MIT License
5 stars 1 forks source link

Fix broken tests #5

Open cthoyt opened 6 years ago

cthoyt commented 6 years ago

Hey we need to figure out what's going on in the tests. One thing to note is that the old artifactory namespace https://arty.scai.fraunhofer.de/artifactory/bel/namespace/human-phenotype-ontology/human-phenotyoe-ontology-YYYYMMDD.belns has been updated to https://arty.scai.fraunhofer.de/artifactory/bel/namespace/hp/hp-YYYYMMDD.belns

Cojabi commented 6 years ago

when i try to run tox i get 4 errors. Always when creating a Manager due to the lack of an pybel module.

>> from pybel.resources.arty import get_latest_arty_namespace ImportError: No module named 'pybel.resources.arty'

Did you change something in pybel there?

cthoyt commented 6 years ago

Since I don't have access to push stuff on Artifactory anymore, I decided it's not a good idea to maintain it as a core feature of PyBEL. I sliced all of those features out to a new repository, pybel-artifactory.

I replaced all of the calls with hard-coded links, centralized in constants.py and removed dependencies. Let me know what you think

cthoyt commented 6 years ago

Okay it looks like now we're back to just the one test failing.

cthoyt commented 5 years ago

Can we take a look into this again