Skip to content

Improve pattern text downgrade license test#143102

Draft
parkertimmins wants to merge 4 commits intoelastic:mainfrom
parkertimmins:parker/pattern-text-downgrade-test-improve
Draft

Improve pattern text downgrade license test#143102
parkertimmins wants to merge 4 commits intoelastic:mainfrom
parkertimmins:parker/pattern-text-downgrade-test-improve

Conversation

@parkertimmins
Copy link
Contributor

Update the pattern_text downgrade test so that it include adding docs and querying for docs.
Specifically the test now does the following:

  1. Create data stream with trial license and pattern text field
  2. Index docs, verify they're searchable.
  3. Downgrade to basic license.
  4. Index more docs in same backing index, verify all docs searchable, verify settings unchanged.
  5. Rollover the data stream, verify the new backing index has disable_templating=true.
  6. Index more docs into the new backing index, verify all docs searchable across both indices.
  7. Search with "fields": ["pattern_field"] to verify the valueFetcher() code path works across both backing indices.

@parkertimmins parkertimmins added >non-issue :StorageEngine/Mapping The storage related side of mappings v9.4.0 labels Feb 25, 2026
List<String> allDocs = new ArrayList<>(allDocsBeforeRollover);
allDocs.addAll(postRolloverDocs);
assertSearchReturnsValues(allDocs);

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added fields loading query so that it would have caught the bug fixed in #142767 . (This test should fail until that PR is merged)

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

Labels

>non-issue :StorageEngine/Mapping The storage related side of mappings v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant