chore(cli): drop self-explanatory comments

lib/cli/tests/authentication/test_auth_main.py

@@ -333,7 +333,6 @@ class TestAuthenticationCommand:
     @patch("crewai_core.auth.oauth2.httpx.post")
     def test_poll_for_token_error(self, mock_post):
         """Test the method to poll for token (error path)."""
-        # Setup mock to return error
         mock_response_error = MagicMock()
         mock_response_error.status_code = 400
         mock_response_error.json.return_value = {

lib/cli/tests/authentication/test_utils.py

@@ -12,7 +12,6 @@ class TestUtils(unittest.TestCase):
     def test_validate_jwt_token(self, mock_jwt, mock_pyjwkclient):
         mock_jwt.decode.return_value = {"exp": 1719859200}
 
-        # Create signing key object mock with a .key attribute
         mock_pyjwkclient.return_value.get_signing_key_from_jwt.return_value = MagicMock(
             key="mock_signing_key"
         )

lib/cli/tests/deploy/test_validate.py

@@ -164,7 +164,6 @@ def test_poetry_lock_is_accepted(tmp_path: Path) -> None:
 
 def test_stale_lockfile_warns(tmp_path: Path) -> None:
     _scaffold_standard_crew(tmp_path)
-    # Make lockfile older than pyproject.
     lock = tmp_path / "uv.lock"
     pyproject = tmp_path / "pyproject.toml"
     old_time = pyproject.stat().st_mtime - 60

lib/cli/tests/skills/test_main.py

@@ -41,14 +41,9 @@ def skill_command():
             yield cmd
 
 
-# ---------------------------------------------------------------------------
-# create
-# ---------------------------------------------------------------------------
-
 class TestSkillCreate:
     def test_create_in_project(self, skill_command, tmp_path):
         with in_temp_dir():
-            # Simulate being inside a project
             Path("pyproject.toml").write_text("[tool.poetry]\nname = 'test'\n")
             skill_command.create("my-skill")
             assert Path("skills/my-skill/SKILL.md").exists()
@@ -75,10 +70,6 @@ class TestSkillCreate:
                 skill_command.create("existing-skill", in_project=False)
 
 
-# ---------------------------------------------------------------------------
-# install
-# ---------------------------------------------------------------------------
-
 class TestSkillInstall:
     def _zip_skill(self, name: str) -> bytes:
         buf = io.BytesIO()
@@ -118,10 +109,6 @@ class TestSkillInstall:
             assert Path("skills/my-skill/SKILL.md").exists()
 
 
-# ---------------------------------------------------------------------------
-# publish
-# ---------------------------------------------------------------------------
-
 class TestSkillPublish:
     def test_publish_no_skill_md(self, skill_command):
         with in_temp_dir():
@@ -155,7 +142,6 @@ class TestSkillPublish:
                 mock_resp.status_code = 200
                 mock_resp.json.return_value = {}
                 mock_client.publish_skill.return_value = mock_resp
-                # No org set → should SystemExit (no org_name in settings)
                 with patch("crewai_cli.skills.main.Settings") as mock_settings_cls:
                     mock_settings_cls.return_value.org_name = None
                     mock_settings_cls.return_value.enterprise_base_url = None
@@ -184,15 +170,10 @@ class TestSkillPublish:
             assert call_kwargs.kwargs["version"] == "1.0.0"
 
 
-# ---------------------------------------------------------------------------
-# list_cached
-# ---------------------------------------------------------------------------
-
 class TestSkillListCached:
     def test_list_cached_empty(self, skill_command, capsys):
         with in_temp_dir():
             skill_command.list_cached()
-            # Should not raise
 
     def test_list_cached_shows_project_skills(self, skill_command, capsys):
         with in_temp_dir():
@@ -202,4 +183,3 @@ class TestSkillListCached:
                 "---\nname: my-skill\nversion: 0.5.0\ndescription: A skill.\n---\nBody."
             )
             skill_command.list_cached()
-            # Should complete without error

lib/cli/tests/test_crew_test.py

@@ -83,7 +83,6 @@ def test_test_crew_called_process_error(mock_subprocess_run, click):
 @mock.patch("crewai_cli.evaluate_crew.click")
 @mock.patch("crewai_cli.evaluate_crew.subprocess.run")
 def test_test_crew_unexpected_exception(mock_subprocess_run, click):
-    # Arrange
     n_iterations = 5
     mock_subprocess_run.side_effect = Exception("Unexpected error")
     evaluate_crew.evaluate_crew(n_iterations, "gpt-4o")

lib/cli/tests/test_settings_command.py

@@ -35,7 +35,6 @@ class TestSettingsCommand(unittest.TestCase):
 
         self.settings_command.list()
 
-        # Tests that the table is created skipping hidden settings
         mock_table_instance.add_row.assert_has_calls(
             [
                 call(
@@ -48,7 +47,6 @@ class TestSettingsCommand(unittest.TestCase):
             ]
         )
 
-        # Tests that the table is printed
         mock_console.print.assert_called_once_with(mock_table_instance)
 
     def test_set_valid_keys(self):

lib/cli/tests/test_token_manager.py

@@ -146,7 +146,6 @@ class TestAtomicFileOperations(unittest.TestCase):
         self.temp_dir = tempfile.mkdtemp()
         self.original_get_path = TokenManager._get_secure_storage_path
 
-        # Patch to use temp directory
         def mock_get_path() -> Path:
             return Path(self.temp_dir)
 
@@ -182,7 +181,6 @@ class TestAtomicFileOperations(unittest.TestCase):
         mock_get_key.return_value = Fernet.generate_key()
         tm = TokenManager()
 
-        # Create file first
         file_path = Path(self.temp_dir) / "test.txt"
         file_path.write_bytes(b"original")
 
@@ -231,7 +229,6 @@ class TestAtomicFileOperations(unittest.TestCase):
 
         tm._atomic_write_secure_file("test.txt", b"content")
 
-        # Check no temp files remain
         temp_files = list(Path(self.temp_dir).glob(".test.txt.*"))
         self.assertEqual(len(temp_files), 0)
 
@@ -285,7 +282,6 @@ class TestAtomicFileOperations(unittest.TestCase):
         mock_get_key.return_value = Fernet.generate_key()
         tm = TokenManager()
 
-        # Should not raise
         tm._delete_secure_file("nonexistent.txt")
 
 

lib/cli/tests/tools/test_main.py

@@ -27,9 +27,7 @@ def in_temp_dir():
 
 @pytest.fixture
 def tool_command():
-    # Create a temporary directory for each test to avoid token storage conflicts
     with tempfile.TemporaryDirectory() as temp_dir:
-        # Mock the secure storage path to use the temp directory
         with patch.object(
             TokenManager, "_get_secure_storage_path", return_value=Path(temp_dir)
         ):
@@ -97,7 +95,6 @@ def test_install_success(
         env=unittest.mock.ANY,
     )
 
-    # Verify _print_current_organization was called
     mock_print_org.assert_called_once()