Using @settings on a test without @given is completely pointless.

PlaytikaOSS · Oct 16, 2023 · 7a38c73 · 7a38c73
1 parent 304c8f5
commit 7a38c73
Showing 1 changed file with 0 additions and 5 deletions.
diff --git a/tests/test_cmab.py b/tests/test_cmab.py
@@ -144,7 +144,6 @@ def test_cmab_init_with_wrong_blr_models(a, b):
         )
 
 
-@settings(deadline=500)
 def test_cmab_update(n_samples=100, n_features=3):
     actions = np.random.choice(["a1", "a2"], size=n_samples).tolist()
     rewards = np.random.choice([0, 1], size=n_samples).tolist()
@@ -176,7 +175,6 @@ def run_update(context):
     run_update(context=context)
 
 
-@settings(deadline=500)
 def test_cmab_update_not_all_actions(n_samples=100, n_feat=3):
     actions = np.random.choice(["a3", "a4"], size=n_samples).tolist()
     rewards = np.random.choice([0, 1], size=n_samples).tolist()
@@ -279,7 +277,6 @@ def test_cmab_predict_shape_mismatch(a_int):
         mab.predict(context=[])
 
 
-@settings(deadline=500)
 def test_cmab_predict_with_forbidden_actions(n_features=3):
     def run_predict(mab):
         context = np.random.uniform(low=-1.0, high=1.0, size=(1000, n_features))
@@ -438,7 +435,6 @@ def test_cmab_bai_predict(n_samples, n_features):
     assert len(selected_actions) == len(probs) == len(weighted_sums) == n_samples
 
 
-@settings(deadline=500)
 def test_cmab_bai_update(n_samples=100, n_features=3):
     actions = np.random.choice(["a1", "a2"], size=n_samples).tolist()
     rewards = np.random.choice([0, 1], size=n_samples).tolist()
@@ -581,7 +577,6 @@ def test_cmab_cc_predict(n_samples, n_features):
     assert len(selected_actions) == len(probs) == len(weighted_sums) == n_samples
 
 
-@settings(deadline=500)
 def test_cmab_cc_update(n_samples=100, n_features=3):
     actions = np.random.choice(["a1", "a2"], size=n_samples).tolist()
     rewards = np.random.choice([0, 1], size=n_samples).tolist()