Update RuntimeField to use IInlineScript

src/Nest/CommonOptions/Scripting/InlineScript.cs

@@ -8,6 +8,7 @@
 namespace Nest
 {
 	[InterfaceDataContract]
+	[ReadAs(typeof(InlineScript))]
 	public interface IInlineScript : IScript
 	{
 		[DataMember(Name ="source")]

src/Nest/Mapping/RuntimeFields/RuntimeField.cs

@@ -2,6 +2,7 @@
 // Elasticsearch B.V licenses this file to you under the Apache 2.0 License.
 // See the LICENSE file in the project root for more information
 
+using System;
 using System.Runtime.Serialization;
 using Elasticsearch.Net.Utf8Json;
 
@@ -22,7 +23,7 @@ public interface IRuntimeField
 		/// The script to be evaluated for field calculation at search time.
 		/// </summary>
 		[DataMember(Name = "script")]
-		IStoredScript Script { get; set; }
+		IInlineScript Script { get; set; }
 
 		/// <summary>
 		/// The datatype of the runtime field.
@@ -36,7 +37,7 @@ public class RuntimeField : IRuntimeField
 		/// <inheritdoc />
 		public string Format { get; set; }
 		/// <inheritdoc />
-		public IStoredScript Script { get; set; }
+		public IInlineScript Script { get; set; }
 		/// <inheritdoc />
 		public FieldType Type { get; set; }
 	}
@@ -47,13 +48,19 @@ public class RuntimeFieldDescriptor
 		public RuntimeFieldDescriptor(FieldType fieldType) => Self.Type = fieldType;
 
 		string IRuntimeField.Format { get; set; }
-		IStoredScript IRuntimeField.Script { get; set; }
+		IInlineScript IRuntimeField.Script { get; set; }
 		FieldType IRuntimeField.Type { get; set; }
 
+		/// <inheritdoc cref="IRuntimeField.Format" />
 		public RuntimeFieldDescriptor Format(string format) => Assign(format, (a, v) => a.Format = v);
-
-		public RuntimeFieldDescriptor Script(IStoredScript script) => Assign(script, (a, v) => a.Script = v);
 
-		public RuntimeFieldDescriptor Script(string source) => Assign(source, (a, v) => a.Script = new PainlessScript(source));
+		/// <inheritdoc cref="IRuntimeField.Script" />
+		public RuntimeFieldDescriptor Script(IInlineScript script) => Assign(script, (a, v) => a.Script = v);
+
+		/// <inheritdoc cref="IRuntimeField.Script" />
+		public RuntimeFieldDescriptor Script(string source) => Assign(source, (a, v) => a.Script = new InlineScript(source));
+
+		/// <inheritdoc cref="IRuntimeField.Script" />
+		public RuntimeFieldDescriptor Script(Func<InlineScriptDescriptor, IInlineScript> selector) => Assign(selector, (a, v) => a.Script = v?.Invoke(new InlineScriptDescriptor()));
 	}
 }

tests/Tests/ClientConcepts/HighLevel/Mapping/FluentMapping.doc.cs

@@ -463,7 +463,6 @@ public void MappingRuntimeFields()
 							type = "keyword",
 							script = new
 							{
-								lang = "painless",
 								source = "emit(doc['@timestamp'].value.dayOfWeekEnum.getDisplayName(TextStyle.FULL, Locale.ROOT))"
 							}
 						}
@@ -487,6 +486,46 @@ public void MappingRuntimeFields()
 
 			//hide
 			Expect(expected).FromRequest(createIndexResponse);
+
+			/**
+			 * One may also include and use parameters in the script.
+			 */
+			createIndexResponse = _client.Indices.Create("myindex", c => c
+				.Map<Company>(m => m
+					.RuntimeFields(rtf => rtf
+						.RuntimeField("birthDayOfWeek", FieldType.Keyword, f => f
+							.Script(s => s
+								.Source("emit(doc['@timestamp'].value.dayOfWeekEnum.getDisplayName(TextStyle.FULL, Locale.ROOT) + params.suffix)")
+								.Params(p => p.Add("suffix", " with a suffix."))
+								)))
+				)
+			);
+
+			//json
+			var finalExpected = new
+			{
+				mappings = new
+				{
+					runtime = new
+					{
+						birthDayOfWeek = new
+						{
+							type = "keyword",
+							script = new
+							{
+								source = "emit(doc['@timestamp'].value.dayOfWeekEnum.getDisplayName(TextStyle.FULL, Locale.ROOT) + params.suffix)",
+								@params = new Dictionary<string, string>
+								{
+									{"suffix", " with a suffix." }
+								}
+							}
+						}
+					}
+				}
+			};
+
+			//hide
+			Expect(finalExpected).FromRequest(createIndexResponse);
 		}
 	}
 }

tests/Tests/Indices/MappingManagement/PutMapping/PutMappingApiTest.cs

@@ -3,6 +3,7 @@
 // See the LICENSE file in the project root for more information
 
 using System;
+using System.Collections.Generic;
 using Elastic.Elasticsearch.Xunit.XunitPlumbing;
 using Elasticsearch.Net;
 using Nest;
@@ -344,7 +345,7 @@ public class PutMappingWithRuntimeFieldsTests : ApiTestBase<ReadOnlyCluster, Put
 	{
 		// These test serialisation only. Integration tests take place in RuntimeFieldsTests.cs
 
-		private const string ScriptValue = "emit(doc['@timestamp'].value.dayOfWeekEnum.getDisplayName(TextStyle.FULL, Locale.ROOT))";
+		private const string ScriptValue = "emit(doc['@timestamp'].value.dayOfWeekEnum.getDisplayName(TextStyle.FULL, Locale.ROOT) + params.foo";
 
 		public PutMappingWithRuntimeFieldsTests(ReadOnlyCluster cluster, EndpointUsage usage) : base(cluster, usage) { }
 
@@ -357,15 +358,18 @@ public PutMappingWithRuntimeFieldsTests(ReadOnlyCluster cluster, EndpointUsage u
 			RuntimeFields = new RuntimeFields
 			{
 				{ "runtime_date", new RuntimeField { Type = FieldType.Date, Format = "yyyy-MM-dd" } },
-				{ "runtime_scripted", new RuntimeField { Type = FieldType.Keyword, Script = new PainlessScript(ScriptValue) } }
+				{ "runtime_scripted", new RuntimeField { Type = FieldType.Keyword, Script = new InlineScript(ScriptValue) { Lang = "painless", Params = new Dictionary<string, object>
+				{
+					{ "foo", " is a good day." }
+				}}}}
 			}
 		};
 
 		protected override Func<PutMappingDescriptor<Project>, IPutMappingRequest> Fluent => d => d
 			.Index(CallIsolatedValue)
 			.RuntimeFields(rtf => rtf
 				.RuntimeField("runtime_date", FieldType.Date, rf => rf.Format("yyyy-MM-dd"))
-				.RuntimeField("runtime_scripted", FieldType.Keyword, rf=> rf.Script(new PainlessScript(ScriptValue))));
+				.RuntimeField("runtime_scripted", FieldType.Keyword, rf=> rf.Script(s => s.Source(ScriptValue).Lang(ScriptLang.Painless).Params(p => p.Add("foo", " is a good day.")))));
 
 		protected override LazyResponses ClientUsage() => Calls(
 			(client, f) => client.Indices.PutMapping(f),
@@ -388,8 +392,12 @@ protected override LazyResponses ClientUsage() => Calls(
 					type = "keyword",
 					script = new
 					{
+						source = ScriptValue,
 						lang = "painless",
-						source = ScriptValue
+						@params = new Dictionary<string, string>
+						{
+							{ "foo", " is a good day." }
+						}
 					}
 				}
 			}

tests/Tests/Mapping/RuntimeFields/RuntimeFieldsTests.cs

@@ -44,7 +44,7 @@ public RuntimeFieldsTests(WritableCluster cluster, EndpointUsage usage) : base(n
 									{RuntimeFieldNameOne, new RuntimeField
 									{
 										Type = FieldType.Keyword,
-										Script = new PainlessScript(ScriptValue)
+										Script = new InlineScript(ScriptValue)
 									}},
 									{RuntimeFieldNameTwo, new RuntimeField
 									{
@@ -108,7 +108,7 @@ public RuntimeFieldsTests(WritableCluster cluster, EndpointUsage usage) : base(n
 								{RuntimeFieldNameOne, new RuntimeField
 								{
 									Type = FieldType.Keyword,
-									Script = new PainlessScript(ScriptValue)
+									Script = new InlineScript(ScriptValue)
 								}},
 								{RuntimeFieldNameTwo, new RuntimeField
 								{

tests/Tests/Search/Search/SearchApiTests.cs

@@ -681,7 +681,6 @@ public SearchApiRuntimeFieldsTests(ReadOnlyCluster cluster, EndpointUsage usage)
 				{
 					script = new
 					{
-						lang = "painless",
 						source = RuntimeFieldScript
 					},
 					type = "keyword"
@@ -710,10 +709,11 @@ public SearchApiRuntimeFieldsTests(ReadOnlyCluster cluster, EndpointUsage usage)
 				.And(RuntimeFieldName),
 			RuntimeFields = new RuntimeFields
 			{
-				{ RuntimeFieldName, new RuntimeField
+				{
+					RuntimeFieldName, new RuntimeField
 					{
 						Type = FieldType.Keyword,
-						Script = new PainlessScript(RuntimeFieldScript)
+						Script = new InlineScript(RuntimeFieldScript)
 					}
 				}
 			}

tests/Tests/Search/SearchingRuntimeFields.doc.cs

@@ -129,7 +129,6 @@ public void SearchQueryRuntimeFields()
 					{
 						script = new
 						{
-							lang = "painless",
 							source = "if (doc['type'].size() != 0) {emit(doc['type'].value.toUpperCase())}"
 						},
 						type = "keyword"
@@ -155,7 +154,7 @@ public void SearchQueryRuntimeFields()
 					{ "search_runtime_field", new RuntimeField
 						{
 							Type = FieldType.Keyword,
-							Script = new PainlessScript("if (doc['type'].size() != 0) {emit(doc['type'].value.toUpperCase())}")
+							Script = new InlineScript("if (doc['type'].size() != 0) {emit(doc['type'].value.toUpperCase())}")
 						}
 					}
 				}

tests/Tests/XPack/Transform/TransformApiWithSettingsTests.cs

@@ -347,7 +347,6 @@ protected override LazyResponses ClientUsage() => Calls(
 						{
 							script = new
 							{
-								lang = "painless",
 								source = RuntimeFieldScript
 							},
 							type = "keyword"
@@ -410,7 +409,7 @@ protected override LazyResponses ClientUsage() => Calls(
 						{ RuntimeFieldName, new RuntimeField
 							{
 								Type = FieldType.Keyword,
-								Script = new PainlessScript(RuntimeFieldScript)
+								Script = new InlineScript(RuntimeFieldScript)
 							}
 						}
 					}