fixes tests fixtures

Author: pcrespov

packages/pytest-simcore/src/pytest_simcore/helpers/webserver_users.py

@@ -45,15 +45,16 @@ async def _create_user_in_db(
     data = data or {}
     data.setdefault("status", UserStatus.ACTIVE.name)
     data.setdefault("role", UserRole.USER.name)
+    data.setdefault("password", DEFAULT_TEST_PASSWORD)
 
-    raw_password = DEFAULT_TEST_PASSWORD
+    raw_password = data["password"]
 
     # inject in db
     user = await exit_stack.enter_async_context(
         insert_and_get_row_lifespan(  # pylint:disable=contextmanager-generator-missing-cleanup
             sqlalchemy_async_engine,
             table=users_table,
-            values=random_user(password=raw_password, **data),
+            values=random_user(**data),
             pk_col=users_table.c.id,
         )
     )

services/web/server/tests/unit/with_dbs/03/invitations/test_invitations.py

@@ -95,7 +95,7 @@ async def test_invalid_invitation_if_guest_is_already_registered_in_product(
 
     # user exists, and we skip product registration to do this test
     mocker.patch(
-        "pytest_simcore.helpers.webserver_login.auto_add_user_to_product_group",
+        "pytest_simcore.helpers.webserver_users.groups_service.auto_add_user_to_product_group",
         return_value=f"Mocked in {__file__}. SKIPPED auto_add_user_to_product_group",
         autospec=True,
     )