Skip to content

Commit 229f036

Browse files
committed
Moved countTestedAtomTypes() to the end, to fix the checking if all atom types have been tests; also added a comment that is must be the last method
1 parent 105850e commit 229f036

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

src/test/org/openscience/cdk/atomtype/CDKAtomTypeMatcherTest.java

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3515,10 +3515,6 @@ public void testPentaMethylPhosphane() throws Exception {
35153515
assertAtomTypes(testedAtomTypes, expectedTypes, mol);
35163516
}
35173517

3518-
@Test public void countTestedAtomTypes() {
3519-
super.countTestedAtomTypes(testedAtomTypes);
3520-
}
3521-
35223518
@Test public void testForDuplicateDefinitions() {
35233519
super.testForDuplicateDefinitions();
35243520
}
@@ -3666,4 +3662,12 @@ public void test_Mn_metallic() throws Exception {
36663662
String[] expectedTypes = {"Mn.metallic"};
36673663
assertAtomTypes(testedAtomTypes, expectedTypes, mol);
36683664
}
3665+
3666+
/*
3667+
* This method *must* be the last method in the class.
3668+
*/
3669+
@Test public void countTestedAtomTypes() {
3670+
super.countTestedAtomTypes(testedAtomTypes);
3671+
}
3672+
36693673
}

0 commit comments

Comments
 (0)