travis-surveillance 0.0.4 → 0.0.5

data/lib/travis/surveillance/project.rb

@@ -1,5 +1,15 @@
 module Travis
   module Surveillance
+    class Builds < Array
+      def <<(item)
+        super
+
+        # A bit expensive, but for now it'll do.
+        self.sort_by! { |i| i.id }
+        self.slice!(0..-11) if self.size > 10
+      end
+    end
+
     class Project
       ATTRIBUTES = [:description, :id, :slug]
       attr_accessor *ATTRIBUTES
@@ -22,7 +32,6 @@ module Travis
 
         build = Build.new(json.merge({'project' => self}))
         builds << build
-        builds.sort_by! { |b| b.id }
         build
       end
 
@@ -35,7 +44,7 @@ module Travis
       end
 
       def builds
-        @builds ||= []
+        @builds ||= Builds.new
       end
 
       def failed?
@@ -87,7 +96,7 @@ module Travis
       def populate
         self.attributes = details = get_details
 
-        get_builds.reverse.each do |build_json|
+        get_builds[0..9].reverse.each do |build_json|
           add_build(build_json)
         end if details['last_build_id']
       end

data/lib/travis/surveillance/version.rb

@@ -1,5 +1,5 @@
 module Travis
   module Surveillance
-    VERSION = "0.0.4"
+    VERSION = "0.0.5"
   end
 end

data/spec/travis/surveillance/project_spec.rb

@@ -25,6 +25,13 @@ describe Travis::Surveillance::Project do
     @project.url.must_equal "http://travis-ci.org/dylanegan/travis-surveillance"
   end
 
+  it "should limit the stored builds to 10" do
+    @project.stub :build_for, nil do
+      21.times { @project.add_build({ 'id' => 1 }) }
+      @project.builds.size.must_equal 10
+    end
+  end
+
   describe "with history" do
     before do
       @history = Travis::Surveillance::Project.new("dylanegan/travis-surveillance-existing")

data/spec/travis/surveillance/surveyor_spec.rb

@@ -59,7 +59,8 @@ describe Travis::Surveillance::Surveyor do
       end
     end
 
-    describe "when job:finished is received before OMFG I SHOULD JUST FIX THE INITIAL LOAD" do
+    # This should not be of concern now that the project is loaded on startup
+    describe "when job:finished is received before build:started" do
       it "should handle it well" do
         @surveyor.socket.simulate_received('job:finished', pusher_json_for(@project.slug, 'job:finished'), 'common')
         @project.builds.last.jobs.last.running?.must_equal false

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: travis-surveillance
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
   prerelease: 
 platform: ruby
 authors: