Skip to content

Commit d6df5eb

Browse files
committed
small fixes
1 parent 7573f0c commit d6df5eb

File tree

4 files changed

+23
-9
lines changed

4 files changed

+23
-9
lines changed

backend/app/services/implementations/user_service.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -409,6 +409,9 @@ async def update_user_data_by_id(self, user_id: str, user_data_update: UserDataU
409409
for field in simple_fields:
410410
if field in update_data:
411411
setattr(user_data, field, update_data[field])
412+
# Sync first_name and last_name to User table for consistency
413+
if field in ("first_name", "last_name"):
414+
setattr(db_user, field, update_data[field])
412415

413416
# Handle pronouns (array field)
414417
if "pronouns" in update_data:

backend/tests/unit/test_availability_service.py

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,11 +23,15 @@
2323

2424
# Test DB Configuration
2525
POSTGRES_DATABASE_URL = os.getenv("POSTGRES_TEST_DATABASE_URL")
26+
2627
if not POSTGRES_DATABASE_URL:
27-
raise RuntimeError(
28-
"POSTGRES_TEST_DATABASE_URL is not set. Please export a Postgres URL, e.g. "
29-
"postgresql+psycopg2://postgres:postgres@db:5432/llsc_test"
28+
# Skip all tests in this file if Postgres isn't available
29+
pytest.skip(
30+
"POSTGRES_TEST_DATABASE_URL not set. "
31+
"These tests require a Postgres database. Set POSTGRES_TEST_DATABASE_URL to run them.",
32+
allow_module_level=True,
3033
)
34+
3135
engine = create_engine(POSTGRES_DATABASE_URL)
3236
TestingSessionLocal = sessionmaker(autocommit=False, autoflush=False, bind=engine)
3337

backend/tests/unit/test_match_service_timezone.py

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,11 +21,15 @@
2121

2222
# Test DB Configuration
2323
POSTGRES_DATABASE_URL = os.getenv("POSTGRES_TEST_DATABASE_URL")
24+
2425
if not POSTGRES_DATABASE_URL:
25-
raise RuntimeError(
26-
"POSTGRES_TEST_DATABASE_URL is not set. Please export a Postgres URL, e.g. "
27-
"postgresql+psycopg2://postgres:postgres@db:5432/llsc_test"
26+
# Skip all tests in this file if Postgres isn't available
27+
pytest.skip(
28+
"POSTGRES_TEST_DATABASE_URL not set. "
29+
"These tests require a Postgres database. Set POSTGRES_TEST_DATABASE_URL to run them.",
30+
allow_module_level=True,
2831
)
32+
2933
engine = create_engine(POSTGRES_DATABASE_URL)
3034
TestingSessionLocal = sessionmaker(autocommit=False, autoflush=False, bind=engine)
3135

backend/tests/unit/test_ranking_service.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,13 @@
1212

1313
# Postgres-only configuration (migrations assumed to be applied)
1414
POSTGRES_DATABASE_URL = os.getenv("POSTGRES_TEST_DATABASE_URL")
15+
1516
if not POSTGRES_DATABASE_URL:
16-
raise RuntimeError(
17-
"POSTGRES_DATABASE_URL is not set. Please export a Postgres URL, e.g. "
18-
"postgresql+psycopg2://postgres:postgres@db:5432/llsc_test"
17+
# Skip all tests in this file if Postgres isn't available
18+
pytest.skip(
19+
"POSTGRES_TEST_DATABASE_URL not set. "
20+
"These tests require a Postgres database. Set POSTGRES_TEST_DATABASE_URL to run them.",
21+
allow_module_level=True,
1922
)
2023
engine = create_engine(POSTGRES_DATABASE_URL)
2124
TestingSessionLocal = sessionmaker(autocommit=False, autoflush=False, bind=engine)

0 commit comments

Comments
 (0)