@spinnaker/core 0.14.2 → 0.15.0

package/src/pipeline/status/Artifact.spec.tsx

@@ -1,11 +1,12 @@
-import React from 'react';
-import { ShallowWrapper, shallow } from 'enzyme';
 import { mock } from 'angular';
-import { REACT_MODULE } from '../../reactShims';
-
-import { IArtifact } from '../../domain';
+import type { ShallowWrapper } from 'enzyme';
+import { shallow } from 'enzyme';
+import React from 'react';
 
-import { Artifact, IArtifactProps } from './Artifact';
+import type { IArtifactProps } from './Artifact';
+import { Artifact } from './Artifact';
+import type { IArtifact } from '../../domain';
+import { REACT_MODULE } from '../../reactShims';
 
 const ARTIFACT_TYPE = 'docker/image';
 const ARTIFACT_NAME = 'example.com/container';

package/src/pipeline/status/ArtifactList.spec.tsx

@@ -1,12 +1,13 @@
-import React from 'react';
-import { ShallowWrapper, shallow } from 'enzyme';
 import { mock } from 'angular';
-import { REACT_MODULE } from '../../reactShims';
-
-import { IArtifact } from '../../domain';
+import type { ShallowWrapper } from 'enzyme';
+import { shallow } from 'enzyme';
+import React from 'react';
 
 import { Artifact } from './Artifact';
-import { ArtifactList, IArtifactListProps } from './ArtifactList';
+import type { IArtifactListProps } from './ArtifactList';
+import { ArtifactList } from './ArtifactList';
+import type { IArtifact } from '../../domain';
+import { REACT_MODULE } from '../../reactShims';
 
 const ARTIFACT_TYPE = 'docker/image';
 const ARTIFACT_NAME = 'example.com/container';

package/src/pipeline/status/ExecutionParameters.spec.tsx

@@ -1,11 +1,12 @@
-import React from 'react';
-import { ShallowWrapper, shallow } from 'enzyme';
 import { mock } from 'angular';
+import type { ShallowWrapper } from 'enzyme';
+import { shallow } from 'enzyme';
+import React from 'react';
 
+import type { IDisplayableParameter, IExecutionParametersProps } from './ExecutionParameters';
+import { ExecutionParameters } from './ExecutionParameters';
 import { REACT_MODULE } from '../../reactShims';
 
-import { IExecutionParametersProps, ExecutionParameters, IDisplayableParameter } from './ExecutionParameters';
-
 describe('<ExecutionParameters/>', () => {
   let component: ShallowWrapper<IExecutionParametersProps>;
 

package/src/pipeline/status/ResolvedArtifactList.spec.tsx

@@ -1,12 +1,13 @@
-import React from 'react';
-import { ShallowWrapper, shallow } from 'enzyme';
 import { mock } from 'angular';
-import { REACT_MODULE } from '../../reactShims';
+import type { ShallowWrapper } from 'enzyme';
+import { shallow } from 'enzyme';
+import React from 'react';
 
-import { IArtifact, IExpectedArtifact } from '../../domain';
 import { Artifact } from './Artifact';
-
-import { ResolvedArtifactList, IResolvedArtifactListProps } from './ResolvedArtifactList';
+import type { IResolvedArtifactListProps } from './ResolvedArtifactList';
+import { ResolvedArtifactList } from './ResolvedArtifactList';
+import type { IArtifact, IExpectedArtifact } from '../../domain';
+import { REACT_MODULE } from '../../reactShims';
 
 const ARTIFACT_TYPE = 'docker/image';
 const ARTIFACT_NAME = 'example.com/container';

package/src/plugins/deck.plugin.spec.ts

@@ -1,7 +1,9 @@
-import { SearchResultType, searchResultTypeRegistry } from '../search/searchResult';
-import { IDeckPlugin, registerPluginExtensions } from './deck.plugin';
+import type { IDeckPlugin } from './deck.plugin';
+import { registerPluginExtensions } from './deck.plugin';
 import { HelpContentsRegistry } from '../help';
 import { Registry } from '../registry';
+import type { SearchResultType } from '../search/searchResult';
+import { searchResultTypeRegistry } from '../search/searchResult';
 
 describe('deck plugin registerPluginExtensions', () => {
   it('returns a promise', async () => {

package/src/plugins/plugin.registry.spec.ts

@@ -1,9 +1,11 @@
-import { IStageTypeConfig } from '../domain';
-import { IDeckPlugin } from './deck.plugin';
+import { mock } from 'angular';
+
 import { RequestBuilder } from '../api';
-import { IPluginMetaData, PluginRegistry } from './plugin.registry';
+import type { IDeckPlugin } from './deck.plugin';
+import type { IStageTypeConfig } from '../domain';
+import type { IPluginMetaData } from './plugin.registry';
+import { PluginRegistry } from './plugin.registry';
 import { Registry } from '../registry';
-import { mock } from 'angular';
 
 const fakePromise = (result: any = undefined) => () => new Promise((resolve) => resolve(result));
 

package/src/presentation/forms/SpinFormik.spec.tsx

@@ -1,6 +1,7 @@
-import React from 'react';
 import { mount } from 'enzyme';
-import { FormikProps } from 'formik';
+import type { FormikProps } from 'formik';
+import React from 'react';
+
 import { SpinFormik } from './SpinFormik';
 
 describe('SpinFormik', () => {

package/src/presentation/forms/fields/FormikFormField.spec.tsx

@@ -1,11 +1,11 @@
-import { FormikProps } from 'formik/dist/types';
-import React from 'react';
 import { mount } from 'enzyme';
+import type { FormikProps } from 'formik/dist/types';
+import React from 'react';
 
+import type { IFormInputProps } from '../..';
 import {
   FormikFormField,
   FormikSpelContextProvider,
-  IFormInputProps,
   ReactSelectInput,
   SpelToggle,
   SpinFormik,

package/src/presentation/forms/hooks/useSaveRestoreMutuallyExclusiveFields.hook.spec.tsx

@@ -1,10 +1,11 @@
 import { mount } from 'enzyme';
-import { useSaveRestoreMutuallyExclusiveFields } from './useSaveRestoreMutuallyExclusiveFields.hook';
-import { Formik, FormikProps } from 'formik';
+import type { Formik, FormikProps } from 'formik';
 import React from 'react';
+
+import { SpinFormik } from '../SpinFormik';
 import { FormikFormField } from '../fields';
 import { SelectInput } from '../inputs';
-import { SpinFormik } from '../SpinFormik';
+import { useSaveRestoreMutuallyExclusiveFields } from './useSaveRestoreMutuallyExclusiveFields.hook';
 
 function PizzaComponent() {
   return (

package/src/presentation/forms/inputs/ChecklistInput.spec.tsx

@@ -1,5 +1,5 @@
-import React from 'react';
 import { mount } from 'enzyme';
+import React from 'react';
 
 import { ChecklistInput } from './ChecklistInput';
 

package/src/presentation/forms/inputs/RadioButtonInput.spec.tsx

@@ -1,5 +1,6 @@
-import React from 'react';
 import { mount } from 'enzyme';
+import React from 'react';
+
 import { RadioButtonInput } from './RadioButtonInput';
 
 const noop = () => {};

package/src/presentation/forms/inputs/SelectInput.spec.tsx

@@ -1,5 +1,6 @@
-import React from 'react';
 import { mount } from 'enzyme';
+import React from 'react';
+
 import { SelectInput } from './SelectInput';
 
 const noop = () => {};

package/src/presentation/forms/inputs/hooks/useInternalValidator.hook.spec.tsx

@@ -1,7 +1,8 @@
-import React from 'react';
 import { mount } from 'enzyme';
-import { IValidator } from '../../../forms/validation';
-import { IFormInputProps, IFormInputValidation } from '../interface';
+import React from 'react';
+
+import type { IValidator } from '../../../forms/validation';
+import type { IFormInputProps, IFormInputValidation } from '../interface';
 import { useInternalValidator } from './useInternalValidator.hook';
 
 function TestInputComponent(props: IFormInputProps & { validator?: IValidator; revalidateDeps?: any[] }) {

package/src/presentation/forms/validation/FormValidator.spec.ts

@@ -1,6 +1,6 @@
-import { Validators } from './validators';
 import { FormValidator } from './FormValidator';
-import { IValidator, IArrayItemValidator } from './validation';
+import type { IArrayItemValidator, IValidator } from './validation';
+import { Validators } from './validators';
 
 const { maxValue, arrayNotEmpty } = Validators;
 

package/src/presentation/forms/validation/useValidationData.spec.tsx

@@ -1,8 +1,9 @@
-import { asyncMessage, errorMessage, infoMessage, messageMessage, successMessage, warningMessage } from './categories';
+import { mount } from 'enzyme';
 import React from 'react';
 
-import { mount } from 'enzyme';
-import { IValidationData, useValidationData } from './useValidationData';
+import { asyncMessage, errorMessage, infoMessage, messageMessage, successMessage, warningMessage } from './categories';
+import type { IValidationData } from './useValidationData';
+import { useValidationData } from './useValidationData';
 
 interface IComponentProps {
   validationMessage: React.ReactNode;

package/src/presentation/hooks/useContainerClassNames.hook.spec.tsx

@@ -1,5 +1,6 @@
-import React from 'react';
 import { mount } from 'enzyme';
+import React from 'react';
+
 import { useContainerClassNames } from './useContainerClassNames.hook';
 
 const TestComponent = ({ classNames }: { classNames: string[] }) => {

package/src/presentation/hooks/useData.hook.spec.tsx

@@ -1,7 +1,8 @@
-import React from 'react';
 import { mount } from 'enzyme';
-import { IUseLatestPromiseResult } from './useLatestPromise.hook';
+import React from 'react';
+
 import { useData } from './useData.hook';
+import type { IUseLatestPromiseResult } from './useLatestPromise.hook';
 
 describe('useData hook', () => {
   function TestComponent(props: any) {

package/src/presentation/hooks/useDebouncedValue.hook.spec.tsx

@@ -1,7 +1,7 @@
+import { mount } from 'enzyme';
 import React from 'react';
 
 import { useDebouncedValue } from './useDebouncedValue.hook';
-import { mount } from 'enzyme';
 
 describe('useDebouncedValue hook', () => {
   beforeEach(() => jasmine.clock().install());

package/src/presentation/hooks/useDeepObjectDiff.hook.spec.tsx

@@ -1,5 +1,6 @@
-import React from 'react';
 import { mount } from 'enzyme';
+import React from 'react';
+
 import { useDeepObjectDiff } from './useDeepObjectDiff.hook';
 
 const { useEffect } = React;

package/src/presentation/hooks/useEventListener.hook.spec.tsx

@@ -1,5 +1,6 @@
-import React from 'react';
 import { mount } from 'enzyme';
+import React from 'react';
+
 import { useEventListener } from './useEventListener.hook';
 
 const TestComponent = ({

package/src/presentation/hooks/useForceUpdate.hook.spec.tsx

@@ -1,6 +1,7 @@
-import React from 'react';
 import { mount } from 'enzyme';
+import React from 'react';
 import { Subject } from 'rxjs';
+
 import { useForceUpdate } from '..';
 
 describe('useForceUpdate', () => {

package/src/presentation/hooks/useInterval.hook.spec.tsx

@@ -1,5 +1,6 @@
-import React from 'react';
 import { mount } from 'enzyme';
+import React from 'react';
+
 import { useInterval } from './useInterval.hook';
 
 describe('useInterval hook', () => {

package/src/presentation/hooks/useIsMountedRef.hook.spec.tsx

@@ -1,7 +1,7 @@
+import { mount } from 'enzyme';
 import React from 'react';
 
 import { useIsMountedRef } from '..';
-import { mount } from 'enzyme';
 
 describe('useIsMountedRef hook', () => {
   let useIsMountedRefSpy: jasmine.Spy;

package/src/presentation/hooks/useLatestCallback.hook.spec.tsx

@@ -1,5 +1,6 @@
-import React from 'react';
 import { mount } from 'enzyme';
+import React from 'react';
+
 import { useLatestCallback } from './useLatestCallback.hook';
 
 const TestComponent = ({

package/src/presentation/hooks/useLatestPromise.hook.spec.tsx

@@ -1,6 +1,8 @@
-import React from 'react';
 import { mount } from 'enzyme';
-import { IUseLatestPromiseResult, useLatestPromise } from './useLatestPromise.hook';
+import React from 'react';
+
+import type { IUseLatestPromiseResult } from './useLatestPromise.hook';
+import { useLatestPromise } from './useLatestPromise.hook';
 
 describe('useLatestPromise hook', () => {
   // Remove the the refresh function for .isEqual assertions
@@ -68,9 +70,13 @@ describe('useLatestPromise hook', () => {
     expect(spy).toHaveBeenCalledTimes(2);
 
     deferred.reject('error');
+    let caught = false;
     try {
       await deferred.promise;
-    } catch (error) {}
+    } catch (error) {
+      caught = true;
+    }
+    expect(caught).toBe(true);
     component.setProps({});
 
     expect(spy).toHaveBeenCalledTimes(3);

package/src/presentation/hooks/useMountStatusRef.hook.spec.tsx

@@ -1,5 +1,7 @@
-import React, { RefObject } from 'react';
 import { mount } from 'enzyme';
+import type { RefObject } from 'react';
+import React from 'react';
+
 import { useMountStatusRef } from './useMountStatusRef.hook';
 
 describe('useMountStatusRef', () => {

package/src/presentation/hooks/usePollingData.hook.spec.tsx

@@ -1,6 +1,7 @@
-import React, { useEffect } from 'react';
 import { mount } from 'enzyme';
-import { IUseLatestPromiseResult } from './useLatestPromise.hook';
+import React, { useEffect } from 'react';
+
+import type { IUseLatestPromiseResult } from './useLatestPromise.hook';
 import { usePollingData } from './usePollingData.hook';
 
 describe('usePollingData hook', () => {

package/src/presentation/hooks/usePrevious.hook.spec.tsx

@@ -1,5 +1,6 @@
-import React from 'react';
 import { mount } from 'enzyme';
+import React from 'react';
+
 import { usePrevious } from './usePrevious.hook';
 
 const TestComponent = ({ value, valueCallback }: { value: any; valueCallback: (value: any) => any }) => {

package/src/presentation/navigation/pageNavigator.component.spec.ts

@@ -1,10 +1,10 @@
-import $ from 'jquery';
+import UI_ROUTER from '@uirouter/angularjs';
 import { mock } from 'angular';
+import $ from 'jquery';
 
+import type { INavigationPage } from './PageNavigationState';
 import { PAGE_NAVIGATOR_COMPONENT } from './pageNavigator.component';
-import { INavigationPage } from './PageNavigationState';
 import { ScrollToService } from '../../utils/scrollTo/scrollTo.service';
-import UI_ROUTER from '@uirouter/angularjs';
 
 describe('Component: Page Navigator', () => {
   let $compile: ng.ICompileService, $scope: ng.IScope, $timeout: ng.ITimeoutService, elem: JQuery;

package/src/presentation/spel/SpelInput.spec.tsx

@@ -1,8 +1,9 @@
-import React from 'react';
 import { mount } from 'enzyme';
-import { IFormInputProps, IStageForSpelPreview, IValidator } from '..';
-import { SpelService } from './SpelService';
+import React from 'react';
+
+import type { IFormInputProps, IStageForSpelPreview, IValidator } from '..';
 import { SpelInput } from './SpelInput';
+import { SpelService } from './SpelService';
 
 function defer() {
   let resolve: Function, reject: Function;
@@ -208,10 +209,14 @@ describe('<SpelInput/>', () => {
       expect(mockValidate).toHaveBeenCalledTimes(1);
       expect(mockValidate.calls.mostRecent().returnValue).toMatch('Async: ');
 
+      let caught = false;
       deferred.reject('something bad happened');
       try {
         await deferred.promise;
-      } catch (error) {}
+      } catch (error) {
+        caught = true;
+      }
+      expect(caught).toBe(true);
       component.setProps({});
 
       expect(mockValidate).toHaveBeenCalledTimes(2);

package/src/presentation/spel/SpelService.spec.ts

@@ -1,5 +1,5 @@
-import { RequestBuilder } from '../../api';
 import { SpelService } from './SpelService';
+import { RequestBuilder } from '../../api';
 
 describe('SpelService', () => {
   it('extracts "result" from the payload', async () => {

package/src/scheduler/SchedulerFactory.spec.ts

@@ -1,5 +1,6 @@
 import * as angular from 'angular';
-import { ITimeoutService, mock } from 'angular';
+import type { ITimeoutService } from 'angular';
+import { mock } from 'angular';
 import { SchedulerFactory } from './SchedulerFactory';
 
 describe('scheduler', function () {

package/src/search/infrastructure/infrastructureSearch.service.ts

@@ -20,8 +20,9 @@ export interface ISearchResultSet<T extends ISearchResult = ISearchResult> {
   query?: string;
 }
 
+export type ISearchResultFormatter = (entry: ISearchResult, fromRoute?: boolean) => string | PromiseLike<string>;
 export interface IProviderResultFormatter {
-  [category: string]: (entry: ISearchResult, fromRoute?: boolean) => string;
+  [category: string]: ISearchResultFormatter;
 }
 
 export class InfrastructureSearcher {
@@ -65,7 +66,7 @@ export class InfrastructureSearcher {
     if (!type) {
       return this.$q.when('');
     }
-    let formatter = type.displayFormatter;
+    let formatter: ISearchResultFormatter = type.displayFormatter;
 
     if (this.providerServiceDelegate.hasDelegate(entry.provider, 'search.resultFormatter')) {
       const providerFormatter: IProviderResultFormatter = this.providerServiceDelegate.getDelegate<

package/src/search/widgets/Filter.spec.tsx

@@ -1,8 +1,10 @@
+import type { ReactWrapper } from 'enzyme';
+import { mount } from 'enzyme';
 import React from 'react';
-import { ReactWrapper, mount } from 'enzyme';
 
-import { IFilterProps, Filter } from './Filter';
-import { IFilterType } from './SearchFilterTypeRegistry';
+import type { IFilterProps } from './Filter';
+import { Filter } from './Filter';
+import type { IFilterType } from './SearchFilterTypeRegistry';
 
 describe('<Filter/>', () => {
   let component: ReactWrapper<IFilterProps, any>;

package/src/search/widgets/Filters.spec.tsx

@@ -1,8 +1,10 @@
+import type { ReactWrapper } from 'enzyme';
+import { mount } from 'enzyme';
 import React from 'react';
-import { ReactWrapper, mount } from 'enzyme';
 
-import { IFilterType } from './SearchFilterTypeRegistry';
-import { IFiltersLayout, IFiltersProps, Filters } from './Filters';
+import type { IFiltersLayout, IFiltersProps } from './Filters';
+import { Filters } from './Filters';
+import type { IFilterType } from './SearchFilterTypeRegistry';
 
 describe('<Filters/>', () => {
   let component: ReactWrapper<IFiltersProps, any>;

package/src/search/widgets/Search.spec.tsx

@@ -1,8 +1,10 @@
+import type { ReactWrapper } from 'enzyme';
+import { mount } from 'enzyme';
 import React from 'react';
-import { ReactWrapper, mount } from 'enzyme';
 
+import type { ISearchProps } from './Search';
+import { Search } from './Search';
 import { SearchFilterTypeRegistry } from './SearchFilterTypeRegistry';
-import { ISearchProps, Search } from './Search';
 
 describe('<Search/>', () => {
   SearchFilterTypeRegistry.register({ key: 'account', name: 'Account' });

package/src/securityGroup/index.ts

@@ -1,3 +1,4 @@
-export * from './securityGroupReader.service';
 export * from './SecurityGroupWriter';
 export * from './label/FirewallLabels';
+export * from './securityGroupReader.service';
+export * from './securityGroupSearchResultType';

package/src/securityGroup/securityGroupReader.service.spec.ts

@@ -1,20 +1,18 @@
-import { mockHttpClient } from '../api/mock/jasmine';
 import { mock } from 'angular';
 
-import { Application } from '../application/application.model';
+import { mockHttpClient } from '../api/mock/jasmine';
+import type { Application } from '../application/application.model';
 import { ApplicationModelBuilder } from '../application/applicationModel.builder';
 import { InfrastructureCaches } from '../cache';
-import { ISecurityGroup } from '../domain';
-import {
+import type { ISecurityGroup } from '../domain';
+import type {
   ISecurityGroupDetail,
-  SECURITY_GROUP_READER,
-  SecurityGroupReader,
   ISecurityGroupsByAccountSourceData,
+  SecurityGroupReader,
 } from './securityGroupReader.service';
-import {
-  SECURITY_GROUP_TRANSFORMER_SERVICE,
-  SecurityGroupTransformerService,
-} from './securityGroupTransformer.service';
+import { SECURITY_GROUP_READER } from './securityGroupReader.service';
+import type { SecurityGroupTransformerService } from './securityGroupTransformer.service';
+import { SECURITY_GROUP_TRANSFORMER_SERVICE } from './securityGroupTransformer.service';
 
 describe('Service: securityGroupReader', function () {
   let $q: ng.IQService, $scope: ng.IRootScopeService, reader: SecurityGroupReader;

package/src/serverGroup/configure/common/deployInitializer.component.spec.ts

@@ -1,6 +1,10 @@
-import { IComponentControllerService, mock } from 'angular';
-import { DeployInitializerController, DEPLOY_INITIALIZER_COMPONENT } from './deployInitializer.component';
-import { Application, ApplicationModelBuilder } from '../../../application';
+import type { IComponentControllerService } from 'angular';
+import { mock } from 'angular';
+
+import type { Application } from '../../../application';
+import { ApplicationModelBuilder } from '../../../application';
+import type { DeployInitializerController } from './deployInitializer.component';
+import { DEPLOY_INITIALIZER_COMPONENT } from './deployInitializer.component';
 
 describe('Component: deployInitializer', () => {
   let ctrl: DeployInitializerController, $componentController: IComponentControllerService, application: Application;

package/src/serverGroup/configure/common/v2instanceArchetypeSelector.component.ts

@@ -54,7 +54,6 @@ class V2InstanceArchetypeSelectorController implements IComponentController {
       $scope.selectedInstanceProfile = null;
     }
     this.command.viewState.instanceProfile = type;
-    this.onProfileChanged && this.onProfileChanged(type);
     $scope.instanceProfiles.forEach((profile: IInstanceTypeCategory) => {
       if (profile.type === type) {
         $scope.selectedInstanceProfile = profile;
@@ -69,6 +68,7 @@ class V2InstanceArchetypeSelectorController implements IComponentController {
         }
       }
     });
+    this.onProfileChanged && this.onProfileChanged(type);
   };
 
   private updateInstanceType = () => {

package/src/serverGroup/details/capacity/CapacityDetailsSection.spec.tsx

@@ -1,5 +1,5 @@
-import React from 'react';
 import { mount } from 'enzyme';
+import React from 'react';
 
 import { CapacityDetailsSection } from './CapacityDetailsSection';
 

package/src/serverGroup/details/scalingActivities/ScalingActivitiesModal.spec.ts

@@ -1,4 +1,5 @@
-import { groupScalingActivities, IScalingEventSummary } from './ScalingActivitiesModal';
+import type { IScalingEventSummary } from './ScalingActivitiesModal';
+import { groupScalingActivities } from './ScalingActivitiesModal';
 
 describe('Group Scaling Activities', () => {
   it('groups activities by cause, parsing availability zone from details, sorted by start date, newest first', () => {

package/src/serverGroup/details/serverGroupWarningMessage.service.spec.ts

@@ -1,8 +1,8 @@
-import { ServerGroupWarningMessageService } from './serverGroupWarningMessage.service';
+import type { Application } from '../../application/application.model';
 import { ApplicationModelBuilder } from '../../application/applicationModel.builder';
-import { IServerGroup } from '../../domain';
-import { Application } from '../../application/application.model';
-import { IConfirmationModalParams } from '../../confirmationModal/confirmationModal.service';
+import type { IConfirmationModalParams } from '../../confirmationModal/confirmationModal.service';
+import type { IServerGroup } from '../../domain';
+import { ServerGroupWarningMessageService } from './serverGroupWarningMessage.service';
 
 describe('ServerGroupWarningMessageService', () => {
   let app: Application, serverGroup: IServerGroup;

package/src/serverGroup/serverGroupWriter.service.spec.ts

@@ -1,15 +1,17 @@
-import { mockHttpClient } from '../api/mock/jasmine';
 import { mock, noop } from 'angular';
-import { MockHttpClient } from '../api/mock/mockHttpClient';
-import { SERVER_GROUP_WRITER, ServerGroupWriter } from './serverGroupWriter.service';
-import {
-  IServerGroupCommand,
-  IServerGroupCommandViewState,
-} from './configure/common/serverGroupCommandBuilder.service';
-import { ITaskCommand } from '../task/taskExecutor';
+
+import { mockHttpClient } from '../api/mock/jasmine';
+import type { MockHttpClient } from '../api/mock/mockHttpClient';
 import { Application } from '../application/application.model';
 import { ApplicationModelBuilder } from '../application/applicationModel.builder';
 import { ApplicationDataSourceRegistry } from '../application/service/ApplicationDataSourceRegistry';
+import type {
+  IServerGroupCommand,
+  IServerGroupCommandViewState,
+} from './configure/common/serverGroupCommandBuilder.service';
+import type { ServerGroupWriter } from './serverGroupWriter.service';
+import { SERVER_GROUP_WRITER } from './serverGroupWriter.service';
+import type { ITaskCommand } from '../task/taskExecutor';
 
 interface IApplicationTask {
   refresh: () => void;

package/src/slack/SlackReader.spec.ts

@@ -1,5 +1,6 @@
+import type { ISlackChannel } from './SlackReader';
+import { SlackReader } from './SlackReader';
 import { mockHttpClient } from '../api/mock/jasmine';
-import { ISlackChannel, SlackReader } from './SlackReader';
 
 const mockSlackChannels: ISlackChannel[] = [
   {