Merge pull request 'fix(user): simplify add_user' (#28) from irl/cloud-api:add_user into main

Reviewed-on: sr2/cloud-api#28

src/user/service.py

@@ -1,8 +1,5 @@
 """
 Module specific business logic for user module
-
-Exports:
- - add_user_to_db: Creates a User record from OIDC claims, or updates user details
 """
 
 from typing import Any
@@ -17,7 +14,7 @@ from src.user.schemas import OIDCUser
 from src.user.models import User
 
 
-async def add_user_to_db(db: Session, user_claims: dict[str, Any]) -> int:
+async def add_user(db: Session, user_claims: dict[str, Any]) -> int:
 	try:
 		valid_user = OIDCUser(
 			first_name=user_claims["given_name"],
@@ -26,7 +23,7 @@ async def add_user_to_db(db: Session, user_claims: dict[str, Any]) -> int:
 			oidc_id=user_claims["sub"],
 		)
 	except Exception as e:
-		print(e)
+		logging.exception(e)
 		raise UnprocessableContentException("Invalid or missing OIDC data")
 
 	db_user = db.query(User).filter(User.oidc_id == valid_user.oidc_id).first()
@@ -37,19 +34,12 @@ async def add_user_to_db(db: Session, user_claims: dict[str, Any]) -> int:
 		user_id = user_model.id
 		db.commit()
 		return user_id
-	else:
-		user_id = db_user.id
-		change = False
-		if db_user.first_name != valid_user.first_name:
-			db_user.first_name = valid_user.first_name
-			change = True
-		if db_user.last_name != valid_user.last_name:
-			db_user.last_name = valid_user.last_name
-			change = True
-		if change:
-			db.add(db_user)
-			db.commit()
-		return user_id
+
+	user_id = db_user.id
+	db_user.first_name = valid_user.first_name
+	db_user.last_name = valid_user.last_name
+	db.commit()
+	return user_id
 
 
 async def send_invitation(user_email: str, org_name: str, org_id: int):