fix: map creation source to service_account

Author: hamir-suspect

guard/lib/guard/grpc_servers/user_server.ex

@@ -664,6 +664,7 @@ defmodule Guard.GrpcServers.UserServer do
   defp map_creation_source(user) do
     case user[:creation_source] do
       :okta -> User.User.CreationSource.value(:OKTA)
+      :service_account -> User.User.CreationSource.value(:SERVICE_ACCOUNT)
       _ -> User.User.CreationSource.value(:NOT_SET)
     end
   end

guard/test/guard/grpc_servers/user_server_test.exs

@@ -1407,14 +1407,18 @@ defmodule Guard.GrpcServers.UserServerTest do
                  id: user_id,
                  name: user_name,
                  email: user_email,
-                 repository_providers: []
+                 repository_providers: [],
+                 creation_source: creation_source
                }
              } = response
 
       assert user_id == user.id
       assert user_name == "Test Service Account"
       assert user_email == user.email
       assert user.creation_source == :service_account
+      # Verify that SERVICE_ACCOUNT enum value (2) is returned
+      assert creation_source == InternalApi.User.User.CreationSource.value(:SERVICE_ACCOUNT)
+      assert creation_source == 2
     end
 
     test "should not return repository providers for service accounts", %{grpc_channel: channel} do
@@ -1516,7 +1520,33 @@ defmodule Guard.GrpcServers.UserServerTest do
       # All should be service accounts with no repository providers
       Enum.each(users, fn user ->
         assert user.repository_providers == []
+        # Verify creation_source is SERVICE_ACCOUNT (2)
+        assert user.creation_source ==
+                 InternalApi.User.User.CreationSource.value(:SERVICE_ACCOUNT)
+
+        assert user.creation_source == 2
       end)
     end
+
+    test "should return creation_source as SERVICE_ACCOUNT for service accounts", %{
+      grpc_channel: channel
+    } do
+      {:ok, %{service_account: _service_account, user: user}} =
+        Support.Factories.ServiceAccountFactory.insert()
+
+      request = User.DescribeRequest.new(user_id: user.id)
+
+      {:ok, response} = channel |> Stub.describe(request)
+
+      assert %User.DescribeResponse{
+               user: %User.User{
+                 creation_source: creation_source
+               }
+             } = response
+
+      # Verify creation_source is exactly SERVICE_ACCOUNT (enum value 2)
+      assert creation_source == InternalApi.User.User.CreationSource.value(:SERVICE_ACCOUNT)
+      assert creation_source == 2
+    end
   end
 end