GitHub/mermaid
1
0
Fork 0
mirror of https://github.com/mermaid-js/mermaid.git synced 2025-01-28 07:03:17 +08:00
Code Issues Projects Releases Wiki Activity

Merge pull request #985 from knsv/bug/984_bracket_syntax_properties_reversed_order

Browse Source 
bug/984 Class diagram: properties with bracket syntax are reversed in order
This commit is contained in:
Knut Sveidqvist 2019-10-15 18:39:20 +02:00 committed by GitHub
commit 51a89c80aa
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 24 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/diagrams/class/classDb.js
View File

@ -80,9 +80,10 @@ export const addMember = function(className, member) {
} }
}; };
export const addMembers = function(className, MembersArr) { export const addMembers = function(className, members) {
if (Array.isArray(MembersArr)) { if (Array.isArray(members)) {
MembersArr.forEach(member => addMember(className, member)); members.reverse();
members.forEach(member => addMember(className, member));
} }
}; };

20
src/diagrams/class/classDiagram.spec.js
View File

@ -262,5 +262,25 @@ describe('class diagram, ', function() {
expect(testClass.methods.length).toBe(1); expect(testClass.methods.length).toBe(1);
expect(testClass.annotations[0]).toBe('interface'); expect(testClass.annotations[0]).toBe('interface');
}); });
it('should add bracket members in right order', function() {
const str =
'classDiagram\n' +
'class Class1 {\n' +
'int : test\n' +
'string : foo\n' +
'test()\n' +
'foo()\n' +
'}';
parser.parse(str);
const testClass = parser.yy.getClass('Class1');
expect(testClass.members.length).toBe(2);
expect(testClass.methods.length).toBe(2);
expect(testClass.members[0]).toBe('int : test');
expect(testClass.members[1]).toBe('string : foo');
expect(testClass.methods[0]).toBe('test()');
expect(testClass.methods[1]).toBe('foo()');
});
}); });
}); });