Angular 18 upgrade

package.json

@@ -30,11 +30,11 @@
   },
   "private": true,
   "dependencies": {
-    "@alfresco/adf-content-services": "8.1.0-14225774087",
-    "@alfresco/adf-core": "8.1.0-14225774087",
-    "@alfresco/adf-extensions": "8.1.0-14225774087",
-    "@alfresco/eslint-plugin-eslint-angular": "8.1.0-14225774087",
-    "@alfresco/js-api": "9.1.0-14225774087",
+    "@alfresco/adf-content-services": "7.0.0-alpha.9-13833690161",
+    "@alfresco/adf-core": "7.0.0-alpha.9-13833690161",
+    "@alfresco/adf-extensions": "7.0.0-alpha.9-13833690161",
+    "@alfresco/eslint-plugin-eslint-angular": "7.0.0-alpha.9-13833690161",
+    "@alfresco/js-api": "8.0.0-alpha.9-13833690161",
     "@angular/animations": "18.2.9",
     "@angular/cdk": "18.2.9",
     "@angular/common": "18.2.9",
@@ -49,7 +49,7 @@
     "@fontsource/open-sans": "^5.1.0",
     "@mat-datetimepicker/core": "14.0.0",
     "@ngrx/effects": "18.1.1",
-    "@ngrx/operators": "18.1.1",
+    "@ngrx/operators": "^18.0.0",
     "@ngrx/router-store": "18.1.1",
     "@ngrx/store": "18.1.1",
     "@ngrx/store-devtools": "18.1.1",
@@ -67,7 +67,7 @@
     "zone.js": "0.14.10"
   },
   "devDependencies": {
-    "@alfresco/adf-cli": "8.1.0-14225774087",
+    "@alfresco/adf-cli": "7.0.0-alpha.9-13833690161",
     "@angular-devkit/build-angular": "18.2.9",
     "@angular-devkit/core": "18.2.9",
     "@angular-devkit/schematics": "18.2.9",
@@ -76,7 +76,7 @@
     "@angular-eslint/eslint-plugin-template": "18.4.3",
     "@angular-eslint/schematics": "18.4.3",
     "@angular-eslint/template-parser": "18.4.3",
-    "@angular/cli": "18.2.15",
+    "@angular/cli": "~18.2.0",
     "@angular/compiler-cli": "18.2.9",
     "@angular/language-service": "18.2.9",
     "@cspell/eslint-plugin": "^8.14.1",

projects/aca-content/src/lib/components/dl-custom-components/datatable-cell-badges/datatable-cell-badges.component.spec.ts

@@ -32,7 +32,7 @@ import { NodeEntry } from '@alfresco/js-api';
 import { of } from 'rxjs';
 import { By } from '@angular/platform-browser';
 import { ContentActionType } from '@alfresco/adf-extensions';
-import { HttpClientModule } from '@angular/common/http';
+import { provideHttpClient, withInterceptorsFromDi } from '@angular/common/http';
 import { StoreModule } from '@ngrx/store';
 
 const mockNode = {
@@ -59,7 +59,6 @@ describe('DatatableCellBadgesComponent', () => {
       imports: [
         TranslateModule.forRoot(),
         AuthModule.forRoot(),
-        HttpClientModule,
         StoreModule.forRoot(
           { app: (state) => state },
           {
@@ -76,7 +75,7 @@ describe('DatatableCellBadgesComponent', () => {
           }
         )
       ],
-      providers: [Actions]
+      providers: [Actions, provideHttpClient(withInterceptorsFromDi())]
     });
 
     fixture = TestBed.createComponent(DatatableCellBadgesComponent);

projects/aca-content/src/lib/components/dl-custom-components/name-column/name-column.component.spec.ts

@@ -26,7 +26,7 @@ import { CustomNameColumnComponent } from './name-column.component';
 import { Actions } from '@ngrx/effects';
 import { StoreModule } from '@ngrx/store';
 import { ComponentFixture, TestBed } from '@angular/core/testing';
-import { HttpClientModule } from '@angular/common/http';
+import { provideHttpClient, withInterceptorsFromDi } from '@angular/common/http';
 import { TranslateModule } from '@ngx-translate/core';
 import { By } from '@angular/platform-browser';
 import { AuthModule } from '@alfresco/adf-core';
@@ -49,7 +49,6 @@ describe('CustomNameColumnComponent', () => {
   beforeEach(() => {
     TestBed.configureTestingModule({
       imports: [
-        HttpClientModule,
         TranslateModule.forRoot(),
         CustomNameColumnComponent,
         MockDatatableCellBadgesComponent,
@@ -70,7 +69,7 @@ describe('CustomNameColumnComponent', () => {
           }
         )
       ],
-      providers: [Actions]
+      providers: [Actions, provideHttpClient(withInterceptorsFromDi())]
     });
 
     fixture = TestBed.createComponent(CustomNameColumnComponent);

projects/aca-content/src/lib/components/home/home.component.spec.ts

@@ -25,7 +25,7 @@
 import { HomeComponent } from './home.component';
 import { ComponentFixture, TestBed } from '@angular/core/testing';
 import { Router } from '@angular/router';
-import { HttpClientModule } from '@angular/common/http';
+import { provideHttpClient, withInterceptorsFromDi } from '@angular/common/http';
 import { RouterTestingModule } from '@angular/router/testing';
 import { AppSettingsService } from '@alfresco/aca-shared';
 
@@ -36,7 +36,8 @@ describe('HomeComponent', () => {
 
   beforeEach(() => {
     TestBed.configureTestingModule({
-      imports: [HttpClientModule, RouterTestingModule, HomeComponent]
+      imports: [RouterTestingModule, HomeComponent],
+      providers: [provideHttpClient(withInterceptorsFromDi())]
     });
     appSettings = TestBed.inject(AppSettingsService);
     spyOnProperty(appSettings, 'landingPage', 'get').and.returnValue('/my-mock-landing-page');

projects/aca-content/src/lib/testing/app-testing.module.ts

@@ -25,25 +25,25 @@
 import { NgModule } from '@angular/core';
 import { TranslateModule } from '@ngx-translate/core';
 import { NoopAnimationsModule } from '@angular/platform-browser/animations';
-import { TranslationService, TranslationMock, AuthenticationService, PageTitleService } from '@alfresco/adf-core';
+import { AuthenticationService, PageTitleService, TranslationMock, TranslationService } from '@alfresco/adf-core';
 import { AlfrescoApiService, AlfrescoApiServiceMock, DiscoveryApiService, SearchQueryBuilderService } from '@alfresco/adf-content-services';
 import { RepositoryInfo, VersionInfo } from '@alfresco/js-api';
-import { HttpClientModule } from '@angular/common/http';
+import { provideHttpClient, withInterceptorsFromDi } from '@angular/common/http';
 import { StoreModule } from '@ngrx/store';
 import { appReducer } from '../store/reducers/app.reducer';
 import { RouterTestingModule } from '@angular/router/testing';
 import { EffectsModule } from '@ngrx/effects';
 import { INITIAL_STATE } from '../store/initial-state';
-import { BehaviorSubject, Observable, Subject, of } from 'rxjs';
+import { BehaviorSubject, Observable, of, Subject } from 'rxjs';
 import { ContentManagementService } from '../services/content-management.service';
 import { DocumentBasePageService } from '@alfresco/aca-shared';
 import { MatSnackBarModule } from '@angular/material/snack-bar';
 import { MatDialogModule } from '@angular/material/dialog';
 
 @NgModule({
+  exports: [RouterTestingModule, TranslateModule],
   imports: [
     NoopAnimationsModule,
-    HttpClientModule,
     RouterTestingModule,
     TranslateModule.forRoot(),
     StoreModule.forRoot(
@@ -60,7 +60,6 @@ import { MatDialogModule } from '@angular/material/dialog';
     MatSnackBarModule,
     MatDialogModule
   ],
-  exports: [RouterTestingModule, TranslateModule],
   providers: [
     SearchQueryBuilderService,
     { provide: AlfrescoApiService, useClass: AlfrescoApiServiceMock },
@@ -96,7 +95,8 @@ import { MatDialogModule } from '@angular/material/dialog';
     {
       provide: PageTitleService,
       useValue: {}
-    }
+    },
+    provideHttpClient(withInterceptorsFromDi())
   ]
 })
 export class AppTestingModule {}

projects/aca-content/viewer/src/lib/services/viewer.service.spec.ts

@@ -29,7 +29,7 @@ import { FavoritePaging, NodePaging, SharedLinkPaging } from '@alfresco/js-api';
 import { ViewerService } from './viewer.service';
 import { of } from 'rxjs';
 import { TranslateModule } from '@ngx-translate/core';
-import { HttpClientModule } from '@angular/common/http';
+import { provideHttpClient, withInterceptorsFromDi } from '@angular/common/http';
 
 const list = {
   list: {
@@ -69,8 +69,14 @@ describe('ViewerService', () => {
 
   beforeEach(() => {
     TestBed.configureTestingModule({
-      imports: [TranslateModule.forRoot(), HttpClientModule],
-      providers: [{ provide: TranslationService, useClass: TranslationMock }, ViewerService, UserPreferencesService, ContentApiService]
+      imports: [TranslateModule.forRoot()],
+      providers: [
+        { provide: TranslationService, useClass: TranslationMock },
+        ViewerService,
+        UserPreferencesService,
+        ContentApiService,
+        provideHttpClient(withInterceptorsFromDi())
+      ]
     });
 
     preferences = TestBed.inject(UserPreferencesService);

projects/aca-shared/src/lib/routing/plugin-enabled.guard.spec.ts

@@ -26,7 +26,8 @@ import { AppConfigService } from '@alfresco/adf-core';
 import { TestBed } from '@angular/core/testing';
 import { PluginEnabledGuard } from './plugin-enabled.guard';
 import { ActivatedRouteSnapshot, Router, RouterStateSnapshot } from '@angular/router';
-import { HttpClientTestingModule } from '@angular/common/http/testing';
+import { provideHttpClientTesting } from '@angular/common/http/testing';
+import { provideHttpClient, withInterceptorsFromDi } from '@angular/common/http';
 
 describe('PluginEnabledGuard', () => {
   let getSpy: jasmine.Spy<(key: string, defaultValue?: boolean) => boolean>;
@@ -34,7 +35,8 @@ describe('PluginEnabledGuard', () => {
 
   beforeEach(() => {
     TestBed.configureTestingModule({
-      imports: [HttpClientTestingModule]
+      imports: [],
+      providers: [provideHttpClient(withInterceptorsFromDi()), provideHttpClientTesting()]
     });
     getSpy = spyOn(TestBed.inject(AppConfigService), 'get');
     route = new ActivatedRouteSnapshot();

projects/aca-shared/src/lib/services/app.service.spec.ts

@@ -26,15 +26,15 @@ import { AppService } from './app.service';
 import { TestBed } from '@angular/core/testing';
 import {
   AuthenticationService,
+  NotificationService,
   PageTitleService,
+  StorageService,
   TranslationMock,
   TranslationService,
-  UserPreferencesService,
-  NotificationService,
-  StorageService
+  UserPreferencesService
 } from '@alfresco/adf-core';
 import { BehaviorSubject, Observable, of, Subject } from 'rxjs';
-import { HttpClientModule } from '@angular/common/http';
+import { provideHttpClient, withInterceptorsFromDi } from '@angular/common/http';
 import {
   AlfrescoApiService,
   AlfrescoApiServiceMock,
@@ -73,7 +73,7 @@ describe('AppService', () => {
 
   beforeEach(() => {
     TestBed.configureTestingModule({
-      imports: [CommonModule, HttpClientModule, TranslateModule.forRoot(), RouterTestingModule.withRoutes([]), MatDialogModule, MatSnackBarModule],
+      imports: [CommonModule, TranslateModule.forRoot(), RouterTestingModule.withRoutes([]), MatDialogModule, MatSnackBarModule],
       providers: [
         SearchQueryBuilderService,
         provideMockStore({}),
@@ -121,7 +121,8 @@ describe('AppService', () => {
             setStoragePrefix: () => null,
             getPropertyKey: (property: string) => `prefix__${property}`
           }
-        }
+        },
+        provideHttpClient(withInterceptorsFromDi())
       ]
     });
 

projects/aca-shared/src/lib/testing/lib-testing-module.ts

@@ -27,7 +27,7 @@ import { TranslateLoader, TranslateModule } from '@ngx-translate/core';
 import { NoopAnimationsModule } from '@angular/platform-browser/animations';
 import { TranslateLoaderService, TranslationMock, TranslationService } from '@alfresco/adf-core';
 import { AlfrescoApiService, AlfrescoApiServiceMock } from '@alfresco/adf-content-services';
-import { HttpClientModule } from '@angular/common/http';
+import { provideHttpClient, withInterceptorsFromDi } from '@angular/common/http';
 import { RouterTestingModule } from '@angular/router/testing';
 import { EffectsModule } from '@ngrx/effects';
 import { StoreModule } from '@ngrx/store';
@@ -93,10 +93,10 @@ export class DocumentBasePageServiceMock extends DocumentBasePageService {
 }
 
 @NgModule({
+  exports: [TranslateModule],
   imports: [
     NoopAnimationsModule,
     CommonModule,
-    HttpClientModule,
     RouterTestingModule,
     MatIconTestingModule,
     StoreModule,
@@ -119,10 +119,10 @@ export class DocumentBasePageServiceMock extends DocumentBasePageService {
       }
     })
   ],
-  exports: [TranslateModule],
   providers: [
     { provide: AlfrescoApiService, useClass: AlfrescoApiServiceMock },
-    { provide: TranslationService, useClass: TranslationMock }
+    { provide: TranslationService, useClass: TranslationMock },
+    provideHttpClient(withInterceptorsFromDi())
   ]
 })
 export class LibTestingModule {}