Skip to content

Commit

Permalink
refactor(@dataflow): interface rename 'FlowInNode' 'FlowOutNode'
Browse files Browse the repository at this point in the history
replace 'BareFlowInNode' as 'FlowInNode'
replace 'BareFlowOutNode' as 'FlowOutNode'

BREAKING CHANGE: replace 'BareFlowInNode' as 'FlowInNode', replace 'BareFlowOutNode' as 'FlowOutNode'
  • Loading branch information
ElonH committed May 17, 2020
1 parent 217700a commit b552309
Show file tree
Hide file tree
Showing 9 changed files with 30 additions and 30 deletions.
6 changes: 3 additions & 3 deletions src/app/@dataflow/core/ajax-flow.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { CacheFlow } from './cache-flow';
import { DataFlowNode, BareFlowInNode, BareFlowOutNode, CombErr } from './bare-flow';
import { DataFlowNode, FlowInNode, FlowOutNode, CombErr } from './bare-flow';
import { Observable, of } from 'rxjs';
import { ajax, AjaxRequest, AjaxResponse } from 'rxjs/ajax';
import { catchError, map } from 'rxjs/operators';
Expand All @@ -8,8 +8,8 @@ import { FlowSupNode } from './superset-flow';
export type AjaxFlowNode = [AjaxResponse | object, Error[]];

export abstract class AjaxFlow<
Tin extends BareFlowInNode,
Tout extends BareFlowOutNode,
Tin extends FlowInNode,
Tout extends FlowOutNode,
Tsup extends FlowSupNode = Tin & Tout
> extends CacheFlow<Tin, Tout, Tsup> {
// protected cacheSupport: boolean;
Expand Down
4 changes: 2 additions & 2 deletions src/app/@dataflow/core/bare-flow.spec.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { BareFlow, DataFlowNode, BareFlowInNode, BareFlowOutNode, CombErr } from './bare-flow';
import { BareFlow, DataFlowNode, FlowInNode, FlowOutNode, CombErr } from './bare-flow';
import { TestScheduler } from 'rxjs/testing';
import { Observable, of } from 'rxjs';

Expand All @@ -19,7 +19,7 @@ describe('BareFlow', () => {
const pre = cold('a----', values);
const expected = 'a----';

const rst = new (class extends BareFlow<BareFlowInNode, BareFlowOutNode> {
const rst = new (class extends BareFlow<FlowInNode, FlowOutNode> {
public prerequest$ = pre;
protected request(pre: DataFlowNode): Observable<DataFlowNode> {
throw new Error('Method not implemented.');
Expand Down
8 changes: 4 additions & 4 deletions src/app/@dataflow/core/bare-flow.ts
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
import { Observable, of } from 'rxjs';
import { switchMap, take, tap, startWith, distinctUntilChanged, skipWhile } from 'rxjs/operators';

export interface BareFlowInNode {}
export interface BareFlowOutNode {}
export type DataFlowNode = [BareFlowInNode, Error[]]; // TODO: drop support
export interface FlowInNode {}
export interface FlowOutNode {}
export type DataFlowNode = [FlowInNode, Error[]]; // TODO: drop support
export type CombErr<T> = [T, Error[]];

export abstract class BareFlow<Tin extends BareFlowInNode, Tout extends BareFlowOutNode> {
export abstract class BareFlow<Tin extends FlowInNode, Tout extends FlowOutNode> {
public abstract prerequest$: Observable<CombErr<Tin>>;
protected abstract request(pre: CombErr<Tin>): Observable<CombErr<Tout>>;
private bareData$: Observable<CombErr<Tout>>;
Expand Down
14 changes: 7 additions & 7 deletions src/app/@dataflow/core/cache-flow.spec.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { CacheFlow } from './cache-flow';
import { TestScheduler } from 'rxjs/testing';
import { DataFlowNode, BareFlowInNode, CombErr, BareFlowOutNode } from './bare-flow';
import { DataFlowNode, FlowInNode, CombErr, FlowOutNode } from './bare-flow';
import { Observable, of } from 'rxjs';

describe('CacheFlow', () => {
Expand All @@ -17,10 +17,10 @@ describe('CacheFlow', () => {
it('prerequest twice(different value), request same value, only output onece', () => {
scheduler.run((helpers) => {
const { cold, hot, expectObservable, expectSubscriptions, flush } = helpers;
interface TestIn extends BareFlowInNode {
interface TestIn extends FlowInNode {
ab: number;
}
interface TestOut extends BareFlowOutNode {
interface TestOut extends FlowOutNode {
k: number;
}
interface TestSup extends TestOut, TestIn {}
Expand Down Expand Up @@ -51,10 +51,10 @@ describe('CacheFlow', () => {
it('check cache validation', () => {
scheduler.run((helpers) => {
const { cold, hot, expectObservable, expectSubscriptions, flush } = helpers;
interface TestIn extends BareFlowInNode {
interface TestIn extends FlowInNode {
ab: number;
}
interface TestOut extends BareFlowOutNode {
interface TestOut extends FlowOutNode {
k: number;
}
interface TestSup extends TestOut, TestIn {}
Expand Down Expand Up @@ -97,10 +97,10 @@ describe('CacheFlow', () => {
it('disable cache', () => {
scheduler.run((helpers) => {
const { cold, hot, expectObservable, expectSubscriptions, flush } = helpers;
interface TestIn extends BareFlowInNode {
interface TestIn extends FlowInNode {
ab: number;
}
interface TestOut extends BareFlowOutNode {
interface TestOut extends FlowOutNode {
cd: number;
}
interface TestSup extends TestOut, TestIn {}
Expand Down
6 changes: 3 additions & 3 deletions src/app/@dataflow/core/cache-flow.ts
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
import { DataFlowNode, BareFlowInNode, BareFlowOutNode, CombErr } from './bare-flow';
import { DataFlowNode, FlowInNode, FlowOutNode, CombErr } from './bare-flow';
import { Observable, iif, of } from 'rxjs';
import { SupersetFlow, FlowSupNode } from './superset-flow';
import { tap, take, map } from 'rxjs/operators';

export abstract class CacheFlow<
Tin extends BareFlowInNode,
Tout extends BareFlowOutNode,
Tin extends FlowInNode,
Tout extends FlowOutNode,
Tsup extends FlowSupNode = Tin & Tout
> extends SupersetFlow<Tin, Tout, Tsup> {
protected abstract requestCache(pre: CombErr<Tin>): Observable<CombErr<Tout>>;
Expand Down
4 changes: 2 additions & 2 deletions src/app/@dataflow/core/superset-flow.spec.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { SupersetFlow } from './superset-flow';
import { TestScheduler } from 'rxjs/testing';
import { DataFlowNode, BareFlowInNode, BareFlowOutNode } from './bare-flow';
import { DataFlowNode, FlowInNode, FlowOutNode } from './bare-flow';
import { Observable, of } from 'rxjs';

describe('SupersetFlow', () => {
Expand All @@ -23,7 +23,7 @@ describe('SupersetFlow', () => {
const pre = cold('a--b-', values);
const expected = 'c--d-';

const rst = new (class extends SupersetFlow<BareFlowInNode, BareFlowOutNode> {
const rst = new (class extends SupersetFlow<FlowInNode, FlowOutNode> {
public prerequest$ = pre;
protected request(pre: DataFlowNode): Observable<DataFlowNode> {
return of([{ cd: pre[0]['ab'] + 1 }, []]);
Expand Down
6 changes: 3 additions & 3 deletions src/app/@dataflow/core/superset-flow.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { BareFlow, BareFlowInNode, BareFlowOutNode, CombErr } from './bare-flow';
import { BareFlow, FlowInNode, FlowOutNode, CombErr } from './bare-flow';
import { Observable } from 'rxjs';
import {
tap,
Expand All @@ -13,8 +13,8 @@ import {
export interface FlowSupNode {}

export abstract class SupersetFlow<
Tin extends BareFlowInNode,
Tout extends BareFlowOutNode,
Tin extends FlowInNode,
Tout extends FlowOutNode,
Tsup extends FlowSupNode = Tin & Tout
> extends BareFlow<Tin, Tout> {
private boostrapPrerequest$: Observable<CombErr<Tin>>;
Expand Down
4 changes: 2 additions & 2 deletions src/app/@dataflow/extra/name-validation.ts
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
import { SupersetFlow, DataFlowNode, BareFlowInNode, CombErr } from '../core';
import { SupersetFlow, DataFlowNode, FlowInNode, CombErr } from '../core';
import { Observable, of } from 'rxjs';
import { IUser, UsersFlowNode } from './users-flow';

export interface NameValidationPreNode extends UsersFlowNode {
currentName: string;
}

export interface NameValidationNode extends BareFlowInNode {
export interface NameValidationNode extends FlowInNode {
nameValid: boolean;
}

Expand Down
8 changes: 4 additions & 4 deletions src/app/@dataflow/extra/users-flow.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { Observable, of } from 'rxjs';
import { BareFlow, DataFlowNode, BareFlowInNode, CombErr } from '../core';
import { BareFlow, DataFlowNode, FlowInNode, CombErr } from '../core';

export interface IRcloneServer {
url: string;
Expand All @@ -11,15 +11,15 @@ export interface IUser extends IRcloneServer {
name: string;
}

export interface UsersFlowNode extends BareFlowInNode {
export interface UsersFlowNode extends FlowInNode {
users: IUser[];
}

export abstract class UsersFlow extends BareFlow<BareFlowInNode, UsersFlowNode> {
export abstract class UsersFlow extends BareFlow<FlowInNode, UsersFlowNode> {
public static readonly defaultUser: IUser[] = [
{ name: 'localhost', url: 'http://localhost:5572' },
];
protected request(pre: CombErr<BareFlowInNode>): Observable<CombErr<UsersFlowNode>> {
protected request(pre: CombErr<FlowInNode>): Observable<CombErr<UsersFlowNode>> {
const dataRaw = localStorage.getItem('users');
if (dataRaw) return of([{ users: JSON.parse(dataRaw) }, []]);
localStorage.setItem('users', JSON.stringify(UsersFlow.defaultUser));
Expand Down

0 comments on commit b552309

Please sign in to comment.