Skip to content

Commit

Permalink
Extra merge corrections in Lineage part
Browse files Browse the repository at this point in the history
  • Loading branch information
zaychenko-sergei committed Sep 11, 2023
1 parent 7efa1e2 commit 0118447
Show file tree
Hide file tree
Showing 10 changed files with 41 additions and 18 deletions.
6 changes: 2 additions & 4 deletions src/app/api/mock/dataset.mock.ts
Original file line number Diff line number Diff line change
Expand Up @@ -238,8 +238,7 @@ export const MOCK_NODES = [
data: {
id: "did:odf:z4k88e8qmphemqz8ZfEio3bGRrAjoKtP83U22XidbGtHrUcEghj",
name: "com.naturalearthdata.admin0.countries.50m",
kind: "ROOT",
isRoot: true,
kind: DatasetKind.Root,
isCurrent: true,
color: "#7aa3e5",
accountName: "kamu",
Expand All @@ -263,8 +262,7 @@ export const MOCK_NODES = [
data: {
id: "did:odf:z4k88e8h7woj2Njge7MpGneHHEo8nuSA6npPrAroFpETcukCenC",
name: "com.naturalearthdata.admin0.countries",
kind: "DERIVATIVE",
isRoot: false,
kind: DatasetKind.Derivative,
isCurrent: false,
color: "#a8385d",
accountName: "kamu",
Expand Down
2 changes: 2 additions & 0 deletions src/app/auth/login/login.component.scss
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
@import 'var';

a {
text-decoration: none;
color: $app-dark;
Expand Down
1 change: 1 addition & 0 deletions src/app/auth/login/login.component.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import { Observable, shareReplay } from "rxjs";
@Component({
selector: "app-login",
templateUrl: "./login.component.html",
styleUrls: ["./login.component.scss"],
changeDetection: ChangeDetectionStrategy.OnPush,
})
export class LoginComponent implements OnInit {
Expand Down
14 changes: 9 additions & 5 deletions src/app/components/lineage-graph/lineage-graph.component.html
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
[attr.width]="node.dimension.width"
[attr.height]="node.dimension.height * 1.1"
[attr.rx]="5"
[attr.fill]="node.data.isRoot ? '#ffa500' : '#add8e6'"
[attr.fill]="node.data.kind === DatasetKind.Root ? '#ffa500' : '#add8e6'"
[attr.cursor]="'pointer'"
[attr.stroke]="'black'"
[attr.stroke-width]="node.data.isCurrent ? 2 : 1"
Expand Down Expand Up @@ -70,14 +70,18 @@
[attr.y]="-12"
[attr.width]="20"
[attr.height]="20"
[attr.href]="node.data.isRoot ? '../assets/images/public.png' : '../assets/images/private.png'"
[attr.href]="
node.data.kind === DatasetKind.Root
? '../assets/images/public.png'
: '../assets/images/private.png'
"
/>
</svg:g>
<svg:g [attr.transform]="'translate(' + node.dimension.width + ', 0)'">
<svg:rect
[attr.x]="node.data.isRoot ? -20 : -40"
[attr.x]="node.data.kind === DatasetKind.Root ? -20 : -40"
y="-12"
[attr.width]="node.data.isRoot ? 40 : 80"
[attr.width]="node.data.kind === DatasetKind.Root ? 40 : 80"
height="24"
rx="10"
[attr.stroke]="'#262626'"
Expand All @@ -91,7 +95,7 @@
fill="black"
style="font-size: 12px"
>
{{ node.data.isRoot ? "Root" : "Derivative" }}
{{ node.data.kind }}
</svg:text>
</svg:g>
</svg:g>
Expand Down
4 changes: 3 additions & 1 deletion src/app/components/lineage-graph/lineage-graph.component.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import {
import { Edge, MiniMapPosition } from "@swimlane/ngx-graph";
import { ClusterNode, Node } from "@swimlane/ngx-graph/lib/models/node.model";
import { Observable } from "rxjs";
import { AccountDetailsFragment } from "src/app/api/kamu.graphql.interface";
import { AccountDetailsFragment, DatasetKind } from "src/app/api/kamu.graphql.interface";
import AppValues from "src/app/common/app.values";
import { AccountService } from "src/app/services/account.service";

Expand Down Expand Up @@ -42,7 +42,9 @@ export class LineageGraphComponent implements OnChanges, OnInit {
public miniMapPosition: MiniMapPosition;
public graphClusters: ClusterNode[];
public graphNodes: Node[];

public readonly DEFAULT_AVATAR_URL = AppValues.DEFAULT_AVATAR_URL;
public readonly DatasetKind: typeof DatasetKind = DatasetKind;

constructor(private accountService: AccountService) {}

Expand Down
Original file line number Diff line number Diff line change
@@ -1,10 +1,15 @@
import { DatasetKind } from "src/app/api/kamu.graphql.interface";

export enum LineageNodeAccess {
PRIVATE = "private",
PUBLIC = "public",
}

export interface LineageNodeData {
id: string;
name: string;
kind: DatasetKind;
isRoot: boolean;
isCurrent: boolean;
owner: string;
access: LineageNodeAccess;
accountName: string;
}
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ import { DatasetBasicsFragment, DatasetKind } from "src/app/api/kamu.graphql.int
import { BaseComponent } from "src/app/common/base.component";
import { LineageUpdate } from "../../dataset.subscriptions.interface";
import { DatasetSubscriptionsService } from "../../dataset.subscriptions.service";
import { LineageNodeAccess, LineageNodeData } from "./lineage-model";

@Component({
selector: "app-lineage",
Expand Down Expand Up @@ -104,11 +105,10 @@ export class LineageComponent extends BaseComponent implements OnInit {
id: dataset.id,
name: dataset.name,
kind: dataset.kind,
isRoot: dataset.kind === DatasetKind.Root,
isCurrent: dataset.id === currentDataset.id,
access: "private",
access: LineageNodeAccess.PRIVATE,
accountName: dataset.owner.name,
},
} as LineageNodeData,
});
}

Expand Down
2 changes: 1 addition & 1 deletion src/app/dataset-view/dataset.component.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -257,7 +257,7 @@ describe("DatasetComponent", () => {
component.onClickLineageNode(mockNode);

const mockNodeData: LineageNodeData = mockNode.data as LineageNodeData;
expect(selectDatasetSpy).toHaveBeenCalledWith(mockNodeData.owner, mockNodeData.name);
expect(selectDatasetSpy).toHaveBeenCalledWith(mockNodeData.accountName, mockNodeData.name);
});

it("should check navigate to overview tab", () => {
Expand Down
2 changes: 1 addition & 1 deletion src/app/dataset-view/dataset.component.ts
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ export class DatasetComponent extends BaseProcessingComponent implements OnInit,

public onClickLineageNode(node: Node): void {
const nodeData: LineageNodeData = node.data as LineageNodeData;
this.onSelectDataset(nodeData.owner, nodeData.name);
this.onSelectDataset(nodeData.accountName, nodeData.name);
}

private initDatasetViewByType(datasetInfo: DatasetInfo, currentPage: number): void {
Expand Down
13 changes: 12 additions & 1 deletion src/app/search/mock.data.ts
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,10 @@ import {
MergeKind,
} from "../dataset-view/additional-components/metadata-component/components/add-polling-source/add-polling-source-form.types";
import { DatasetHistoryUpdate } from "../dataset-view/dataset.subscriptions.interface";
import {
LineageNodeAccess,
LineageNodeData,
} from "../dataset-view/additional-components/lineage-component/lineage-model";

export const mockPageBasedInfo: PageBasedInfo = {
currentPage: 1,
Expand Down Expand Up @@ -954,5 +958,12 @@ export const mockHistoryEditPollingSourceService: DatasetHistoryUpdate = {
export const mockNode: Node = {
id: "testId",
label: "british-columbia.case-details.hm",
data: mockDatasetBasicsDerivedFragment,
data: {
id: mockDatasetBasicsDerivedFragment.id,
name: mockDatasetBasicsDerivedFragment.name,
kind: mockDatasetBasicsDerivedFragment.kind,
isCurrent: false,
access: LineageNodeAccess.PUBLIC,
accountName: mockDatasetBasicsDerivedFragment.owner.name,
} as LineageNodeData,
};

0 comments on commit 0118447

Please sign in to comment.