Fix lint and coverage: top-level imports, pragma no cover on Robot import

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: neha-sanse

percy/__init__.py

@@ -3,8 +3,8 @@
 
 # Robot Framework support — graceful when robotframework is not installed
 try:
-    from percy.robot_library import PercyLibrary
-except ImportError:
+    from percy.robot_library import PercyLibrary  # pragma: no cover
+except ImportError:  # pragma: no cover
     pass
 
 # import snapshot command

tests/test_robot_library.py

@@ -1,16 +1,12 @@
 """Tests for Robot Framework library integration.
 
-These tests import robot_library directly (not via percy.__init__) to
-avoid triggering the playwright import in percy.screenshot at module level.
-The keyword methods use lazy imports so playwright is only needed at call time.
+These tests mock playwright at module level so they can run without
+playwright installed, since percy.screenshot imports it at top level.
 """
 import sys
 from unittest.mock import MagicMock, patch
 
-import pytest
-
-
-# Mock playwright before any percy imports so screenshot.py can load
+# Mock playwright before any percy imports
 _mock_playwright = MagicMock()
 _mock_playwright._repo_version.version = "1.50.0"
 _mock_playwright.sync_api.Error = Exception
@@ -19,54 +15,51 @@
 sys.modules.setdefault("playwright._repo_version", _mock_playwright._repo_version)
 sys.modules.setdefault("playwright.sync_api", _mock_playwright.sync_api)
 
+from percy.robot_library import (  # noqa: E402 pylint: disable=wrong-import-position
+    PercyLibrary,
+    _parse_bool,
+    _parse_csv,
+    _parse_json,
+    _parse_widths,
+)
+
 
 class TestParseHelpers:
     def test_parse_bool_none(self):
-        from percy.robot_library import _parse_bool
         assert _parse_bool(None) is None
 
     def test_parse_bool_true(self):
-        from percy.robot_library import _parse_bool
         assert _parse_bool("True") is True
         assert _parse_bool("true") is True
         assert _parse_bool("1") is True
 
     def test_parse_bool_false(self):
-        from percy.robot_library import _parse_bool
         assert _parse_bool("False") is False
         assert _parse_bool("no") is False
 
     def test_parse_widths_string(self):
-        from percy.robot_library import _parse_widths
         assert _parse_widths("375,768,1280") == [375, 768, 1280]
 
     def test_parse_widths_none(self):
-        from percy.robot_library import _parse_widths
         assert _parse_widths(None) is None
 
     def test_parse_csv_string(self):
-        from percy.robot_library import _parse_csv
         assert _parse_csv("regression, homepage") == ["regression", "homepage"]
 
     def test_parse_json_string(self):
-        from percy.robot_library import _parse_json
         assert _parse_json('{"key": true}') == {"key": True}
 
     def test_parse_json_none(self):
-        from percy.robot_library import _parse_json
         assert _parse_json(None) is None
 
 
 class TestPercyLibraryKeywords:
     def test_import_succeeds(self):
-        from percy.robot_library import PercyLibrary
         assert PercyLibrary is not None
 
     @patch("percy.robot_library._get_screenshot_module")
     @patch("percy.robot_library.BuiltIn")
     def test_percy_snapshot_keyword(self, mock_builtin, mock_get_mod):
-        from percy.robot_library import PercyLibrary
-
         mock_mod = MagicMock()
         mock_get_mod.return_value = mock_mod
         mock_builtin.return_value.get_library_instance.return_value = MagicMock()
@@ -75,28 +68,24 @@ def test_percy_snapshot_keyword(self, mock_builtin, mock_get_mod):
         lib.percy_snapshot_keyword("Homepage", widths="375,1280", labels="regression,v2")
 
         mock_mod.percy_snapshot.assert_called_once()
-        _, kwargs = mock_mod.percy_snapshot.call_args
-        assert kwargs["widths"] == [375, 1280]
-        assert kwargs["labels"] == ["regression", "v2"]
+        call_kwargs = mock_mod.percy_snapshot.call_args[1]
+        assert call_kwargs["widths"] == [375, 1280]
+        assert call_kwargs["labels"] == ["regression", "v2"]
 
     @patch("percy.robot_library._get_screenshot_module")
     def test_percy_is_running(self, mock_get_mod):
-        from percy.robot_library import PercyLibrary
-
         mock_mod = MagicMock()
         mock_get_mod.return_value = mock_mod
 
-        mock_mod._is_percy_enabled.return_value = {"session_type": "web"}
+        mock_mod._is_percy_enabled.return_value = {"session_type": "web"}  # pylint: disable=protected-access
         lib = PercyLibrary()
         assert lib.percy_is_running_keyword() is True
 
-        mock_mod._is_percy_enabled.return_value = False
+        mock_mod._is_percy_enabled.return_value = False  # pylint: disable=protected-access
         assert lib.percy_is_running_keyword() is False
 
     @patch("percy.robot_library._get_screenshot_module")
     def test_create_region(self, mock_get_mod):
-        from percy.robot_library import PercyLibrary
-
         mock_mod = MagicMock()
         mock_mod.create_region.return_value = {"algorithm": "ignore", "elementSelector": {"elementCSS": ".ad"}}
         mock_get_mod.return_value = mock_mod