Forgot to handle the rev case when I removed it as a param

Note: in the regular usage, we do nothing is visible is false, and we pass copies into the CLIPLINE method anyways, so there's no point reversing it back
NREL · Aug 12, 2024 · 955463a · 955463a · nrel-bot-2b · Aug 12, 2024
1 parent 251ea56
commit 955463a
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/tst/EnergyPlus/unit/SolarShading.unit.cc b/tst/EnergyPlus/unit/SolarShading.unit.cc
@@ -6266,6 +6266,7 @@ TEST_F(EnergyPlusFixture, CLIPLINE_Full)
 
         Real64 x1 = t.line_ori.p1.x;
         Real64 y1 = t.line_ori.p1.y;
+        bool is_rev = x0 > x1;
 
         std::string const msg = fmt::format("From ({}, {}) to ({}, {})", t.line_ori.p0.x, t.line_ori.p0.y, t.line_ori.p1.x, t.line_ori.p1.y);
 
@@ -6279,6 +6280,10 @@ TEST_F(EnergyPlusFixture, CLIPLINE_Full)
             EXPECT_DOUBLE_EQ(t.line_new.p1.y, y1) << msg;
         } else {
             EXPECT_FALSE(visible) << msg;
+            if (is_rev) {
+                std::swap(x0, x1);
+                std::swap(y0, y1);
+            }
             EXPECT_DOUBLE_EQ(t.line_ori.p0.x, x0) << msg;
             EXPECT_DOUBLE_EQ(t.line_ori.p0.y, y0) << msg;
             EXPECT_DOUBLE_EQ(t.line_ori.p1.x, x1) << msg;