Skip SQL integration tests for now

russcam · russcam · commit a3d0faf5e952 · 2019-07-15T13:59:59.000+10:00
Relates: elastic/elasticsearch#44320 This commit skips the SQL integration tests for the moment. (cherry picked from commit 2490f71)
diff --git a/src/Tests/Tests/ClientConcepts/HighLevel/Serialization/ExtendingNestTypes.doc.cs b/src/Tests/Tests/ClientConcepts/HighLevel/Serialization/ExtendingNestTypes.doc.cs
@@ -48,9 +48,10 @@ public MyPluginProperty(string name, string language)
 			[PropertyName("numeric")]
 			public bool Numeric { get; set; }
 		}
-
+		
 
 		[U (Skip = "TODO: Does not work with utf8json")]
+		// hide
 		public void InjectACustomIPropertyImplementation()
 		{
 			/**
diff --git a/src/Tests/Tests/XPack/Sql/ClearSqlCursor/ClearSqlCursorApiTests.cs b/src/Tests/Tests/XPack/Sql/ClearSqlCursor/ClearSqlCursorApiTests.cs
@@ -11,7 +11,9 @@
 
 namespace Tests.XPack.Sql.ClearSqlCursor
 {
-	[SkipVersion("<6.4.0", "")]
+	//[SkipVersion("<6.4.0", "")]
+	// TODO: unskip when https://github.com/elastic/elasticsearch/issues/44320 is fixed
+	[SkipVersion(">1.0.0", "open issue https://github.com/elastic/elasticsearch/issues/44320")]
 	public class ClearSqlCursorApiTests
 		: ApiIntegrationTestBase<XPackCluster, ClearSqlCursorResponse, IClearSqlCursorRequest, ClearSqlCursorDescriptor, ClearSqlCursorRequest>
 	{
diff --git a/src/Tests/Tests/XPack/Sql/QuerySql/QuerySqlApiTests.cs b/src/Tests/Tests/XPack/Sql/QuerySql/QuerySqlApiTests.cs
@@ -11,7 +11,9 @@
 
 namespace Tests.XPack.Sql.QuerySql
 {
-	[SkipVersion("<6.4.0", "")]
+	//[SkipVersion("<6.4.0", "")]
+	// TODO: unskip when https://github.com/elastic/elasticsearch/issues/44320 is fixed
+	[SkipVersion(">1.0.0", "open issue https://github.com/elastic/elasticsearch/issues/44320")]
 	public class QuerySqlApiTests : ApiIntegrationTestBase<XPackCluster, QuerySqlResponse, IQuerySqlRequest, QuerySqlDescriptor, QuerySqlRequest>
 	{
 		private static readonly string SqlQuery =
diff --git a/src/Tests/Tests/XPack/Sql/TranslateSql/TranslateSqlQueryApiTests.cs b/src/Tests/Tests/XPack/Sql/TranslateSql/TranslateSqlQueryApiTests.cs
@@ -11,7 +11,9 @@
 
 namespace Tests.XPack.Sql.TranslateSql
 {
-	[SkipVersion("<6.4.0", "")]
+	//[SkipVersion("<6.4.0", "")]
+	// TODO: unskip when https://github.com/elastic/elasticsearch/issues/44320 is fixed
+	[SkipVersion(">1.0.0", "open issue https://github.com/elastic/elasticsearch/issues/44320")]
 	public class TranslateSqlApiTests
 		: ApiIntegrationTestBase<XPackCluster, TranslateSqlResponse, ITranslateSqlRequest, TranslateSqlDescriptor, TranslateSqlRequest>
 	{

Original file line number	Diff line number	Diff line change
`@@ -48,9 +48,10 @@ public MyPluginProperty(string name, string language)`
`48`	`48`	`[PropertyName("numeric")]`
`49`	`49`	`public bool Numeric { get; set; }`
`50`	`50`	`}`
`51`		`-`
	`51`	`+`
`52`	`52`
`53`	`53`	`[U (Skip = "TODO: Does not work with utf8json")]`
	`54`	`+ // hide`
`54`	`55`	`public void InjectACustomIPropertyImplementation()`
`55`	`56`	`{`
`56`	`57`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,9 @@`
`11`	`11`
`12`	`12`	`namespace Tests.XPack.Sql.ClearSqlCursor`
`13`	`13`	`{`
`14`		`- [SkipVersion("<6.4.0", "")]`
	`14`	`+ //[SkipVersion("<6.4.0", "")]`
	`15`	`+ // TODO: unskip when https://github.com/elastic/elasticsearch/issues/44320 is fixed`
	`16`	`+ [SkipVersion(">1.0.0", "open issue https://github.com/elastic/elasticsearch/issues/44320")]`
`15`	`17`	`public class ClearSqlCursorApiTests`
`16`	`18`	`: ApiIntegrationTestBase<XPackCluster, ClearSqlCursorResponse, IClearSqlCursorRequest, ClearSqlCursorDescriptor, ClearSqlCursorRequest>`
`17`	`19`	`{`