use an empty total time when the build has not started yet so the UI knows

2 files changed, 6 insertions, 2 deletions

diff --git a/app/serializers/analytics_build_entity.rb b/app/serializers/analytics_build_entity.rb
index abefcd5cc02..206a7eadbcf 100644
--- a/app/serializers/analytics_build_entity.rb
+++ b/app/serializers/analytics_build_entity.rb
@@ -13,7 +13,7 @@ class AnalyticsBuildEntity < Grape::Entity
   end
 
   expose :duration, as: :total_time do |build|
-    distance_of_time_as_hash(build.duration.to_f)
+    build_started?(build) ? distance_of_time_as_hash(build.duration.to_f) : {}
   end
 
   expose :branch do
@@ -37,4 +37,8 @@ class AnalyticsBuildEntity < Grape::Entity
   def url_to(route, build, id = nil)
     public_send("#{route}_url", build.project.namespace, build.project, id || build)
   end
+
+  def build_started?(build)
+    build.duration && build[:started_at]
+  end
 end
diff --git a/app/serializers/entity_date_helper.rb b/app/serializers/entity_date_helper.rb
index 3cc98fb18a1..9607ad55a8b 100644
--- a/app/serializers/entity_date_helper.rb
+++ b/app/serializers/entity_date_helper.rb
@@ -2,7 +2,7 @@ module EntityDateHelper
   include ActionView::Helpers::DateHelper
 
   def interval_in_words(diff)
-    return 'not started' unless diff
+    return 'Not started' unless diff
 
     "#{distance_of_time_in_words(Time.now, diff)} ago"
   end