Skip to content

Change HLRC GetIndexResponseTests to AbstractResponseTestCase#66934

Merged
martijnvg merged 1 commit intoelastic:7.xfrom
martijnvg:fix_get_index_response_hlrc_test
Jan 5, 2021
Merged

Change HLRC GetIndexResponseTests to AbstractResponseTestCase#66934
martijnvg merged 1 commit intoelastic:7.xfrom
martijnvg:fix_get_index_response_hlrc_test

Conversation

@martijnvg
Copy link
Copy Markdown
Member

This was done in master via #47364, but that change was never backported to 7.x.
This change only includes the changes to GetIndexResponseTests from that pr.

This should also fix the failure in #66653,
since no fields are shuffled in the server side response.

Closes #66653

This was done in master via elastic#47364, but that change was never backported to 7.x.
This change only includes the changes to GetIndexResponseTests from that pr.

This should also fix the failure in elastic#66653,
since no fields are shuffled in the server side response.

Closes elastic#66653
@martijnvg martijnvg added >test Issues or PRs that are addressing/adding tests :Data Management/Indices APIs DO NOT USE. Use ":Distributed/Indices APIs" or ":StorageEngine/Templates" instead. v7.12.0 v7.11.1 labels Jan 4, 2021
@elasticmachine elasticmachine added the Team:Data Management (obsolete) DO NOT USE. This team no longer exists. label Jan 4, 2021
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-features (Team:Core/Features)

@martijnvg martijnvg changed the title Move HLRC GetIndexResponseTests to AbstractResponseTestCase Change HLRC GetIndexResponseTests to AbstractResponseTestCase Jan 4, 2021
Copy link
Copy Markdown
Contributor

@danhermann danhermann left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Data Management/Indices APIs DO NOT USE. Use ":Distributed/Indices APIs" or ":StorageEngine/Templates" instead. Team:Data Management (obsolete) DO NOT USE. This team no longer exists. >test Issues or PRs that are addressing/adding tests v7.11.1 v7.12.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants