feat: dataverse export #909
Conversation
| def dataverse_demo(client): | ||
| """Configure environment to use Dataverse demo environment.""" | ||
| client.set_value( | ||
| 'dataverse', 'access_token', '4ca13597-cf43-4815-8763-b64994058c19' |
There was a problem hiding this comment.
Access token should probably be extracted into env variable and encrypted for the CI.
There was a problem hiding this comment.
This a free account on their demo server (same as the one above for Zenodo). I'd prefer to keep it like this for easier execution of tests in our local machines.
| @@ -269,8 +269,26 @@ def test_dataset_reimport_removed_dataset(runner, project, sleep_after): | |||
|
|
|||
|
|
|||
| @pytest.mark.integration | |||
| def test_dataset_import_preserve_names(runner, project, sleep_after): | |||
| """Test import keeps original file names.""" | |||
| DOI = '10.7910/DVN/F4NUMR' | |||
There was a problem hiding this comment.
Let's not define constants in local scopes, otherwise pylint is crying. Just make it normal variable (lower case).
| url_parts = url_parts._replace(path=path, query=query_params) | ||
| return urllib.parse.urlunparse(url_parts) | ||
|
|
||
| def _post(self, url, json=None, data=None, files=None): |
There was a problem hiding this comment.
We should make this more robust, with #902 merged there is a new utility for retrying.
With it you could:
with retry() as session:
session.post(...)
| @@ -269,8 +269,26 @@ def test_dataset_reimport_removed_dataset(runner, project, sleep_after): | |||
|
|
|||
|
|
|||
| @pytest.mark.integration | |||
There was a problem hiding this comment.
With #902, there is a new decorator with which you can specify integration tests for a rerun
@flaky(...)
...
This should make IT more robust in case of failure when running this in parallel.
m-alisafaee
force-pushed
the
668-dataverse-export
branch
from
7efeda7 to
a09fe48
Compare
Description
Enables exporting datasets to Dataverse. Fixes wrong file names after import.
Fixes #908
Fixes #668