Skip to content
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

DownloadFilesIT/downloadAllFilesRestricted Failing on build 555 #7195

Closed
djbrooke opened this issue Aug 13, 2020 · 4 comments · Fixed by #7198
Closed

DownloadFilesIT/downloadAllFilesRestricted Failing on build 555 #7195

djbrooke opened this issue Aug 13, 2020 · 4 comments · Fixed by #7198
Milestone

Comments

@djbrooke
Copy link
Contributor

We are down to one failing test 😄

On build 555 of Jenkins (not linking here so the bots don't find it) there's more info.

@djbrooke djbrooke added this to the Dataverse 5 milestone Aug 13, 2020
@landreev
Copy link
Contributor

Copying what I said on slack; this appears to be this message hard-coded in DownloadFilesIT.java:

downloadFiles3.then().assertThat()
                .statusCode(FORBIDDEN.getStatusCode())
                .body("status", equalTo("ERROR"))
                .body("message", equalTo("'/api/v1/access/dataset/%3ApersistentId' you are not authorized to access this object via this api endpoint. Please check your code for typos, or consult our API guide at http://guides.dataverse.org."));

and the actual message has changed, after the "error handler" PR got merged. So it simply needs to be adjusted accordingly.

@scolapasta
Copy link
Contributor

Do we even need to check the exact message? I think confirming that we got the write status code and error would be enough? (i.e. we shouldn't have to fix this test every time we decide to change the messaging)

@landreev
Copy link
Contributor

Agree. I don't think it's necessary at all, to check the exact message.

@landreev
Copy link
Contributor

(it's in QA now - but I don't think it needs any QA other than having it pass the tests; which has been confirmed: https://jenkins.dataverse.org/job/IQSS-Dataverse-Develop-PR/job/PR-7198/)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants