fix(Employee): Use official's email address if names are blank (#140)

lib/orbit/employee.ex

@@ -51,7 +51,10 @@ defmodule Orbit.Employee do
 
   @spec display_name(t()) :: String.t()
   def display_name(employee) do
-    "#{employee.preferred_first || employee.first_name} #{employee.last_name}"
+    case String.trim("#{employee.preferred_first || employee.first_name} #{employee.last_name}") do
+      "" -> employee.email
+      name -> name
+    end
   end
 
   @spec get_by_badge_serial(String.t()) :: t() | nil

test/orbit/employee_test.exs

@@ -61,6 +61,18 @@ defmodule Orbit.EmployeeTest do
 
       assert Employee.display_name(employee) == "First Name"
     end
+
+    test "uses email if names are all blank" do
+      employee =
+        build(:employee, %{
+          preferred_first: nil,
+          first_name: "",
+          last_name: "",
+          email: "[email protected]"
+        })
+
+      assert Employee.display_name(employee) == "[email protected]"
+    end
   end
 
   describe "get_by_badge_serial/1" do