From 962c116bb2156dcfd211af8ef36b78d9a116f13d Mon Sep 17 00:00:00 2001 From: crusader Date: Sat, 19 May 2018 22:06:59 +0900 Subject: [PATCH] ing --- package.json | 7 +- src/app/app-rpc.module.ts | 13 +- .../noauth-probe-page-routing.module.ts | 2 +- .../noauth-probe-page.component.html | 3 +- .../noauth-probe/noauth-probe-page.module.ts | 2 +- src/packages/noauth-probe/component/index.ts | 5 + .../noauth-probe-list.component.html} | 30 ++-- .../noauth-probe-list.component.spec.ts} | 12 +- .../component/noauth-probe-list.component.ts | 42 +++++ src/packages/noauth-probe/container/index.ts | 5 + ...noauth-probe-list-container.component.html | 1 + ...uth-probe-list-container.component.spec.ts | 25 +++ .../noauth-probe-list-container.component.ts | 60 +++++++ .../noauth-probe-logger.module.ts | 0 .../noauth-probe-rpc.module.ts | 0 .../noauth-probe-store.module.ts | 0 .../noauth-probe.constant.ts | 0 .../{noauth => noauth-probe}/noauth.module.ts | 7 +- .../{noauth => noauth-probe}/service/index.ts | 0 .../service/noauth-probe.service.spec.ts | 0 .../service/noauth-probe.service.ts | 0 .../{noauth => noauth-probe}/store/index.ts | 16 +- .../store/noauth-probe/index.ts | 0 .../store/noauth-probe/noauth-probe.action.ts | 0 .../noauth-probe/noauth-probe.effect.spec.ts | 0 .../store/noauth-probe/noauth-probe.effect.ts | 0 .../noauth-probe/noauth-probe.reducer.ts | 28 ++- .../store/noauth-probe/noauth-probe.state.ts | 29 ++++ .../subscriber/index.ts | 0 .../subscriber/noauth-probe.subscriber.ts | 0 src/packages/noauth/component/index.ts | 5 - .../noauth/component/list/list.component.ts | 128 -------------- .../store/noauth-probe/noauth-probe.state.ts | 15 -- .../component/list/filter/filter.component.ts | 3 +- tsconfig.json | 13 +- tslint.json | 159 +++--------------- 36 files changed, 277 insertions(+), 333 deletions(-) create mode 100644 src/packages/noauth-probe/component/index.ts rename src/packages/{noauth/component/list/list.component.html => noauth-probe/component/noauth-probe-list.component.html} (69%) rename src/packages/{noauth/component/list/list.component.spec.ts => noauth-probe/component/noauth-probe-list.component.spec.ts} (51%) create mode 100644 src/packages/noauth-probe/component/noauth-probe-list.component.ts create mode 100644 src/packages/noauth-probe/container/index.ts create mode 100644 src/packages/noauth-probe/container/noauth-probe-list-container.component.html create mode 100644 src/packages/noauth-probe/container/noauth-probe-list-container.component.spec.ts create mode 100644 src/packages/noauth-probe/container/noauth-probe-list-container.component.ts rename src/packages/{noauth => noauth-probe}/noauth-probe-logger.module.ts (100%) rename src/packages/{noauth => noauth-probe}/noauth-probe-rpc.module.ts (100%) rename src/packages/{noauth => noauth-probe}/noauth-probe-store.module.ts (100%) rename src/packages/{noauth => noauth-probe}/noauth-probe.constant.ts (100%) rename src/packages/{noauth => noauth-probe}/noauth.module.ts (78%) rename src/packages/{noauth => noauth-probe}/service/index.ts (100%) rename src/packages/{noauth => noauth-probe}/service/noauth-probe.service.spec.ts (100%) rename src/packages/{noauth => noauth-probe}/service/noauth-probe.service.ts (100%) rename src/packages/{noauth => noauth-probe}/store/index.ts (52%) rename src/packages/{noauth => noauth-probe}/store/noauth-probe/index.ts (100%) rename src/packages/{noauth => noauth-probe}/store/noauth-probe/noauth-probe.action.ts (100%) rename src/packages/{noauth => noauth-probe}/store/noauth-probe/noauth-probe.effect.spec.ts (100%) rename src/packages/{noauth => noauth-probe}/store/noauth-probe/noauth-probe.effect.ts (100%) rename src/packages/{noauth => noauth-probe}/store/noauth-probe/noauth-probe.reducer.ts (83%) create mode 100644 src/packages/noauth-probe/store/noauth-probe/noauth-probe.state.ts rename src/packages/{noauth => noauth-probe}/subscriber/index.ts (100%) rename src/packages/{noauth => noauth-probe}/subscriber/noauth-probe.subscriber.ts (100%) delete mode 100644 src/packages/noauth/component/index.ts delete mode 100644 src/packages/noauth/component/list/list.component.ts delete mode 100644 src/packages/noauth/store/noauth-probe/noauth-probe.state.ts diff --git a/package.json b/package.json index eaf6a7c..4bfb386 100644 --- a/package.json +++ b/package.json @@ -26,14 +26,14 @@ "@loafer/decorator": "^0.0.1", "@loafer/ng-logger": "^0.0.1", "@loafer/ng-rest": "^0.0.1", - "@loafer/ng-rpc": "^0.0.1", + "@loafer/ng-rpc": "^0.0.3", "@ngrx/core": "^1.2.0", "@ngrx/effects": "^5.2.0", "@ngrx/entity": "^5.2.0", "@ngrx/router-store": "^5.2.0", "@ngrx/store": "^5.2.0", "@ngrx/store-devtools": "^5.2.0", - "@overflow/commons-typescript": "^0.0.3", + "@overflow/commons-typescript": "^0.0.5", "angular-l10n": "^4.1.5", "angularx-qrcode": "^1.1.0", "chart.js": "^2.7.2", @@ -69,9 +69,12 @@ "karma-jasmine": "^1.1.1", "karma-jasmine-html-reporter": "^0.2.2", "karma-coverage-istanbul-reporter": "^1.4.2", + "prettier": "^1.12.1", "protractor": "~5.3.1", "ts-node": "~4.0.1", "tslint": "~5.9.1", + "tslint-config-prettier": "^1.12.0", + "tslint-config-standard": "^7.0.0", "typescript": "~2.5.3" }, "description": "This project was generated with [Angular CLI](https://github.com/angular/angular-cli) version 1.3.0.", diff --git a/src/app/app-rpc.module.ts b/src/app/app-rpc.module.ts index 7e7d167..df5e3b5 100644 --- a/src/app/app-rpc.module.ts +++ b/src/app/app-rpc.module.ts @@ -1,5 +1,5 @@ import { NgModule } from '@angular/core'; -import { RPCModule } from '@loafer/ng-rpc'; +import { RPCModule } from '@loafer/ng-rpc/ng-rpc.module'; import { RPC_CODEC, @@ -13,6 +13,13 @@ import { import { environment } from '../environments/environment'; +export function getRPCClientCodec() { + return new JSONRPCClientCodec(); +} + +export function getRPCClientWebsocketRWC(config: RxWebsocketSubjectConfig) { + return new RPCClientWebsocketRWC(config); +} @NgModule({ exports: [ @@ -24,10 +31,10 @@ import { environment } from '../environments/environment'; providers: [ {provide: 'WEBAPP_RPC_CONFIG', useValue: environment.webappRPCConfig}, - {provide: RPC_CODEC, useFactory: () => new JSONRPCClientCodec()}, + {provide: RPC_CODEC, useFactory: getRPCClientCodec}, { provide: RPC_RWC, - useFactory: (config: RxWebsocketSubjectConfig) => new RPCClientWebsocketRWC(config), + useFactory: getRPCClientWebsocketRWC, deps: ['WEBAPP_RPC_CONFIG'] }, diff --git a/src/app/pages/probes/noauth-probe/noauth-probe-page-routing.module.ts b/src/app/pages/probes/noauth-probe/noauth-probe-page-routing.module.ts index 5e7062b..360f666 100644 --- a/src/app/pages/probes/noauth-probe/noauth-probe-page-routing.module.ts +++ b/src/app/pages/probes/noauth-probe/noauth-probe-page-routing.module.ts @@ -2,7 +2,7 @@ import { NgModule } from '@angular/core'; import { Routes, RouterModule } from '@angular/router'; import { NoAuthProbePageComponent } from './noauth-probe-page.component'; import { ListComponent as ProbeListComponent } from 'packages/probe/component/list/list.component'; -import { ListComponent as NoauthListComponent } from 'packages/noauth/component/list/list.component'; +import { NoAuthProbeListComponent } from 'packages/noauth-probe/component/noauth-probe-list.component'; import { DownloadComponent } from 'packages/probe/component/download/download.component'; const routes: Routes = [ diff --git a/src/app/pages/probes/noauth-probe/noauth-probe-page.component.html b/src/app/pages/probes/noauth-probe/noauth-probe-page.component.html index d5187b7..53697d2 100644 --- a/src/app/pages/probes/noauth-probe/noauth-probe-page.component.html +++ b/src/app/pages/probes/noauth-probe/noauth-probe-page.component.html @@ -1,2 +1 @@ -container로 교체 - \ No newline at end of file + \ No newline at end of file diff --git a/src/app/pages/probes/noauth-probe/noauth-probe-page.module.ts b/src/app/pages/probes/noauth-probe/noauth-probe-page.module.ts index 5023f44..6511f2d 100644 --- a/src/app/pages/probes/noauth-probe/noauth-probe-page.module.ts +++ b/src/app/pages/probes/noauth-probe/noauth-probe-page.module.ts @@ -1,6 +1,6 @@ import { NgModule } from '@angular/core'; import { CommonModule } from '@angular/common'; -import { NoauthModule } from 'packages/noauth/noauth.module'; +import { NoauthModule } from 'packages/noauth-probe/noauth.module'; import { NoAuthProbePageComponent } from './noauth-probe-page.component'; import { NoAuthProbePageRoutingModule } from './noauth-probe-page-routing.module'; import { TabbarModule } from 'app/commons/component/layout/tabbar/app.tabbar.module'; diff --git a/src/packages/noauth-probe/component/index.ts b/src/packages/noauth-probe/component/index.ts new file mode 100644 index 0000000..8dc365d --- /dev/null +++ b/src/packages/noauth-probe/component/index.ts @@ -0,0 +1,5 @@ +import { NoAuthProbeListComponent } from './noauth-probe-list.component'; + +export const COMPONENTS = [ + NoAuthProbeListComponent, +]; diff --git a/src/packages/noauth/component/list/list.component.html b/src/packages/noauth-probe/component/noauth-probe-list.component.html similarity index 69% rename from src/packages/noauth/component/list/list.component.html rename to src/packages/noauth-probe/component/noauth-probe-list.component.html index 94066ca..06d6b3c 100644 --- a/src/packages/noauth/component/list/list.component.html +++ b/src/packages/noauth-probe/component/noauth-probe-list.component.html @@ -11,16 +11,16 @@ - + - {{rowData.data.tempProbeKey}} - {{rowData.meta.host.name}} - {{rowData.meta.host.os}} - {{rowData.data.createDate | date: 'dd/MM/yyyy'}} + {{rowData.tempProbeKey}} + {{rowData.descriptions.host.name}} + {{rowData.descriptions.host.os}} + {{rowData.createDate | date: 'dd/MM/yyyy'}} @@ -30,39 +30,39 @@
- Platform: {{rowData.meta.host.platform}} + Platform: {{rowData.descriptions.host.platform}}
- Platform family: {{rowData.meta.host.platformFamily}} + Platform family: {{rowData.descriptions.host.platformFamily}}
- Kernel: {{rowData.meta.host.kernelVersion}} + Kernel: {{rowData.descriptions.host.kernelVersion}}
- HostID: {{rowData.meta.host.hostID}} + HostID: {{rowData.descriptions.host.hostID}}
- NIC: {{rowData.meta.network.name}} + NIC: {{rowData.descriptions.network.name}}
- Network Address: {{rowData.meta.network.address}} + Network Address: {{rowData.descriptions.network.address}}
- Gateway: {{rowData.meta.network.gateway}} + Gateway: {{rowData.descriptions.network.gateway}}
- Mac Address: {{rowData.meta.network.macAddress}} + Mac Address: {{rowData.descriptions.network.macAddress}}
- - + +
diff --git a/src/packages/noauth/component/list/list.component.spec.ts b/src/packages/noauth-probe/component/noauth-probe-list.component.spec.ts similarity index 51% rename from src/packages/noauth/component/list/list.component.spec.ts rename to src/packages/noauth-probe/component/noauth-probe-list.component.spec.ts index beacd15..cc06c94 100644 --- a/src/packages/noauth/component/list/list.component.spec.ts +++ b/src/packages/noauth-probe/component/noauth-probe-list.component.spec.ts @@ -1,20 +1,20 @@ import { async, ComponentFixture, TestBed } from '@angular/core/testing'; -import { ListComponent } from './list.component'; +import { NoAuthProbeListComponent } from './noauth-probe-list.component'; -describe('ListComponent', () => { - let component: ListComponent; - let fixture: ComponentFixture; +describe('NoAuthProbeListComponent', () => { + let component: NoAuthProbeListComponent; + let fixture: ComponentFixture; beforeEach(async(() => { TestBed.configureTestingModule({ - declarations: [ ListComponent ] + declarations: [ NoAuthProbeListComponent ] }) .compileComponents(); })); beforeEach(() => { - fixture = TestBed.createComponent(ListComponent); + fixture = TestBed.createComponent(NoAuthProbeListComponent); component = fixture.componentInstance; fixture.detectChanges(); }); diff --git a/src/packages/noauth-probe/component/noauth-probe-list.component.ts b/src/packages/noauth-probe/component/noauth-probe-list.component.ts new file mode 100644 index 0000000..2e987b6 --- /dev/null +++ b/src/packages/noauth-probe/component/noauth-probe-list.component.ts @@ -0,0 +1,42 @@ +import { Component, Input, Output, EventEmitter, AfterContentInit } from '@angular/core'; +import { NoAuthProbe } from '@overflow/commons-typescript/model/noauth'; +import { ConfirmationService, Message } from 'primeng/primeng'; +import { MessageService } from 'primeng/components/common/messageservice'; + +@Component({ + selector: 'of-noauth-list', + templateUrl: './noauth-probe-list.component.html', + providers: [ConfirmationService, MessageService] +}) +export class NoAuthProbeListComponent { + @Input() noauthProbes: NoAuthProbe[]; + @Output() accept = new EventEmitter(); + @Output() deny = new EventEmitter(); + msgs: Message[]; + + constructor( + private confirmationService: ConfirmationService, + private messageService: MessageService + ) { + } + + onAcceptOrDeny(isAccept: boolean, selected: NoAuthProbe) { + console.log(selected); + this.msgs = []; + const title = isAccept ? + 'Are you sure to accept this Probe?' : 'Are you sure to deny this Probe'; + const message = isAccept ? + 'Start collecting data as a Probe.' : 'It will be permanently deleted.'; + + this.confirmationService.confirm({ + header: title, + message: message, + icon: isAccept ? 'fa-check' : 'fa fa-trash', + accept: () => { + isAccept ? this.accept.emit(selected) : this.deny.emit(selected); + }, + reject: () => { + } + }); + } +} diff --git a/src/packages/noauth-probe/container/index.ts b/src/packages/noauth-probe/container/index.ts new file mode 100644 index 0000000..77e3b4b --- /dev/null +++ b/src/packages/noauth-probe/container/index.ts @@ -0,0 +1,5 @@ +import { NoAuthProbeListContainerComponent } from './noauth-probe-list-container.component'; + +export const CONTAINER_COMPONENTS = [ + NoAuthProbeListContainerComponent, +]; diff --git a/src/packages/noauth-probe/container/noauth-probe-list-container.component.html b/src/packages/noauth-probe/container/noauth-probe-list-container.component.html new file mode 100644 index 0000000..4dc70c5 --- /dev/null +++ b/src/packages/noauth-probe/container/noauth-probe-list-container.component.html @@ -0,0 +1 @@ + diff --git a/src/packages/noauth-probe/container/noauth-probe-list-container.component.spec.ts b/src/packages/noauth-probe/container/noauth-probe-list-container.component.spec.ts new file mode 100644 index 0000000..d2a3436 --- /dev/null +++ b/src/packages/noauth-probe/container/noauth-probe-list-container.component.spec.ts @@ -0,0 +1,25 @@ +import { async, ComponentFixture, TestBed } from '@angular/core/testing'; + +import { NoAuthProbeListContainerComponent } from './noauth-probe-list-container.component'; + +describe('NoAuthProbeListContainerComponent', () => { + let component: NoAuthProbeListContainerComponent; + let fixture: ComponentFixture; + + beforeEach(async(() => { + TestBed.configureTestingModule({ + declarations: [ NoAuthProbeListContainerComponent ] + }) + .compileComponents(); + })); + + beforeEach(() => { + fixture = TestBed.createComponent(NoAuthProbeListContainerComponent); + component = fixture.componentInstance; + fixture.detectChanges(); + }); + + it('should create', () => { + expect(component).toBeTruthy(); + }); +}); diff --git a/src/packages/noauth-probe/container/noauth-probe-list-container.component.ts b/src/packages/noauth-probe/container/noauth-probe-list-container.component.ts new file mode 100644 index 0000000..dc77050 --- /dev/null +++ b/src/packages/noauth-probe/container/noauth-probe-list-container.component.ts @@ -0,0 +1,60 @@ +import { Component, OnInit } from '@angular/core'; +import { AfterContentInit, OnDestroy } from '@angular/core/src/metadata/lifecycle_hooks'; +import { Store, select } from '@ngrx/store'; +import { Observable } from 'rxjs/Observable'; +import 'rxjs/add/operator/map'; + +import { AuthSelector } from 'packages/member/store'; +import { Domain } from '@overflow/commons-typescript/model/domain'; +import * as ListStore from '../store/noauth-probe'; +import { NoAuthProbeSelector } from '../store'; +import { NoAuthProbe } from '@overflow/commons-typescript/model/noauth'; +import { ConfirmationService } from 'primeng/primeng'; +import { MessageService } from 'primeng/components/common/messageservice'; + +@Component({ + selector: 'of-noauth-list-container', + templateUrl: './noauth-probe-list-container.component.html', + providers: [ConfirmationService, MessageService] +}) +export class NoAuthProbeListContainerComponent implements OnInit, AfterContentInit, OnDestroy { + noauthProbes$: Observable; + + constructor( + private store: Store, + ) { + + } + + ngOnInit() { + // this.noauthProbes$ = this.store.pipe(select(NoAuthProbeSelector.select('noAuthProbes'))).map((_noauthProbes: NoAuthProbe[]) => { + this.noauthProbes$ = this.store.pipe(select(NoAuthProbeSelector.selectAll)).map((_noauthProbes: NoAuthProbe[]) => { + if (null === _noauthProbes) { + return null; + } + _noauthProbes.forEach(_noauthProbe => { + _noauthProbe.descriptions = JSON.parse(_noauthProbe.description); + }); + return _noauthProbes; + }); + } + + ngAfterContentInit() { + this.store.select(AuthSelector.select('domain')).subscribe( + (domain: Domain) => { + this.store.dispatch(new ListStore.ReadAllByDomain(domain)); + } + ); + } + + ngOnDestroy() { + } + + accept(selected) { + this.store.dispatch(new ListStore.Accept(selected)); + } + + deny(selected) { + this.store.dispatch(new ListStore.Deny(selected)); + } +} diff --git a/src/packages/noauth/noauth-probe-logger.module.ts b/src/packages/noauth-probe/noauth-probe-logger.module.ts similarity index 100% rename from src/packages/noauth/noauth-probe-logger.module.ts rename to src/packages/noauth-probe/noauth-probe-logger.module.ts diff --git a/src/packages/noauth/noauth-probe-rpc.module.ts b/src/packages/noauth-probe/noauth-probe-rpc.module.ts similarity index 100% rename from src/packages/noauth/noauth-probe-rpc.module.ts rename to src/packages/noauth-probe/noauth-probe-rpc.module.ts diff --git a/src/packages/noauth/noauth-probe-store.module.ts b/src/packages/noauth-probe/noauth-probe-store.module.ts similarity index 100% rename from src/packages/noauth/noauth-probe-store.module.ts rename to src/packages/noauth-probe/noauth-probe-store.module.ts diff --git a/src/packages/noauth/noauth-probe.constant.ts b/src/packages/noauth-probe/noauth-probe.constant.ts similarity index 100% rename from src/packages/noauth/noauth-probe.constant.ts rename to src/packages/noauth-probe/noauth-probe.constant.ts diff --git a/src/packages/noauth/noauth.module.ts b/src/packages/noauth-probe/noauth.module.ts similarity index 78% rename from src/packages/noauth/noauth.module.ts rename to src/packages/noauth-probe/noauth.module.ts index 288ad9f..c7675bb 100644 --- a/src/packages/noauth/noauth.module.ts +++ b/src/packages/noauth-probe/noauth.module.ts @@ -1,11 +1,14 @@ import { NgModule } from '@angular/core'; import { CommonModule } from '@angular/common'; +import { PrimeNGModules } from 'packages/commons/prime-ng/prime-ng.module'; + import { NoAuthProbeStoreModule } from './noauth-probe-store.module'; import { NoAuthProbeRPCModule } from './noauth-probe-rpc.module'; import { NoAuthProbeLoggerModule } from './noauth-probe-logger.module'; import { COMPONENTS } from './component'; +import { CONTAINER_COMPONENTS } from './container'; + import { SERVICES } from './service'; -import { PrimeNGModules } from '../commons/prime-ng/prime-ng.module'; @NgModule({ imports: [ @@ -17,9 +20,11 @@ import { PrimeNGModules } from '../commons/prime-ng/prime-ng.module'; ], declarations: [ COMPONENTS, + CONTAINER_COMPONENTS, ], exports: [ COMPONENTS, + CONTAINER_COMPONENTS, ], providers: [ SERVICES, diff --git a/src/packages/noauth/service/index.ts b/src/packages/noauth-probe/service/index.ts similarity index 100% rename from src/packages/noauth/service/index.ts rename to src/packages/noauth-probe/service/index.ts diff --git a/src/packages/noauth/service/noauth-probe.service.spec.ts b/src/packages/noauth-probe/service/noauth-probe.service.spec.ts similarity index 100% rename from src/packages/noauth/service/noauth-probe.service.spec.ts rename to src/packages/noauth-probe/service/noauth-probe.service.spec.ts diff --git a/src/packages/noauth/service/noauth-probe.service.ts b/src/packages/noauth-probe/service/noauth-probe.service.ts similarity index 100% rename from src/packages/noauth/service/noauth-probe.service.ts rename to src/packages/noauth-probe/service/noauth-probe.service.ts diff --git a/src/packages/noauth/store/index.ts b/src/packages/noauth-probe/store/index.ts similarity index 52% rename from src/packages/noauth/store/index.ts rename to src/packages/noauth-probe/store/index.ts index 44d3653..507df04 100644 --- a/src/packages/noauth/store/index.ts +++ b/src/packages/noauth-probe/store/index.ts @@ -21,9 +21,17 @@ export const EFFECTS = [ NoAuthProbeStore.Effects, ]; -export const selectNoAuthProbeState = createFeatureSelector(MODULE.name); +export const selectFeatureState = createFeatureSelector(MODULE.name); -export const NoAuthProbeSelector = new StateSelector(createSelector( - selectNoAuthProbeState, +export const selectNoAuthProbeState = createSelector( + selectFeatureState, (state: State) => state.noAuthProbe -)); +); + + +// export const NoAuthProbeSelector = new StateSelector(createSelector( +// selectNoAuthProbeState, +// (state: State) => state.noAuthProbe +// )); + +export const NoAuthProbeSelector = NoAuthProbeStore.adapter.getSelectors(selectNoAuthProbeState); diff --git a/src/packages/noauth/store/noauth-probe/index.ts b/src/packages/noauth-probe/store/noauth-probe/index.ts similarity index 100% rename from src/packages/noauth/store/noauth-probe/index.ts rename to src/packages/noauth-probe/store/noauth-probe/index.ts diff --git a/src/packages/noauth/store/noauth-probe/noauth-probe.action.ts b/src/packages/noauth-probe/store/noauth-probe/noauth-probe.action.ts similarity index 100% rename from src/packages/noauth/store/noauth-probe/noauth-probe.action.ts rename to src/packages/noauth-probe/store/noauth-probe/noauth-probe.action.ts diff --git a/src/packages/noauth/store/noauth-probe/noauth-probe.effect.spec.ts b/src/packages/noauth-probe/store/noauth-probe/noauth-probe.effect.spec.ts similarity index 100% rename from src/packages/noauth/store/noauth-probe/noauth-probe.effect.spec.ts rename to src/packages/noauth-probe/store/noauth-probe/noauth-probe.effect.spec.ts diff --git a/src/packages/noauth/store/noauth-probe/noauth-probe.effect.ts b/src/packages/noauth-probe/store/noauth-probe/noauth-probe.effect.ts similarity index 100% rename from src/packages/noauth/store/noauth-probe/noauth-probe.effect.ts rename to src/packages/noauth-probe/store/noauth-probe/noauth-probe.effect.ts diff --git a/src/packages/noauth/store/noauth-probe/noauth-probe.reducer.ts b/src/packages/noauth-probe/store/noauth-probe/noauth-probe.reducer.ts similarity index 83% rename from src/packages/noauth/store/noauth-probe/noauth-probe.reducer.ts rename to src/packages/noauth-probe/store/noauth-probe/noauth-probe.reducer.ts index c3382e0..2fb6c17 100644 --- a/src/packages/noauth/store/noauth-probe/noauth-probe.reducer.ts +++ b/src/packages/noauth-probe/store/noauth-probe/noauth-probe.reducer.ts @@ -15,6 +15,7 @@ import { import { State, initialState, + adapter, } from './noauth-probe.state'; import { NoAuthProbe } from '@overflow/commons-typescript/model/noauth'; @@ -30,21 +31,19 @@ export function reducer(state = initialState, action: Actions): State { } case ActionType.ReadAllByDomainSuccess: { - return { + return adapter.addAll(action.payload, { ...state, error: null, pending: false, - noAuthProbes: action.payload, - }; + }); } case ActionType.ReadAllByDomainFailure: { - return { + return adapter.removeAll({ ...state, error: action.payload, pending: false, - noAuthProbes: null, - }; + }); } case ActionType.Accept: { @@ -56,12 +55,12 @@ export function reducer(state = initialState, action: Actions): State { } case ActionType.AcceptSuccess: { - return { - ...state, + const s = adapter.removeAll(state); + return adapter.addAll(action.payload, { + ...s, error: null, pending: false, - noAuthProbes: action.payload, - }; + }); } case ActionType.AcceptFailure: { @@ -81,12 +80,12 @@ export function reducer(state = initialState, action: Actions): State { } case ActionType.DenySuccess: { - return { - ...state, + const s = adapter.removeAll(state); + return adapter.addAll(action.payload, { + ...s, error: null, pending: false, - noAuthProbes: action.payload, - }; + }); } case ActionType.DenyFailure: { @@ -97,7 +96,6 @@ export function reducer(state = initialState, action: Actions): State { }; } case ActionType.OnConnect: { - return { ...state, }; diff --git a/src/packages/noauth-probe/store/noauth-probe/noauth-probe.state.ts b/src/packages/noauth-probe/store/noauth-probe/noauth-probe.state.ts new file mode 100644 index 0000000..854d7ba --- /dev/null +++ b/src/packages/noauth-probe/store/noauth-probe/noauth-probe.state.ts @@ -0,0 +1,29 @@ +import { EntityState, EntityAdapter, createEntityAdapter } from '@ngrx/entity'; + +import { RPCClientError } from '@loafer/ng-rpc/protocol'; + +import { NoAuthProbe } from '@overflow/commons-typescript/model/noauth'; + +export interface State extends EntityState { + error: RPCClientError | null; + pending: boolean; +} + +export const adapter: EntityAdapter = createEntityAdapter(); + +// export interface State { +// error: RPCClientError | null; +// pending: boolean; +// noAuthProbes: NoAuthProbe[] | null; +// } + +// export const initialState: State = { +// error: null, +// pending: false, +// noAuthProbes: null, +// }; + +export const initialState: State = adapter.getInitialState({ + error: null, + pending: false, +}); diff --git a/src/packages/noauth/subscriber/index.ts b/src/packages/noauth-probe/subscriber/index.ts similarity index 100% rename from src/packages/noauth/subscriber/index.ts rename to src/packages/noauth-probe/subscriber/index.ts diff --git a/src/packages/noauth/subscriber/noauth-probe.subscriber.ts b/src/packages/noauth-probe/subscriber/noauth-probe.subscriber.ts similarity index 100% rename from src/packages/noauth/subscriber/noauth-probe.subscriber.ts rename to src/packages/noauth-probe/subscriber/noauth-probe.subscriber.ts diff --git a/src/packages/noauth/component/index.ts b/src/packages/noauth/component/index.ts deleted file mode 100644 index bd78b54..0000000 --- a/src/packages/noauth/component/index.ts +++ /dev/null @@ -1,5 +0,0 @@ -import { ListComponent } from './list/list.component'; - -export const COMPONENTS = [ - ListComponent, -]; diff --git a/src/packages/noauth/component/list/list.component.ts b/src/packages/noauth/component/list/list.component.ts deleted file mode 100644 index e9ce44d..0000000 --- a/src/packages/noauth/component/list/list.component.ts +++ /dev/null @@ -1,128 +0,0 @@ -import { Component, OnInit, AfterViewInit, ViewChild } from '@angular/core'; -import { AfterContentInit, OnDestroy } from '@angular/core/src/metadata/lifecycle_hooks'; -import { Router } from '@angular/router'; -import { Store, select } from '@ngrx/store'; -import { AuthSelector } from 'packages/member/store'; -import { RPCClientError } from '@loafer/ng-rpc/protocol'; -import { Domain } from '@overflow/commons-typescript/model/domain'; -import * as ListStore from '../../store/noauth-probe'; -import { NoAuthProbeSelector } from '../../store'; -import { NoAuthProbe } from '@overflow/commons-typescript/model/noauth'; -import { ConfirmationService, Message } from 'primeng/primeng'; -import { MessageService } from 'primeng/components/common/messageservice'; -import { Subscription } from 'rxjs/Subscription'; - -class NoauthProbeDesc { - host: { - name: string; - os: string; - platform: string; - platformFamily: string; - platformVersion: string; - kernelVersion: string; - hostID: string; - }; - network: { - name: string; - address: string; - gateway: string; - macAddress: string; - }; -} - -class NoauthProbeResult { - id: number; - data: NoAuthProbe; - meta: NoauthProbeDesc; -} - - -@Component({ - selector: 'of-noauth-list', - templateUrl: './list.component.html', - providers: [ConfirmationService, MessageService] -}) -export class ListComponent implements OnInit, AfterContentInit, OnDestroy { - noAuthProbesSubscription$: Subscription; - noAuthProbes$ = this.store.pipe(select(NoAuthProbeSelector.select('noAuthProbes'))); - noauthProbes: NoauthProbeResult[]; - msgs: Message[]; - - - constructor( - private router: Router, - private store: Store, - private confirmationService: ConfirmationService, - private messageService: MessageService - ) { - } - - ngOnInit() { - this.noAuthProbesSubscription$ = this.noAuthProbes$.subscribe( - (result: NoAuthProbe[]) => { - if (result) { - this.arrangeData(result); - } - }, - (error: RPCClientError) => { - console.log(error.response.message); - } - ); - } - - ngAfterContentInit() { - this.store.select(AuthSelector.select('domain')).subscribe( - (domain: Domain) => { - this.store.dispatch(new ListStore.ReadAllByDomain(domain)); - } - ); - } - - ngOnDestroy() { - this.noAuthProbesSubscription$.unsubscribe(); - } - - arrangeData(list) { - if (!list || list.length === 0) { - return; - } - this.noauthProbes = []; - for (const np of list) { - const desc: NoauthProbeDesc = JSON.parse(np.description); - const result: NoauthProbeResult = { - id: np.id, - data: np, - meta: desc - }; - this.noauthProbes.push(result); - } - } - - onAcceptOrDeny(isAccept: boolean, selected: NoAuthProbe) { - console.log(selected); - this.msgs = []; - const title = isAccept ? - 'Are you sure to accept this Probe?' : 'Are you sure to deny this Probe'; - const message = isAccept ? - 'Start collecting data as a Probe.' : 'It will be permanently deleted.'; - - this.confirmationService.confirm({ - header: title, - message: message, - icon: isAccept ? 'fa-check' : 'fa fa-trash', - accept: () => { - isAccept ? this.handleAccept(selected) : this.handleDeny(selected); - }, - reject: () => { - } - }); - } - - handleAccept(selected) { - this.store.dispatch(new ListStore.Accept(selected)); - } - - handleDeny(selected) { - this.store.dispatch(new ListStore.Deny(selected)); - } -} diff --git a/src/packages/noauth/store/noauth-probe/noauth-probe.state.ts b/src/packages/noauth/store/noauth-probe/noauth-probe.state.ts deleted file mode 100644 index f5d0a4b..0000000 --- a/src/packages/noauth/store/noauth-probe/noauth-probe.state.ts +++ /dev/null @@ -1,15 +0,0 @@ -import { RPCClientError } from '@loafer/ng-rpc/protocol'; - -import { NoAuthProbe } from '@overflow/commons-typescript/model/noauth'; - -export interface State { - error: RPCClientError | null; - pending: boolean; - noAuthProbes: NoAuthProbe[] | null; -} - -export const initialState: State = { - error: null, - pending: false, - noAuthProbes: null, -}; diff --git a/src/packages/sensor/component/list/filter/filter.component.ts b/src/packages/sensor/component/list/filter/filter.component.ts index 9ca9b1f..e3b340f 100644 --- a/src/packages/sensor/component/list/filter/filter.component.ts +++ b/src/packages/sensor/component/list/filter/filter.component.ts @@ -4,8 +4,7 @@ import { Router } from '@angular/router'; @Component({ selector: 'of-target-filter', - templateUrl: './filter.component.html', - styleUrls: ['./filter.component.scss'] + templateUrl: './filter.component.html' }) export class FilterComponent implements OnInit { diff --git a/tsconfig.json b/tsconfig.json index f6106ad..e2ff9fd 100644 --- a/tsconfig.json +++ b/tsconfig.json @@ -19,6 +19,17 @@ "lib": [ "es2017", "dom" - ] + ], + // /* Additional Checks */ + // "noUnusedLocals": true /* Report errors on unused locals. */, + // "noUnusedParameters": true /* Report errors on unused parameters. */, + // "noImplicitReturns": true /* Report error when not all code paths in function return a value. */, + // "noFallthroughCasesInSwitch": true /* Report errors for fallthrough cases in switch statement. */, + + // /* Debugging Options */ + // "traceResolution": false /* Report module resolution log messages. */, + // "listEmittedFiles": false /* Print names of generated files part of the compilation. */, + // "listFiles": false /* Print names of files part of the compilation. */, + // "pretty": true /* Stylize errors and messages using color and context. */, } } diff --git a/tslint.json b/tslint.json index a36a2fe..b954a5d 100644 --- a/tslint.json +++ b/tslint.json @@ -1,139 +1,34 @@ { - "rulesDirectory": [ - "node_modules/codelyzer" - ], + "extends": ["tslint:latest", "tslint-config-prettier", "tslint-immutable"], "rules": { - "arrow-return-shorthand": true, - "callable-types": true, - "class-name": true, - "comment-format": [ - true, - "check-space" - ], - "curly": true, - "eofline": true, - "forin": true, - "import-blacklist": [ - true, - "rxjs" - ], - "import-spacing": true, - "indent": [ - true, - "spaces" - ], - "interface-over-type-literal": true, - "label-position": true, - "max-line-length": [ - true, - 140 - ], - "member-access": false, - "member-ordering": [ - true, - { - "order": [ - "static-field", - "instance-field", - "static-method", - "instance-method" - ] - } - ], - "no-arg": true, - "no-bitwise": true, - "no-console": [ - true, - "debug", - "info", - "time", - "timeEnd", - "trace" - ], - "no-construct": true, - "no-debugger": true, - "no-duplicate-super": true, - "no-empty": false, - "no-empty-interface": true, - "no-eval": true, - "no-inferrable-types": [ - true, - "ignore-params" - ], - "no-misused-new": true, - "no-non-null-assertion": true, - "no-shadowed-variable": true, - "no-string-literal": false, - "no-string-throw": true, - "no-switch-case-fall-through": true, - "no-trailing-whitespace": true, - "no-unnecessary-initializer": true, - "no-unused-expression": true, - "no-use-before-define": ["error", { "functions": true, "classes": true }], + "interface-name": [true, "never-prefix"], + // TODO: allow devDependencies only in **/*.spec.ts files: + // waiting on https://github.com/palantir/tslint/pull/3708 + "no-implicit-dependencies": [true, "dev"], + + /* tslint-immutable rules */ + // Recommended built-in rules "no-var-keyword": true, - "object-literal-sort-keys": false, - "one-line": [ + "no-parameter-reassignment": true, + "typedef": [true, "call-signature"], + + // Immutability rules + "readonly-keyword": true, + "readonly-array": true, + "no-let": true, + "no-object-mutation": true, + "no-delete": true, + "no-method-signature": true, + + // Functional style rules + "no-this": true, + "no-class": true, + "no-mixed-interface": true, + "no-expression-statement": [ true, - "check-open-brace", - "check-catch", - "check-else", - "check-whitespace" + { "ignore-prefix": ["console.", "process.exit"] } ], - "prefer-const": true, - "quotemark": [ - true, - "single" - ], - "radix": true, - "semicolon": [ - true, - "always" - ], - "triple-equals": [ - true, - "allow-null-check" - ], - "typedef-whitespace": [ - true, - { - "call-signature": "nospace", - "index-signature": "nospace", - "parameter": "nospace", - "property-declaration": "nospace", - "variable-declaration": "nospace" - } - ], - "typeof-compare": true, - "unified-signatures": true, - "variable-name": false, - "whitespace": [ - true, - "check-branch", - "check-decl", - "check-operator", - "check-separator", - "check-type" - ], - "directive-selector": [ - true, - "attribute", - "of", - "camelCase" - ], - "component-selector": [ - true, - "element", - "of", - "kebab-case" - ], - "use-input-property-decorator": true, - "use-output-property-decorator": true, - "use-host-property-decorator": true, - "no-input-rename": true, - "no-output-rename": true, - "use-life-cycle-interface": true, - "use-pipe-transform-interface": true, - "component-class-suffix": true, - "directive-class-suffix": true + "no-if-statement": true + /* end tslint-immutable rules */ } }