smart-tests-cli 2.11.2__tar.gz → 2.12.1__tar.gz

{smart_tests_cli-2.11.2/smart_tests_cli.egg-info → smart_tests_cli-2.12.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: smart-tests-cli
-Version: 2.11.2
+Version: 2.12.1
 Summary: Smart Tests CLI
 Author-email: CloudBees <support@cloudbees.com>
 License: Apache Software License v2

{smart_tests_cli-2.11.2 → smart_tests_cli-2.12.1}/smart_tests/commands/record/build.py

@@ -384,9 +384,17 @@ def build(
         header = ["Name", "Path", "HEAD Commit"]
         rows = [[w.name, w.dir, w.commit_hash] for w in ws]
         click.echo(tabulate(rows, header, tablefmt="github"))
-        click.echo(
-            f"\nVisit https://app.launchableinc.com/organizations/{org}/workspaces/"
-            f"{workspace}/data/builds/{build_id} to view this build and its test sessions")
+        url = _get_build_url(build_id, org, workspace)
+        click.echo(f"\nVisit {url} to view this build and its test sessions")
+
+    def _get_build_url(build_id, org, workspace) -> str:
+        cbp_workspace = client.get_cbp_workspace()
+        if cbp_workspace:
+            org_cbp_id, ws_cbp_id = cbp_workspace
+            url = f"https://cloudbees.io/{org_cbp_id}/{ws_cbp_id}/smart-tests/data/builds/{build_id}"
+        else:
+            url = f"https://app.launchableinc.com/organizations/{org}/workspaces/{workspace}/data/builds/{build_id}"
+        return url
 
     # all the logics at the high level
     if len(commits) == 0:

{smart_tests_cli-2.11.2 → smart_tests_cli-2.12.1}/smart_tests/commands/record/tests.py

@@ -484,11 +484,20 @@ class RecordTests:
         if duration == 0:
             click.secho("\nTotal test duration is 0."
                         "\nPlease check whether the test duration times in report files are correct.", fg="yellow")
+        url = self._get_test_session_url()
         click.echo(
-            f"\nVisit https://app.launchableinc.com/organizations/{self.org}/workspaces/"
-            f"{self.workspace}/test-sessions/{self.test_session_id} to view uploaded test results "
+            f"\nVisit {url} to view uploaded test results "
             f"(or run `launchable inspect tests --test-session-id {self.test_session_id}`)")
 
+    def _get_test_session_url(self) -> str:
+        cbp_workspace = self.client.get_cbp_workspace()
+        if cbp_workspace:
+            org_cbp_id, ws_cbp_id = cbp_workspace
+            return f"https://cloudbees.io/{org_cbp_id}/{ws_cbp_id}/smart-tests/test-sessions/{self.test_session_id}"
+        else:
+            return (f"https://app.launchableinc.com/organizations/{self.org}/workspaces/"
+                    f"{self.workspace}/test-sessions/{self.test_session_id}")
+
 
 tests = Group(name="tests", callback=RecordTests, help="Record test results")
 

{smart_tests_cli-2.11.2 → smart_tests_cli-2.12.1}/smart_tests/commands/subset.py

@@ -603,8 +603,10 @@ class Subset(TestPathWriter):
         elif self.fallback_mode == FallbackMode.RANDOM_SAMPLE:
             target_fraction = float(self.target) if self.target is not None else 1.0
             click.echo(
-                f"Warning: Smart Tests could not retrieve a subset. Falling back to local random sample at {
-                    target_fraction:.0%}.", err=True, )
+                f"Warning: Smart Tests could not retrieve a subset. "
+                f"Falling back to local random sample at {target_fraction:.0%}.",
+                err=True,
+            )
             return SubsetResult.from_random_sample(self.test_paths, target_fraction)
         else:
             return SubsetResult.from_test_paths(self.test_paths)

{smart_tests_cli-2.11.2 → smart_tests_cli-2.12.1}/smart_tests/commands/view/flaky_tests.py

@@ -61,10 +61,10 @@ def flaky_tests(
         type=parse_datetime_with_timezone,
         metavar="DATE"
     )] = None,
-    test_suite: Annotated[str | None, typer.Option(
-        "--test-suite",
-        help="Test suite name filter (e.g., 'unit-tests')",
-        metavar="NAME"
+    test_path: Annotated[str | None, typer.Option(
+        "--test-path",
+        help="Test path filter (full path string, e.g., 'class=MyTest')",
+        metavar="PATH"
     )] = None,
     limit: Annotated[int | None, typer.Option(
         "--limit",
@@ -83,11 +83,11 @@ def flaky_tests(
     if weeks:
         params["weeks"] = str(weeks)
     if from_date:
-        params["from"] = str(from_date)
+        params["from"] = from_date.datetime().strftime("%Y-%m-%d")
     if to_date:
-        params["to"] = str(to_date)
-    if test_suite:
-        params["test-suite"] = test_suite
+        params["to"] = to_date.datetime().strftime("%Y-%m-%d")
+    if test_path:
+        params["test-path"] = test_path
     if limit:
         params["limit"] = str(limit)
 

{smart_tests_cli-2.11.2 → smart_tests_cli-2.12.1}/smart_tests/commands/view/test_results.py

@@ -18,9 +18,9 @@ from ...utils.typer_types import DateTimeWithTimezone, parse_datetime_with_timez
 
 class TestStatus(str, Enum):
     """Test execution status"""
-    PASSED = "PASSED"
-    FAILED = "FAILED"
-    SKIPPED = "SKIPPED"
+    SUCCESS = "SUCCESS"
+    FAILURE = "FAILURE"
+    SKIP = "SKIP"
     FLAKE = "FLAKE"
 
     @staticmethod
@@ -30,7 +30,7 @@ class TestStatus(str, Enum):
             if member.value.lower() == value.lower():
                 return member
         raise BadCmdLineException(
-            f"Invalid status: '{value}'. Valid values: PASSED, FAILED, SKIPPED, FLAKE"
+            f"Invalid status: '{value}'. Valid values: SUCCESS, FAILURE, SKIP, FLAKE"
         )
 
 
@@ -45,7 +45,7 @@ def test_results(
     )],
     status: Annotated[TestStatus | None, typer.Option(
         "--status",
-        help="Filter by test status (PASSED, FAILED, SKIPPED, FLAKE)",
+        help="Filter by test status (SUCCESS, FAILURE, SKIP, FLAKE)",
         type=TestStatus.from_str,
         metavar="STATUS"
     )] = None,
@@ -102,9 +102,9 @@ def test_results(
     if test_suite:
         params["test-suite"] = test_suite
     if from_date:
-        params["from"] = str(from_date)
+        params["from"] = from_date.datetime().strftime("%Y-%m-%d")
     if to_date:
-        params["to"] = str(to_date)
+        params["to"] = to_date.datetime().strftime("%Y-%m-%d")
     if logs:
         params["include-logs"] = "true"
     if limit:

{smart_tests_cli-2.11.2 → smart_tests_cli-2.12.1}/smart_tests/test_runners/dotnet.py

@@ -1,8 +1,10 @@
 import glob
 import os
 from typing import Annotated, List
+from urllib.parse import unquote
 
 import click
+from junitparser import TestCase, TestSuite  # type: ignore
 
 import smart_tests.args4p.typer as typer
 from smart_tests.commands.record.tests import RecordTests
@@ -11,6 +13,10 @@ from smart_tests.test_runners import smart_tests
 from smart_tests.test_runners.nunit import nunit_parse_func
 from smart_tests.testpath import TestPath
 
+NUNIT_FORMAT = "nunit"
+JUNIT_FORMAT = "junit"
+SUPPORTED_FORMATS = (JUNIT_FORMAT, NUNIT_FORMAT)
+
 
 # main subset logic
 def do_subset(client: Subset, bare):
@@ -78,6 +84,53 @@ def subset(
     do_subset(client, bare)
 
 
+def _clean_field(value):
+    """
+    Trim tab and other characters that are not appropriate as part of a test path.
+    """
+    if not value:
+        return value
+    for token in value.split("\t"):
+        token = token.strip()
+        if token:
+            return unquote(token)
+    return ""
+
+
+def _junit_path_builder(file_path_normalizer):
+    """
+    Build a TestPath from a JUnit <testcase> element so the resulting structure
+    matches the one produced by the NUnit parser:
+
+        Assembly -> TestSuite -> ... -> TestSuite -> TestCase
+
+    Assembly is taken from the <testsuite> name (e.g. "rocket-car-dotnet.dll")
+    that `dotnet test --logger junit` emits.
+    """
+
+    def build(case: TestCase, suite: TestSuite, report_file: str) -> TestPath:
+        test_path: TestPath = []
+
+        assembly = _clean_field(suite._elem.attrib.get("name"))
+        if assembly:
+            test_path.append({"type": "Assembly", "name": assembly})
+
+        classname = case._elem.attrib.get("classname") or suite._elem.attrib.get("classname")
+        classname = _clean_field(classname)
+        if classname:
+            for part in classname.split("."):
+                if part:
+                    test_path.append({"type": "TestSuite", "name": part})
+
+        case_name = _clean_field(case._elem.attrib.get("name"))
+        if case_name:
+            test_path.append({"type": "TestCase", "name": case_name})
+
+        return test_path
+
+    return build
+
+
 @smart_tests.record.tests
 def record_tests(
     client: RecordTests,
@@ -85,10 +138,21 @@ def record_tests(
         multiple=True,
         help="Test report files to process"
     )],
+    format: Annotated[str, typer.Option(
+        "--format",
+        help=f"Test report format. One of: {', '.join(SUPPORTED_FORMATS)}.",
+    )] = NUNIT_FORMAT,
 ):
     """
-    Alpha: Supports only NUnit report formats.
+    Alpha: Supports NUnit (default) and JUnit report formats.
     """
+    if format not in SUPPORTED_FORMATS:
+        click.secho(
+            f"Unsupported --format value: {format}. Supported formats: {', '.join(SUPPORTED_FORMATS)}",
+            fg='red',
+            err=True)
+        raise typer.Exit(1)
+
     for file in files:
         match = False
         for t in glob.iglob(file, recursive=True):
@@ -100,7 +164,10 @@ def record_tests(
         if not match:
             click.echo(f"No matches found: {file}", err=True)
 
-    # Note: we support only Nunit test report format now.
-    # If we need to support another format e.g) JUnit, trc, then we'll add a option
-    client.parse_func = nunit_parse_func
+    if format == NUNIT_FORMAT:
+        client.parse_func = nunit_parse_func
+    else:
+        client.path_builder = _junit_path_builder(client.file_path_normalizer)
+        client.junitxml_parse_func = None
+
     client.run()

{smart_tests_cli-2.11.2 → smart_tests_cli-2.12.1}/smart_tests/utils/smart_tests_client.py

@@ -24,6 +24,7 @@ class SmartTestsClient:
         self.tracking_client = tracking_client
         self.organization, self.workspace = ensure_org_workspace()
         self._workspace_state_cache: Dict[str, str | bool] | None = None
+        self._cbp_workspace_cache: tuple[str, str] | None = None
 
     def request(
         self,
@@ -105,6 +106,27 @@ class SmartTestsClient:
         state = self._get_workspace_state()
         return state.get('pts_v2', False)
 
+    def get_cbp_workspace(self) -> tuple[str, str] | None:
+        """
+        Returns (org_cbp_id, workspace_cbp_id) if this is a CBP workspace, else None.
+        """
+        state = self._get_workspace_state()
+        if not state.get('is_cbp_workspace', False):
+            return None
+
+        if self._cbp_workspace_cache is not None:
+            return self._cbp_workspace_cache
+
+        try:
+            res = self.request("get", "cbp-workspace")
+            res.raise_for_status()
+            data = res.json()
+            self._cbp_workspace_cache = (data['organizationCbpId'], data['workspaceCbpId'])
+            return self._cbp_workspace_cache
+        except Exception as e:
+            self.print_exception_and_recover(e, "Failed to get CBP workspace info")
+            return None
+
     def _get_workspace_state(self) -> dict:
         """
         Get the current state of the workspace.
@@ -119,6 +141,7 @@ class SmartTestsClient:
             self._workspace_state_cache = {
                 'fail_fast_mode': state.get('isFailFastMode', False),
                 'pts_v2': state.get('isPtsV2Enabled', False),
+                'is_cbp_workspace': state.get('isCbpWorkspace', False),
             }
             return self._workspace_state_cache
         except Exception as e:

{smart_tests_cli-2.11.2 → smart_tests_cli-2.12.1/smart_tests_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: smart-tests-cli
-Version: 2.11.2
+Version: 2.12.1
 Summary: Smart Tests CLI
 Author-email: CloudBees <support@cloudbees.com>
 License: Apache Software License v2

{smart_tests_cli-2.11.2 → smart_tests_cli-2.12.1}/smart_tests_cli.egg-info/SOURCES.txt

@@ -374,6 +374,7 @@ src/test/java/com/launchableinc/ingest/commits/BoundedExecutorServiceTest.java
 src/test/java/com/launchableinc/ingest/commits/CommitGraphCollectorTest.java
 src/test/java/com/launchableinc/ingest/commits/ConcurrentConsumerTest.java
 src/test/java/com/launchableinc/ingest/commits/FileChunkStreamerTest.java
+src/test/java/com/launchableinc/ingest/commits/GitFileTest.java
 src/test/java/com/launchableinc/ingest/commits/MainTest.java
 src/test/java/com/launchableinc/ingest/commits/PassThroughTreeReceiverImpl.java
 src/test/java/com/launchableinc/ingest/commits/ProgressReporterTest.java
@@ -484,6 +485,10 @@ tests/data/cypress/empty.xml
 tests/data/cypress/record_test_result.json
 tests/data/cypress/subset_result.json
 tests/data/cypress/test-result.xml
+tests/data/dotnet/junit-result-with-metadata.xml
+tests/data/dotnet/junit-result.xml
+tests/data/dotnet/record_test_result-junit-with-metadata.json
+tests/data/dotnet/record_test_result-junit.json
 tests/data/dotnet/record_test_result.json
 tests/data/dotnet/test-result.xml
 tests/data/file/record_test_result.json

{smart_tests_cli-2.11.2 → smart_tests_cli-2.12.1}/src/main/java/com/launchableinc/ingest/commits/GitFile.java

@@ -5,10 +5,10 @@ import org.eclipse.jgit.lib.ObjectLoader;
 import org.eclipse.jgit.lib.ObjectReader;
 
 import java.io.IOException;
-import java.io.InputStreamReader;
 import java.io.OutputStream;
-import java.io.Reader;
+import java.nio.ByteBuffer;
 import java.nio.charset.CharacterCodingException;
+import java.nio.charset.CodingErrorAction;
 import java.util.function.Supplier;
 
 import static java.nio.charset.StandardCharsets.UTF_8;
@@ -68,12 +68,10 @@ final class GitFile implements VirtualFile {
    */
   public boolean isText() throws IOException {
     try {
-      char[] c = new char[1024];
-      try (Reader r = new InputStreamReader(open().openStream(), UTF_8)) {
-        while (r.read(c)!= -1) {
-          // Read the file until EOF.
-        }
-      }
+      UTF_8.newDecoder()
+          .onMalformedInput(CodingErrorAction.REPORT)
+          .onUnmappableCharacter(CodingErrorAction.REPORT)
+          .decode(ByteBuffer.wrap(open().getCachedBytes(Integer.MAX_VALUE)));
       return true;
     } catch (CharacterCodingException e) {
       return false;

{smart_tests_cli-2.11.2 → smart_tests_cli-2.12.1}/src/test/java/com/launchableinc/ingest/commits/AllTests.java

@@ -12,6 +12,7 @@ import org.junit.runners.Suite.SuiteClasses;
     FileChunkStreamerTest.class,
     MainTest.class,
     SSLBypassTest.class,
+    GitFileTest.class,
     ProgressReporterTest.class
 })
 public class AllTests {}

smart_tests_cli-2.12.1/src/test/java/com/launchableinc/ingest/commits/GitFileTest.java

@@ -0,0 +1,51 @@
+package com.launchableinc.ingest.commits;
+
+import static com.google.common.truth.Truth.assertThat;
+
+import java.io.File;
+import org.eclipse.jgit.api.Git;
+import org.eclipse.jgit.lib.ObjectId;
+import org.eclipse.jgit.lib.ObjectInserter;
+import org.eclipse.jgit.lib.Repository;
+import org.junit.Rule;
+import org.junit.Test;
+import org.junit.rules.TemporaryFolder;
+import org.junit.runner.RunWith;
+import org.junit.runners.JUnit4;
+
+import static org.eclipse.jgit.lib.Constants.OBJ_BLOB;
+
+@RunWith(JUnit4.class)
+public class GitFileTest {
+
+  @Rule public TemporaryFolder tmp = new TemporaryFolder();
+
+  @Test
+  public void isText_textFile_returnsTrue() throws Exception {
+    assertThat(gitFileWithContent("public class Hello {}".getBytes()).isText()).isTrue();
+  }
+
+  @Test
+  public void isText_binaryFile_returnsFalse() throws Exception {
+    // 0xFF 0xFE is invalid UTF-8
+    assertThat(gitFileWithContent(new byte[]{(byte) 0xFF, (byte) 0xFE, 0x00, 0x01}).isText()).isFalse();
+  }
+
+  @Test
+  public void isText_utf8MultiByte_returnsTrue() throws Exception {
+    assertThat(gitFileWithContent("日本語テスト".getBytes(java.nio.charset.StandardCharsets.UTF_8)).isText()).isTrue();
+  }
+
+  private GitFile gitFileWithContent(byte[] content) throws Exception {
+    File repoDir = tmp.newFolder();
+    try (Git git = Git.init().setDirectory(repoDir).call()) {
+      Repository repo = git.getRepository();
+      ObjectId blobId;
+      try (ObjectInserter inserter = repo.newObjectInserter()) {
+        blobId = inserter.insert(OBJ_BLOB, content);
+        inserter.flush();
+      }
+      return new GitFile("test", "file", blobId, repo::newObjectReader);
+    }
+  }
+}

{smart_tests_cli-2.11.2 → smart_tests_cli-2.12.1}/tests/commands/test_view_flaky_tests.py

@@ -197,34 +197,41 @@ class ViewFlakyTestsTest(CliTestCase):
         )
         self.assert_success(result)
 
-        # Verify the request was made and query parameters were passed
+        # Verify the request was made and query parameters were plain YYYY-MM-DD dates
         self.assertGreater(len(responses.calls), 0)
         self.assertIn("/view/flaky-tests", responses.calls[0].request.url)
-        self.assertIn("from=", responses.calls[0].request.url)
-        self.assertIn("to=", responses.calls[0].request.url)
+        self.assertIn("from=2026-04-08", responses.calls[0].request.url)
+        self.assertIn("to=2026-04-14", responses.calls[0].request.url)
 
     @responses.activate
     @mock.patch.dict(os.environ, {"SMART_TESTS_TOKEN": CliTestCase.smart_tests_token})
-    def test_flaky_tests_with_test_suite(self):
-        """Test flaky tests query with test-suite filter"""
+    def test_flaky_tests_with_test_path(self):
+        """Test flaky tests query with test-path filter"""
         mock_json_response = {
             "data": {
                 "weeks": [
                     {
-                        "weekDate": "2026-W19",
-                        "calculationStatus": "CALCULATED",
-                        "calculationTime": "2026-05-11T10:30:00Z",
-                        "flakyTests": [],
-                        "flakyTestCount": 0
-                    }
-                ]
-            },
+                        "id": "0197c5cc-8468-73f0-ade2-619e21d20049",
+                        "testPath": (
+                            "class=com.cloudbees.plugin.spec.CreateOrUpdateObjectsSuite"
+                            "#testcase=C000001 Sanity. Create service with deployment definitions from yaml file "
+                        ),
+                        "status": "SUCCESS",
+                        "totalDuration": 145087,
+                        "passed": 1,
+                        "failed": 0,
+                        "skipped": 0,
+                        "session": {
+                            "id": 4595735,
+                            "buildId": 4196923,
+                            "branch": "master",
+                            "links": [],
+                            "createdAt": "2025-07-01T07:50:22.322649Z"},
+                        "createdAt": "2025-07-01T11:43:24.246347Z"}]},
             "metadata": {
-                "weeksRequested": 1,
-                "weeksReturned": 1,
-                "latestWeek": "2026-W19"
-            }
-        }
+                        "totalCount": 3,
+                        "limit": 50,
+                "offset": 0}}
 
         responses.add(
             responses.GET,
@@ -233,13 +240,22 @@ class ViewFlakyTestsTest(CliTestCase):
             status=200,
         )
 
-        result = self.cli("view", "flaky-tests", "--test-suite", "unit-tests", mix_stderr=False)
+        result = self.cli(
+            "view",
+            "flaky-tests",
+            "--test-path",
+            "class=com.cloudbees.plugin.spec.CreateOrUpdateObjectsSuite"
+            "#testcase=C000001 Sanity. Create service with deployment definitions from yaml file ",
+            mix_stderr=False)
         self.assert_success(result)
 
-        # Verify the request was made and query parameter was passed
+# Verify the request was made and query parameter was passed
         self.assertGreater(len(responses.calls), 0)
         self.assertIn("/view/flaky-tests", responses.calls[0].request.url)
-        self.assertIn("test-suite=unit-tests", responses.calls[0].request.url)
+        self.assertIn(
+            "test-path=class%3Dcom.cloudbees.plugin.spec.CreateOrUpdateObjectsSuite"
+            "%23testcase%3DC000001+Sanity.+Create+service+with+deployment+definitions+from+yaml+file+",
+            responses.calls[0].request.url)
 
     @responses.activate
     @mock.patch.dict(os.environ, {"SMART_TESTS_TOKEN": CliTestCase.smart_tests_token})