fix: resolve test warnings and update CI to use pnpm

   - Add @Injectable() to TestableTranslateService in http-loader to fix
     Angular DI deprecation warning
   - Fix fakeAsync test in translate.pipe-standalone.spec.ts that was
     returning the function instead of executing it
   - Add expect() assertions to onTranslationRefresh tests that only had
     done() callbacks
   - Update GitHub Actions to use pnpm instead of npm (matching project's
     packageManager setting)
   - Add lint step to CI pipeline

Author: CodeAndWeb

.github/workflows/ngx-translate-test.yml

@@ -15,15 +15,18 @@ jobs:
 
         strategy:
             matrix:
-                node-version: [20.x, 22.x, 24.x]
+                node-version: [20.x, 22.x]
 
         steps:
             - uses: actions/checkout@v4
+            - name: Install pnpm
+              uses: pnpm/action-setup@v4
             - name: Use Node.js ${{ matrix.node-version }}
               uses: actions/setup-node@v4
               with:
                   node-version: ${{ matrix.node-version }}
-                  cache: "npm"
-            - run: npm ci
-            - run: npm run build-all
-            - run: npm run test-ci
+                  cache: "pnpm"
+            - run: pnpm install --frozen-lockfile
+            - run: pnpm run lint
+            - run: pnpm run build-all
+            - run: pnpm run test-ci

projects/http-loader/src/test-helper/testable-translate-service.ts

@@ -1,4 +1,4 @@
-import { ClassProvider, Provider } from "@angular/core";
+import { ClassProvider, Injectable, Provider } from "@angular/core";
 import {
     provideTranslateService,
     RootTranslateServiceConfig,
@@ -7,6 +7,7 @@ import {
     TranslateService,
 } from "@ngx-translate/core";
 
+@Injectable()
 export class TestableTranslateService extends TranslateService {
     public getCurrentLoader(): TranslateLoader {
         return this.currentLoader;

projects/ngx-translate/src/tests/translate.pipe-standalone.spec.ts

@@ -55,19 +55,19 @@ describe("TranslatePipe (standalone)", () => {
     });
 
     describe("should update translations on lang change - async", () => {
-        it("should detect changes with OnPush", () =>
-            fakeAsync(() => {
-                prepare({ provide: TranslateLoader, useClass: DelayedFakeLoader });
+        it("should detect changes with OnPush", fakeAsync(() => {
+            prepare({ provide: TranslateLoader, useClass: DelayedFakeLoader });
 
-                fixture.detectChanges();
-                expect(fixture.debugElement.nativeElement.innerHTML).toEqual("TEST");
+            fixture.detectChanges();
+            expect(fixture.debugElement.nativeElement.innerHTML).toEqual("TEST");
 
-                translate.use("en");
-                fixture.detectChanges();
-                expect(fixture.debugElement.nativeElement.innerHTML).toEqual("TEST");
+            translate.use("en");
+            fixture.detectChanges();
+            expect(fixture.debugElement.nativeElement.innerHTML).toEqual("TEST");
 
-                tick(10);
-                expect(fixture.debugElement.nativeElement.innerHTML).toEqual("This is a test");
-            }));
+            tick(10);
+            fixture.detectChanges();
+            expect(fixture.debugElement.nativeElement.innerHTML).toEqual("This is a test");
+        }));
     });
 });

projects/ngx-translate/src/tests/translate.service.spec.ts

@@ -1566,6 +1566,7 @@ describe("TranslateService.onTranslationRefresh", () => {
         translate.use("en");
 
         translate.onTranslationRefresh.subscribe(() => {
+            expect(translate.currentLang).toEqual("fr");
             done();
         });
 
@@ -1577,6 +1578,7 @@ describe("TranslateService.onTranslationRefresh", () => {
         translate.use("en");
 
         translate.onTranslationRefresh.subscribe(() => {
+            expect(translate.instant("TEST")).toEqual("Updated");
             done();
         });
 
@@ -1594,6 +1596,7 @@ describe("TranslateService.onTranslationRefresh", () => {
         translate.onTranslationRefresh.subscribe(() => {
             emitCount++;
             if (emitCount === 1) {
+                expect(translate.instant("TEST")).toEqual("Updated German");
                 done();
             }
         });
@@ -1609,6 +1612,7 @@ describe("TranslateService.onTranslationRefresh", () => {
         translate.setFallbackLang("de");
 
         translate.onTranslationRefresh.subscribe(() => {
+            expect(translate.getFallbackLang()).toEqual("fr");
             done();
         });