-
Notifications
You must be signed in to change notification settings - Fork 501
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
API test suite failing for "develop" branch post 4.9.2 #5025
Comments
They are all related to the changes in #4565 around builtinuser. Thanks for running this Phil! |
- Fixes one test and moves the error down on another
-Fixes at least one broken IT test
Pull request #5036 looks good to me. Moving to QA. @matthew-a-dunlap will be adding some testing notes. |
Half of the reason the IT tests were failing is due to the IT Tests looking for email in builtinuser, when it no longer lives there. The non-test changes were to:
Testing this means confirming that both of these two pieces of code work as expected. Also maybe try changing a password and another basic account test or two to around email. |
I just ran the API test suite on the "phoenix" server and according to https://build.hmdc.harvard.edu:8443/job/phoenix.dataverse.org-apitest-develop/ sometime between August 24th (df3e31f) and September 6th (d149475) the API test suite started failing. Here's a screenshot:
The phoenix server is described in the Dev Guide at http://guides.dataverse.org/en/4.9.2/developers/testing.html#the-phoenix-server
The text was updated successfully, but these errors were encountered: