Add dummy targets to render pipelines (gpuweb#3807)

Also allow manual dispatch of push workflow, useful for deployment of WIP branches.

.github/workflows/push.yml

@@ -1,6 +1,7 @@
 name: Push CI
 
 on:
+  workflow_dispatch:
   push:
     branches: [main]
 

src/webgpu/api/operation/labels.spec.ts

@@ -106,6 +106,14 @@ const kTestFunctions: { [name: string]: TestFunction } = {
         module,
         entryPoint: 'foo',
       },
+      // Specify a color attachment so we have at least one render target. Otherwise, details here
+      // are not relevant to this test.
+      fragment: {
+        targets: [{ format: 'rgba8unorm' }],
+        module: t.device.createShaderModule({
+          code: `@fragment fn main() -> @location(0) vec4f { return vec4f(0); }`,
+        }),
+      },
     });
     t.expect(renderPipeline.label === label);
   },
@@ -143,6 +151,14 @@ const kTestFunctions: { [name: string]: TestFunction } = {
         module,
         entryPoint: 'foo',
       },
+      // Specify a color attachment so we have at least one render target. Otherwise, details here
+      // are not relevant to this test.
+      fragment: {
+        targets: [{ format: 'rgba8unorm' }],
+        module: t.device.createShaderModule({
+          code: `@fragment fn main() -> @location(0) vec4f { return vec4f(0); }`,
+        }),
+      },
     });
     t.expect(renderPipeline.label === label);
   },

src/webgpu/api/validation/capability_checks/limits/limit_utils.ts

@@ -718,12 +718,20 @@ export class LimitTestsImpl extends GPUTestBase {
   }
 
   _createRenderPipelineDescriptor(module: GPUShaderModule): GPURenderPipelineDescriptor {
+    const { device } = this;
     return {
       layout: 'auto',
       vertex: {
         module,
         entryPoint: 'mainVS',
       },
+      // Specify a color attachment so we have at least one render target.
+      fragment: {
+        targets: [{ format: 'rgba8unorm' }],
+        module: device.createShaderModule({
+          code: `@fragment fn main() -> @location(0) vec4f { return vec4f(0); }`,
+        }),
+      },
     };
   }
 

src/webgpu/api/validation/render_pipeline/misc.spec.ts

@@ -49,6 +49,10 @@ state (and thus has no color state), and can be created with or without depth st
         '',
       ] as const)
       .combine('hasColor', [false, true])
+      .unless(({ depthStencilFormat, hasColor }) => {
+        // Render pipeline needs at least one attachement
+        return hasColor === false && depthStencilFormat === '';
+      })
   )
   .fn(t => {
     const { isAsync, depthStencilFormat, hasColor } = t.params;

src/webgpu/api/validation/render_pipeline/shader_module.spec.ts

@@ -83,6 +83,13 @@ g.test('invalid,vertex')
           : t.createInvalidShaderModule(),
         entryPoint: 'main',
       },
+      // Specify a color attachment so we have at least one render target.
+      fragment: {
+        targets: [{ format: 'rgba8unorm' }],
+        module: t.device.createShaderModule({
+          code: `@fragment fn main() -> @location(0) vec4f { return vec4f(0); }`,
+        }),
+      },
     });
   });