Remove yaml skips older than 7.0 (#40183)

These are now obsolete in 8.0, and make adding new skips complicated. Also found some very old skips (5.4!)
elastic · Apr 2, 2019 · 6acdddc · 6acdddc
1 parent f5ff021
commit 6acdddc
Show file tree

Hide file tree

Showing 135 changed files with 117 additions and 564 deletions.
diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/bulk/10_basic.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/bulk/10_basic.yml
@@ -1,9 +1,7 @@
 ---
 "Array of objects":
 
-  - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
 
   - do:
       bulk:
@@ -29,9 +27,7 @@
 ---
 "Empty _id":
 
-  - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
 
   - do:
       bulk:
@@ -65,9 +61,7 @@
 "empty action":
 
   - skip:
-      version: " - 6.99.99"
       features: headers
-      reason: types are required in requests before 7.0.0
 
   - do:
       catch: /Malformed action\/metadata line \[3\], expected FIELD_NAME but found \[END_OBJECT\]/

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/bulk/20_list_of_strings.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/bulk/20_list_of_strings.yml
@@ -1,9 +1,7 @@
 ---
 "List of strings":
 
-  - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
   - do:
       bulk:
         refresh: true

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/bulk/30_big_string.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/bulk/30_big_string.yml
@@ -1,9 +1,7 @@
 ---
 "One big string":
 
-  - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
   - do:
       bulk:
         refresh: true

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/bulk/40_source.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/bulk/40_source.yml
@@ -1,9 +1,7 @@
 ---
 "Source filtering":
 
-  - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
   - do:
       index:
         refresh: true

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/bulk/50_refresh.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/bulk/50_refresh.yml
@@ -1,9 +1,7 @@
 ---
 "refresh=true immediately makes changes are visible in search":
 
-  - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
   - do:
       bulk:
         refresh: true
@@ -21,9 +19,7 @@
 ---
 "refresh=empty string immediately makes changes are visible in search":
 
-  - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
   - do:
       bulk:
         refresh: ""
@@ -42,9 +38,7 @@
 ---
 "refresh=wait_for waits until changes are visible in search":
 
-  - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
   - do:
       bulk:
         refresh: wait_for

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/bulk/70_mix_typeless_typeful.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/bulk/70_mix_typeless_typeful.yml
@@ -1,10 +1,6 @@
 ---
 "bulk without types on an index that has types":
 
- - skip:
-      version: " - 6.99.99"
-      reason: Typeless APIs were introduced in 7.0.0
-
  - do:
       indices.create: # not using include_type_name: false on purpose
           include_type_name: true

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/bulk/81_cas_with_types.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/bulk/81_cas_with_types.yml
@@ -1,10 +1,6 @@
 ---
 "Compare And Swap Sequence Numbers":
 
- - skip:
-      version: " - 6.6.99"
-      reason:  cas operations with sequence numbers was added in 6.7
-
  - do:
       index:
           index:  test_1

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/cat.thread_pool/10_basic.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/cat.thread_pool/10_basic.yml
@@ -1,8 +1,5 @@
 ---
 "Test cat thread_pool output":
-  - skip:
-        version: " - 6.99.99"
-        reason: this API was changed in a backwards-incompatible fashion in 7.0.0 so we need to skip in a mixed cluster
 
   - do:
       cat.thread_pool: {}

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/cluster.health/10_basic.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/cluster.health/10_basic.yml
@@ -94,9 +94,6 @@
 
 ---
 "cluster health basic test, one index with wait for no initializing shards":
-  - skip:
-      version: " - 6.1.99"
-      reason: "wait_for_no_initializing_shards is introduced in 6.2.0"
 
   - do:
       indices.create:

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/cluster.state/10_basic.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/cluster.state/10_basic.yml
@@ -7,9 +7,6 @@
 
 ---
 "get cluster state returns cluster_uuid at the top level":
-  - skip:
-      version:  " - 6.3.99"
-      reason:   "cluster state including cluster_uuid at the top level is new in v6.4.0 and higher"
 
   - do:
       cluster.state:

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/cluster.state/20_filtering.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/cluster.state/20_filtering.yml
@@ -155,9 +155,6 @@ setup:
 
 ---
 "Filtering the cluster state returns cluster_uuid at the top level regardless of metric filters":
-  - skip:
-      version:  " - 6.3.99"
-      reason:   "cluster state including cluster_uuid at the top level is new in v6.4.0 and higher"
 
   # Get the current cluster_uuid
   - do:

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/cluster.stats/10_basic.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/cluster.stats/10_basic.yml
@@ -32,9 +32,6 @@
 
 ---
 "get cluster stats returns cluster_uuid at the top level":
-  - skip:
-      version: " - 6.99.99"
-      reason:  "cluster stats including cluster_uuid at the top level is new in v6.5.0 and higher"
 
   - do:
       cluster.stats: {}
@@ -69,9 +66,6 @@
 
 ---
 "get cluster stats returns discovery types":
-  - skip:
-      version: " - 6.99.99"
-      reason:  "discovery types are added for v7.0.0"
 
   - do:
       cluster.stats: {}

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/create/10_with_id.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/create/10_with_id.yml
@@ -1,8 +1,6 @@
 ---
 "Create with ID":
- - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
  - do:
       create:
           index:  test_1

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/create/15_without_id.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/create/15_without_id.yml
@@ -1,8 +1,6 @@
 ---
 "Create without ID":
- - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
  - do:
       catch: param
       create:

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/create/35_external_version.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/create/35_external_version.yml
@@ -1,8 +1,6 @@
 ---
 "External version":
- - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
  - do:
       catch:              bad_request
       create:

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/create/40_routing.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/create/40_routing.yml
@@ -1,8 +1,6 @@
 ---
 "Routing":
- - skip:
-     version: " - 6.99.99"
-     reason: types are required in requests before 7.0.0
+
  - do:
      indices.create:
        index:    test_1

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/create/60_refresh.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/create/60_refresh.yml
@@ -1,8 +1,6 @@
 ---
 "Refresh":
- - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
  - do:
       indices.create:
           index:    test_1
@@ -44,9 +42,7 @@
 
 ---
 "When refresh url parameter is an empty string that means \"refresh immediately\"":
- - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
  - do:
       create:
           index:   test_1
@@ -66,9 +62,7 @@
 
 ---
 "refresh=wait_for waits until changes are visible in search":
- - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
  - do:
       index:
           index:   create_60_refresh_1

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/create/70_nested.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/create/70_nested.yml
@@ -1,8 +1,6 @@
 ---
 setup:
-  - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
   - do:
       indices.create:
           index: test_1
@@ -16,9 +14,7 @@ setup:
 
 ---
 "Indexing a doc with No. nested objects less or equal to index.mapping.nested_objects.limit should succeed":
-  - skip:
-      version: " - 6.99.99"
-      reason:  index.mapping.nested_objects setting has been added in 7.0.0
+
   - do:
       create:
           index:  test_1
@@ -29,9 +25,7 @@ setup:
 
 ---
 "Indexing a doc with No. nested objects more than index.mapping.nested_objects.limit should fail":
-  - skip:
-      version: " - 6.99.99"
-      reason:  index.mapping.nested_objects setting has been added in 7.0.0
+
   - do:
       catch: /The number of nested documents has exceeded the allowed limit of \[2\]. This limit can be set by changing the \[index.mapping.nested_objects.limit\] index level setting\./
       create:

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/create/71_nested_with_types.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/create/71_nested_with_types.yml
@@ -15,9 +15,7 @@ setup:
 
 ---
 "Indexing a doc with No. nested objects less or equal to index.mapping.nested_objects.limit should succeed":
-  - skip:
-      version: " - 6.99.99"
-      reason:  index.mapping.nested_objects setting has been added in 7.0.0
+
   - do:
       create:
           index:  test_1
@@ -29,9 +27,7 @@ setup:
 
 ---
 "Indexing a doc with No. nested objects more than index.mapping.nested_objects.limit should fail":
-  - skip:
-      version: " - 6.99.99"
-      reason:  index.mapping.nested_objects setting has been added in 7.0.0
+
   - do:
       catch: /The number of nested documents has exceeded the allowed limit of \[2\]. This limit can be set by changing the \[index.mapping.nested_objects.limit\] index level setting\./
       create:

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/delete/10_basic.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/delete/10_basic.yml
@@ -1,9 +1,7 @@
 ---
 "Basic":
 
- - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
 
  - do:
       index:

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/delete/11_shard_header.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/delete/11_shard_header.yml
@@ -1,9 +1,7 @@
 ---
 "Delete check shard header":
 
- - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
 
  - do:
       indices.create:

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/delete/12_result.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/delete/12_result.yml
@@ -1,9 +1,7 @@
 ---
 "Delete result field":
 
- - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
 
  - do:
       index:

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/delete/20_cas.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/delete/20_cas.yml
@@ -1,9 +1,7 @@
 ---
 "Internal version":
 
- - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
 
  - do:
       index:

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/delete/25_external_version.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/delete/25_external_version.yml
@@ -1,9 +1,7 @@
 ---
 "External version":
 
- - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
 
  - do:
       index:

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/delete/26_external_gte_version.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/delete/26_external_gte_version.yml
@@ -1,9 +1,7 @@
 ---
 "External GTE version":
 
- - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
 
  - do:
       index:

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/delete/30_routing.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/delete/30_routing.yml
@@ -1,9 +1,7 @@
 ---
 "Routing":
 
- - skip:
-      version: " - 6.99.99"
-      reason: types are required in requests before 7.0.0
+
 
  - do:
       indices.create: