tbt276 - build count (#1357)

* build_count added to repo repr.

* filtering out PRs from build count

Author: GbArc

lib/travis/api/v3/renderer/repository.rb

@@ -1,12 +1,12 @@
 module Travis::API::V3
   class Renderer::Repository < ModelRenderer
     representation(:minimal,  :id, :name, :slug)
-    representation(:standard, :id, :name, :slug, :description, :github_id, :vcs_id, :vcs_type, :github_language, :active, :private, :owner, :owner_name, :vcs_name, :default_branch, :starred, :managed_by_installation, :active_on_org, :migration_status, :history_migration_status, :shared, :config_validation, :server_type, :scan_failed_at)
-    representation(:experimental, :id, :name, :slug, :description, :vcs_id, :vcs_type, :github_id, :github_language, :active, :private, :owner, :default_branch, :starred, :current_build, :last_started_build, :next_build_number, :server_type, :scan_failed_at)
+    representation(:standard, :id, :name, :slug, :description, :github_id, :vcs_id, :vcs_type, :github_language, :active, :private, :owner, :owner_name, :vcs_name, :default_branch, :starred, :managed_by_installation, :active_on_org, :migration_status, :history_migration_status, :shared, :config_validation, :server_type, :scan_failed_at, :build_count)
+    representation(:experimental, :id, :name, :slug, :description, :vcs_id, :vcs_type, :github_id, :github_language, :active, :private, :owner, :default_branch, :starred, :current_build, :last_started_build, :next_build_number, :server_type, :scan_failed_at, :build_count)
     representation(:internal, :id, :name, :slug, :github_id, :vcs_id, :vcs_type, :active, :private, :owner, :default_branch, :private_key, :token, :user_settings, :server_type, :scan_failed_at)
     representation(:list, :id, :name, :slug, :active, :private, :owner, :vcs_id, :vcs_type, :server_type)
-    representation(:minimal_with_build, :id, :name, :slug, :active, :private, :owner, :vcs_id, :vcs_type, :server_type, :managed_by_installation, :last_started_build, :current_build)
-    representation(:additional, :allow_migration)
+    representation(:minimal_with_build, :id, :name, :slug, :active, :private, :owner, :vcs_id, :vcs_type, :server_type, :managed_by_installation, :last_started_build, :current_build, :build_count)
+    representation(:additional, :allow_migration, :build_count)
 
     hidden_representations(:experimental, :internal)
 
@@ -37,6 +37,10 @@ def default_branch
       }
     end
 
+    def build_count
+      model.builds.where.not(event_type: :pull_request).length
+    end
+
     def current_build
       build = model.current_build
       build if access_control.visible? build

spec/v3/services/owner/find_spec.rb

@@ -207,7 +207,8 @@
           "active_on_org"     => nil,
           "migration_status"  => nil,
           "history_migration_status"  => nil,
-          "config_validation" => false
+          "config_validation" => false,
+          "build_count"       => 0
         }]
       }}
     end
@@ -313,7 +314,8 @@
           "active_on_org"   => nil,
           "migration_status" => nil,
           "history_migration_status"  => nil,
-          "config_validation" => false
+          "config_validation" => false,
+          "build_count"     => 0
         }]
       }}
     end

spec/v3/services/repositories/for_current_user_spec.rb

@@ -101,10 +101,11 @@
           "name"             => "master"},
         "starred"            => false,
         "managed_by_installation"=>false,
-        "active_on_org"=>nil,
-        "migration_status" => nil,
+        "active_on_org"      =>nil,
+        "migration_status"   => nil,
         "history_migration_status" => nil,
-        "config_validation" => false
+        "config_validation"  => false,
+        "build_count"        => 3
         }]
     }}
   end

spec/v3/services/repositories/for_owner_spec.rb

@@ -134,7 +134,8 @@
           "active_on_org"    => nil,
           "migration_status" => nil,
           "history_migration_status"  => nil,
-          "config_validation" => false
+          "config_validation" => false,
+          "build_count"      => 3
         }]}}
   end
 
@@ -210,6 +211,7 @@
         "migration_status"  => nil,
         "history_migration_status"  => nil,
         "config_validation" => false,
+        "build_count"       => 3,
         "last_started_build"=>{
           "@type"          =>"build",
           "@href"          =>"/v3/build/#{build.id}",
@@ -348,6 +350,7 @@
         "migration_status" => nil,
         "history_migration_status"  => nil,
         "config_validation" => false,
+        "build_count" => 3,
         "current_build" => {
           "@type"               => "build",
           "@href"               => "/v3/build/#{build.id}",
@@ -528,7 +531,8 @@
         "active_on_org"   => nil,
         "migration_status" => nil,
         "history_migration_status"  => nil,
-        "config_validation" => false}, {
+        "config_validation" => false,
+        "build_count"      => 3}, {
         "@type"           => "repository",
         "@href"           => "/v3/repo/#{repo2.id}",
         "@representation" => "standard",
@@ -590,7 +594,9 @@
           "active_on_org"  =>nil,
           "migration_status" => nil,
           "history_migration_status" => nil,
-          "config_validation" => false}]}
+          "config_validation" => false,
+          "build_count"     => 0
+        }]}
   end
 
   describe "shared repository for collaborator, authenticated as user with access" do
@@ -684,7 +690,8 @@
           "active_on_org"    => nil,
           "migration_status" => nil,
           "history_migration_status"  => nil,
-          "config_validation" => false
+          "config_validation" => false,
+          "build_count" => 1
         }]}}
   end
 

spec/v3/services/repository/find_spec.rb

@@ -168,7 +168,8 @@
       "managed_by_installation" => false,
       "migration_status"   => nil,
       "history_migration_status" => nil,
-      "config_validation"  => false
+      "config_validation"  => false,
+      "build_count" => 3
     })}
   end
 

spec/v3/services/v2_subscription/executions_spec.rb

@@ -162,7 +162,8 @@
               "history_migration_status"=>nil,
               "scan_failed_at"=>nil,
               "shared"=>false,
-              "config_validation"=>false
+              "config_validation"=>false,
+              "build_count"=> 3
             }
           }
         ]