Remove inheritance for code-generated types (#6611) (#6612)

* Remove inheritence for code gen of classes

* Seal request types

* Update serialization after inheritance removal

* Prevent parallel tests which seem to hang

* Fix search API test assertion

* Fix multi search test

* Rename and move some files

* Queries derive from Query type

* Update tests

* Update test file

Co-authored-by: Steve Gordon <sgordon@hotmail.co.uk>

Author: github-actions[bot]

src/Elastic.Clients.Elasticsearch/Api/BulkRequest.cs

@@ -13,7 +13,7 @@ namespace Elastic.Clients.Elasticsearch
 {
 	public partial class BulkRequest : IStreamSerializable
 	{
-		protected IRequest Self => this;
+		internal IRequest Self => this;
 
 		public BulkOperationsCollection Operations { get; set; }
 

src/Elastic.Clients.Elasticsearch/Api/CountRequest.cs

@@ -7,7 +7,7 @@
 
 namespace Elastic.Clients.Elasticsearch
 {
-	public partial class CountRequest<TDocument> : CountRequest
+	public sealed partial class CountRequest<TDocument> : CountRequest
 	{
 		//protected CountRequest<TDocument> TypedSelf => this;
 

src/Elastic.Clients.Elasticsearch/Common/Aggregations/Aggregate.cs

@@ -0,0 +1,16 @@
+// Licensed to Elasticsearch B.V under one or more agreements.
+// 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.
+
+namespace Elastic.Clients.Elasticsearch.Aggregations;
+
+public interface IAggregate { }
+
+/// <summary>
+/// Base class for all aggregates.
+/// </summary>
+public abstract class Aggregate : IAggregate
+{
+}
+
+

src/Elastic.Clients.Elasticsearch/Common/Aggregations/Aggregation.cs

@@ -0,0 +1,27 @@
+// Licensed to Elasticsearch B.V under one or more agreements.
+// 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.
+
+namespace Elastic.Clients.Elasticsearch.Aggregations;
+
+public interface IAggregation
+{
+	string? Name { get; }
+}
+
+/// <summary>
+/// Base class for all aggregations.
+/// </summary>
+public abstract class Aggregation : IAggregation
+{
+	public abstract string? Name { get; internal set; }
+
+	//always evaluate to false so that each side of && equation is evaluated
+	public static bool operator false(Aggregation _) => false;
+
+	//always evaluate to false so that each side of && equation is evaluated
+	public static bool operator true(Aggregation _) => false;
+
+	public static Aggregation operator &(Aggregation left, Aggregation right) =>
+		new AggregationCombinator(null, left, right);
+}

src/Elastic.Clients.Elasticsearch/Common/Aggregations/AggregationBase.cs

@@ -10,19 +10,20 @@ namespace Elastic.Clients.Elasticsearch.Aggregations;
 /// <summary>
 /// Combines aggregations into a single list of aggregations.
 /// </summary>
-internal class AggregationCombinator : AggregationBase
+internal class AggregationCombinator : Aggregation
 {
-	public AggregationCombinator(string name, AggregationBase left, AggregationBase right) : base(name)
+	public AggregationCombinator(string name, Aggregation left, Aggregation right)
 	{
 		AddAggregation(left);
 		AddAggregation(right);
+		Name = name;
 	}
 
-	internal List<AggregationBase> Aggregations { get; } = new List<AggregationBase>();
+	public override string? Name { get; internal set; }
 
-	//internal override void WrapInContainer(AggregationContainer container) { }
+	internal List<Aggregation> Aggregations { get; } = new List<Aggregation>();
 
-	private void AddAggregation(AggregationBase agg)
+	private void AddAggregation(Aggregation agg)
 	{
 		switch (agg)
 		{

src/Elastic.Clients.Elasticsearch/Common/Aggregations/AggregationCombinator.cs

@@ -9,7 +9,7 @@
 namespace Elastic.Clients.Elasticsearch.Aggregations;
 
 /// <summary>
-/// Describes aggregations that we would like to execute on Elasticsearch.
+/// Describes aggregations to execute as part of a search.
 /// </summary>
 public sealed class AggregationDictionary : IsADictionaryBase<string, AggregationContainer>
 {
@@ -24,9 +24,9 @@ public AggregationDictionary(Dictionary<string, AggregationContainer> dictionary
 	public static implicit operator AggregationDictionary(Dictionary<string, AggregationContainer> dictionary) =>
 		new(dictionary);
 
-	public static implicit operator AggregationDictionary(AggregationBase aggregator)
+	public static implicit operator AggregationDictionary(Aggregation aggregator)
 	{
-		AggregationBase b;
+		Aggregation b;
 		if (aggregator is AggregationCombinator combinator)
 		{
 			var dict = new AggregationDictionary();

src/Elastic.Clients.Elasticsearch/Common/Aggregations/AggregationDictionary.cs

@@ -7,18 +7,6 @@
 
 namespace Elastic.Clients.Elasticsearch.QueryDsl
 {
-	// TODO: FieldNameQueryConvertor (see FieldNameQueryFormatter)
-	public interface IFieldNameQuery
-	{
-		Field Field { get; set; }
-	}
-
-	public abstract class FieldNameQueryBase : QueryBase, IFieldNameQuery
-	{
-		[JsonIgnore]
-		public Field Field { get; set; }
-	}
-
 	public interface IQuery
 	{
 		/// <summary>
@@ -53,19 +41,19 @@ public interface IQuery
 		/// <summary>
 		///     Whether the query should be treated as writable. Used when determining how to combine queries.
 		/// </summary>
-		[JsonIgnore]
-		bool IsWritable { get; }
+		//[JsonIgnore]
+		//bool IsWritable { get; }
 
 		/// <summary>
 		///     The name of the query. Allows you to retrieve for each document what part of the query it matched on.
 		/// </summary>
 		//string Name { get; set; }
 	}
 
-	public abstract partial class QueryBase : IQuery
+	public abstract partial class Query : IQuery
 	{
-		[JsonIgnore]
-		public bool IsWritable => throw new NotImplementedException();
+		//[JsonIgnore]
+		//public bool IsWritable => throw new NotImplementedException();
 
 		////protected abstract bool Conditionless { get; }
 		//[JsonIgnore]
@@ -80,10 +68,10 @@ public abstract partial class QueryBase : IQuery
 		//bool IQuery.Conditionless => Conditionless;
 
 		//always evaluate to false so that each side of && equation is evaluated
-		public static bool operator false(QueryBase a) => false;
+		public static bool operator false(Query a) => false;
 
 		//always evaluate to false so that each side of && equation is evaluated
-		public static bool operator true(QueryBase a) => false;
+		public static bool operator true(Query a) => false;
 
 		//public static QueryBase operator &(QueryBase leftQuery, QueryBase rightQuery) => Combine(leftQuery, rightQuery, (l, r) => l && r);
 

src/Elastic.Clients.Elasticsearch/Common/Query/QueryBase.cs renamed to src/Elastic.Clients.Elasticsearch/Common/Query/Query.cs

@@ -10,7 +10,7 @@ namespace Elastic.Clients.Elasticsearch
 
 	internal static class AggregationContainerSerializationHelper
 	{
-		public static AggregationContainer ReadContainer<T>(ref Utf8JsonReader reader, JsonSerializerOptions options) where T : AggregationBase
+		public static AggregationContainer ReadContainer<T>(ref Utf8JsonReader reader, JsonSerializerOptions options) where T : Aggregation
 		{
 			var variant = JsonSerializer.Deserialize<T?>(ref reader, options);
 
@@ -19,7 +19,7 @@ public static AggregationContainer ReadContainer<T>(ref Utf8JsonReader reader, J
 			return container;
 		}
 
-		public static AggregationContainer ReadContainer<T>(string variantName, ref Utf8JsonReader reader, JsonSerializerOptions options) where T : AggregationBase
+		public static AggregationContainer ReadContainer<T>(string variantName, ref Utf8JsonReader reader, JsonSerializerOptions options) where T : Aggregation
 		{
 			var variant = JsonSerializer.Deserialize<T?>(ref reader, options);