Changeset 0fad93 for src/Graph/unittests

Timestamp:

Dec 3, 2012, 9:50:01 AM (12 years ago)

Author:

Frederik Heber <heber@…>

Branches:

Action_Thermostats, Add_AtomRandomPerturbation, Add_FitFragmentPartialChargesAction, Add_RotateAroundBondAction, Add_SelectAtomByNameAction, Added_ParseSaveFragmentResults, AddingActions_SaveParseParticleParameters, Adding_Graph_to_ChangeBondActions, Adding_MD_integration_tests, Adding_ParticleName_to_Atom, Adding_StructOpt_integration_tests, AtomFragments, Automaking_mpqc_open, AutomationFragmentation_failures, Candidate_v1.5.4, Candidate_v1.6.0, Candidate_v1.6.1, ChangeBugEmailaddress, ChangingTestPorts, ChemicalSpaceEvaluator, CombiningParticlePotentialParsing, Combining_Subpackages, Debian_Package_split, Debian_package_split_molecuildergui_only, Disabling_MemDebug, Docu_Python_wait, EmpiricalPotential_contain_HomologyGraph, EmpiricalPotential_contain_HomologyGraph_documentation, Enable_parallel_make_install, Enhance_userguide, Enhanced_StructuralOptimization, Enhanced_StructuralOptimization_continued, Example_ManyWaysToTranslateAtom, Exclude_Hydrogens_annealWithBondGraph, FitPartialCharges_GlobalError, Fix_BoundInBox_CenterInBox_MoleculeActions, Fix_ChargeSampling_PBC, Fix_ChronosMutex, Fix_FitPartialCharges, Fix_FitPotential_needs_atomicnumbers, Fix_ForceAnnealing, Fix_IndependentFragmentGrids, Fix_ParseParticles, Fix_ParseParticles_split_forward_backward_Actions, Fix_PopActions, Fix_QtFragmentList_sorted_selection, Fix_Restrictedkeyset_FragmentMolecule, Fix_StatusMsg, Fix_StepWorldTime_single_argument, Fix_Verbose_Codepatterns, Fix_fitting_potentials, Fixes, ForceAnnealing_goodresults, ForceAnnealing_oldresults, ForceAnnealing_tocheck, ForceAnnealing_with_BondGraph, ForceAnnealing_with_BondGraph_continued, ForceAnnealing_with_BondGraph_continued_betteresults, ForceAnnealing_with_BondGraph_contraction-expansion, FragmentAction_writes_AtomFragments, FragmentMolecule_checks_bonddegrees, GeometryObjects, Gui_Fixes, Gui_displays_atomic_force_velocity, ImplicitCharges, IndependentFragmentGrids, IndependentFragmentGrids_IndividualZeroInstances, IndependentFragmentGrids_IntegrationTest, IndependentFragmentGrids_Sole_NN_Calculation, JobMarket_RobustOnKillsSegFaults, JobMarket_StableWorkerPool, JobMarket_unresolvable_hostname_fix, MoreRobust_FragmentAutomation, ODR_violation_mpqc_open, PartialCharges_OrthogonalSummation, PdbParser_setsAtomName, PythonUI_with_named_parameters, QtGui_reactivate_TimeChanged_changes, Recreated_GuiChecks, Rewrite_FitPartialCharges, RotateToPrincipalAxisSystem_UndoRedo, SaturateAtoms_findBestMatching, SaturateAtoms_singleDegree, StoppableMakroAction, Subpackage_CodePatterns, Subpackage_JobMarket, Subpackage_LinearAlgebra, Subpackage_levmar, Subpackage_mpqc_open, Subpackage_vmg, Switchable_LogView, ThirdParty_MPQC_rebuilt_buildsystem, TrajectoryDependenant_MaxOrder, TremoloParser_IncreasedPrecision, TremoloParser_MultipleTimesteps, TremoloParser_setsAtomName, Ubuntu_1604_changes, stable

Children:

5197e5

Parents:

91207d

git-author:

Frederik Heber <heber@…> (09/20/12 16:31:44)

git-committer:

Frederik Heber <heber@…> (12/03/12 09:50:01)

Message:

Renamed CheckAgainstAdjacencyFile -> AdjacencyList.

• operator() is now CheckAgainstSubset().

Location:

src/Graph/unittests

Files:

• AdjacencyListUnitTest.cpp (moved) (moved from src/Graph/unittests/CheckAgainstAdjacencyFileUnitTest.cpp ) (11 diffs)
• AdjacencyListUnitTest.hpp (moved) (moved from src/Graph/unittests/CheckAgainstAdjacencyFileUnitTest.hpp ) (6 diffs)
• Makefile.am (modified) (3 diffs)

src/Graph/unittests/AdjacencyListUnitTest.cpp

@@ -22 +22 @@
 
 /*
- * CheckAgainstAdjacencyFileUnitTest.cpp
+ * AdjacencyListUnitTest.cpp
  *
  *  Created on: Oct 17, 2011
@@ -33 +33 @@
 #endif
 
-#include "CheckAgainstAdjacencyFileUnitTest.hpp"
+#include "AdjacencyListUnitTest.hpp"
 
 #include <cppunit/CompilerOutputter.h>
@@ -52 +52 @@
 #include "Element/element.hpp"
 #include "Element/periodentafel.hpp"
-#include "Graph/CheckAgainstAdjacencyFile.hpp"
+#include "Graph/AdjacencyList.hpp"
 #include "molecule.hpp"
 #include "World.hpp"
@@ -63 +63 @@
 /********************************************** Test classes **************************************/
 // Registers the fixture into the 'registry'
-CPPUNIT_TEST_SUITE_REGISTRATION( CheckAgainstAdjacencyFileTest );
+CPPUNIT_TEST_SUITE_REGISTRATION( AdjacencyListTest );
 
 static std::string adjacencyfile ="\
@@ -115 +115 @@
 
 // set up and tear down
-void CheckAgainstAdjacencyFileTest::setUp()
+void AdjacencyListTest::setUp()
 {
   // failing asserts should be thrown
@@ -148 +148 @@
 }
 
-void CheckAgainstAdjacencyFileTest::tearDown()
+void AdjacencyListTest::tearDown()
 {
   comparisonMap.clear();
@@ -161 +161 @@
 }
 
-/** Unit tests for CheckAgainstAdjacencyFile::CreateExternalMap().
- *
- */
-void CheckAgainstAdjacencyFileTest::CreateExternalMapTest()
+/** Unit tests for AdjacencyList::CreateExternalMap().
+ *
+ */
+void AdjacencyListTest::CreateExternalMapTest()
 {
   std::stringstream input;
-  CheckAgainstAdjacencyFile fileChecker(input);
+  AdjacencyList fileChecker(input);
   fileChecker.CreateExternalMap(atomIds);
 
@@ -190 +190 @@
 }
 
-/** Unit tests for CheckAgainstAdjacencyFile::ParseInInternalMap().
- *
- */
-void CheckAgainstAdjacencyFileTest::ParseInInternalMapTest()
+/** Unit tests for AdjacencyList::ParseInInternalMap().
+ *
+ */
+void AdjacencyListTest::ParseInInternalMapTest()
 {
   std::stringstream input(adjacencyfile);
-  CheckAgainstAdjacencyFile fileChecker(input);
+  AdjacencyList fileChecker(input);
   std::vector<atomId_t> noids;
   fileChecker.CreateExternalMap(noids);
@@ -218 +218 @@
 }
 
-/** Unit tests for CheckAgainstAdjacencyFile::CompareInternalExternalMap().
- *
- */
-void CheckAgainstAdjacencyFileTest::CompareInternalExternalMapTest()
+/** Unit tests for AdjacencyList::CompareInternalExternalMap().
+ *
+ */
+void AdjacencyListTest::CompareInternalExternalMapTest()
 {
   std::stringstream input(adjacencyfile);
-  CheckAgainstAdjacencyFile fileChecker(input);
+  AdjacencyList fileChecker(input);
 
   // assert equality before parsing (empty sets should always return true)
@@ -240 +240 @@
 }
 
-/** Unit tests for CheckAgainstAdjacencyFile::operator()().
- *
- */
-void CheckAgainstAdjacencyFileTest::operatorTest()
+/** Unit tests for AdjacencyList::operator()().
+ *
+ */
+void AdjacencyListTest::CheckAgainstSubsetTest()
 {
   {
     // parse right
     std::stringstream input(adjacencyfile);
-    CheckAgainstAdjacencyFile fileChecker(input);
-    CPPUNIT_ASSERT( fileChecker(atomIds) );
+    AdjacencyList fileChecker(input);
+    CPPUNIT_ASSERT( fileChecker.CheckAgainstSubset(atomIds) );
   }
   {
     // parse wrong1 (more atoms in the world than in file, hence wrong)
     std::stringstream input(wrongadjacencyfile1);
-    CheckAgainstAdjacencyFile fileChecker(input);
-    CPPUNIT_ASSERT( !fileChecker(atomIds) );
+    AdjacencyList fileChecker(input);
+    CPPUNIT_ASSERT( !fileChecker.CheckAgainstSubset(atomIds) );
   }
   {
@@ -264 +264 @@
     // parse wrong1 (more atoms in the world than in file, hence wrong)
     std::stringstream input(wrongadjacencyfile1);
-    CheckAgainstAdjacencyFile fileChecker(input);
-    CPPUNIT_ASSERT( fileChecker(validids) );
+    AdjacencyList fileChecker(input);
+    CPPUNIT_ASSERT( fileChecker.CheckAgainstSubset(validids) );
   }
   {
     // parse wrong2 (there is no atom 10, but present in file. hence true)
     std::stringstream input(wrongadjacencyfile2);
-    CheckAgainstAdjacencyFile fileChecker(input);
-    CPPUNIT_ASSERT( !fileChecker(atomIds) );
+    AdjacencyList fileChecker(input);
+    CPPUNIT_ASSERT( !fileChecker.CheckAgainstSubset(atomIds) );
   }
   {
     // parse wrong3 (6 is not connected)
     std::stringstream input(wrongadjacencyfile3);
-    CheckAgainstAdjacencyFile fileChecker(input);
-    CPPUNIT_ASSERT( !fileChecker(atomIds) );
-  }
-}
+    AdjacencyList fileChecker(input);
+    CPPUNIT_ASSERT( !fileChecker.CheckAgainstSubset(atomIds) );
+  }
+}

src/Graph/unittests/AdjacencyListUnitTest.hpp

@@ -1 +1 @@
 /*
- * CheckAgainstAdjacencyFileUnitTest.hpp
+ * AdjacencyListUnitTest.hpp
  *
  *  Created on: Oct 17, 2011
@@ -6 +6 @@
  */
 
-#ifndef CHECKADJACENCYAGAINSTFILEUNITTEST_HPP_
-#define CHECKADJACENCYAGAINSTFILEUNITTEST_HPP_
+#ifndef UNITTEST_HPP_
+#define UNITTEST_HPP_
 
 // include config.h
@@ -17 +17 @@
 #include <cppunit/extensions/HelperMacros.h>
 
-#include "Graph/CheckAgainstAdjacencyFile.hpp"
+#include "Graph/AdjacencyList.hpp"
 #include "types.hpp"
 
@@ -27 +27 @@
 class World;
 
-class CheckAgainstAdjacencyFileTest : public CppUnit::TestFixture
+class AdjacencyListTest : public CppUnit::TestFixture
 {
-  CPPUNIT_TEST_SUITE( CheckAgainstAdjacencyFileTest );
+  CPPUNIT_TEST_SUITE( AdjacencyListTest );
   CPPUNIT_TEST ( CreateExternalMapTest );
   CPPUNIT_TEST ( ParseInInternalMapTest );
   CPPUNIT_TEST ( CompareInternalExternalMapTest );
-  CPPUNIT_TEST ( operatorTest );
+  CPPUNIT_TEST ( CheckAgainstSubsetTest );
   CPPUNIT_TEST_SUITE_END();
 
@@ -43 +43 @@
   void ParseInInternalMapTest();
   void CompareInternalExternalMapTest();
-  void operatorTest();
+  void CheckAgainstSubsetTest();
 
 private:
-  CheckAgainstAdjacencyFile::AtomBondMap comparisonMap;
+  AdjacencyList::AtomBondMap comparisonMap;
   std::vector<atom *> atoms;
   std::vector<atomId_t> atomIds;
@@ -52 +52 @@
 };
 
-#endif /* CHECKADJACENCYAGAINSTFILEUNITTEST_HPP_ */
+#endif /* UNITTEST_HPP_ */

src/Graph/unittests/Makefile.am

@@ -3 +3 @@
 
 GRAPHTESTSSOURCES = \
-        ../Graph/unittests/BondGraphUnitTest.cpp \
-        ../Graph/unittests/CheckAgainstAdjacencyFileUnitTest.cpp
+        ../Graph/unittests/AdjacencyListUnitTest.cpp \
+        ../Graph/unittests/BondGraphUnitTest.cpp
 
 GRAPHTESTSHEADERS = \
-        ../Graph/unittests/BondGraphUnitTest.hpp \
-        ../Graph/unittests/CheckAgainstAdjacencyFileUnitTest.hpp
+        ../Graph/unittests/AdjacencyListUnitTest.hpp \
+        ../Graph/unittests/BondGraphUnitTest.hpp
 
 GRAPHTESTS = \
-        BondGraphUnitTest \
-        CheckAgainstAdjacencyFileUnitTest
+        AdjacencyListUnitTest \
+        BondGraphUnitTest
 
 
@@ -32 +32 @@
 
 
+AdjacencyListUnitTest_SOURCES = $(top_srcdir)/src/unittests/UnitTestMain.cpp \
+        ../Graph/unittests/AdjacencyListUnitTest.cpp \
+        ../Graph/unittests/AdjacencyListUnitTest.hpp
+AdjacencyListUnitTest_LDADD = ${ALLLIBS}
+
 BondGraphUnitTest_SOURCES = $(top_srcdir)/src/unittests/UnitTestMain.cpp \
         ../Graph/unittests/BondGraphUnitTest.cpp \
@@ -37 +42 @@
 BondGraphUnitTest_LDADD = ${GRAPHLIBS}
 
-CheckAgainstAdjacencyFileUnitTest_SOURCES = $(top_srcdir)/src/unittests/UnitTestMain.cpp \
-        ../Graph/unittests/CheckAgainstAdjacencyFileUnitTest.cpp \
-        ../Graph/unittests/CheckAgainstAdjacencyFileUnitTest.hpp
-CheckAgainstAdjacencyFileUnitTest_LDADD = ${ALLLIBS}
-