generator-jhipster 7.2.0 → 7.4.1

package/generators/client/templates/angular/src/main/webapp/app/home/home.component.spec.ts.ejs

@@ -37,135 +37,133 @@ import { LoginService } from 'app/login/login.service';
 
 import { HomeComponent } from './home.component';
 
-describe('Component Tests', () => {
-  describe('Home Component', () => {
-    let comp: HomeComponent;
-    let fixture: ComponentFixture<HomeComponent>;
-    let mockAccountService: AccountService;
+describe('Home Component', () => {
+  let comp: HomeComponent;
+  let fixture: ComponentFixture<HomeComponent>;
+  let mockAccountService: AccountService;
 <%_ if (!authenticationTypeOauth2) { _%>
-    let mockRouter: Router;
+  let mockRouter: Router;
 <%_ } else { _%>
-    let mockLoginService: LoginService;
+  let mockLoginService: LoginService;
 <%_ } _%>
-    const account: Account = {
-      activated: true,
-      authorities: [],
-      email: '',
-      firstName: null,
-      langKey: '',
-      lastName: null,
-      login: 'login',
-      imageUrl: null,
-    };
-
-    beforeEach(
-      waitForAsync(() => {
-        TestBed.configureTestingModule({
-          declarations: [HomeComponent],
-          providers: [AccountService, <% if (!authenticationTypeOauth2) { %>Router<% } else { %>LoginService<% } %>],
-        })
-          .overrideTemplate(HomeComponent, '')
-          .compileComponents();
+  const account: Account = {
+    activated: true,
+    authorities: [],
+    email: '',
+    firstName: null,
+    langKey: '',
+    lastName: null,
+    login: 'login',
+    imageUrl: null,
+  };
+
+  beforeEach(
+    waitForAsync(() => {
+      TestBed.configureTestingModule({
+        declarations: [HomeComponent],
+        providers: [AccountService, <% if (!authenticationTypeOauth2) { %>Router<% } else { %>LoginService<% } %>],
       })
-    );
-
-    beforeEach(() => {
-      fixture = TestBed.createComponent(HomeComponent);
-      comp = fixture.componentInstance;
-      mockAccountService = TestBed.inject(AccountService);
-      mockAccountService.identity = jest.fn(() => of(null));
-      mockAccountService.getAuthenticationState = jest.fn(() => of(null));
+        .overrideTemplate(HomeComponent, '')
+        .compileComponents();
+    })
+  );
+
+  beforeEach(() => {
+    fixture = TestBed.createComponent(HomeComponent);
+    comp = fixture.componentInstance;
+    mockAccountService = TestBed.inject(AccountService);
+    mockAccountService.identity = jest.fn(() => of(null));
+    mockAccountService.getAuthenticationState = jest.fn(() => of(null));
 <%_ if (!authenticationTypeOauth2) { _%>
-      mockRouter = TestBed.inject(Router);
+    mockRouter = TestBed.inject(Router);
 <%_ } else { _%>
-      mockLoginService = TestBed.inject(LoginService);
+    mockLoginService = TestBed.inject(LoginService);
 <%_ } _%>
-    });
+  });
 
-    describe('ngOnInit', () => {
-      it('Should synchronize account variable with current account', () => {
+  describe('ngOnInit', () => {
+    it('Should synchronize account variable with current account', () => {
 <%_ if (!authenticationTypeOauth2) { _%>
-        // GIVEN
-        const authenticationState = new Subject<Account | null>();
-        mockAccountService.getAuthenticationState = jest.fn(() => authenticationState.asObservable());
+      // GIVEN
+      const authenticationState = new Subject<Account | null>();
+      mockAccountService.getAuthenticationState = jest.fn(() => authenticationState.asObservable());
 
-        // WHEN
-        comp.ngOnInit();
+      // WHEN
+      comp.ngOnInit();
 
-        // THEN
-        expect(comp.account).toBeNull();
+      // THEN
+      expect(comp.account).toBeNull();
 
-        // WHEN
-        authenticationState.next(account);
+      // WHEN
+      authenticationState.next(account);
 
-        // THEN
-        expect(comp.account).toEqual(account);
+      // THEN
+      expect(comp.account).toEqual(account);
 
-        // WHEN
-        authenticationState.next(null);
+      // WHEN
+      authenticationState.next(null);
 
-        // THEN
-        expect(comp.account).toBeNull();
+      // THEN
+      expect(comp.account).toBeNull();
 <%_ } else { _%>
-        // GIVEN
-        mockAccountService.identity = jest.fn(() => of(account));
+      // GIVEN
+      mockAccountService.identity = jest.fn(() => of(account));
 
-        // WHEN
-        comp.ngOnInit();
+      // WHEN
+      comp.ngOnInit();
 
-        // THEN
-        expect(comp.account).toEqual(account);
+      // THEN
+      expect(comp.account).toEqual(account);
 <%_ } _%>
-      });
     });
+  });
 
-    describe('login', () => {
+  describe('login', () => {
 <%_ if (!authenticationTypeOauth2) { _%>
-      it('Should navigate to /login on login', () => {
-        // WHEN
-        comp.login();
+    it('Should navigate to /login on login', () => {
+      // WHEN
+      comp.login();
 
-        // THEN
-        expect(mockRouter.navigate).toHaveBeenCalledWith(['/login']);
-      });
+      // THEN
+      expect(mockRouter.navigate).toHaveBeenCalledWith(['/login']);
+    });
 <%_ } else { _%>
-      it('Should call loginService.login on login', () => {
-        // WHEN
-        comp.login();
+    it('Should call loginService.login on login', () => {
+      // WHEN
+      comp.login();
 
-        // THEN
-        expect(mockLoginService.login).toHaveBeenCalled();
-      });
-<%_ } _%>
+      // THEN
+      expect(mockLoginService.login).toHaveBeenCalled();
     });
+<%_ } _%>
+  });
 
 <%_ if (!authenticationTypeOauth2) { _%>
-    describe('ngOnDestroy', () => {
-      it('Should destroy authentication state subscription on component destroy', () => {
-        // GIVEN
-        const authenticationState = new Subject<Account | null>();
-        mockAccountService.getAuthenticationState = jest.fn(() => authenticationState.asObservable());
+  describe('ngOnDestroy', () => {
+    it('Should destroy authentication state subscription on component destroy', () => {
+      // GIVEN
+      const authenticationState = new Subject<Account | null>();
+      mockAccountService.getAuthenticationState = jest.fn(() => authenticationState.asObservable());
 
-        // WHEN
-        comp.ngOnInit();
+      // WHEN
+      comp.ngOnInit();
 
-        // THEN
-        expect(comp.account).toBeNull();
+      // THEN
+      expect(comp.account).toBeNull();
 
-        // WHEN
-        authenticationState.next(account);
+      // WHEN
+      authenticationState.next(account);
 
-        // THEN
-        expect(comp.account).toEqual(account);
+      // THEN
+      expect(comp.account).toEqual(account);
 
-        // WHEN
-        comp.ngOnDestroy();
-        authenticationState.next(null);
+      // WHEN
+      comp.ngOnDestroy();
+      authenticationState.next(null);
 
-        // THEN
-        expect(comp.account).toEqual(account);
-      });
+      // THEN
+      expect(comp.account).toEqual(account);
     });
-<%_ } _%>
   });
+<%_ } _%>
 });

package/generators/client/templates/angular/src/main/webapp/app/layouts/main/main.component.spec.ts.ejs

@@ -33,234 +33,232 @@ import { FindLanguageFromKeyPipe } from 'app/shared/language/find-language-from-
 
 import { MainComponent } from './main.component';
 
-describe('Component Tests', () => {
-  describe('MainComponent', () => {
-    let comp: MainComponent;
-    let fixture: ComponentFixture<MainComponent>;
-    let titleService: Title;
+describe('MainComponent', () => {
+  let comp: MainComponent;
+  let fixture: ComponentFixture<MainComponent>;
+  let titleService: Title;
 <%_ if (enableTranslation) { _%>
-    let translateService: TranslateService;
+  let translateService: TranslateService;
   <%_ if (enableI18nRTL) { _%>
-    let findLanguageFromKeyPipe: FindLanguageFromKeyPipe;
+  let findLanguageFromKeyPipe: FindLanguageFromKeyPipe;
   <%_ } _%>
 <%_ } _%>
-    let mockAccountService: AccountService;
-    const routerEventsSubject = new Subject<RouterEvent>();
-    const routerState: any = { snapshot: { root: { data: {} } } };
-    class MockRouter {
-      events = routerEventsSubject;
-      routerState = routerState;
-    }
-
-    beforeEach(
-      waitForAsync(() => {
-        TestBed.configureTestingModule({
+  let mockAccountService: AccountService;
+  const routerEventsSubject = new Subject<RouterEvent>();
+  const routerState: any = { snapshot: { root: { data: {} } } };
+  class MockRouter {
+    events = routerEventsSubject;
+    routerState = routerState;
+  }
+
+  beforeEach(
+    waitForAsync(() => {
+      TestBed.configureTestingModule({
 <%_ if (enableTranslation) { _%>
-          imports: [TranslateModule.forRoot()],
+        imports: [TranslateModule.forRoot()],
 <%_ } _%>
-          declarations: [MainComponent],
-          providers: [
-            Title,
+        declarations: [MainComponent],
+        providers: [
+          Title,
 <%_ if (enableI18nRTL) { _%>
-            FindLanguageFromKeyPipe,
+          FindLanguageFromKeyPipe,
 <%_ } _%>
-            AccountService,
-            {
-              provide: Router,
-              useClass: MockRouter,
-            },
-          ],
-        })
-          .overrideTemplate(MainComponent, '')
-          .compileComponents();
+          AccountService,
+          {
+            provide: Router,
+            useClass: MockRouter,
+          },
+        ],
       })
-    );
-
-    beforeEach(() => {
-      fixture = TestBed.createComponent(MainComponent);
-      comp = fixture.componentInstance;
-      titleService = TestBed.inject(Title);
+        .overrideTemplate(MainComponent, '')
+        .compileComponents();
+    })
+  );
+
+  beforeEach(() => {
+    fixture = TestBed.createComponent(MainComponent);
+    comp = fixture.componentInstance;
+    titleService = TestBed.inject(Title);
 <%_ if (enableTranslation) { _%>
-      translateService = TestBed.inject(TranslateService);
+    translateService = TestBed.inject(TranslateService);
   <%_ if (enableI18nRTL) { _%>
-      findLanguageFromKeyPipe = TestBed.inject(FindLanguageFromKeyPipe);
+    findLanguageFromKeyPipe = TestBed.inject(FindLanguageFromKeyPipe);
   <%_ } _%>
 <%_ } _%>
-      mockAccountService = TestBed.inject(AccountService);
-      mockAccountService.identity = jest.fn(() => of(null));
-      mockAccountService.getAuthenticationState = jest.fn(() => of(null));
-    });
+    mockAccountService = TestBed.inject(AccountService);
+    mockAccountService.identity = jest.fn(() => of(null));
+    mockAccountService.getAuthenticationState = jest.fn(() => of(null));
+  });
 
-    describe('page title', () => {
-      const defaultPageTitle = '<% if (enableTranslation) { %>global.title<% } else { %><%= humanizedBaseName %><% } %>';
-      const parentRoutePageTitle = 'parentTitle';
-      const childRoutePageTitle = 'childTitle';
-      const navigationEnd = new NavigationEnd(1, '', '');
-      const navigationStart = new NavigationStart(1, '');
+  describe('page title', () => {
+    const defaultPageTitle = '<% if (enableTranslation) { %>global.title<% } else { %><%= humanizedBaseName %><% } %>';
+    const parentRoutePageTitle = 'parentTitle';
+    const childRoutePageTitle = 'childTitle';
+    const navigationEnd = new NavigationEnd(1, '', '');
+    const navigationStart = new NavigationStart(1, '');
 <%_ if (enableTranslation) { _%>
-      const langChangeEvent: LangChangeEvent = { lang: '<%= nativeLanguage %>', translations: null };
+    const langChangeEvent: LangChangeEvent = { lang: '<%= nativeLanguage %>', translations: null };
 <%_ } _%>
 
-      beforeEach(() => {
-        routerState.snapshot.root = { data: {} };
+    beforeEach(() => {
+      routerState.snapshot.root = { data: {} };
 <%_ if (enableTranslation) { _%>
-        jest.spyOn(translateService, 'get').mockImplementation((key: string | string[]) => of(`${key as string} translated`));
-        translateService.currentLang = '<%= nativeLanguage %>';
+      jest.spyOn(translateService, 'get').mockImplementation((key: string | string[]) => of(`${key as string} translated`));
+      translateService.currentLang = '<%= nativeLanguage %>';
 <%_ } _%>
-        jest.spyOn(titleService, 'setTitle');
-        comp.ngOnInit();
-      });
+      jest.spyOn(titleService, 'setTitle');
+      comp.ngOnInit();
+    });
 
-      describe('navigation end', () => {
-        it('should set page title to default title if pageTitle is missing on routes', () => {
-          // WHEN
-          routerEventsSubject.next(navigationEnd);
+    describe('navigation end', () => {
+      it('should set page title to default title if pageTitle is missing on routes', () => {
+        // WHEN
+        routerEventsSubject.next(navigationEnd);
 
-          // THEN
+        // THEN
 <%_ if (enableTranslation) { _%>
-          expect(translateService.get).toHaveBeenCalledWith(defaultPageTitle);
+        expect(translateService.get).toHaveBeenCalledWith(defaultPageTitle);
 <%_ } _%>
-          expect(titleService.setTitle).toHaveBeenCalledWith(defaultPageTitle<% if (enableTranslation) { %> + ' translated'<% } %>);
-        });
+        expect(titleService.setTitle).toHaveBeenCalledWith(defaultPageTitle<% if (enableTranslation) { %> + ' translated'<% } %>);
+      });
 
-        it('should set page title to root route pageTitle if there is no child routes', () => {
-          // GIVEN
-          routerState.snapshot.root.data = { pageTitle: parentRoutePageTitle };
+      it('should set page title to root route pageTitle if there is no child routes', () => {
+        // GIVEN
+        routerState.snapshot.root.data = { pageTitle: parentRoutePageTitle };
 
-          // WHEN
-          routerEventsSubject.next(navigationEnd);
+        // WHEN
+        routerEventsSubject.next(navigationEnd);
 
-          // THEN
+        // THEN
 <%_ if (enableTranslation) { _%>
-          expect(translateService.get).toHaveBeenCalledWith(parentRoutePageTitle);
+        expect(translateService.get).toHaveBeenCalledWith(parentRoutePageTitle);
 <%_ } _%>
-          expect(titleService.setTitle).toHaveBeenCalledWith(parentRoutePageTitle<% if (enableTranslation) { %> + ' translated'<% } %>);
-        });
+        expect(titleService.setTitle).toHaveBeenCalledWith(parentRoutePageTitle<% if (enableTranslation) { %> + ' translated'<% } %>);
+      });
 
-        it('should set page title to child route pageTitle if child routes exist and pageTitle is set for child route', () => {
-          // GIVEN
-          routerState.snapshot.root.data = { pageTitle: parentRoutePageTitle };
-          routerState.snapshot.root.firstChild = { data: { pageTitle: childRoutePageTitle } };
+      it('should set page title to child route pageTitle if child routes exist and pageTitle is set for child route', () => {
+        // GIVEN
+        routerState.snapshot.root.data = { pageTitle: parentRoutePageTitle };
+        routerState.snapshot.root.firstChild = { data: { pageTitle: childRoutePageTitle } };
 
-          // WHEN
-          routerEventsSubject.next(navigationEnd);
+        // WHEN
+        routerEventsSubject.next(navigationEnd);
 
-          // THEN
+        // THEN
 <%_ if (enableTranslation) { _%>
-          expect(translateService.get).toHaveBeenCalledWith(childRoutePageTitle);
+        expect(translateService.get).toHaveBeenCalledWith(childRoutePageTitle);
 <%_ } _%>
-          expect(titleService.setTitle).toHaveBeenCalledWith(childRoutePageTitle<% if (enableTranslation) { %> + ' translated'<% } %>);
-        });
+        expect(titleService.setTitle).toHaveBeenCalledWith(childRoutePageTitle<% if (enableTranslation) { %> + ' translated'<% } %>);
+      });
 
-        it('should set page title to parent route pageTitle if child routes exists but pageTitle is not set for child route data', () => {
-          // GIVEN
-          routerState.snapshot.root.data = { pageTitle: parentRoutePageTitle };
-          routerState.snapshot.root.firstChild = { data: {} };
+      it('should set page title to parent route pageTitle if child routes exists but pageTitle is not set for child route data', () => {
+        // GIVEN
+        routerState.snapshot.root.data = { pageTitle: parentRoutePageTitle };
+        routerState.snapshot.root.firstChild = { data: {} };
 
-          // WHEN
-          routerEventsSubject.next(navigationEnd);
+        // WHEN
+        routerEventsSubject.next(navigationEnd);
 
-          // THEN
+        // THEN
 <%_ if (enableTranslation) { _%>
-          expect(translateService.get).toHaveBeenCalledWith(parentRoutePageTitle);
+        expect(translateService.get).toHaveBeenCalledWith(parentRoutePageTitle);
 <%_ } _%>
-          expect(titleService.setTitle).toHaveBeenCalledWith(parentRoutePageTitle<% if (enableTranslation) { %> + ' translated'<% } %>);
-        });
+        expect(titleService.setTitle).toHaveBeenCalledWith(parentRoutePageTitle<% if (enableTranslation) { %> + ' translated'<% } %>);
       });
+    });
 
-      describe('navigation start', () => {
-        it('should not set page title on navigation start', () => {
-          // WHEN
-          routerEventsSubject.next(navigationStart);
+    describe('navigation start', () => {
+      it('should not set page title on navigation start', () => {
+        // WHEN
+        routerEventsSubject.next(navigationStart);
 
-          // THEN
-          expect(titleService.setTitle).not.toHaveBeenCalled();
-        });
+        // THEN
+        expect(titleService.setTitle).not.toHaveBeenCalled();
       });
+    });
 
 <%_ if (enableTranslation) { _%>
-      describe('language change', () => {
-        it('should set page title to default title if pageTitle is missing on routes', () => {
-          // WHEN
-          translateService.onLangChange.emit(langChangeEvent);
-
-          // THEN
-          expect(translateService.get).toHaveBeenCalledWith(defaultPageTitle);
-          expect(titleService.setTitle).toHaveBeenCalledWith(defaultPageTitle + ' translated');
-        });
-
-        it('should set page title to root route pageTitle if there is no child routes', () => {
-          // GIVEN
-          routerState.snapshot.root.data = { pageTitle: parentRoutePageTitle };
-
-          // WHEN
-          translateService.onLangChange.emit(langChangeEvent);
-
-          // THEN
-          expect(translateService.get).toHaveBeenCalledWith(parentRoutePageTitle);
-          expect(titleService.setTitle).toHaveBeenCalledWith(parentRoutePageTitle + ' translated');
-        });
-
-        it('should set page title to child route pageTitle if child routes exist and pageTitle is set for child route', () => {
-          // GIVEN
-          routerState.snapshot.root.data = { pageTitle: parentRoutePageTitle };
-          routerState.snapshot.root.firstChild = { data: { pageTitle: childRoutePageTitle } };
-
-          // WHEN
-          translateService.onLangChange.emit(langChangeEvent);
-
-          // THEN
-          expect(translateService.get).toHaveBeenCalledWith(childRoutePageTitle);
-          expect(titleService.setTitle).toHaveBeenCalledWith(childRoutePageTitle + ' translated');
-        });
-
-        it('should set page title to parent route pageTitle if child routes exists but pageTitle is not set for child route data', () => {
-          // GIVEN
-          routerState.snapshot.root.data = { pageTitle: parentRoutePageTitle };
-          routerState.snapshot.root.firstChild = { data: {} };
-
-          // WHEN
-          translateService.onLangChange.emit(langChangeEvent);
-
-          // THEN
-          expect(translateService.get).toHaveBeenCalledWith(parentRoutePageTitle);
-          expect(titleService.setTitle).toHaveBeenCalledWith(parentRoutePageTitle + ' translated');
-        });
+    describe('language change', () => {
+      it('should set page title to default title if pageTitle is missing on routes', () => {
+        // WHEN
+        translateService.onLangChange.emit(langChangeEvent);
+
+        // THEN
+        expect(translateService.get).toHaveBeenCalledWith(defaultPageTitle);
+        expect(titleService.setTitle).toHaveBeenCalledWith(defaultPageTitle + ' translated');
+      });
+
+      it('should set page title to root route pageTitle if there is no child routes', () => {
+        // GIVEN
+        routerState.snapshot.root.data = { pageTitle: parentRoutePageTitle };
+
+        // WHEN
+        translateService.onLangChange.emit(langChangeEvent);
+
+        // THEN
+        expect(translateService.get).toHaveBeenCalledWith(parentRoutePageTitle);
+        expect(titleService.setTitle).toHaveBeenCalledWith(parentRoutePageTitle + ' translated');
+      });
+
+      it('should set page title to child route pageTitle if child routes exist and pageTitle is set for child route', () => {
+        // GIVEN
+        routerState.snapshot.root.data = { pageTitle: parentRoutePageTitle };
+        routerState.snapshot.root.firstChild = { data: { pageTitle: childRoutePageTitle } };
+
+        // WHEN
+        translateService.onLangChange.emit(langChangeEvent);
+
+        // THEN
+        expect(translateService.get).toHaveBeenCalledWith(childRoutePageTitle);
+        expect(titleService.setTitle).toHaveBeenCalledWith(childRoutePageTitle + ' translated');
       });
-<%_ } _%>
-    });
-<%_ if (enableTranslation) { _%>
 
-    describe('page language attribute', () => {
-      it('should change page language attribute on language change', () => {
+      it('should set page title to parent route pageTitle if child routes exists but pageTitle is not set for child route data', () => {
         // GIVEN
-        comp.ngOnInit();
+        routerState.snapshot.root.data = { pageTitle: parentRoutePageTitle };
+        routerState.snapshot.root.firstChild = { data: {} };
 
         // WHEN
+        translateService.onLangChange.emit(langChangeEvent);
+
+        // THEN
+        expect(translateService.get).toHaveBeenCalledWith(parentRoutePageTitle);
+        expect(titleService.setTitle).toHaveBeenCalledWith(parentRoutePageTitle + ' translated');
+      });
+    });
+<%_ } _%>
+  });
+<%_ if (enableTranslation) { _%>
+
+  describe('page language attribute', () => {
+    it('should change page language attribute on language change', () => {
+      // GIVEN
+      comp.ngOnInit();
+
+      // WHEN
   <%_ if (enableI18nRTL) { _%>
-        findLanguageFromKeyPipe.isRTL = jest.fn(() => false);
+      findLanguageFromKeyPipe.isRTL = jest.fn(() => false);
   <%_ } _%>
-        translateService.onLangChange.emit({ lang: 'lang1', translations: null });
+      translateService.onLangChange.emit({ lang: 'lang1', translations: null });
 
-        // THEN
-        expect(document.querySelector('html')?.getAttribute('lang')).toEqual('lang1');
+      // THEN
+      expect(document.querySelector('html')?.getAttribute('lang')).toEqual('lang1');
   <%_ if (enableI18nRTL) { _%>
-        expect(document.querySelector('html')?.getAttribute('dir')).toEqual('ltr');
+      expect(document.querySelector('html')?.getAttribute('dir')).toEqual('ltr');
   <%_ } _%>
 
-        // WHEN
+      // WHEN
   <%_ if (enableI18nRTL) { _%>
-        findLanguageFromKeyPipe.isRTL = jest.fn(() => true);
+      findLanguageFromKeyPipe.isRTL = jest.fn(() => true);
   <%_ } _%>
-        translateService.onLangChange.emit({ lang: 'lang2', translations: null });
+      translateService.onLangChange.emit({ lang: 'lang2', translations: null });
 
-        // THEN
-        expect(document.querySelector('html')?.getAttribute('lang')).toEqual('lang2');
+      // THEN
+      expect(document.querySelector('html')?.getAttribute('lang')).toEqual('lang2');
   <%_ if (enableI18nRTL) { _%>
-        expect(document.querySelector('html')?.getAttribute('dir')).toEqual('rtl');
+      expect(document.querySelector('html')?.getAttribute('dir')).toEqual('rtl');
   <%_ } _%>
-      });
     });
-<%_ } _%>
   });
+<%_ } _%>
 });