Update 'buffer,usage' test in createBindGroup.spec.ts (#1824)

This PR updates the buffer,usage test to handle the case that the buffer's
type is `storage` or `read-only-storage`. In such cases, the usage should
be STORAGE.

Issue: #884

src/webgpu/api/validation/createBindGroup.spec.ts

@@ -750,14 +750,17 @@ g.test('storage_texture,mip_level_count')
 g.test('buffer,usage')
   .desc(
     `
-    Test that the buffer usage contains 'uniform' if the BindGroup entry defines
-    buffer and it's type is 'uniform'.
+    Test that the buffer usage contains 'UNIFORM' if the BindGroup entry defines buffer and it's
+    type is 'uniform', and the buffer usage contains 'STORAGE' if the BindGroup entry's buffer type
+    is 'storage'|read-only-storage'.
   `
   )
   .params(u =>
     u //
+      .combine('type', kBufferBindingTypes)
       // If usage0 and usage1 are the same, the usage being test is a single usage. Otherwise, it's
       // a combined usage.
+      .beginSubcases()
       .combine('usage0', kBufferUsages)
       .combine('usage1', kBufferUsages)
       .unless(
@@ -767,7 +770,7 @@ g.test('buffer,usage')
       )
   )
   .fn(async t => {
-    const { usage0, usage1 } = t.params;
+    const { type, usage0, usage1 } = t.params;
 
     const usage = usage0 | usage1;
 
@@ -776,7 +779,7 @@ g.test('buffer,usage')
         {
           binding: 0,
           visibility: GPUShaderStage.COMPUTE,
-          buffer: { type: 'uniform' },
+          buffer: { type },
         },
       ],
     });
@@ -786,7 +789,12 @@ g.test('buffer,usage')
       usage,
     });
 
-    const isValid = GPUBufferUsage.UNIFORM & usage;
+    let isValid = false;
+    if (type === 'uniform') {
+      isValid = GPUBufferUsage.UNIFORM & usage ? true : false;
+    } else if (type === 'storage' || type === 'read-only-storage') {
+      isValid = GPUBufferUsage.STORAGE & usage ? true : false;
+    }
 
     t.expectValidationError(() => {
       t.device.createBindGroup({