IQSS / dataverse-client-python

Python library for writing clients that use APIs from Dataverse
http://guides.dataverse.org/en/latest/api
Apache License 2.0
32 stars 27 forks source link

test_connect_unknown_failure gives SSLError when running against dev1 #47

Open mdehollander opened 6 years ago

mdehollander commented 6 years ago
dataverse/test/test_dataverse.py::TestUtils::test_get_element PASSED
dataverse/test/test_dataverse.py::TestUtils::test_get_elements PASSED
dataverse/test/test_dataverse.py::TestUtils::test_format_term PASSED
dataverse/test/test_dataverse.py::TestUtils::test_format_term_replace PASSED
dataverse/test/test_dataverse.py::TestConnection::test_connect PASSED
dataverse/test/test_dataverse.py::TestConnection::test_connect_unauthorized PASSED
dataverse/test/test_dataverse.py::TestConnection::test_connect_unknown_failure FAILED
dataverse/test/test_dataverse.py::TestConnection::test_create_dataverse PASSED
dataverse/test/test_dataverse.py::TestConnection::test_delete_dataverse PASSED
dataverse/test/test_dataverse.py::TestConnection::test_get_dataverses PASSED
dataverse/test/test_dataverse.py::TestConnection::test_get_dataverse PASSED
dataverse/test/test_dataverse.py::TestDataset::test_init PASSED
dataverse/test/test_dataverse.py::TestDataset::test_init_from_xml PASSED
dataverse/test/test_dataverse.py::TestDatasetOperations::test_create_dataset PASSED
dataverse/test/test_dataverse.py::TestDatasetOperations::test_add_dataset_from_xml PASSED
dataverse/test/test_dataverse.py::TestDatasetOperations::test_add_files PASSED
dataverse/test/test_dataverse.py::TestDatasetOperations::test_upload_file PASSED
dataverse/test/test_dataverse.py::TestDatasetOperations::test_display_atom_entry PASSED
dataverse/test/test_dataverse.py::TestDatasetOperations::test_display_dataset_statement PASSED
dataverse/test/test_dataverse.py::TestDatasetOperations::test_delete_a_file PASSED
dataverse/test/test_dataverse.py::TestDatasetOperations::test_delete_a_dataset PASSED
dataverse/test/test_dataverse.py::TestDatasetOperations::test_publish_dataset SKIPPED

Complete log: https://paste.ubuntu.com/p/8PjCBf5pxN/

pdurbin commented 6 years ago

@mdehollander thanks for opening this issue and I'm glad #44 is resolved.

I took a quick look at the stack trace and the test but I can't figure out why it's failing. I'm not even sure what the purpose of this test is.

rliebz commented 6 years ago

I think what the test does is—

Looks like we're getting an SSL error instead, but the real issue is that we should only be hitting the mock endpoint. Not sure what's changed.

pdurbin commented 6 years ago

@rliebz oh, interesting. Thanks for explaining this!

@mdehollander is running the tests against https://dev1.dataverse.org rather than any sort of mock endpoint so perhaps this test doesn't make any sense in that context? I don't know. I guess we should figure out if it passes or fails against the mock API endpoint.