fixes angular tests

Signed-off-by: Maksym Trofimenko <maksym@container-registry.com>
container-registry · Jan 19, 2025 · 13acb4c · 13acb4c
1 parent 3aef93a
commit 13acb4c
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 11 deletions.
diff --git a/src/portal/src/app/app.module.ts b/src/portal/src/app/app.module.ts
@@ -77,7 +77,6 @@ class MyMissingTranslationHandler implements MissingTranslationHandler {
         HttpClientModule,
         HarborRoutingModule,
         CookieModule.forRoot(),
-        SharedModule.forRoot(),
     ],
     providers: [
         AppConfigService,

diff --git a/...oject/tag-feature-integration/immutable-tag/add-rule/add-immutable-rule.component.spec.ts b/...oject/tag-feature-integration/immutable-tag/add-rule/add-immutable-rule.component.spec.ts
@@ -6,6 +6,7 @@ import { ErrorHandler } from '../../../../../shared/units/error-handler';
 import { InlineAlertComponent } from '../../../../../shared/components/inline-alert/inline-alert.component';
 import { AddImmutableRuleComponent } from './add-immutable-rule.component';
 import { SharedTestingModule } from '../../../../../shared/shared.module';
+import { CallbackPipe } from '../../../../../shared/pipes/callback.pipe';
 
 describe('AddRuleComponent', () => {
     let component: AddImmutableRuleComponent;
@@ -40,7 +41,11 @@ describe('AddRuleComponent', () => {
     };
     beforeEach(async () => {
         await TestBed.configureTestingModule({
-            declarations: [AddImmutableRuleComponent, InlineAlertComponent],
+            declarations: [
+                AddImmutableRuleComponent,
+                InlineAlertComponent,
+                CallbackPipe,
+            ],
             schemas: [CUSTOM_ELEMENTS_SCHEMA],
             imports: [SharedTestingModule],
             providers: [

diff --git a/...pp/base/project/tag-feature-integration/tag-retention/add-rule/add-rule.component.spec.ts b/...pp/base/project/tag-feature-integration/tag-retention/add-rule/add-rule.component.spec.ts
@@ -13,6 +13,8 @@ import { HttpClientTestingModule } from '@angular/common/http/testing';
 import { TagRetentionService } from '../tag-retention.service';
 import { ErrorHandler } from '../../../../../shared/units/error-handler';
 import { InlineAlertComponent } from '../../../../../shared/components/inline-alert/inline-alert.component';
+import { CallbackPipe } from '../../../../../shared/pipes/callback.pipe';
+
 describe('AddRuleComponent', () => {
     let component: AddRuleComponent;
     let fixture: ComponentFixture<AddRuleComponent>;
@@ -30,7 +32,11 @@ describe('AddRuleComponent', () => {
                 NoopAnimationsModule,
                 HttpClientTestingModule,
             ],
-            declarations: [AddRuleComponent, InlineAlertComponent],
+            declarations: [
+                AddRuleComponent,
+                CallbackPipe,
+                InlineAlertComponent,
+            ],
             providers: [
                 TranslateService,
                 ErrorHandler,

diff --git a/src/portal/src/app/shared/shared.module.ts b/src/portal/src/app/shared/shared.module.ts
@@ -252,14 +252,7 @@ ClarityIcons.add({
         },
     ],
 })
-export class SharedModule {
-    static forRoot() {
-        return {
-            ngModule: SharedModule,
-            providers: [],
-        };
-    }
-}
+export class SharedModule {}
 
 // this module is only for testing, you should only import this module in *.spec.ts files
 @NgModule({