Merge pull request #1164 from cloudbees-oss/add-get-tests-from-guess-field

Konboi · web-flow · commit f58730097ee7 · 2025-10-21T13:24:20.000+09:00
Add get tests from guess field
diff --git a/launchable/commands/subset.py b/launchable/commands/subset.py
@@ -378,6 +378,7 @@ def __init__(self, app: Application):
             self.output_handler = self._default_output_handler
             self.exclusion_output_handler = self._default_exclusion_output_handler
             self.is_get_tests_from_previous_sessions = is_get_tests_from_previous_sessions
+            self.is_get_tests_from_guess = is_get_tests_from_guess
             self.is_output_exclusion_rules = is_output_exclusion_rules
             self.is_get_tests_from_guess = is_get_tests_from_guess
             super(Optimize, self).__init__(app=app)
@@ -486,6 +487,7 @@ def get_payload(
                 },
                 "ignoreNewTests": ignore_new_tests,
                 "getTestsFromPreviousSessions": self.is_get_tests_from_previous_sessions,
+                "getTestsFromGuess": self.is_get_tests_from_guess,
             }
 
             if target is not None:
diff --git a/tests/cli_test_case.py b/tests/cli_test_case.py
@@ -43,6 +43,9 @@ def setUp(self):
         if not hasattr(self, 'test_files_dir'):
             self.test_files_dir = self.get_test_files_dir()
 
+        # not to use cached configuration between tests
+        responses.reset()
+
         responses.add(
             responses.POST,
             "{}/intake/organizations/{}/workspaces/{}/builds/{}/test_sessions".format(
diff --git a/tests/data/adb/subset_result.json b/tests/data/adb/subset_result.json
@@ -11,5 +11,6 @@
   "goal": {"type": "subset-by-percentage", "percentage": 0.1},
   "ignoreNewTests": false,
   "session": { "id": "16" },
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/ant/subset_result.json b/tests/data/ant/subset_result.json
@@ -7,5 +7,6 @@
   "session": { "id": "16" },
   "goal": {"type": "subset-by-percentage", "percentage": 0.1},
   "ignoreNewTests": false,
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/bazel/subset_result.json b/tests/data/bazel/subset_result.json
@@ -41,5 +41,6 @@
   "goal": {"type": "subset-by-percentage", "percentage": 0.1},
   "ignoreNewTests": false,
   "session": { "id": "16" },
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/behave/subset_result.json b/tests/data/behave/subset_result.json
@@ -8,5 +8,6 @@
   "session": {
       "id":  "16"
   },
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/ctest/subset_result.json b/tests/data/ctest/subset_result.json
@@ -9,5 +9,6 @@
   "goal": {"type": "subset-by-percentage", "percentage": 0.1},
   "ignoreNewTests": false,
   "session": { "id": "16" },
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/cts/subset_result.json b/tests/data/cts/subset_result.json
@@ -30,6 +30,7 @@
     "id": "16"
   },
   "ignoreNewTests": false,
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false,
   "goal": {
     "type": "subset-by-percentage",
diff --git a/tests/data/cypress/subset_result.json b/tests/data/cypress/subset_result.json
@@ -13,5 +13,6 @@
     ]
   ],
   "testRunner": "cypress",
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/go_test/subset_result.json b/tests/data/go_test/subset_result.json
@@ -6,4 +6,5 @@
                 [{"name": "rocket-car-gotest", "type": "class"}, {"name": "TestExample3", "type": "testcase"}],
                 [{"name": "rocket-car-gotest", "type": "class"}, {"name": "TestExample4", "type": "testcase"}]],
   "testRunner": "go-test",
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false}
diff --git a/tests/data/googletest/subset_result.json b/tests/data/googletest/subset_result.json
@@ -17,5 +17,6 @@
   "goal": {"type": "subset-by-percentage", "percentage": 0.1},
   "ignoreNewTests": false,
   "session": { "id": "16" },
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/jest/subset_result.json b/tests/data/jest/subset_result.json
@@ -37,5 +37,6 @@
   "session": { "id": "16" },
   "goal": {"type": "subset-by-percentage", "percentage": 0.1},
   "ignoreNewTests": false,
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/maven/subset_by_absolute_time_result.json b/tests/data/maven/subset_by_absolute_time_result.json
@@ -12,5 +12,6 @@
   },
   "goal": {"type": "subset-by-absolute-time", "duration": 5400},
   "ignoreNewTests": false,
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/maven/subset_by_confidence_result.json b/tests/data/maven/subset_by_confidence_result.json
@@ -12,5 +12,6 @@
   },
   "goal": {"type": "subset-by-confidence", "percentage": 0.9},
   "ignoreNewTests": false,
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/maven/subset_from_file_result.json b/tests/data/maven/subset_from_file_result.json
@@ -26,5 +26,6 @@
   "session": {
       "id":  "16"
   },
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/maven/subset_result.json b/tests/data/maven/subset_result.json
@@ -12,5 +12,6 @@
   "session": {
       "id":  "16"
   },
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/maven/subset_scan_test_compile_lst_result.json b/tests/data/maven/subset_scan_test_compile_lst_result.json
@@ -16,5 +16,6 @@
   "session": {
       "id":  "16"
   },
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/nunit/subset_result.json b/tests/data/nunit/subset_result.json
@@ -46,5 +46,6 @@
   "goal": {"type": "subset-by-percentage", "percentage": 0.1},
   "ignoreNewTests": false,
   "session": { "id": "16" },
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/pytest/subset_result.json b/tests/data/pytest/subset_result.json
@@ -129,5 +129,6 @@
   },
   "goal": {"type": "subset-by-percentage", "percentage": 0.1},
   "ignoreNewTests": false,
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/robot/subset_result.json b/tests/data/robot/subset_result.json
@@ -29,5 +29,6 @@
   "goal": {"type": "subset-by-percentage", "percentage": 0.1},
   "ignoreNewTests": false,
   "session": { "id": "16" },
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": false
 }
diff --git a/tests/data/xctest/subset_result.json b/tests/data/xctest/subset_result.json
@@ -5,6 +5,7 @@
     "id": "16"
   },
   "ignoreNewTests": false,
+  "getTestsFromGuess": false,
   "getTestsFromPreviousSessions": true,
   "useServerSideOptimizationTarget": true
 }
diff --git a/tests/test_runners/test_raw.py b/tests/test_runners/test_raw.py
@@ -67,6 +67,7 @@ def test_subset(self):
                 'session': {'id': str(self.session_id)},
                 "goal": {"type": "subset-by-percentage", "percentage": 0.1},
                 "ignoreNewTests": False,
+                "getTestsFromGuess": False,
                 "getTestsFromPreviousSessions": False,
             })
             # Check split output
@@ -123,6 +124,7 @@ def test_subset_get_tests_from_previous_sessions(self):
             'session': {'id': str(self.session_id)},
             "goal": {"type": "subset-by-percentage", "percentage": 0.1},
             "ignoreNewTests": False,
+            "getTestsFromGuess": False,
             "getTestsFromPreviousSessions": True,
         })
 

Original file line number	Diff line number	Diff line change
`@@ -11,5 +11,6 @@`
`11`	`11`	`"goal": {"type": "subset-by-percentage", "percentage": 0.1},`
`12`	`12`	`"ignoreNewTests": false,`
`13`	`13`	`"session": { "id": "16" },`
	`14`	`+ "getTestsFromGuess": false,`
`14`	`15`	`"getTestsFromPreviousSessions": false`
`15`	`16`	`}`
Original file line number	Diff line number	Diff line change
`@@ -7,5 +7,6 @@`
`7`	`7`	`"session": { "id": "16" },`
`8`	`8`	`"goal": {"type": "subset-by-percentage", "percentage": 0.1},`
`9`	`9`	`"ignoreNewTests": false,`
	`10`	`+ "getTestsFromGuess": false,`
`10`	`11`	`"getTestsFromPreviousSessions": false`
`11`	`12`	`}`
Original file line number	Diff line number	Diff line change
`@@ -41,5 +41,6 @@`
`41`	`41`	`"goal": {"type": "subset-by-percentage", "percentage": 0.1},`
`42`	`42`	`"ignoreNewTests": false,`
`43`	`43`	`"session": { "id": "16" },`
	`44`	`+ "getTestsFromGuess": false,`
`44`	`45`	`"getTestsFromPreviousSessions": false`
`45`	`46`	`}`
Original file line number	Diff line number	Diff line change
`@@ -8,5 +8,6 @@`
`8`	`8`	`"session": {`
`9`	`9`	`"id": "16"`
`10`	`10`	`},`
	`11`	`+ "getTestsFromGuess": false,`
`11`	`12`	`"getTestsFromPreviousSessions": false`
`12`	`13`	`}`
Original file line number	Diff line number	Diff line change
`@@ -9,5 +9,6 @@`
`9`	`9`	`"goal": {"type": "subset-by-percentage", "percentage": 0.1},`
`10`	`10`	`"ignoreNewTests": false,`
`11`	`11`	`"session": { "id": "16" },`
	`12`	`+ "getTestsFromGuess": false,`
`12`	`13`	`"getTestsFromPreviousSessions": false`
`13`	`14`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,5 +13,6 @@`
`13`	`13`	`]`
`14`	`14`	`],`
`15`	`15`	`"testRunner": "cypress",`
	`16`	`+ "getTestsFromGuess": false,`
`16`	`17`	`"getTestsFromPreviousSessions": false`
`17`	`18`	`}`