fix(@schematics/angular): generate services without a .service extension/type

To align with the updated style guide, Angular v20 will generate services
without a `.service` file extension type for all service related
files by default. Projects will automatically use this naming convention.
Projects can however opt-out by setting the `type` option to `Service`
for the service schematic. This can be done as a default in the `angular.json`
or directly on the commandline via `--type=Service` when executing `ng generate`.
As an example, `example.service.ts` will now be named `example.ts`. Additionally,
the TypeScript class name will be `Example` instead of the previous `ExampleService`.

Author: clydin

packages/schematics/angular/service/files/__name@dasherize@if-flat__/__name@dasherize__.__type@dasherize__.spec.ts.template

@@ -0,0 +1,16 @@
+import { TestBed } from '@angular/core/testing';
+
+import { <%= classify(name) %><%= classify(type) %> } from './<%= dasherize(name) %><%= type ? '.' + dasherize(type) : '' %>';
+
+describe('<%= classify(name) %><%= classify(type) %>', () => {
+  let service: <%= classify(name) %><%= classify(type) %>;
+
+  beforeEach(() => {
+    TestBed.configureTestingModule({});
+    service = TestBed.inject(<%= classify(name) %><%= classify(type) %>);
+  });
+
+  it('should be created', () => {
+    expect(service).toBeTruthy();
+  });
+});

packages/schematics/angular/service/files/__name@dasherize@if-flat__/__name@dasherize__.service.ts.template packages/schematics/angular/service/files/__name@dasherize@if-flat__/__name@dasherize__.__type@dasherize__.ts.template

@@ -3,7 +3,7 @@ import { Injectable } from '@angular/core';
 @Injectable({
   providedIn: 'root'
 })
-export class <%= classify(name) %>Service {
+export class <%= classify(name) %><%= classify(type) %> {
 
   constructor() { }
 }

packages/schematics/angular/service/files/__name@dasherize@if-flat__/__name@dasherize__.service.spec.ts.template

@@ -15,6 +15,9 @@ export default function (options: ServiceOptions): Rule {
   const flat = options.flat;
   options.flat = true;
 
+  // Schematic templates require a defined type value
+  options.type ??= '';
+
   return generateFromFiles(options, {
     'if-flat': (s: string) => (flat ? '' : s),
   });

packages/schematics/angular/service/index.ts

@@ -46,15 +46,15 @@ describe('Service Schematic', () => {
 
     const tree = await schematicRunner.runSchematic('service', options, appTree);
     const files = tree.files;
-    expect(files).toContain('/projects/bar/src/app/foo/foo.service.spec.ts');
-    expect(files).toContain('/projects/bar/src/app/foo/foo.service.ts');
+    expect(files).toContain('/projects/bar/src/app/foo/foo.spec.ts');
+    expect(files).toContain('/projects/bar/src/app/foo/foo.ts');
   });
 
   it('service should be tree-shakeable', async () => {
     const options = { ...defaultOptions };
 
     const tree = await schematicRunner.runSchematic('service', options, appTree);
-    const content = tree.readContent('/projects/bar/src/app/foo/foo.service.ts');
+    const content = tree.readContent('/projects/bar/src/app/foo/foo.ts');
     expect(content).toMatch(/providedIn: 'root'/);
   });
 
@@ -63,15 +63,33 @@ describe('Service Schematic', () => {
 
     const tree = await schematicRunner.runSchematic('service', options, appTree);
     const files = tree.files;
-    expect(files).toContain('/projects/bar/src/app/foo/foo.service.ts');
-    expect(files).not.toContain('/projects/bar/src/app/foo/foo.service.spec.ts');
+    expect(files).toContain('/projects/bar/src/app/foo/foo.ts');
+    expect(files).not.toContain('/projects/bar/src/app/foo/foo.spec.ts');
   });
 
   it('should respect the sourceRoot value', async () => {
     const config = JSON.parse(appTree.readContent('/angular.json'));
     config.projects.bar.sourceRoot = 'projects/bar/custom';
     appTree.overwrite('/angular.json', JSON.stringify(config, null, 2));
     appTree = await schematicRunner.runSchematic('service', defaultOptions, appTree);
-    expect(appTree.files).toContain('/projects/bar/custom/app/foo/foo.service.ts');
+    expect(appTree.files).toContain('/projects/bar/custom/app/foo/foo.ts');
+  });
+
+  it('should respect the type option', async () => {
+    const options = { ...defaultOptions, type: 'Service' };
+    const tree = await schematicRunner.runSchematic('service', options, appTree);
+    const content = tree.readContent('/projects/bar/src/app/foo/foo.service.ts');
+    const testContent = tree.readContent('/projects/bar/src/app/foo/foo.service.spec.ts');
+    expect(content).toContain('export class FooService');
+    expect(testContent).toContain("describe('FooService'");
+  });
+
+  it('should allow empty string in the type option', async () => {
+    const options = { ...defaultOptions, type: '' };
+    const tree = await schematicRunner.runSchematic('service', options, appTree);
+    const content = tree.readContent('/projects/bar/src/app/foo/foo.ts');
+    const testContent = tree.readContent('/projects/bar/src/app/foo/foo.spec.ts');
+    expect(content).toContain('export class Foo');
+    expect(testContent).toContain("describe('Foo'");
   });
 });

packages/schematics/angular/service/index_spec.ts

@@ -39,6 +39,10 @@
       "type": "boolean",
       "description": "Skip the generation of a unit test file `spec.ts` for the service.",
       "default": false
+    },
+    "type": {
+      "type": "string",
+      "description": "Append a custom type to the service's filename. For example, if you set the type to `service`, the file will be named `my-service.service.ts`."
     }
   },
   "required": ["name", "project"]

packages/schematics/angular/service/schema.json

@@ -7,12 +7,14 @@
  */
 
 import {
+  FileOperator,
   Rule,
   Tree,
   apply,
   applyTemplates,
   chain,
   filter,
+  forEach,
   mergeWith,
   move,
   noop,
@@ -31,6 +33,7 @@ export interface GenerateFromFilesOptions {
   project: string;
   skipTests?: boolean;
   templateFilesDirectory?: string;
+  type?: string;
 }
 
 export function generateFromFiles(
@@ -56,6 +59,16 @@ export function generateFromFiles(
         ...options,
         ...extraTemplateValues,
       }),
+      !options.type
+        ? forEach(((file) => {
+            return file.path.includes('..')
+              ? {
+                  content: file.content,
+                  path: file.path.replace('..', '.'),
+                }
+              : file;
+          }) as FileOperator)
+        : noop(),
       move(parsedPath.path + (options.flat ? '' : '/' + strings.dasherize(options.name))),
     ]);
 

packages/schematics/angular/utility/generate-from-files.ts

@@ -17,7 +17,7 @@ export async function libraryConsumptionSetup(): Promise<void> {
     export class MyLibComponent {}`,
     './src/app/app.ts': `
     import { Component } from '@angular/core';
-    import { MyLibService, MyLibComponent } from 'my-lib';
+    import { MyLibComponent } from 'my-lib';
 
     @Component({
       standalone: true,
@@ -28,8 +28,7 @@ export async function libraryConsumptionSetup(): Promise<void> {
     export class App {
       title = 'test-project';
 
-      constructor(myLibService: MyLibService) {
-        console.log(myLibService);
+      constructor() {
       }
     }
   `,

tests/legacy-cli/e2e/tests/build/library/setup.ts

@@ -9,8 +9,8 @@ export default function () {
   return (
     ng('generate', 'service', 'test-service')
       .then(() => expectFileToExist(serviceDir))
-      .then(() => expectFileToExist(join(serviceDir, 'test-service.service.ts')))
-      .then(() => expectFileToExist(join(serviceDir, 'test-service.service.spec.ts')))
+      .then(() => expectFileToExist(join(serviceDir, 'test-service.ts')))
+      .then(() => expectFileToExist(join(serviceDir, 'test-service.spec.ts')))
 
       // Try to run the unit tests.
       .then(() => ng('test', '--watch=false'))

tests/legacy-cli/e2e/tests/generate/service/service-basic.ts

@@ -2,10 +2,10 @@ import { prependToFile, replaceInFile } from '../../utils/fs';
 import { ng } from '../../utils/process';
 
 export default async function () {
-  await ng('generate', 'service', 'user');
+  await ng('generate', 'service', 'user-service');
 
   // Update the application to use the new service
-  await prependToFile('src/app/app.ts', "import { UserService } from './user.service';");
+  await prependToFile('src/app/app.ts', "import { UserService } from './user-service';");
 
   await replaceInFile(
     'src/app/app.ts',