TT-317 remove feature manager service

src/app/app-routing.module.ts

@@ -11,8 +11,6 @@ import { HomeComponent } from './modules/home/home.component';
 import { LoginComponent } from './modules/login/login.component';
 import { CustomerComponent } from './modules/customer-management/pages/customer.component';
 import { UsersComponent } from './modules/users/pages/users.component';
-import { TechnologyReportComponent } from './modules/technology-report/pages/technology-report.component';
-import { TechnologiesReportGuard } from './guards/technologies-report-guard/technologies-report.guard';
 
 const routes: Routes = [
   {
@@ -26,7 +24,6 @@ const routes: Routes = [
       { path: 'activities-management', component: ActivitiesManagementComponent },
       { path: 'customers-management', canActivate: [AdminGuard], component: CustomerComponent },
       { path: 'users', canActivate: [AdminGuard], component: UsersComponent },
-      { path: 'technology-report', canActivate: [AdminGuard, TechnologiesReportGuard], component: TechnologyReportComponent },
       { path: '', pathMatch: 'full', redirectTo: 'time-clock' },
     ],
   },

src/app/modules/shared/components/sidebar/sidebar.component.spec.ts

@@ -5,14 +5,12 @@ import { RouterTestingModule } from '@angular/router/testing';
 import { Router, Routes } from '@angular/router';
 import { TimeClockComponent } from '../../../time-clock/pages/time-clock.component';
 import { of } from 'rxjs';
-import { FeatureManagerService } from '../../feature-toggles/feature-toggle-manager.service';
 import { UserInfoService } from 'src/app/modules/user/services/user-info.service';
 
 describe('SidebarComponent', () => {
   let component: SidebarComponent;
   let fixture: ComponentFixture<SidebarComponent>;
   let azureAdB2CServiceStubInjected;
-  let featureManagerServiceStubInjected: FeatureManagerService;
   let userInfoService: UserInfoService;
   let router;
   const routes: Routes = [{ path: 'time-clock', component: TimeClockComponent }];
@@ -47,7 +45,6 @@ describe('SidebarComponent', () => {
   beforeEach(() => {
     fixture = TestBed.createComponent(SidebarComponent);
     azureAdB2CServiceStubInjected = TestBed.inject(AzureAdB2CService);
-    featureManagerServiceStubInjected = TestBed.inject(FeatureManagerService);
     userInfoService = TestBed.inject(UserInfoService);
     component = fixture.componentInstance;
     fixture.detectChanges();
@@ -86,21 +83,4 @@ describe('SidebarComponent', () => {
     });
   });
 
-  it('List Technologies item is added when feature flag "ui-list-technologies" is enabled for user', () => {
-    spyOn(featureManagerServiceStubInjected, 'isToggleEnabledForUser').and.returnValue(of(true));
-    const itemsSidebar = [];
-
-    component.toggleListTechnologies(itemsSidebar);
-
-    expect(itemsSidebar.length).toBe(1);
-  });
-
-  it('List Technologies item is not added when feature flag "ui-list-technologies" is disabled for user', () => {
-    spyOn(featureManagerServiceStubInjected, 'isToggleEnabledForUser').and.returnValue(of(false));
-    const itemsSidebar = [];
-
-    component.toggleListTechnologies(itemsSidebar);
-
-    expect(itemsSidebar.length).toBe(0);
-  });
 });

src/app/modules/shared/components/sidebar/sidebar.component.ts

@@ -3,7 +3,6 @@ import { ItemSidebar } from './models/item-sidebar.model';
 import { NavigationStart, Router } from '@angular/router';
 import { Observable, Subscription } from 'rxjs';
 import { filter, map } from 'rxjs/operators';
-import { FeatureManagerService } from '../../feature-toggles/feature-toggle-manager.service';
 import { UserInfoService } from 'src/app/modules/user/services/user-info.service';
 
 @Component({
@@ -19,7 +18,6 @@ export class SidebarComponent implements OnInit, OnDestroy {
   constructor(
     private router: Router,
     private userInfoService: UserInfoService,
-    private featureManagerService: FeatureManagerService,
   ) {
     this.navStart = this.router.events.pipe(
       filter((evt) => evt instanceof NavigationStart)
@@ -29,7 +27,6 @@ export class SidebarComponent implements OnInit, OnDestroy {
   ngOnInit(): void {
     this.toggleSideBar();
     this.sidebarItems$ = this.getSidebarItems().subscribe();
-    this.toggleListTechnologies(this.itemsSidebar);
     this.highlightMenuOption(this.router.routerState.snapshot.url);
     this.navStart.subscribe((evt) => {
       this.highlightMenuOption(evt.url);
@@ -68,22 +65,6 @@ export class SidebarComponent implements OnInit, OnDestroy {
     );
   }
 
-  toggleListTechnologies(itemsSidebar: ItemSidebar[]) {
-    this.featureManagerService
-      .isToggleEnabledForUser('ui-list-technologies')
-      .subscribe((enabled) => {
-        if (enabled === true) {
-          const listTechnologiesItem = {
-            route: '/technology-report',
-            icon: 'fas fa-user',
-            text: 'Technology Report',
-            active: false,
-          };
-          itemsSidebar.push(listTechnologiesItem);
-        }
-      });
-  }
-
   highlightMenuOption(route) {
     this.itemsSidebar.map((item) => (item.active = false));
     this.itemsSidebar.filter((item) => item.route === route).map((item) => (item.active = true));

src/app/modules/shared/feature-toggles/feature-toggle-general/feature-toggle-general.service.spec.ts

@@ -23,18 +23,6 @@ describe('FeatureToggleGeneralService', () => {
     expect(featureToggleGeneralService).toBeTruthy();
   });
 
-  const params = [{ bool: false }, { bool: true }];
-  params.map((param) => {
-    it(`isActivated should return a boolean ${param.bool}`, () => {
-      const toggleName = FeatureToggle.SWITCH_GROUP;
-      featureManagerService.isToggleEnabledForUser = () => of(param.bool);
-
-      featureToggleGeneralService.isActivated(toggleName).subscribe((enabled) => {
-        expect(enabled).toBe(param.bool);
-      });
-    });
-  });
-
   it('getActivated return a FeatureToggleModel', () => {
     const anyNotMatchingFilter = new TargetingFeatureFilterModel(
       { Audience: { Groups: ['a-group'], Users: ['user-a'] } },

src/app/modules/shared/feature-toggles/feature-toggle-general/feature-toggle-general.service.ts

@@ -11,10 +11,6 @@ import { FeatureToggleModel } from '../feature-toggle.model';
 export class FeatureToggleGeneralService {
   constructor(private featureManagerService: FeatureManagerService) {}
 
-  isActivated(featureToggle: FeatureToggle): Observable<boolean> {
-    return this.featureManagerService.isToggleEnabledForUser(featureToggle);
-  }
-
   getActivated(): Observable<FeatureToggleModel[]>{
     return this.featureManagerService.getAllFeatureToggleEnableForUser();
   }

src/app/modules/shared/feature-toggles/feature-toggle-manager.service.spec.ts

@@ -26,18 +26,6 @@ describe('FeatureToggleManager', () => {
       spyOn(fakeFeatureToggleProvider, 'getFeatureToggle').and.returnValue(of(aFeatureToggle));
       service = new FeatureManagerService(fakeFeatureToggleProvider);
     });
-    it('manager uses feature provider to build feature toggle model', async () => {
-      service.isToggleEnabled(featureToggleKey, featureToggleLabel).subscribe((value) => {
-
-        expect(fakeFeatureToggleProvider).toHaveBeenCalledWith(featureToggleKey, featureToggleLabel);
-      });
-    });
-
-    it('manager extracts enabled attribute from feature toggle model', async () => {
-      service.isToggleEnabled(featureToggleKey, featureToggleLabel).subscribe((value) => {
-        expect(value).toEqual(aFeatureToggle.enabled);
-      });
-    });
   });
 
 
@@ -62,19 +50,6 @@ describe('FeatureToggleManager', () => {
       );
       getFeatureToggleSpy = spyOn(fakeFeatureToggleProvider, 'getFeatureToggle').and.returnValue(of(aToggleWithFilters));
       service = new FeatureManagerService(fakeFeatureToggleProvider);
-      spyOn(service, 'isToggleEnabled').and.returnValue(of(true));
-    });
-
-    it('manager uses feature provider to build feature toggle model', async () => {
-      service.isToggleEnabledForUser(featureToggleKey, featureToggleLabel).subscribe((value) => {
-        expect(getFeatureToggleSpy).toHaveBeenCalledWith(featureToggleKey, featureToggleLabel);
-      });
-    });
-
-    it('given a feature toggle with filters which match the verification, then the response is true', async () => {
-      service.isToggleEnabledForUser(featureToggleKey, featureToggleLabel).subscribe((value) => {
-        expect(value).toEqual(true);
-      });
     });
 
     it('given a feature toggle with filters which do not match the verification, then the response is false', async () => {
@@ -87,9 +62,6 @@ describe('FeatureToggleManager', () => {
       spyOn(fakeFeatureToggleProvider, 'getFeatureToggle').and.returnValue(of(aToggleWithFilters));
       service = new FeatureManagerService(fakeFeatureToggleProvider);
 
-      service.isToggleEnabledForUser(featureToggleKey, featureToggleLabel).subscribe((value) => {
-        expect(value).toEqual(false);
-      });
     });
 
     it('Get empty when getAllFeatureToggle() return empty', () => {

src/app/modules/shared/feature-toggles/feature-toggle-manager.service.ts

@@ -11,31 +11,6 @@ export class FeatureManagerService {
 
   constructor(private featureToggleProvider: FeatureToggleProvider) { }
 
-  public isToggleEnabled(toggleName: string, toggleLabel?: string): Observable<boolean> {
-    return this.featureToggleProvider
-      .getFeatureToggle(toggleName, toggleLabel)
-      .pipe(map((featureToggle) => featureToggle.enabled));
-  }
-
-  public isToggleEnabledForUser(toggleName: string, toggleLabel?: string): Observable<boolean> {
-    const matchesFilters$: Observable<boolean> = this.featureToggleProvider
-    .getFeatureToggle(toggleName, toggleLabel)
-    .pipe(
-      map(featureToggle => featureToggle.filters),
-      map(filters => filters.map(filter => filter.evaluate())),
-      map(filterEvaluations => filterEvaluations.includes(true))
-    );
-
-    const result$: Observable<boolean> = zip(
-      this.isToggleEnabled(toggleName, toggleLabel),
-      matchesFilters$
-    ).pipe(
-      map(([enabled, enabledForUser]) => enabled && enabledForUser)
-    );
-
-    return result$;
-  }
-
   public getAllFeatureToggleEnableForUser(): Observable<FeatureToggleModel[]> {
     return from(this.featureToggleProvider.getAllFeatureToggle()).pipe(
       map((allFeatureToggle) =>