#684 Fix applying default class to flowchart nodes

This commit is contained in:
Marc Faber
2020-02-24 01:31:39 +01:00
parent 1c07e550bb
commit d67e49400f
4 changed files with 81 additions and 2 deletions

View File

@@ -36,7 +36,7 @@ export const addVertices = function(vert, g, svgId) {
* Variable for storing the classes for the vertex
* @type {string}
*/
let classStr = '';
let classStr = 'default';
if (vertex.classes.length > 0) {
classStr = vertex.classes.join(' ');
}

View File

@@ -127,6 +127,40 @@ describe('the flowchart renderer', function() {
expect(addedNodes[0][1]).toHaveProperty('labelStyle', expectedLabelStyle);
});
});
it(`should add default class to all nodes which do not have another class assigned`, function() {
const addedNodes = [];
const mockG = {
setNode: function(id, object) {
addedNodes.push([id, object]);
}
};
addVertices(
{
v1: {
type: 'rect',
id: 'defaultNode',
classes: [],
styles: [],
text: 'my vertex text'
},
v2: {
type: 'rect',
id: 'myNode',
classes: ['myClass'],
styles: [],
text: 'my vertex text'
}
},
mockG,
'svg-id'
);
expect(addedNodes).toHaveLength(2);
expect(addedNodes[0][0]).toEqual('defaultNode');
expect(addedNodes[0][1]).toHaveProperty('class', 'default');
expect(addedNodes[1][0]).toEqual('myNode');
expect(addedNodes[1][1]).toHaveProperty('class', 'myClass');
});
});
describe('when adding edges to a graph', function() {