Skip to content

Commit 8b81345

Browse files
committed
Fix: Ordering of statements in test
1 parent dd17c98 commit 8b81345

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

test/test_pipeline/components/regression/test_base.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,6 @@ def test_default_boston(self):
3333
if self.__class__ == BaseRegressionComponentTest:
3434
return
3535

36-
fixture = self.res["default_boston"]
37-
places = self.res.get("default_boston_places", 7)
38-
3936
for _ in range(2):
4037

4138
with ignore_warnings(regressor_warnings):
@@ -46,20 +43,23 @@ def test_default_boston(self):
4643

4744
score = sklearn.metrics.r2_score(y_true=targets, y_pred=predictions)
4845

46+
# Special treatment for Gaussian Process Regression
4947
if "default_boston_le_ge" in self.res:
50-
# Special treatment for Gaussian Process Regression
51-
self.assertLessEqual(score, self.res["default_boston_le_ge"][0])
52-
self.assertGreaterEqual(score, self.res["default_boston_le_ge"][1])
48+
upper, lower = self.res["default_boston_le_ge"]
49+
self.assertLessEqual(score, upper)
50+
self.assertGreaterEqual(score, lower)
5351

5452
else:
53+
fixture = self.res["default_boston"]
54+
places = self.res.get("default_boston_places", 7)
55+
5556
if score < -1e10:
56-
print(f"score = {score}, fixture = {fixture}")
5757
score = np.log(-score)
5858
fixture = np.log(-fixture)
5959

6060
self.assertAlmostEqual(fixture, score, places)
6161

62-
if self.res.get("boston_n_calls"):
62+
if "boston_n_calls" in self.res:
6363
self.assertEqual(self.res["boston_n_calls"], n_calls)
6464

6565
def test_default_boston_iterative_fit(self):

0 commit comments

Comments
 (0)