Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(530)

Unified Diff: scripts/slave/recipe_modules/auto_bisect/resources/significantly_different_test.py

Issue 2247373002: Refactor stages 1, 2 and test_api overhaul. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Addressing all early feedback. Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: scripts/slave/recipe_modules/auto_bisect/resources/significantly_different_test.py
diff --git a/scripts/slave/recipe_modules/auto_bisect/resources/significantly_different_test.py b/scripts/slave/recipe_modules/auto_bisect/resources/significantly_different_test.py
deleted file mode 100755
index 052ba2d922cc19ecd95136e530797e239a01e5a0..0000000000000000000000000000000000000000
--- a/scripts/slave/recipe_modules/auto_bisect/resources/significantly_different_test.py
+++ /dev/null
@@ -1,59 +0,0 @@
-#!/usr/bin/env python
-"""Tests for significantly_different."""
-
-import json
-import os
-import sys
-import unittest
-
-# pylint: disable=relative-import
-import significantly_different
-
-
-class SignificantlyDifferentTest(unittest.TestCase):
-
- def setUp(self):
- self.conda_path = None
- try:
- import scipy # pylint: disable=unused-variable
- self.conda_path = sys.executable
- except ImportError:
- if os.path.exists(os.path.expanduser('~/conda-test/bin/python')):
- self.conda_path = os.path.expanduser('~/conda-test/bin/python')
-
- def test_basic_case(self):
- try:
- sample_A = [1, 2, 3, 3, 2, 1]
- sample_B = [1, 2, 2, 2, 5, 0]
- results = significantly_different.main(
- ['', json.dumps(sample_A), json.dumps(sample_B), '0.05'],
- self.conda_path)
- except significantly_different.ScipyNotInstalledError:
- # This is meant to let presubmit pass on CQ bots :( because they don't
- # have scipy either directly or thorugh anaconda.
- return
-
- self.assertAlmostEqual(
- 0.40073980338363635,
- results['mann_p_value'])
- self.assertEqual(results['first_sample'], sample_A)
- self.assertEqual(results['second_sample'], sample_B)
-
- def test_single_value(self):
- try:
- sample_A = [1, 1, 1, 1, 1, 1]
- sample_B = [1, 1, 1]
- results = significantly_different.main(
- ['', json.dumps(sample_A), json.dumps(sample_B), '0.05'],
- self.conda_path)
- except significantly_different.ScipyNotInstalledError:
- # This is meant to let presubmit pass on CQ bots :( because they don't
- # have scipy either directly or thorugh anaconda.
- return
- self.assertIsNone(results['mann_p_value'])
- self.assertFalse(results['significantly_different'])
- self.assertEqual(results['first_sample'], sample_A)
- self.assertEqual(results['second_sample'], sample_B)
-
-if __name__ == '__main__':
- unittest.main()

Powered by Google App Engine
This is Rietveld 408576698