Update-by-query rest tests: fixed bad yaml and deleted a client-dependent test

modules/reindex/src/test/resources/rest-api-spec/test/update_by_query/10_basic.yaml

@@ -20,7 +20,8 @@
   - match: {noops: 0}
   - match: {throttled_millis: 0}
   - gte: { took: 0 }
-  - is_false: created # Update by query can't create
+  # Update by query can't create
+  - is_false: created
   - is_false: task
 
 ---
@@ -70,7 +71,8 @@
         body:    { "text": "test" }
   - do:
       indices.refresh: {}
-  - do: # Creates a new version for reindex to miss on scan.
+  # Creates a new version for reindex to miss on scan.
+  - do:
       index:
         index:   test
         type:    foo
@@ -111,7 +113,8 @@
         body:    { "text": "test" }
   - do:
       indices.refresh: {}
-  - do: # Creates a new version for reindex to miss on scan.
+  # Creates a new version for reindex to miss on scan.
+  - do:
       index:
         index:   test
         type:    foo

modules/reindex/src/test/resources/rest-api-spec/test/update_by_query/20_validation.yaml

@@ -12,20 +12,6 @@
         index: test
         conflicts: cat
 
----
-"invalid scroll_size fails":
-  - do:
-      index:
-        index:   test
-        type:    test
-        id:      1
-        body:    { "text": "test" }
-  - do:
-      catch: /Failed to parse int parameter \[scroll_size\] with value \[cat\]/
-      update_by_query:
-        index: test
-        scroll_size: cat
-
 ---
 "invalid size fails":
   - do: