Skip to content

Commit c430f6d

Browse files
committed
fixed unit tests
1 parent b07351f commit c430f6d

File tree

5 files changed

+22
-16
lines changed

5 files changed

+22
-16
lines changed

java-client/src/test/java/co/elastic/clients/elasticsearch/model/BehaviorsTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -113,10 +113,10 @@ public void testAdditionalPropertyOnContainer() {
113113

114114
{
115115
SortOptions so = SortOptions.of(_0 -> _0
116-
.script(_1 -> _1.script(_2 -> _2.inline(_3 -> _3.source("blah"))))
116+
.script(_1 -> _1.script(_3 -> _3.source("blah")))
117117
);
118118
so = checkJsonRoundtrip(so, "{\"_script\":{\"script\":{\"source\":\"blah\"}}}");
119-
assertEquals("blah", so.script().script().inline().source());
119+
assertEquals("blah", so.script().script().source());
120120

121121
}
122122

java-client/src/test/java/co/elastic/clients/elasticsearch/model/UnionTests.java

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -46,28 +46,24 @@ public void testScriptDeserializer() {
4646
// A union discriminated by its field names (source -> inline, id -> stored)
4747
{
4848
Script s = Script.of(_1 -> _1
49-
.inline(_2 -> _2
5049
.source("a script")
51-
)
5250
);
5351
s = checkJsonRoundtrip(s, "{\"source\":\"a script\"}");
54-
assertEquals("a script", s.inline().source());
52+
assertEquals("a script", s.source());
5553
}
5654

5755
{
5856
Script s = Script.of(_1 -> _1
59-
.stored(_2 -> _2
6057
.id("script_id")
61-
)
6258
);
6359
s = checkJsonRoundtrip(s, "{\"id\":\"script_id\"}");
64-
assertEquals("script_id", s.stored().id());
60+
assertEquals("script_id", s.id());
6561
}
6662

6763
{
6864
// Test shortcut property
6965
Script s = fromJson("\"a script\"", Script.class);
70-
assertEquals("a script", s.inline().source());
66+
assertEquals("a script", s.source());
7167
}
7268
}
7369

java-client/src/test/java/co/elastic/clients/elasticsearch/spec_issues/SpecIssuesTest.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -98,9 +98,7 @@ public void i0298_runtimeMappings() throws Exception {
9898
RuntimeField runtimeField = RuntimeField.of(rf -> rf
9999
.type(RuntimeFieldType.Double)
100100
.script(Script.of(s -> s
101-
.inline(i -> i.
102-
source("emit(doc['price'].value * 1.19)")
103-
)
101+
.source("emit(doc['price'].value * 1.19)")
104102
))
105103
);
106104

java-client/src/test/java/co/elastic/clients/json/JsonpUtilsTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ public void testJsonString() {
179179
{
180180
IndicesPrivileges priv = IndicesPrivileges.of(i -> i
181181
.names("bar")
182-
.query(q -> q._custom("template", RoleTemplateScript.of(s -> s.stored(v -> v.id("foo")))))
182+
.query(q -> q._custom("template", RoleTemplateScript.of(s -> s.id("foo"))))
183183
.privileges(IndexPrivilege.All.jsonValue())
184184
);
185185

@@ -188,7 +188,7 @@ public void testJsonString() {
188188
assertEquals(json, toJson(priv));
189189

190190
priv = fromJson(json, IndicesPrivileges.class);
191-
assertEquals("foo", priv.query()._custom().to(RoleTemplateScript.class).stored().id());
191+
assertEquals("foo", priv.query()._custom().to(RoleTemplateScript.class).id());
192192
}
193193

194194
// Array value

java-client/src/test/java/co/elastic/clients/json/WithJsonTest.java

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919

2020
package co.elastic.clients.json;
2121

22-
import co.elastic.clients.elasticsearch._types.InlineScript;
22+
import co.elastic.clients.elasticsearch._types.Script;
2323
import co.elastic.clients.elasticsearch._types.SlicedScroll;
2424
import co.elastic.clients.elasticsearch._types.mapping.Property;
2525
import co.elastic.clients.elasticsearch._types.mapping.TextProperty;
@@ -28,6 +28,7 @@
2828
import co.elastic.clients.elasticsearch.core.IndexRequest;
2929
import co.elastic.clients.elasticsearch.core.SearchResponse;
3030
import co.elastic.clients.elasticsearch.indices.PutIndicesSettingsRequest;
31+
import co.elastic.clients.elasticsearch.security.RoleTemplateScript;
3132
import co.elastic.clients.testkit.ModelTestCase;
3233
import org.junit.jupiter.api.Test;
3334

@@ -123,7 +124,7 @@ public void testTypeWithParent() {
123124

124125
String json = "{\"source\": \"return doc;\"}";
125126

126-
InlineScript is = InlineScript.of(b -> b
127+
Script is = Script.of(b -> b
127128
.withJson(new StringReader(json))
128129
);
129130

@@ -170,5 +171,16 @@ public void testInternallyTaggedUnion() {
170171
TextProperty tp = p.text();
171172
assertEquals("lowercase", tp.fields().get("some_field").keyword().normalizer());
172173
}
174+
175+
@Test
176+
public void testExternalTaggedUnion() {
177+
178+
RoleTemplateScript withSource = RoleTemplateScript.of(j -> j.withJson(new StringReader("{\"source\": {\"match\": {\"category\": \"click\"}}}")));
179+
assertTrue(withSource.source().isQueryObject());
180+
RoleTemplateScript withStringSource = RoleTemplateScript.of(j -> j.withJson(new StringReader("{\"source\": \"string\"}")));
181+
assertTrue(withStringSource.source().isQueryString());
182+
RoleTemplateScript withStoredScript = RoleTemplateScript.of(j -> j.withJson(new StringReader("{\"id\": \"foo\"}")));
183+
assertTrue(!withStoredScript.id().isEmpty());
184+
}
173185
}
174186

0 commit comments

Comments
 (0)