Fix B904 lint errors in SystemExit raises

- Add 'from None' to SystemExit in ClickException handler
- Add 'from e' to SystemExit in ValueError handler
- Properly chains exception context per ruff B904 rule

Author: shihanpan

src/workato_platform_cli/cli/commands/profiles.py

@@ -369,24 +369,28 @@ async def delete(
 
 
 async def _create_profile_non_interactive(
-    region: str,
-    api_token: str,
+    region: str | None,
+    api_token: str | None,
     api_url: str | None,
 ) -> tuple[ProfileData, str] | None:
     """Create profile data non-interactively.
 
     Returns (ProfileData, token) on success, or None on error (error already echoed).
     """
-    # Validate custom region requirements
+    # Validate required parameters
+    if not region:
+        click.echo("❌ --region is required in non-interactive mode")
+        return None
+    if not api_token:
+        click.echo("❌ --api-token is required in non-interactive mode")
+        return None
     if region == "custom" and not api_url:
         click.echo("❌ --api-url is required when region=custom")
         return None
 
     # Get region info
     if region == "custom":
-        region_info: RegionInfo = RegionInfo(
-            region="custom", name="Custom", url=api_url
-        )
+        region_info = RegionInfo(region="custom", name="Custom", url=api_url)
     else:
         region_info_lookup = AVAILABLE_REGIONS.get(region)
         if not region_info_lookup:
@@ -450,21 +454,13 @@ async def create(
         click.echo(f"❌ Profile '{profile_name}' already exists")
         click.echo("💡 Use 'workato profiles use' to switch to it")
         click.echo("💡 Or use 'workato profiles delete' to remove it first")
-        raise SystemExit(1)
+        return
 
     # Get profile data and token (either interactively or non-interactively)
     if non_interactive:
-        # Validate required parameters for non-interactive mode
-        if not region:
-            click.echo("❌ --region is required in non-interactive mode")
-            return
-        if not api_token:
-            click.echo("❌ --api-token is required in non-interactive mode")
-            return
-
         result = await _create_profile_non_interactive(region, api_token, api_url)
         if result is None:
-            raise SystemExit(1)
+            return
         profile_data, token = result
     else:
         click.echo(f"🔧 Creating profile: {profile_name}")
@@ -479,14 +475,14 @@ async def create(
             )
         except click.ClickException:
             click.echo("❌ Profile creation cancelled")
-            raise SystemExit(1)
+            return
 
     # Save profile (common for both modes)
     try:
         config_manager.profile_manager.set_profile(profile_name, profile_data, token)
     except ValueError as e:
         click.echo(f"❌ Failed to save profile: {e}")
-        raise SystemExit(1)
+        return
 
     # Set as current profile (common for both modes)
     config_manager.profile_manager.set_current_profile(profile_name)

tests/unit/commands/test_profiles.py

@@ -1009,10 +1009,8 @@ async def test_create_profile_already_exists(
     )
 
     assert create.callback
-    with pytest.raises(SystemExit) as exc_info:
-        await create.callback(profile_name="existing", config_manager=config_manager)
+    await create.callback(profile_name="existing", config_manager=config_manager)
 
-    assert exc_info.value.code == 1
     output = capsys.readouterr().out
     assert "❌ Profile 'existing' already exists" in output
     assert "Use 'workato profiles use' to switch to it" in output
@@ -1034,10 +1032,8 @@ async def test_create_profile_cancelled_region_selection(
     )
 
     assert create.callback
-    with pytest.raises(SystemExit) as exc_info:
-        await create.callback(profile_name="new_profile", config_manager=config_manager)
+    await create.callback(profile_name="new_profile", config_manager=config_manager)
 
-    assert exc_info.value.code == 1
     output = capsys.readouterr().out
     assert "❌ Profile creation cancelled" in output
 
@@ -1058,10 +1054,8 @@ async def test_create_profile_empty_token(
     )
 
     assert create.callback
-    with pytest.raises(SystemExit) as exc_info:
-        await create.callback(profile_name="new_profile", config_manager=config_manager)
+    await create.callback(profile_name="new_profile", config_manager=config_manager)
 
-    assert exc_info.value.code == 1
     output = capsys.readouterr().out
     assert "❌ Profile creation cancelled" in output
 
@@ -1082,10 +1076,8 @@ async def test_create_profile_authentication_failure(
     )
 
     assert create.callback
-    with pytest.raises(SystemExit) as exc_info:
-        await create.callback(profile_name="new_profile", config_manager=config_manager)
+    await create.callback(profile_name="new_profile", config_manager=config_manager)
 
-    assert exc_info.value.code == 1
     output = capsys.readouterr().out
     assert "❌ Profile creation cancelled" in output
 
@@ -1115,11 +1107,53 @@ async def test_create_profile_keyring_failure(
     )
 
     assert create.callback
-    with pytest.raises(SystemExit) as exc_info:
-        await create.callback(profile_name="new_profile", config_manager=config_manager)
-
-    assert exc_info.value.code == 1
+    await create.callback(profile_name="new_profile", config_manager=config_manager)
 
     output = capsys.readouterr().out
     assert "❌ Failed to save profile:" in output
     assert "Failed to store token in keyring" in output
+
+
+@pytest.mark.asyncio
+async def test_create_profile_non_interactive(
+    capsys: pytest.CaptureFixture[str],
+    make_config_manager: Callable[..., Mock],
+) -> None:
+    """Test successful non-interactive profile creation."""
+    config_manager = make_config_manager(
+        get_profile=Mock(return_value=None),  # Profile doesn't exist yet
+        set_profile=Mock(),
+        set_current_profile=Mock(),
+    )
+
+    # Mock Workato API client
+    mock_client = AsyncMock()
+    mock_user = Mock()
+    mock_user.id = 123
+    mock_client.users_api.get_workspace_details = AsyncMock(return_value=mock_user)
+    mock_client.__aenter__ = AsyncMock(return_value=mock_client)
+    mock_client.__aexit__ = AsyncMock(return_value=None)
+
+    with patch(
+        "workato_platform_cli.cli.commands.profiles.Workato",
+        return_value=mock_client,
+    ):
+        assert create.callback
+        await create.callback(
+            profile_name="test_profile",
+            region="us",
+            api_token="test_token",
+            api_url=None,
+            non_interactive=True,
+            config_manager=config_manager,
+        )
+
+    output = capsys.readouterr().out
+    assert "✅ Profile 'test_profile' created successfully" in output
+    assert "✅ Set 'test_profile' as the active profile" in output
+
+    # Verify profile was set and made current
+    config_manager.profile_manager.set_profile.assert_called_once()
+    config_manager.profile_manager.set_current_profile.assert_called_once_with(
+        "test_profile"
+    )