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

Get API test suite passing 😴 😴 😴 #7020

Closed
pdurbin opened this issue Jun 25, 2020 · 1 comment
Closed

Get API test suite passing 😴 😴 😴 #7020

pdurbin opened this issue Jun 25, 2020 · 1 comment
Milestone

Comments

@pdurbin
Copy link
Member

pdurbin commented Jun 25, 2020

[ERROR] FilesIT.testForceReplaceAndUpdate:649 expected:<200> but was:<400> should be fixed in pull request #6968 but there are a variety of other test failing with 500 errors.

As of b97185a I saw the following tests failing on an EC2 instance...

[ERROR] DatasetsIT.testAddRoles:1197 expected:<200> but was:<500>
[ERROR] FilesIT.testReplaceFileBadJson:898 Expected status code <200> doesn't match actual status code <500>.
[ERROR] FilesIT.test_006_ReplaceFileGood:340 Expected status code <200> doesn't match actual status code <500>.

... but these are just examples. If you go to https://jenkins.dataverse.org/job/IQSS-dataverse-develop/test_results_analyzer/ you can see recently failing tests. Here's a screenshot:

Screen Shot 2020-06-25 at 2 00 36 PM

Long term, we'd like to figure the root cause of the 500 errors (believed to be database deadlocks, see #6865) but for now we plan to sprinkle in some sleep statements as necessary. 😴 😴 😴

@pdurbin pdurbin self-assigned this Jun 25, 2020
@pdurbin pdurbin added this to the Dataverse 5 milestone Jun 25, 2020
@pdurbin pdurbin added the Medium label Jun 25, 2020
pdurbin added a commit that referenced this issue Jun 25, 2020
pdurbin added a commit that referenced this issue Jun 26, 2020
pdurbin added a commit that referenced this issue Jun 26, 2020
pdurbin added a commit that referenced this issue Jun 26, 2020
pdurbin added a commit that referenced this issue Jun 26, 2020
pdurbin added a commit that referenced this issue Jun 26, 2020
@pdurbin pdurbin removed their assignment Jun 29, 2020
@djbrooke
Copy link
Contributor

I'm going to close this, as I believe it's been superseded by #6865 and #7069.

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