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.

Sign up for GitHub

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

Jump to bottom

Reproducible failure in SearchTemplateRequestTests #43157

Closed
romseygeek opened this issue Jun 12, 2019 · 2 comments
Closed

Reproducible failure in SearchTemplateRequestTests #43157

romseygeek opened this issue Jun 12, 2019 · 2 comments
Assignees
@jdconrad
Labels
:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >test-failure Triaged test failures from CI

Comments

@romseygeek
Copy link
Contributor 

./gradlew :modules:lang-mustache:test --tests "org.elasticsearch.script.mustache.SearchTemplateRequestTests.testEqualsAndHashcode" -Dtests.seed=ED8940E3CD13BA8E -Dtests.security.manager=true -Dtests.locale=kl-GL -Dtests.timezone=America/Menominee -Dcompiler.java=12 -Druntime.java=11

org.elasticsearch.script.mustache.SearchTemplateRequestTests > testEqualsAndHashcode FAILED
    java.lang.AssertionError: SearchTemplateRequest mutation should not be equal to original
    Expected: not <org.elasticsearch.script.mustache.SearchTemplateRequest@ee43a3d7>
         but: was <org.elasticsearch.script.mustache.SearchTemplateRequest@ee43a3d7>

Fails on master, passes on 7x
@romseygeek romseygeek added :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >test-failure Triaged test failures from CI labels Jun 12, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@jdconrad jdconrad mentioned this issue Jun 21, 2019
Merged
jdconrad added a commit to jdconrad/elasticsearch that referenced this issue Jun 25, 2019
@jdconrad
Fix search template request (elastic#43509)
d640fb5 
A seed was hit in (elastic#43157) that caused mutateInstance to generate an identical 
instance. This change prevents that.
@jdconrad
Copy link
Contributor

Fixed by (#43509)

jkakavas pushed a commit to jkakavas/elasticsearch that referenced this issue Jun 27, 2019
@jdconrad @jkakavas
Fix search template request (elastic#43509)
d488a72 
A seed was hit in (elastic#43157) that caused mutateInstance to generate an identical 
instance. This change prevents that.
mayya-sharipova pushed a commit that referenced this issue Jul 4, 2019
@jdconrad @mayya-sharipova
Fix search template request (#43509)
fc38926 
A seed was hit in (#43157) that caused mutateInstance to generate an identical 
instance. This change prevents that.
jtibshirani pushed a commit that referenced this issue Feb 8, 2021
@jdconrad @jtibshirani
Fix search template request (#43509)
ecc1500 
A seed was hit in (#43157) that caused mutateInstance to generate an identical 
instance. This change prevents that.
@jtibshirani jtibshirani mentioned this issue Feb 8, 2021
Closed
jtibshirani pushed a commit that referenced this issue Feb 8, 2021
@jdconrad @jtibshirani
Fix search template request (#43509)
c8e12ec 
A seed was hit in (#43157) that caused mutateInstance to generate an identical 
instance. This change prevents that.
jtibshirani pushed a commit that referenced this issue Feb 8, 2021
@jdconrad @jtibshirani
Fix search template request (#43509)
1bbed54 
A seed was hit in (#43157) that caused mutateInstance to generate an identical 
instance. This change prevents that.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jdconrad jdconrad

Labels
:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@romseygeek @jdconrad @elasticmachine