Test pass but not encoder ended in encoder_open_state

Author: andyleiserson

src/webgpu/api/validation/encoding/encoder_open_state.spec.ts

@@ -274,15 +274,13 @@ g.test('render_pass_commands')
     `
     Test that functions of GPURenderPassEncoder generate a validation error if the encoder or the
     pass is already finished.
-
-    - TODO: Consider testing: nothing before command, end before command, end+finish before command.
   `
   )
   .params(u =>
     u
       .combine('command', kRenderPassEncoderCommands)
       .beginSubcases()
-      .combine('finishBeforeCommand', [false, true])
+      .combine('finishBeforeCommand', ['no', 'pass', 'encoder'])
   )
   .fn(t => {
     const { command, finishBeforeCommand } = t.params;
@@ -305,8 +303,10 @@ g.test('render_pass_commands')
 
     const bindGroup = t.createBindGroupForTest();
 
-    if (finishBeforeCommand) {
+    if (finishBeforeCommand !== 'no') {
       renderPass.end();
+    }
+    if (finishBeforeCommand === 'encoder') {
       encoder.finish();
     }
 
@@ -420,7 +420,9 @@ g.test('render_pass_commands')
         default:
           unreachable();
       }
-    }, finishBeforeCommand);
+    }, finishBeforeCommand === 'encoder');
+    // TODO(https://github.com/gpuweb/gpuweb/issues/5207): resolve whether this
+    // is correct or should be changed to `finishBeforeCommand !== 'no'`.
   });
 
 g.test('render_bundle_commands')
@@ -524,15 +526,13 @@ g.test('compute_pass_commands')
     `
     Test that functions of GPUComputePassEncoder generate a validation error if the encoder or the
     pass is already finished.
-
-    - TODO: Consider testing: nothing before command, end before command, end+finish before command.
   `
   )
   .params(u =>
     u
       .combine('command', kComputePassEncoderCommands)
       .beginSubcases()
-      .combine('finishBeforeCommand', [false, true])
+      .combine('finishBeforeCommand', ['no', 'pass', 'encoder'])
   )
   .fn(t => {
     const { command, finishBeforeCommand } = t.params;
@@ -549,8 +549,10 @@ g.test('compute_pass_commands')
 
     const bindGroup = t.createBindGroupForTest();
 
-    if (finishBeforeCommand) {
+    if (finishBeforeCommand !== 'no') {
       computePass.end();
+    }
+    if (finishBeforeCommand === 'encoder') {
       encoder.finish();
     }
 
@@ -594,5 +596,7 @@ g.test('compute_pass_commands')
         default:
           unreachable();
       }
-    }, finishBeforeCommand);
+    }, finishBeforeCommand === 'encoder');
+    // TODO(https://github.com/gpuweb/gpuweb/issues/5207): resolve whether this
+    // is correct or should be changed to `finishBeforeCommand !== 'no'`.
   });

src/webgpu/api/validation/encoding/encoder_state.spec.ts

@@ -275,7 +275,7 @@ g.test('pass_begin_invalid_encoder')
     const querySet = t.trackForCleanup(
       t.device.createQuerySet({
         type: 'timestamp',
-        count: 1,
+        count: 2,
       })
     );
 
@@ -291,7 +291,7 @@ g.test('pass_begin_invalid_encoder')
     const mismatchedQuerySet = t.trackForCleanup(
       t.mismatchedDevice.createQuerySet({
         type: 'timestamp',
-        count: 1,
+        count: 2,
       })
     );