Changeset 3e6b93 for src/Parameters/unittests/ContinuousValueTest.cpp

Timestamp:

Feb 4, 2015, 9:56:43 PM (11 years ago)

Author:

Frederik Heber <heber@…>

Parents:

76a109

git-author:

Frederik Heber <heber@…> (09/23/14 16:34:50)

git-committer:

Frederik Heber <heber@…> (02/04/15 21:56:43)

Message:

Changed checking of Parameter::isValid() on set(), not on get().

• this would fix problems with ActionQueue::OutputAs...() commands that need to get() parameter values after usage by the Action. If files were forced to be non-present before, then written by the Action, the Validator will then fail.

File:

• src/Parameters/unittests/ContinuousValueTest.cpp (modified) (3 diffs)

src/Parameters/unittests/ContinuousValueTest.cpp

@@ -155 +155 @@
   {
     Value<int> test(*ValidIntRange);
-    test.setAsString(toString(5));
-    CPPUNIT_ASSERT_THROW(test.getAsString(), ParameterValueException);
-    test.setAsString(toString(0));
-    std::cout << "The following Assert failures are intended and do not indicate a failure of the test." << std::endl;
-    CPPUNIT_ASSERT_THROW(test.getAsString(), ParameterValueException);
+    std::cout << "The following Assert failures are intended and do not indicate a failure of the test." << std::endl;
+    CPPUNIT_ASSERT_THROW(test.setAsString(toString(5)), ParameterValueException);
+    std::cout << "The following Assert failures are intended and do not indicate a failure of the test." << std::endl;
+    CPPUNIT_ASSERT_THROW(test.setAsString(toString(0)), ParameterValueException);
   }
 
@@ -187 +186 @@
 
   // setting invalid and getting it, throws ParameterValueException
-  test.set(5);
-  std::cout << "The following Assert failures are intended and do not indicate a failure of the test." << std::endl;
-  CPPUNIT_ASSERT_THROW(test.get(), ParameterValueException);
-  test.set(0);
-  std::cout << "The following Assert failures are intended and do not indicate a failure of the test." << std::endl;
-  CPPUNIT_ASSERT_THROW(test.get(), ParameterValueException);
+  std::cout << "The following Assert failures are intended and do not indicate a failure of the test." << std::endl;
+  CPPUNIT_ASSERT_THROW(test.set(5), ParameterValueException);
+  std::cout << "The following Assert failures are intended and do not indicate a failure of the test." << std::endl;
+  CPPUNIT_ASSERT_THROW(test.set(0), ParameterValueException);
 
   // checking all valid ones
@@ -400 +397 @@
   {
     Value<Vector> test(*ValidVectorRange);
-    test.set(Vector(5,0,0));
-    std::cout << "The following Assert failures are intended and do not indicate a failure of the test." << std::endl;
-    CPPUNIT_ASSERT_THROW(test.get(), ParameterValueException);
-    test.set(Vector(5,20,5));
-    std::cout << "The following Assert failures are intended and do not indicate a failure of the test." << std::endl;
-    CPPUNIT_ASSERT_THROW(test.get(), ParameterValueException);
+    std::cout << "The following Assert failures are intended and do not indicate a failure of the test." << std::endl;
+    CPPUNIT_ASSERT_THROW(test.set(Vector(5,0,0)), ParameterValueException);
+    std::cout << "The following Assert failures are intended and do not indicate a failure of the test." << std::endl;
+    CPPUNIT_ASSERT_THROW(test.set(Vector(5,20,5)), ParameterValueException);
   }