pum 1.1.17__tar.gz → 1.1.18__tar.gz

{pum-1.1.17 → pum-1.1.18}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pum
-Version: 1.1.17
+Version: 1.1.18
 Summary: Pum stands for "Postgres Upgrades Manager". It is a Database migration management tool very similar to flyway-db or Liquibase, based on metadata tables.
 Author-email: Denis Rouzaud <denis@opengis.ch>
 License-Expression: GPL-2.0-or-later

{pum-1.1.17 → pum-1.1.18}/pum/role_manager.py

@@ -125,7 +125,7 @@ class Role:
         self.inherit = inherit
         self.description = description
 
-    def permissions(self):
+    def permissions(self) -> list[Permission]:
         """
         Returns the list of permissions associated with the role.
         """
@@ -138,11 +138,16 @@ class Role:
         Returns:
             bool: True if the role exists, False otherwise.
         """
-        SqlContent("SELECT 1 FROM pg_roles WHERE rolname = {name}").execute(
-            connection=connection,
-            commit=False,
-            parameters={"name": psycopg.sql.Literal(self.name)},
-        ).fetchone() is not None
+        return (
+            SqlContent("SELECT 1 FROM pg_roles WHERE rolname = {name}")
+            .execute(
+                connection=connection,
+                commit=False,
+                parameters={"name": psycopg.sql.Literal(self.name)},
+            )
+            .fetchone()
+            is not None
+        )
 
     def create(
         self, connection: psycopg.Connection, grant: bool = False, commit: bool = False

{pum-1.1.17 → pum-1.1.18}/pum.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pum
-Version: 1.1.17
+Version: 1.1.18
 Summary: Pum stands for "Postgres Upgrades Manager". It is a Database migration management tool very similar to flyway-db or Liquibase, based on metadata tables.
 Author-email: Denis Rouzaud <denis@opengis.ch>
 License-Expression: GPL-2.0-or-later

{pum-1.1.17 → pum-1.1.18}/pyproject.toml

@@ -38,7 +38,7 @@ packages = ["pum"]
 
 [tool.setuptools-git-versioning]
 enabled = true
-starting_version = "0.0.0"
+starting_version = "0.1.2"
 
 [tool.setuptools.dynamic]
 readme = {file = ["README.md"], content-type = "text/markdown"}

{pum-1.1.17 → pum-1.1.18}/test/test_config.py

@@ -6,6 +6,8 @@ from packaging.version import parse as parse_version
 from pum.pum_config import PumConfig
 from pum.exceptions import PumConfigError
 from pum.hook import HookHandler
+import os
+import platform
 
 
 class TestConfig(unittest.TestCase):
@@ -98,6 +100,9 @@ class TestConfig(unittest.TestCase):
             PumConfig(base_path=Path("test") / "data" / "parameters", validate=True)
 
     def test_minimum_version(self) -> None:
+        if os.environ.get("CI") and platform.system().lower().startswith("win"):
+            self.skipTest("Skipped on Windows in CI")
+
         with self.assertRaises(PumConfigError):
             PumConfig(
                 base_path=Path("test") / "data" / "single_changelog",

{pum-1.1.17 → pum-1.1.18}/test/test_roles.py

@@ -98,6 +98,20 @@ class TestRoles(unittest.TestCase):
             )
             self.assertTrue(cur.fetchone()[0])
 
+    def test_multiple_installs_roles(self) -> None:
+        """Test granting permissions to roles."""
+        test_dir = Path("test") / "data" / "roles"
+        cfg = PumConfig.from_yaml(test_dir / ".pum.yaml")
+        with psycopg.connect(f"service={self.pg_service}") as conn:
+            Upgrader(cfg).install(connection=conn, roles=True, grant=True, commit=True)
+            cur = conn.cursor()
+            cur.execute(
+                "DROP TABLE public.pum_migrations CASCADE; "
+                "DROP SCHEMA pum_test_data_schema_1 CASCADE; "
+                "DROP SCHEMA pum_test_data_schema_2 CASCADE;"
+            )
+            Upgrader(cfg).install(connection=conn, roles=True, grant=True, commit=True)
+
 
 if __name__ == "__main__":
     unittest.main()