diff options
Diffstat (limited to 'python')
| -rw-r--r-- | python/fatcat_tools/importers/jstor.py | 3 | ||||
| -rw-r--r-- | python/tests/import_jstor.py | 77 |
2 files changed, 78 insertions, 2 deletions
diff --git a/python/fatcat_tools/importers/jstor.py b/python/fatcat_tools/importers/jstor.py index 6ab320c3..9bf4a043 100644 --- a/python/fatcat_tools/importers/jstor.py +++ b/python/fatcat_tools/importers/jstor.py @@ -1,5 +1,4 @@ - import sys import json import sqlite3 @@ -228,7 +227,7 @@ class JstorImporter(EntityImporter): # then try DOI lookup if there is one if not existing and re.ext_ids.doi: try: - existing = self.api.lookup_release(jstor=re.ext_ids.jstor) + existing = self.api.lookup_release(doi=re.ext_ids.doi) except fatcat_client.rest.ApiException as err: if err.status != 404: raise err diff --git a/python/tests/import_jstor.py b/python/tests/import_jstor.py new file mode 100644 index 00000000..b1b86a44 --- /dev/null +++ b/python/tests/import_jstor.py @@ -0,0 +1,77 @@ + +import json, gzip +import pytest +from fatcat_tools.importers import JstorImporter, Bs4XmlFilePusher +from fixtures import api +from bs4 import BeautifulSoup + + +@pytest.fixture(scope="function") +def jstor_importer(api): + with open('tests/files/ISSN-to-ISSN-L.snip.txt', 'r') as issn_file: + yield JstorImporter(api, issn_file, extid_map_file='tests/files/example_map.sqlite3', bezerk_mode=True) + +@pytest.fixture(scope="function") +def jstor_importer_existing(api): + with open('tests/files/ISSN-to-ISSN-L.snip.txt', 'r') as issn_file: + yield JstorImporter(api, issn_file, extid_map_file='tests/files/example_map.sqlite3', bezerk_mode=False) + +def test_jstor_importer(jstor_importer): + last_index = jstor_importer.api.get_changelog(limit=1)[0].index + with open('tests/files/jstor-article-10.2307_111039.xml', 'r') as f: + jstor_importer.bezerk_mode = True + counts = Bs4XmlFilePusher(jstor_importer, f, "article").run() + assert counts['insert'] == 1 + assert counts['exists'] == 0 + assert counts['skip'] == 0 + + # fetch most recent editgroup + change = jstor_importer.api.get_changelog_entry(index=last_index+1) + eg = change.editgroup + assert eg.description + assert "jstor" in eg.description.lower() + assert eg.extra['git_rev'] + assert "fatcat_tools.JstorImporter" in eg.extra['agent'] + + last_index = jstor_importer.api.get_changelog(limit=1)[0].index + with open('tests/files/jstor-article-10.2307_111039.xml', 'r') as f: + jstor_importer.bezerk_mode = False + jstor_importer.reset() + counts = Bs4XmlFilePusher(jstor_importer, f, "article").run() + assert counts['insert'] == 0 + assert counts['exists'] == 1 + assert counts['skip'] == 0 + assert last_index == jstor_importer.api.get_changelog(limit=1)[0].index + +def test_jstor_xml_parse(jstor_importer): + with open('tests/files/jstor-article-10.2307_111039.xml', 'r') as f: + soup = BeautifulSoup(f, "xml") + r = jstor_importer.parse_record(soup.find_all("article")[0]) + + print(r.extra) + assert r.title == "On the Universal Law of Attraction, Including that of Gravitation, as a Particular Case of Approximation Deducible from the Principle that Equal and Similar Particles of Matter Move Similarly, Relatively to Each other. [Abstract]" + assert r.subtitle == None + assert r.original_title == None + assert r.publisher == "The Royal Society" + assert r.release_type == "abstract" + assert r.release_stage == "published" + assert r.license_slug == None + # XXX: assert r.ext_ids.doi == "10.2307/111039" + assert r.ext_ids.jstor == "111039" + assert r.language == "en" + assert r.volume == "5" + assert r.issue == None + assert r.pages == "831-832" + # TODO: None if published jan 1st? + assert str(r.release_date) == "1843-01-01" + assert r.release_year == 1843 + # matched by ISSN, so shouldn't be in there? + #assert extra['container_name'] == "Abstracts of the Papers Communicated to the Royal Society of London" + assert len(r.contribs) == 1 + # XXX: extra['jstor'] stuff + + assert r.contribs[0].raw_name == "John Kinnersley Smythies" + assert r.contribs[0].given_name == "John Kinnersley" + assert r.contribs[0].surname == "Smythies" + + assert not r.refs |