-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add acceptance test for doi manuscript pdf service #1004
Add acceptance test for doi manuscript pdf service #1004
Comments
@markpatton or maybe you are referring to the Files step where the calls to the DOI service are made for |
I also wanted to confirm a doubt I have. I ran a test locally using pass-docker. I entered DOI
Based on the code I see in the UI, I think the UI should show the |
@rpoet-jh Right, I was referring the Files step in particular. I also have observed the same behavior of the ui not seeming to do the right thing. I had meant to make a separate ticket. |
What?
The pass-ui interaction usage of the DOI service which returns available pdfs should have an acceptance test.
Why?
We missed updating the calls to the doi service for CSRF protection by the ui because of a lack of testing.
How?
Acceptance Criteria
A dedicated acceptance test.
Related Issues
The text was updated successfully, but these errors were encountered: