chore: resolve technical debt after v2.0.0 migration (#89)

heyding · web-flow · commit fdb0b71624d3 · 2026-07-25T18:38:57.000+02:00
* chore: resolve technical debt after v2.0.0 migration

spec files (10):
- Replace TranslateModule.forRoot() with provideTranslateService()
  in all *.spec.ts files — TranslateModule was removed in ngx-translate v18,
  causing all unit tests to fail at compile time

package.json:
- Remove autoprefixer (not referenced in postcss.config.json)
- Remove @tailwindcss/aspect-ratio (built into Tailwind v4, already
  documented as removed in CHANGELOG but still listed in devDependencies)

interceptors/retry.interceptor.ts:
- Remove debug console.log from production retry logic

features-demo.component.ts:
- Remove debug console.log from onSelectionChange handler

* chore: reorder import statements in retry interceptor
diff --git a/package.json b/package.json
@@ -88,11 +88,9 @@
     "@angular/compiler-cli": "^22.0.1",
     "@types/jasmine": "^5.1.4",
     "@types/node": "^22.10.2",
-    "@tailwindcss/aspect-ratio": "^0.4.2",
     "@tailwindcss/forms": "^0.5.11",
     "@tailwindcss/postcss": "^4.3.3",
     "@tailwindcss/typography": "^0.5.20",
-    "autoprefixer": "^10.4.23",
     "husky": "^9.1.7",
     "http-server": "^14.1.1",
     "jasmine-core": "^5.5.0",
diff --git a/src/app/app.component.spec.ts b/src/app/app.component.spec.ts
@@ -1,13 +1,13 @@
 import { TestBed } from '@angular/core/testing';
-import { AppComponent } from './app.component';
 import { provideRouter } from '@angular/router';
-import { TranslateModule } from '@ngx-translate/core';
+import { provideTranslateService } from '@ngx-translate/core';
+import { AppComponent } from './app.component';
 
 describe('AppComponent', () => {
   beforeEach(async () => {
     await TestBed.configureTestingModule({
-      imports: [AppComponent, TranslateModule.forRoot()],
-      providers: [provideRouter([])],
+      imports: [AppComponent],
+      providers: [provideRouter([]), provideTranslateService()],
     }).compileComponents();
   });
 
diff --git a/src/app/core/footer/footer.component.spec.ts b/src/app/core/footer/footer.component.spec.ts
@@ -1,14 +1,15 @@
 import { ComponentFixture, TestBed } from '@angular/core/testing';
+import { provideTranslateService } from '@ngx-translate/core';
 import { FooterComponent } from './footer.component';
-import { TranslateModule } from '@ngx-translate/core';
 
 describe('FooterComponent', () => {
   let component: FooterComponent;
   let fixture: ComponentFixture<FooterComponent>;
 
   beforeEach(async () => {
     await TestBed.configureTestingModule({
-      imports: [FooterComponent, TranslateModule.forRoot()],
+      imports: [FooterComponent],
+      providers: [provideTranslateService()],
     }).compileComponents();
 
     fixture = TestBed.createComponent(FooterComponent);
diff --git a/src/app/core/header/header.component.spec.ts b/src/app/core/header/header.component.spec.ts
@@ -1,16 +1,16 @@
 import { ComponentFixture, TestBed } from '@angular/core/testing';
-import { HeaderComponent } from './header.component';
-import { TranslateModule } from '@ngx-translate/core';
 import { provideRouter } from '@angular/router';
+import { provideTranslateService } from '@ngx-translate/core';
+import { HeaderComponent } from './header.component';
 
 describe('HeaderComponent', () => {
   let component: HeaderComponent;
   let fixture: ComponentFixture<HeaderComponent>;
 
   beforeEach(async () => {
     await TestBed.configureTestingModule({
-      imports: [HeaderComponent, TranslateModule.forRoot()],
-      providers: [provideRouter([])],
+      imports: [HeaderComponent],
+      providers: [provideRouter([]), provideTranslateService()],
     }).compileComponents();
 
     fixture = TestBed.createComponent(HeaderComponent);
diff --git a/src/app/core/interceptors/retry.interceptor.ts b/src/app/core/interceptors/retry.interceptor.ts
@@ -1,4 +1,4 @@
-import { HttpInterceptorFn, HttpErrorResponse } from '@angular/common/http';
+import { HttpErrorResponse, HttpInterceptorFn } from '@angular/common/http';
 import { retry, timer } from 'rxjs';
 
 export const retryInterceptor: HttpInterceptorFn = (req, next) => {
@@ -18,7 +18,6 @@ export const retryInterceptor: HttpInterceptorFn = (req, next) => {
 
         // Exponential backoff: 1s, 2s, 4s
         const delayMs = Math.pow(2, retryCount - 1) * 1000;
-        console.log(`Retry attempt ${retryCount} after ${delayMs}ms`);
         return timer(delayMs);
       },
     })
diff --git a/src/app/features/home/components/feature-overview/feature-overview.component.spec.ts b/src/app/features/home/components/feature-overview/feature-overview.component.spec.ts
@@ -1,14 +1,15 @@
 import { ComponentFixture, TestBed } from '@angular/core/testing';
+import { provideTranslateService } from '@ngx-translate/core';
 import { FeatureOverviewComponent } from './feature-overview.component';
-import { TranslateModule } from '@ngx-translate/core';
 
 describe('FeatureOverviewComponent', () => {
   let component: FeatureOverviewComponent;
   let fixture: ComponentFixture<FeatureOverviewComponent>;
 
   beforeEach(async () => {
     await TestBed.configureTestingModule({
-      imports: [FeatureOverviewComponent, TranslateModule.forRoot()],
+      imports: [FeatureOverviewComponent],
+      providers: [provideTranslateService()],
     }).compileComponents();
 
     fixture = TestBed.createComponent(FeatureOverviewComponent);
diff --git a/src/app/features/home/components/language-switch/language-switch.component.spec.ts b/src/app/features/home/components/language-switch/language-switch.component.spec.ts
@@ -1,14 +1,15 @@
 import { ComponentFixture, TestBed } from '@angular/core/testing';
+import { provideTranslateService } from '@ngx-translate/core';
 import { LanguageSwitchComponent } from './language-switch.component';
-import { TranslateModule } from '@ngx-translate/core';
 
 describe('LanguageSwitchComponent', () => {
   let component: LanguageSwitchComponent;
   let fixture: ComponentFixture<LanguageSwitchComponent>;
 
   beforeEach(async () => {
     await TestBed.configureTestingModule({
-      imports: [LanguageSwitchComponent, TranslateModule.forRoot()],
+      imports: [LanguageSwitchComponent],
+      providers: [provideTranslateService()],
     }).compileComponents();
 
     fixture = TestBed.createComponent(LanguageSwitchComponent);
diff --git a/src/app/features/home/components/page-not-found/page-not-found.component.spec.ts b/src/app/features/home/components/page-not-found/page-not-found.component.spec.ts
@@ -1,14 +1,15 @@
 import { ComponentFixture, TestBed } from '@angular/core/testing';
+import { provideTranslateService } from '@ngx-translate/core';
 import { PageNotFoundComponent } from './page-not-found.component';
-import { TranslateModule } from '@ngx-translate/core';
 
 describe('PageNotFoundComponent', () => {
   let component: PageNotFoundComponent;
   let fixture: ComponentFixture<PageNotFoundComponent>;
 
   beforeEach(async () => {
     await TestBed.configureTestingModule({
-      imports: [PageNotFoundComponent, TranslateModule.forRoot()],
+      imports: [PageNotFoundComponent],
+      providers: [provideTranslateService()],
     }).compileComponents();
 
     fixture = TestBed.createComponent(PageNotFoundComponent);
diff --git a/src/app/features/home/pages/about/about.component.spec.ts b/src/app/features/home/pages/about/about.component.spec.ts
@@ -1,14 +1,15 @@
 import { ComponentFixture, TestBed } from '@angular/core/testing';
+import { provideTranslateService } from '@ngx-translate/core';
 import { AboutComponent } from './about.component';
-import { TranslateModule } from '@ngx-translate/core';
 
 describe('AboutComponent', () => {
   let component: AboutComponent;
   let fixture: ComponentFixture<AboutComponent>;
 
   beforeEach(async () => {
     await TestBed.configureTestingModule({
-      imports: [AboutComponent, TranslateModule.forRoot()],
+      imports: [AboutComponent],
+      providers: [provideTranslateService()],
     }).compileComponents();
 
     fixture = TestBed.createComponent(AboutComponent);
diff --git a/src/app/features/home/pages/api-demo/api-demo.component.spec.ts b/src/app/features/home/pages/api-demo/api-demo.component.spec.ts
@@ -1,17 +1,17 @@
-import { ComponentFixture, TestBed } from '@angular/core/testing';
 import { provideHttpClient } from '@angular/common/http';
 import { provideHttpClientTesting } from '@angular/common/http/testing';
+import { ComponentFixture, TestBed } from '@angular/core/testing';
+import { provideTranslateService } from '@ngx-translate/core';
 import { ApiDemoComponent } from './api-demo.component';
-import { TranslateModule } from '@ngx-translate/core';
 
 describe('ApiDemoComponent', () => {
   let component: ApiDemoComponent;
   let fixture: ComponentFixture<ApiDemoComponent>;
 
   beforeEach(async () => {
     await TestBed.configureTestingModule({
-      imports: [ApiDemoComponent, TranslateModule.forRoot()],
-      providers: [provideHttpClient(), provideHttpClientTesting()],
+      imports: [ApiDemoComponent],
+      providers: [provideHttpClient(), provideHttpClientTesting(), provideTranslateService()],
     }).compileComponents();
 
     fixture = TestBed.createComponent(ApiDemoComponent);
diff --git a/src/app/features/home/pages/features-demo/features-demo.component.ts b/src/app/features/home/pages/features-demo/features-demo.component.ts
@@ -93,8 +93,8 @@ export class FeaturesDemoComponent implements OnInit {
     });
   }
 
-  onSelectionChange(selected: User[]): void {
-    console.log('Selected users:', selected);
+  onSelectionChange(_selected: User[]): void {
+    // Selection handled by parent/store if needed
   }
 
   toggleDropdown(): void {
diff --git a/src/app/features/home/pages/home/home.component.spec.ts b/src/app/features/home/pages/home/home.component.spec.ts
@@ -1,17 +1,17 @@
 import { ComponentFixture, TestBed } from '@angular/core/testing';
-import { HomeComponent } from './home.component';
 import { provideStore } from '@ngrx/store';
-import { TranslateModule } from '@ngx-translate/core';
-import { reducers, metaReducers } from '../../../../store/app.reducer';
+import { provideTranslateService } from '@ngx-translate/core';
+import { metaReducers, reducers } from '../../../../store/app.reducer';
+import { HomeComponent } from './home.component';
 
 describe('HomeComponent', () => {
   let component: HomeComponent;
   let fixture: ComponentFixture<HomeComponent>;
 
   beforeEach(async () => {
     await TestBed.configureTestingModule({
-      imports: [HomeComponent, TranslateModule.forRoot()],
-      providers: [provideStore(reducers, { metaReducers })],
+      imports: [HomeComponent],
+      providers: [provideStore(reducers, { metaReducers }), provideTranslateService()],
     }).compileComponents();
 
     fixture = TestBed.createComponent(HomeComponent);
diff --git a/src/app/features/home/pages/login/login.component.spec.ts b/src/app/features/home/pages/login/login.component.spec.ts
@@ -1,11 +1,10 @@
 import { ComponentFixture, TestBed } from '@angular/core/testing';
 import { ReactiveFormsModule } from '@angular/forms';
-import { Router, ActivatedRoute } from '@angular/router';
-import { of } from 'rxjs';
-import { LoginComponent } from './login.component';
+import { ActivatedRoute, Router } from '@angular/router';
+import { provideTranslateService } from '@ngx-translate/core';
 import { AuthService } from '../../../../core/services/auth.service';
 import { ToastService } from '../../../../shared/services/toast.service';
-import { TranslateModule } from '@ngx-translate/core';
+import { LoginComponent } from './login.component';
 
 describe('LoginComponent', () => {
   let component: LoginComponent;
@@ -26,8 +25,9 @@ describe('LoginComponent', () => {
     };
 
     await TestBed.configureTestingModule({
-      imports: [LoginComponent, ReactiveFormsModule, TranslateModule.forRoot()],
+      imports: [LoginComponent, ReactiveFormsModule],
       providers: [
+        provideTranslateService(),
         { provide: AuthService, useValue: mockAuthService },
         { provide: Router, useValue: mockRouter },
         { provide: ActivatedRoute, useValue: mockActivatedRoute },
