Skip to content

Conversation

@jakelandis
Copy link
Contributor

@jakelandis jakelandis commented Mar 9, 2020

This commit addresses a few issues which prevented the
temporary compat test fixture from running.

It is expected that the compat tests will fail with this PR.

@jakelandis jakelandis requested a review from pgomulka March 9, 2020 18:00
@jakelandis jakelandis changed the title Fix NPE in test due to parameter change used by tests Fix issues which keep the temporary compat test fixutre Mar 9, 2020
@jakelandis
Copy link
Contributor Author

@elasticmachine update branch

@jakelandis
Copy link
Contributor Author

@pgomulka feel free to merge this PR to the feature branch.

compile project(':test:framework')
}

test.enabled = false
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note - this line will likely conflict with this line #53228

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have only disabled it temporarily to see if this will try to continue the build

Copy link
Contributor

@pgomulka pgomulka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, we have 283 failures - which is expected

compile project(':test:framework')
}

test.enabled = false
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have only disabled it temporarily to see if this will try to continue the build

@jakelandis jakelandis merged commit e5de47b into elastic:compat_rest_api Mar 10, 2020
@jakelandis jakelandis deleted the compat_rest_api_fix_npe branch March 10, 2020 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants