Skip to content

Commit

Permalink
fix: make constructors open for extension
Browse files Browse the repository at this point in the history
  • Loading branch information
nikku committed Apr 4, 2023
1 parent 466a2ac commit 9c3d27e
Show file tree
Hide file tree
Showing 6 changed files with 57 additions and 7 deletions.
15 changes: 14 additions & 1 deletion lib/BaseModeler.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,17 @@ const modeler = new BaseModeler({
container: 'container'
});

testViewer(modeler);
testViewer(modeler);


const otherModeler = new BaseModeler({
container: 'container'
});

const extendedModeler = new BaseModeler({
container: 'container',
alignToOrigin: false,
propertiesPanel: {
attachTo: '#properties-panel'
}
});
2 changes: 1 addition & 1 deletion lib/BaseViewer.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ export type BaseViewerOptions = {
container?: string|HTMLElement;
moddleExtensions?: ModdleExtensions;
additionalModules?: ModuleDeclaration[];
};
} & Record<string, any>;

export type ModdleElement = Object;

Expand Down
12 changes: 10 additions & 2 deletions lib/BaseViewer.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,9 @@ import BaseViewer, { Events } from './BaseViewer';

import OverlaysModule from 'diagram-js/lib/features/overlays';

let viewer = new BaseViewer();
const viewer = new BaseViewer();

viewer = new BaseViewer({
const configuredViewer = new BaseViewer({
width: 100,
height: 100,
position: 'absolute',
Expand All @@ -23,6 +23,14 @@ viewer = new BaseViewer({

testViewer(viewer);

const extendedViewer = new BaseViewer({
container: 'container',
alignToOrigin: false,
propertiesPanel: {
attachTo: '#properties-panel'
}
});

export function testViewer(viewer: BaseViewer) {
viewer.importXML('<?xml version="1.0" encoding="UTF-8"?>', 'BPMNDiagram_1');

Expand Down
15 changes: 14 additions & 1 deletion lib/Modeler.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,17 @@ const modeler = new Modeler({

testViewer(modeler);

modeler.createDiagram();
modeler.createDiagram();


const otherModeler = new Modeler({
container: 'container'
});

const extendedModeler = new Modeler({
container: 'container',
alignToOrigin: false,
propertiesPanel: {
attachTo: '#properties-panel'
}
});
10 changes: 9 additions & 1 deletion lib/NavigatedViewer.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,12 @@ const viewer = new NavigatedViewer({
container: 'container'
});

testViewer(viewer);
testViewer(viewer);

const extendedViewer = new NavigatedViewer({
container: 'container',
alignToOrigin: false,
propertiesPanel: {
attachTo: '#properties-panel'
}
});
10 changes: 9 additions & 1 deletion lib/Viewer.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,12 @@ const viewer = new Viewer({
container: 'container'
});

testViewer(viewer);
testViewer(viewer);

const extendedViewer = new Viewer({
container: 'container',
alignToOrigin: false,
propertiesPanel: {
attachTo: '#properties-panel'
}
});

0 comments on commit 9c3d27e

Please sign in to comment.