CM-55551 - fix linter check comments

Author: erang-cycode

cycode/cli/files_collector/sca/npm/restore_npm_dependencies.py

@@ -77,7 +77,7 @@ def _create_document_from_lockfile(
         Args:
             document: Original document (package.json).
             lockfile_path: Path to the existing lockfile.
-    
+
         Returns:
             Document with lockfile content if successful, None otherwise.
         """
@@ -96,20 +96,20 @@ def _create_document_from_lockfile(
                 {'path': document.path, 'lockfile': lock_file_name, 'content_size': len(restore_file_content)},
             )
             return Document(relative_restore_file_path, restore_file_content, self.is_git_diff)
-        else:
-            logger.warning(
-                'Lockfile exists but could not read content, %s',
-                {'path': document.path, 'lockfile': lock_file_name, 'lockfile_path': lockfile_path},
-            )
-            return None
+      
+        logger.warning(
+            'Lockfile exists but could not read content, %s',
+            {'path': document.path, 'lockfile': lock_file_name, 'lockfile_path': lockfile_path},
+        )
+        return None
 
     def try_restore_dependencies(self, document: Document) -> Optional[Document]:
         """Override to prevent npm install when any lockfile exists.
 
         The base class uses document.absolute_path which might be None or incorrect.
         We need to use the same path resolution logic as get_manifest_file_path()
         to ensure we check for lockfiles in the correct location.
-        
+ 
         If any lockfile exists (package-lock.json, pnpm-lock.yaml, yarn.lock, deno.lock),
         we use it directly without running npm install to avoid generating invalid lockfiles.
         """
@@ -128,7 +128,8 @@ def try_restore_dependencies(self, document: Document) -> Optional[Document]:
             return super().try_restore_dependencies(document)
 
         # Check for existing lockfiles
-        logger.debug('Checking for existing lockfiles in directory, %s', {'directory': manifest_dir, 'path': document.path})
+        logger.debug('Checking for existing lockfiles in directory, %s', 
+                     {'directory': manifest_dir, 'path': document.path})
         existing_lock_file, checked_lockfiles = self._find_existing_lockfile(manifest_dir)
 
         logger.debug(
@@ -176,6 +177,5 @@ def prepare_manifest_file_path_for_command(manifest_file_path: str) -> str:
             # Handle both cases: with separator (e.g., '/path/to/package.json') and without (e.g., 'package.json')
             if os.sep in manifest_file_path:
                 return manifest_file_path.replace(os.sep + NPM_MANIFEST_FILE_NAME, '')
-            else:
-                return ''
+            return ''
         return manifest_file_path

tests/cli/files_collector/sca/npm/__init__.py

@@ -31,7 +31,7 @@ class TestRestoreNpmDependenciesAlternativeLockfiles:
     """Test that lockfiles prevent npm install from running."""
 
     @pytest.mark.parametrize(
-        'lockfile_name,lockfile_content,expected_content',
+        ('lockfile_name,lockfile_content,expected_content'),
         [
             ('pnpm-lock.yaml', 'lockfileVersion: 5.4\n', 'lockfileVersion: 5.4\n'),
             ('yarn.lock', '# yarn lockfile v1\n', '# yarn lockfile v1\n'),
@@ -90,7 +90,7 @@ def test_no_lockfile_exists_should_proceed_with_normal_flow(
                 return_value=None,
         ) as mock_super:
             # Execute
-            result = restore_npm_dependencies.try_restore_dependencies(document)
+            restore_npm_dependencies.try_restore_dependencies(document)
 
             # Verify: Should call parent's try_restore_dependencies (which will run npm install)
             mock_super.assert_called_once_with(document)
@@ -239,7 +239,8 @@ def test_multiple_lockfiles_should_use_first_found(
     def test_multiple_alternative_lockfiles_should_use_first_found(
             self, restore_npm_dependencies: RestoreNpmDependencies, tmp_path: Path
     ) -> None:
-        """Test that when multiple alternative lockfiles exist (but no package-lock.json), the first one found is used."""
+        """Test that when multiple alternative lockfiles exist (but no package-lock.json), 
+        the first one found is used."""
         package_json_path = tmp_path / 'package.json'
         yarn_lock_path = tmp_path / 'yarn.lock'
         pnpm_lock_path = tmp_path / 'pnpm-lock.yaml'
@@ -285,7 +286,7 @@ def test_lockfile_in_different_directory_should_not_be_found(
                 'try_restore_dependencies',
                 return_value=None,
         ) as mock_super:
-            result = restore_npm_dependencies.try_restore_dependencies(document)
+            restore_npm_dependencies.try_restore_dependencies(document)
 
             # Should proceed with normal flow since lockfile not in same directory
             mock_super.assert_called_once_with(document)