Browse Source

Add missing improvements for boxing module

Patrick-Christopher Mattulat 10 tháng trước cách đây
mục cha
commit
987dbb7b02

+ 11 - 11
test/cases/boxing/BooleanTest.cpp

@@ -48,7 +48,7 @@ namespace
 
   TEST_F(BooleanTest, operator_output_stream)
   {
-    Boolean expression{true};
+    const Boolean expression{true};
     ostringstream _stream{};
     _stream << expression;
 
@@ -71,8 +71,8 @@ namespace
 
   TEST_F(BooleanTest, operator_and)
   {
-    Boolean expressionA{true};
-    Boolean expressionB{true};
+    const Boolean expressionA{true};
+    const Boolean expressionB{true};
 
     ASSERT_TRUE(expressionA && expressionB);
     ASSERT_TRUE(expressionB && expressionA);
@@ -83,8 +83,8 @@ namespace
 
   TEST_F(BooleanTest, operator_and_with_false_result)
   {
-    Boolean expressionA{true};
-    Boolean expressionB{false};
+    const Boolean expressionA{true};
+    const Boolean expressionB{false};
 
     ASSERT_FALSE(expressionA && expressionB);
     ASSERT_FALSE(expressionB && expressionA);
@@ -92,8 +92,8 @@ namespace
 
   TEST_F(BooleanTest, operator_or)
   {
-    Boolean expressionA{false};
-    Boolean expressionB{true};
+    const Boolean expressionA{false};
+    const Boolean expressionB{true};
 
     ASSERT_TRUE(expressionA || expressionB);
     ASSERT_TRUE(expressionB || expressionA);
@@ -102,8 +102,8 @@ namespace
 
   TEST_F(BooleanTest, operator_or_with_false_result)
   {
-    Boolean expressionA{false};
-    Boolean expressionB{false};
+    const Boolean expressionA{false};
+    const Boolean expressionB{false};
 
     ASSERT_FALSE(expressionA || expressionB);
     ASSERT_FALSE(expressionB || expressionA);
@@ -181,8 +181,8 @@ namespace
 
   TEST_F(BooleanTest, XOR_with_positive_result)
   {
-    Boolean x{2 < 3};
-    Boolean y{4 < 3};
+    const Boolean x{2 < 3};
+    const Boolean y{4 < 3};
 
     ASSERT_TRUE(Boolean::XOR(x, y));
     ASSERT_TRUE(Boolean::XOR(y, x));

+ 2 - 2
test/cases/boxing/IntegerTest.cpp

@@ -118,7 +118,7 @@ namespace
           try
           {
             Integer x{9};
-            Integer y{0};
+            const Integer y{0};
 
             x = x / y;
           }
@@ -238,7 +238,7 @@ namespace
           try
           {
             Integer x{9};
-            Integer y{0};
+            const Integer y{0};
 
             x = x /= y;
           }

+ 3 - 3
test/cases/boxing/LongTest.cpp

@@ -109,7 +109,7 @@ namespace
   TEST_F(LongTest, operator_div_with_value)
   {
     const Long x{81};
-    ASSERT_EQ(9, x / (long_type) 9);
+    ASSERT_EQ(9, x / 9);
   }
 
   TEST_F(LongTest, operator_div_by_zero_with_reference)
@@ -119,7 +119,7 @@ namespace
           try
           {
             Long x{9};
-            Long y{0};
+            const Long y{0};
 
             x = x / y;
           }
@@ -239,7 +239,7 @@ namespace
           try
           {
             Long x{9};
-            Long y{0};
+            const Long y{0};
 
             x = x /= y;
           }