Permalink
Browse files

Fixed unit test: because the implementation creates an IMolecule obje…

…ct, the original 'molecule' pointer does not get atoms, and the returned IMolecule object should be assigned to get a molecule with information

Change-Id: I480895ef8a0f308a279f80c869a6efb93c67cbee
  • Loading branch information...
1 parent eae7598 commit 8ff45842644f86a60a9d94290f2d054c8fc205ba @egonw egonw committed Feb 19, 2012
Showing with 2 additions and 2 deletions.
  1. +2 −2 src/test/org/openscience/cdk/io/MDLV2000ReaderTest.java
View
4 src/test/org/openscience/cdk/io/MDLV2000ReaderTest.java
@@ -926,10 +926,10 @@ public void testAtomParity() throws CDKException, IOException{
InputStream in = ClassLoader.getSystemResourceAsStream("data/mdl/mol_testAtomParity.mol");
MDLV2000Reader reader = new MDLV2000Reader(in);
IAtomContainer molecule = DefaultChemObjectBuilder.getInstance().newInstance(IAtomContainer.class);
- reader.read(molecule);
+ molecule = reader.read(molecule);
reader.close();
-
+ Assert.assertEquals(6, molecule.getAtomCount());
boolean chiralCentre = false;
IAtom[] atoms = AtomContainerManipulator.getAtomArray(molecule);
for (IAtom atom : atoms) {

0 comments on commit 8ff4584

Please sign in to comment.