using constructors rather than class const literals

redengin · redengin · commit 2c8d5519cfea · 2019-03-09T01:23:26.000-08:00
modified:   tests/CppUTestExt/TestDoubleTest.cpp
diff --git a/tests/CppUTestExt/TestDoubleTest.cpp b/tests/CppUTestExt/TestDoubleTest.cpp
@@ -108,7 +108,8 @@ TEST_GROUP( UseModel )
   class OutputModel : public TestDouble::IModel
   {
   public:
-    const int value = 2;    ///< don't use 1 as default actual produces true == 1
+    const int value;
+    OutputModel() : value(2) {};  ///< don't use 1 as default actual produces true == 1
 
     virtual void model( TestDouble::AActualCall &actualCall )
     {
@@ -119,7 +120,8 @@ TEST_GROUP( UseModel )
   class ReturnModel : public TestDouble::IModel
   {
   public:
-    const int value = 2;    ///< don't use 1 as default actual produces true == 1
+    const int value;
+    ReturnModel() : value(2) {};  ///< don't use 1 as default actual produces true == 1
 
     virtual void model( TestDouble::AActualCall &actualCall )
     {

Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,8 @@ TEST_GROUP( UseModel )`
`108`	`108`	`class OutputModel : public TestDouble::IModel`
`109`	`109`	`{`
`110`	`110`	`public:`
`111`		`- const int value = 2; ///< don't use 1 as default actual produces true == 1`
	`111`	`+ const int value;`
	`112`	`+ OutputModel() : value(2) {}; ///< don't use 1 as default actual produces true == 1`
`112`	`113`
`113`	`114`	`virtual void model( TestDouble::AActualCall &actualCall )`
`114`	`115`	`{`
`@@ -119,7 +120,8 @@ TEST_GROUP( UseModel )`
`119`	`120`	`class ReturnModel : public TestDouble::IModel`
`120`	`121`	`{`
`121`	`122`	`public:`
`122`		`- const int value = 2; ///< don't use 1 as default actual produces true == 1`
	`123`	`+ const int value;`
	`124`	`+ ReturnModel() : value(2) {}; ///< don't use 1 as default actual produces true == 1`
`123`	`125`
`124`	`126`	`virtual void model( TestDouble::AActualCall &actualCall )`
`125`	`127`	`{`