From a420b61925344422914ebad98e3ffc265e5e209e Mon Sep 17 00:00:00 2001
From: Syed Asad Rahman <s9asad@gmail.com>
Date: Mon, 30 Aug 2010 11:55:51 +0100
Subject: [PATCH] a)Assert cleaned and fixed b)Correct ASSERT imports used

---
 .../algorithm/rgraph/CDKRMapHandlerTest.java  | 23 +++++++++----------
 1 file changed, 11 insertions(+), 12 deletions(-)

diff --git a/src/test/org/openscience/cdk/smsd/algorithm/rgraph/CDKRMapHandlerTest.java b/src/test/org/openscience/cdk/smsd/algorithm/rgraph/CDKRMapHandlerTest.java
index 5b4177f43d8..06232b4e44f 100644
--- a/src/test/org/openscience/cdk/smsd/algorithm/rgraph/CDKRMapHandlerTest.java
+++ b/src/test/org/openscience/cdk/smsd/algorithm/rgraph/CDKRMapHandlerTest.java
@@ -22,7 +22,6 @@
  */
 package org.openscience.cdk.smsd.algorithm.rgraph;
 
-import org.openscience.cdk.smsd.algorithm.rgraph.CDKRMapHandler;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
@@ -32,8 +31,8 @@
 import org.junit.Before;
 import org.junit.BeforeClass;
 import org.junit.Test;
+import org.junit.Assert;
 import org.openscience.cdk.DefaultChemObjectBuilder;
-import static org.junit.Assert.*;
 import org.openscience.cdk.exception.CDKException;
 import org.openscience.cdk.exception.InvalidSmilesException;
 import org.openscience.cdk.interfaces.IAtomContainer;
@@ -76,7 +75,7 @@ public void testGetSource() {
         IAtomContainer expResult = DefaultChemObjectBuilder.getInstance().newInstance(IAtomContainer.class);
         CDKRMapHandler.setSource(expResult);
         IAtomContainer result = CDKRMapHandler.getSource();
-        assertEquals(expResult, result);
+        Assert.assertEquals(expResult, result);
     }
 
     /**
@@ -88,7 +87,7 @@ public void testSetSource() {
         IAtomContainer expResult = DefaultChemObjectBuilder.getInstance().newInstance(IAtomContainer.class);
         CDKRMapHandler.setSource(expResult);
         IAtomContainer result = CDKRMapHandler.getSource();
-        assertEquals(expResult, result);
+        Assert.assertEquals(expResult, result);
     }
 
     /**
@@ -100,7 +99,7 @@ public void testGetTarget() {
         IAtomContainer expResult = DefaultChemObjectBuilder.getInstance().newInstance(IAtomContainer.class);
         CDKRMapHandler.setTarget(expResult);
         IAtomContainer result = CDKRMapHandler.getTarget();
-        assertEquals(expResult, result);
+        Assert.assertEquals(expResult, result);
     }
 
     /**
@@ -112,7 +111,7 @@ public void testSetTarget() {
         IAtomContainer expResult = DefaultChemObjectBuilder.getInstance().newInstance(IAtomContainer.class);
         CDKRMapHandler.setTarget(expResult);
         IAtomContainer result = CDKRMapHandler.getTarget();
-        assertEquals(expResult, result);
+        Assert.assertEquals(expResult, result);
     }
 
     /**
@@ -127,7 +126,7 @@ public void testCalculateOverlapsAndReduce() throws Exception {
         IAtomContainer Molecule2 = sp.parseSmiles("C1CCCC1");
         CDKRMapHandler instance = new CDKRMapHandler();
         instance.calculateOverlapsAndReduce(Molecule1, Molecule2, true);
-        assertNotNull(FinalMappings.getInstance().getSize());
+        Assert.assertNotNull(FinalMappings.getInstance().getSize());
     }
 
     /**
@@ -143,7 +142,7 @@ public void testCalculateOverlapsAndReduceExactMatch() throws Exception {
         CDKRMapHandler instance = new CDKRMapHandler();
         instance.calculateOverlapsAndReduceExactMatch(Molecule1, Molecule2, true);
         // TODO review the generated test code and remove the default call to fail.
-        assertNotNull(FinalMappings.getInstance().getSize());
+        Assert.assertNotNull(FinalMappings.getInstance().getSize());
     }
 
     /**
@@ -159,7 +158,7 @@ public void testGetMappings() throws InvalidSmilesException, CDKException {
         CDKRMapHandler instance = new CDKRMapHandler();
         instance.calculateOverlapsAndReduceExactMatch(Molecule1, Molecule2, true);
         List<Map<Integer, Integer>> result = instance.getMappings();
-        assertEquals(2, result.size());
+        Assert.assertEquals(2, result.size());
     }
 
     /**
@@ -176,7 +175,7 @@ public void testSetMappings() {
         mappings.add(map);
         CDKRMapHandler instance = new CDKRMapHandler();
         instance.setMappings(mappings);
-        assertNotNull(instance.getMappings());
+        Assert.assertNotNull(instance.getMappings());
     }
 
     /**
@@ -189,7 +188,7 @@ public void testIsTimeoutFlag() {
         boolean expResult = true;
         instance.setTimeoutFlag(true);
         boolean result = instance.isTimeoutFlag();
-        assertEquals(expResult, result);
+        Assert.assertEquals(expResult, result);
     }
 
     /**
@@ -201,6 +200,6 @@ public void testSetTimeoutFlag() {
         boolean timeoutFlag = false;
         CDKRMapHandler instance = new CDKRMapHandler();
         instance.setTimeoutFlag(timeoutFlag);
-        assertNotSame(true, instance.isTimeoutFlag());
+        Assert.assertNotSame(true, instance.isTimeoutFlag());
     }
 }