fixup! fix(@schematics/angular): avoid adding web worker snippet to server configuration files

Author: MeAkib

packages/schematics/angular/web-worker/index.ts

@@ -41,8 +41,8 @@ function addSnippet(options: WebWorkerOptions): Rule {
     const siblingModules = host
       .getDir(options.path)
       .subfiles // Find all files that start with the same name, are ts files,
-      // and aren't spec, module, or configuration files (including their server variants).
-      .filter((f) => fileRegExp.test(f) && !/\.(module|spec|config|routes|server)\.ts$/.test(f))
+      // and aren't spec, module, declaration, or configuration files (including their server variants).
+      .filter((f) => fileRegExp.test(f) && !/\.(module|spec|config|routes|server|d)\.ts$/.test(f))
       // Sort alphabetically for consistency.
       .sort();
 

packages/schematics/angular/web-worker/index_spec.ts

@@ -96,8 +96,8 @@ describe('Web Worker Schematic', () => {
   });
 
   it('should add snippet to the component file and not a server configuration file', async () => {
-    appTree = await schematicRunner.runSchematic('server', { project: 'bar' }, appTree);
-    const tree = await schematicRunner.runSchematic('web-worker', defaultOptions, appTree);
+    const serverAppTree = await schematicRunner.runSchematic('server', { project: 'bar' }, appTree);
+    const tree = await schematicRunner.runSchematic('web-worker', defaultOptions, serverAppTree);
 
     const appComponent = tree.readContent('/projects/bar/src/app/app.ts');
     expect(appComponent).toContain(`new Worker(new URL('./${defaultOptions.name}.worker`);
@@ -115,6 +115,16 @@ describe('Web Worker Schematic', () => {
     expect(content).toContain(`new Worker(new URL('./observer.worker`);
   });
 
+  it('should not add snippet to a declaration file', async () => {
+    appTree.create('/projects/bar/src/app/app.d.ts', 'declare const foo: string;\n');
+    const tree = await schematicRunner.runSchematic('web-worker', defaultOptions, appTree);
+
+    expect(tree.readContent('/projects/bar/src/app/app.d.ts')).not.toContain('new Worker');
+    expect(tree.readContent('/projects/bar/src/app/app.ts')).toContain(
+      `new Worker(new URL('./${defaultOptions.name}.worker`,
+    );
+  });
+
   it(`should add 'tsconfig.worker.json' outside of 'src' directory in root app`, async () => {
     const rootAppOptions = { ...appOptions, projectRoot: '', name: 'foo' };
     const workerOptions = { ...defaultOptions, project: 'foo' };