fix dataverse regression introduced in last release

pull/1011/head
Mridul Seth 2021-02-11 21:58:21 +01:00
rodzic cb4621f254
commit 0885204a5d
2 zmienionych plików z 22 dodań i 18 usunięć

Wyświetl plik

@ -73,7 +73,7 @@ class Dataverse(DoiProvider):
parsed_url._replace(path="/api/search", query=search_query)
)
self.log.debug("Querying Dataverse: " + search_url)
data = self.urlopen(search_url).json()
data = self.urlopen(search_url).json()["data"]
if data["count_in_response"] != 1:
self.log.debug(
"Dataverse search query failed!\n - doi: {}\n - url: {}\n - resp: {}\n".format(
@ -103,7 +103,7 @@ class Dataverse(DoiProvider):
)
resp = self.urlopen(url, headers={"accept": "application/json"})
record = resp.json()
record = resp.json()["data"]
for fobj in deep_get(record, "latestVersion.files"):
file_url = "{}/api/access/datafile/{}".format(

Wyświetl plik

@ -60,8 +60,10 @@ def test_detect_dataverse(test_input, expected, requests_mock):
requests_mock.get(
"https://dataverse.harvard.edu/api/search?q=entityId:3323458&type=file",
json={
"count_in_response": 1,
"items": [{"dataset_persistent_id": "doi:10.7910/DVN/6ZXAGT"}],
"data": {
"count_in_response": 1,
"items": [{"dataset_persistent_id": "doi:10.7910/DVN/6ZXAGT"}],
}
},
)
@ -109,20 +111,22 @@ def dv_files(tmpdir):
def test_dataverse_fetch(dv_files, requests_mock):
mock_response = {
"latestVersion": {
"files": [
{"dataFile": {"id": 1}, "label": "some-file.txt"},
{
"dataFile": {"id": 2},
"label": "some-other-file.txt",
"directoryLabel": "directory",
},
{
"dataFile": {"id": 3},
"label": "the-other-file.txt",
"directoryLabel": "directory/subdirectory",
},
]
"data": {
"latestVersion": {
"files": [
{"dataFile": {"id": 1}, "label": "some-file.txt"},
{
"dataFile": {"id": 2},
"label": "some-other-file.txt",
"directoryLabel": "directory",
},
{
"dataFile": {"id": 3},
"label": "the-other-file.txt",
"directoryLabel": "directory/subdirectory",
},
]
}
}
}