Skip to content

Commit 50c3cd4

Browse files
authored
It doesn't look like this test has ever worked, and nobody wants to mess with that code. (#133688)
So let's just drop it - if we ever do dare to change the conversion code, we can add proper tests then.
1 parent db39847 commit 50c3cd4

File tree

2 files changed

+0
-7
lines changed

2 files changed

+0
-7
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -471,9 +471,6 @@ tests:
471471
- class: org.elasticsearch.upgrades.TransformSurvivesUpgradeIT
472472
method: testTransformRollingUpgrade
473473
issue: https://github.com/elastic/elasticsearch/issues/132892
474-
- class: org.elasticsearch.index.mapper.LongFieldMapperTests
475-
method: testFetchCoerced
476-
issue: https://github.com/elastic/elasticsearch/issues/132893
477474
- class: org.elasticsearch.xpack.eql.planner.QueryTranslatorTests
478475
method: testMatchOptimization
479476
issue: https://github.com/elastic/elasticsearch/issues/132894

server/src/test/java/org/elasticsearch/index/mapper/LongFieldMapperTests.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -118,10 +118,6 @@ protected Number randomNumber() {
118118
return randomDoubleBetween(-MAX_SAFE_LONG_FOR_DOUBLE, MAX_SAFE_LONG_FOR_DOUBLE, true);
119119
}
120120

121-
public void testFetchCoerced() throws IOException {
122-
assertFetch(randomFetchTestMapper(), "field", 3.783147882954537E18, randomFetchTestFormat());
123-
}
124-
125121
protected IngestScriptSupport ingestScriptSupport() {
126122
return new IngestScriptSupport() {
127123
@Override

0 commit comments

Comments
 (0)