-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Complete testsuite for upload / creation #4
Comments
Very recently we set up a Jenkins server that spins up Dataverse on AWS EC2. (And that EC2 instance is terminated later that day.) If we want, we could close this issue and focus on testing over at #22. I'm fine with whatever. 😄 |
Approximate order of functions to write tests for. The later functions tend to depend on the earlier ones. The classification & order will be updated as I learn more. Downloading Structure & Metadata
Downloading Files
Creating & Publishing
Uploading
Deleting
I haven't studied/classified yet
|
See also #40 for discussion on how we will use a live Dataverse for this task. Will close when more of the checklist ^ is completed. |
@kuriwaki @wibeasley I just posted a message intended mostly for @JR-1991 about newly published containers for API testing: https://dataverse.zulipchat.com/#narrow/stream/377090-python/topic/containers.20for.20API.20testing You're very welcome to join Zulip and discuss! (That's the "python" stream but I'm happy to create more streams if you want one.) Thanks, as always, for all of your contributions to the |
Great to see it in action! This github actions setup looks like the ideal setup for our test environment. And thank you, @JR-1991. |
@kuriwaki I'm getting a strange error when I upload a file: https://dataverse.zulipchat.com/#narrow/stream/377090-python/topic/containers.20for.20API.20testing/near/355010545 Here's my workflow file if you want to try to iterate on it: https://github.com/IQSS/dataverse-sample-data/actions/runs/4856813925/workflow I am able to create a collection and a dataset, at least, so maybe you could try adding tests for that. |
This looks awesome. |
Yes! @poikilotherm got it working today! Check it out! https://github.com/IQSS/dataverse-sample-data/actions/runs/4872123218/jobs/8689917400#step:8:74 You should be able to use this as a starting point: https://github.com/IQSS/dataverse-sample-data/compare/container-test |
Beep boop. These days I'm actively using https://github.com/gdcc/api-test-runner to spin up Dataverse in containers using GitHub Actions (the "develop" workflow, usually). You're very welcome to copy it, if it helps. After I spin up Dataverse, I run our (Java-based) API test suite against it. Longer term we are working on https://github.com/gdcc/dataverse-action which (once it's ready), we'll publish to the GitHub Marketplace. This is all on topic for our weekly Containerization Working Groups meetings, if you're interested! https://ct.gdcc.io |
A new PR from pyDataverse that spins up Dataverse in containers locally: |
Native APISearch APIData Access APIUse test server (https://demo.dataverse.org/)Use @monogan examples from: http://dx.doi.org/10.7910/DVN/ARKOTISWORD/Deposit APIedit: jump to list of functions to test
The text was updated successfully, but these errors were encountered: