lamindb_setup 1.0.1__tar.gz → 1.0.3__tar.gz

{lamindb_setup-1.0.1 → lamindb_setup-1.0.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: lamindb_setup
-Version: 1.0.1
+Version: 1.0.3
 Summary: Setup & configure LaminDB.
 Author-email: Lamin Labs <open-source@lamin.ai>
 Requires-Python: >=3.10

{lamindb_setup-1.0.1 → lamindb_setup-1.0.3}/lamindb_setup/__init__.py

@@ -33,7 +33,7 @@ Modules & settings:
 
 """
 
-__version__ = "1.0.1"  # denote a release candidate for 0.1.0 with 0.1rc1
+__version__ = "1.0.3"  # denote a release candidate for 0.1.0 with 0.1rc1
 
 import os
 

{lamindb_setup-1.0.1 → lamindb_setup-1.0.3}/lamindb_setup/_connect_instance.py

@@ -394,6 +394,24 @@ def migrate_lnschema_core(
                 if response != "y":
                     print("Aborted.")
                     quit()
+                if isettings.is_on_hub:
+                    from lamindb_setup.core._hub_client import call_with_fallback_auth
+                    from lamindb_setup.core._hub_crud import (
+                        select_collaborator,
+                    )
+
+                    # double check that user is an admin, otherwise will fail below
+                    # due to insufficient SQL permissions with cryptic error
+                    collaborator = call_with_fallback_auth(
+                        select_collaborator,
+                        instance_id=settings.instance._id,
+                        account_id=settings.user._uuid,
+                    )
+                    if collaborator is None or collaborator["role"] != "admin":
+                        raise SystemExit(
+                            "❌ Only admins can deploy migrations, please ensure that you're an"
+                            f" admin: https://lamin.ai/{settings.instance.slug}/settings"
+                        )
                 for table in tables_to_rename:
                     if db_type == "sqlite":
                         cur.execute(

{lamindb_setup-1.0.1 → lamindb_setup-1.0.3}/tests/hub-local/test_all.py

@@ -325,7 +325,10 @@ def test_connect_instance_hub_corrupted_or_expired_credentials(
     create_myinstance, create_testadmin1_session
 ):
     # assume token & password are corrupted or expired
-    ln_setup.settings.user.access_token = "corrupted_or_expired_token"
+    # make realisticly looking token that passes
+    # supafunc is_valid_jwt but is actually not a real token
+    invalid_token = "header1.payload1.signature1"
+    ln_setup.settings.user.access_token = invalid_token
     correct_password = ln_setup.settings.user.password
     ln_setup.settings.user.password = "corrupted_password"
     with pytest.raises(FunctionsHttpError):
@@ -336,7 +339,7 @@ def test_connect_instance_hub_corrupted_or_expired_credentials(
     # now, let's assume only the token is expired or corrupted
     # re-creating the auth client triggers a re-generated token because it
     # excepts the error assuming the token is expired
-    ln_setup.settings.user.access_token = "corrupted_or_expired_token"
+    ln_setup.settings.user.access_token = invalid_token
     ln_setup.settings.user.password = correct_password
     connect_instance_hub(
         owner="testadmin1",
@@ -344,7 +347,7 @@ def test_connect_instance_hub_corrupted_or_expired_credentials(
     )
     # check access_token renewal
     access_token = ln_setup.settings.user.access_token
-    assert access_token != "corrupted_or_expired_token"
+    assert access_token != invalid_token
     # check that the access_token was written to the settings
     ln_setup.settings._user_settings = None
     assert ln_setup.settings.user.access_token == access_token