ci_in_a_can 0.0.25 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 73df3df8205659926b06262cc3b7de93193c4112
-  data.tar.gz: 2915333ad1956d539c0b55548fd2a207e2feb001
+  metadata.gz: 01a156efea17fc2fc3ea80032a133372068c96b4
+  data.tar.gz: f1a685129c6f48b798c93eec378c353701460140
 SHA512:
-  metadata.gz: 2462767fcd35e1a8bb8b3cc96ce9f23abf194945de80d9af9f885123d2e62ab824c7b4deb86289a17aff7ad71e24d596b98e54ad622fb7f132a2c264041513df
-  data.tar.gz: c5f4db5042869b114436fd561a818b1b8328da9a329a6e0ef48e5be711cd3225d2458f0b4c5e5609a9a8eb4af6241df17c8ba1cca4c21eb23ced61d6713c196a
+  metadata.gz: 7aa2a8ecf9eda070a3bb35efaf73c183199aaa95f6e85bdbdb2601f76f09056d2523f1733afcf4dc51cfc36dfa59c50276e9314a93039f0a954f04dba4557c7b
+  data.tar.gz: 5ddaab0fe750ed016db64bbf995a215f1824db6bb183e189a6a75a11473abdc31ab9e11c57415559e9c5905894c741b9999aef02f49bc1c1a2b4fec28a648791

data/README.md

@@ -40,7 +40,8 @@ When any push is made to your Github repository, Github will send a notification
 1.  A default rake task that runs all of your tests.
 2.  An environment variable named GITHUB_AUTH_TOKEN.  This is used to report results back to Github.
 3.  An environment variable named SITE_URL.  This will be the URL of your site, for things like providing Github with a link back to the site to show test results.
-2.  A server that will "just run" your application.  Set up your own server with whatever dependencies your application needs.  
+4.  A server that will "just run" your application.  Set up your own server with whatever dependencies your application needs.  
+5.  An environment variable named PASSPHRASE. You'll need to provide this if you want to make config changes. You may not need to do this, so this step is optional.
 
 
 ## Contributing

data/lib/ci_in_a_can.rb

@@ -3,7 +3,9 @@ require 'uuid'
 require 'subtle'
 
 require_relative "ci_in_a_can/version"
+require_relative "ci_in_a_can/view_models/view_model"
 Dir[File.dirname(__FILE__) + '/ci_in_a_can/*.rb'].each { |file| require file }
+Dir[File.dirname(__FILE__) + '/ci_in_a_can/view_models/*.rb'].each { |file| require file }
 
 module CiInACan
 

data/lib/ci_in_a_can/app.rb

@@ -13,6 +13,12 @@ module CiInACan
     end
 
     post %r{/repo/(.+)} do
+
+      if ENV['PASSPHRASE'] != params[:passphrase].to_s
+        redirect '/'
+        return
+      end
+
       params[:id] = params[:captures].first
       commands = params[:commands].gsub("\r\n", "\n").split("\n")
       commands = commands.map { |x| x.strip }.select { |x| x != '' }
@@ -30,6 +36,8 @@ module CiInACan
       CiInACan::WebContent.full_page_of(
 <<EOF
 <form action="/repo/#{params[:id]}" method="post">
+<label>Passphrase</label>
+<input type="text" name="passphrase">
 <textarea name="commands">
 #{commands}
 </textarea>
@@ -41,91 +49,11 @@ EOF
 
     get '/test_result/:id' do
       test_result = CiInACan::TestResult.find(params[:id])
-
-      CiInACan::WebContent.full_page_of(
-<<EOF
-    <table class="table table-bordered">
-      <tbody>
-      <tr>
-        <td>
-          Id
-        </td>
-        <td>
-          #{test_result.id}
-        </td>
-      </tr>
-      <tr>
-        <td>
-          Repo
-        </td>
-        <td>
-          <a href="/repo/#{test_result.repo}">
-          #{test_result.repo}
-          </a>
-        </td>
-      </tr>
-      <tr>
-        <td>
-          Branch
-        </td>
-        <td>
-          #{test_result.branch}
-        </td>
-      </tr>
-      <tr>
-        <td>
-          Created At
-        </td>
-        <td>
-          #{test_result.created_at.to_s}
-        </td>
-      </tr>
-      <tr>
-        <td>
-          Passed
-        </td>
-        <td>
-          #{test_result.passed ? 'Yes' : 'No'}
-        </td>
-      </tr>
-      <tr>
-        <td>
-          Output
-        </td>
-        <td>
-          #{test_result.output.to_s.gsub("\n", '<br />')}
-        </td>
-      </tr>
-      </tbody>
-    </table>
-EOF
-)
+      CiInACan::WebContent.full_page_of test_result.to_html
     end
 
     get '/' do
-      run_html = CiInACan::LastRunList.all.map do |run|
-<<EOF
-                    <tr>
-                      <td>
-                        #{run.created_at}
-                      </td>
-                      <td>
-                        <a href="/repo/#{run.repo}">
-                        #{run.repo}
-                        </a>
-                      </td>
-                      <td>
-                        #{run.branch}
-                      </td>
-                      <td>
-                        #{run.passed ? 'Yes' : 'No'}
-                      </td>
-                      <td>
-                        <a href="/test_result/#{run.test_result_id}">#{run.sha}</a>
-                      </td>
-                    </tr>
-EOF
-                 end.join("\n")
+      run_html = CiInACan::Run.all.map { |r| r.to_html }.join("\n")
 
       CiInACan::WebContent.full_page_of(
 <<EOF

data/lib/ci_in_a_can/{last_run_list.rb → run.rb}

@@ -1,14 +1,19 @@
 module CiInACan
 
-  module LastRunList
-
-    class TestRunResult
-      def initialize value
-        @value = value
-      end
-      def method_missing(meth, *args, &blk)
-        @value[meth]
-      end
+  class Run
+
+    params_constructor
+
+    attr_accessor :created_at,
+                  :test_result_id,
+                  :passed,
+                  :build_id,
+                  :sha,
+                  :repo,
+                  :branch
+
+    def to_html
+      CiInACan::ViewModels::RunViewModel.new(self).to_html
     end
 
     def self.add build, test_result
@@ -26,7 +31,7 @@ module CiInACan
 
     def self.all
       blah = CiInACan::Persistence.hash_for("test_run_list")
-      blah.sort_by { |x| x[0] }.reverse.map { |x| TestRunResult.new x[1] }
+      blah.sort_by { |x| x[0] }.reverse.map { |x| new x[1] }
     end
 
   end

data/lib/ci_in_a_can/test_result.rb

@@ -34,6 +34,10 @@ module CiInACan
         created_at: created_at
       }.to_json
     end
+    
+    def to_html
+      CiInACan::ViewModels::TestResultViewModel.new(self).to_html
+    end
 
     private
 

data/lib/ci_in_a_can/test_result_notifier.rb

@@ -9,7 +9,7 @@ module CiInACan
     private
 
     def self.report_results_to_github build, test_result
-      CiInACan::LastRunList.add build, test_result
+      CiInACan::Run.add build, test_result
       CiInACan::Github.report_complete_status_for build, test_result
     end
 

data/lib/ci_in_a_can/version.rb

@@ -1,3 +1,3 @@
 module CiInACan
-  VERSION = "0.0.25"
+  VERSION = "0.1.0"
 end

data/lib/ci_in_a_can/view_models/run_view_model.rb

@@ -0,0 +1,35 @@
+module CiInACan
+
+  module ViewModels
+
+    class RunViewModel < ViewModel
+
+      def to_html
+<<EOF
+                    <tr>
+                      <td>
+                        #{created_at}
+                      </td>
+                      <td>
+                        <a href="/repo/#{repo}">
+                        #{repo}
+                        </a>
+                      </td>
+                      <td>
+                        #{branch}
+                      </td>
+                      <td>
+                        #{passed ? 'Yes' : 'No'}
+                      </td>
+                      <td>
+                        <a href="/test_result/#{test_result_id}">#{sha}</a>
+                      </td>
+                    </tr>
+EOF
+      end
+
+    end
+
+  end
+
+end

data/lib/ci_in_a_can/view_models/test_result_view_model.rb

@@ -0,0 +1,70 @@
+module CiInACan
+
+  module ViewModels
+
+    class TestResultViewModel < ViewModel
+
+      def to_html
+<<EOF
+    <table class="table table-bordered">
+      <tbody>
+      <tr>
+        <td>
+          Id
+        </td>
+        <td>
+          #{id}
+        </td>
+      </tr>
+      <tr>
+        <td>
+          Repo
+        </td>
+        <td>
+          <a href="/repo/#{repo}">
+          #{repo}
+          </a>
+        </td>
+      </tr>
+      <tr>
+        <td>
+          Branch
+        </td>
+        <td>
+          #{branch}
+        </td>
+      </tr>
+      <tr>
+        <td>
+          Created At
+        </td>
+        <td>
+          #{created_at.to_s}
+        </td>
+      </tr>
+      <tr>
+        <td>
+          Passed
+        </td>
+        <td>
+          #{passed ? 'Yes' : 'No'}
+        </td>
+      </tr>
+      <tr>
+        <td>
+          Output
+        </td>
+        <td>
+          #{output.to_s.gsub("\n", '<br />')}
+        </td>
+      </tr>
+      </tbody>
+    </table>
+EOF
+      end
+
+    end
+
+  end
+
+end

data/lib/ci_in_a_can/view_models/view_model.rb

@@ -0,0 +1,19 @@
+module CiInACan
+
+  module ViewModels
+
+    class ViewModel
+
+      def initialize value
+        @value = value
+      end
+
+      def method_missing(meth, *args, &blk)
+        @value.send(meth)
+      end
+
+    end
+
+  end
+
+end

data/lib/ci_in_a_can/watcher.rb

@@ -23,10 +23,18 @@ module CiInACan
 
           build = create_a_build_for(new_files.first, working_location)
 
+          delete new_files.first
+
           Runner.run build
+
         end
       end
 
+      def delete file
+        File.delete file
+      rescue
+      end
+
       def create_a_build_for file, working_location
         build = CiInACan::Build.parse File.read(file)
         build.id = UUID.new.generate

data/spec/ci_in_a_can/{last_run_list_spec.rb → run_spec.rb}

@@ -1,6 +1,6 @@
 require_relative '../spec_helper'
 
-describe CiInACan::LastRunList do
+describe CiInACan::Run do
 
   before do
     clear_all_persisted_data
@@ -12,9 +12,9 @@ describe CiInACan::LastRunList do
     let(:test_result) { CiInACan::TestResult.new }
 
     it "should store information that we can pull out later" do
-      CiInACan::LastRunList.add build, test_result
+      CiInACan::Run.add build, test_result
 
-      results = CiInACan::LastRunList.all
+      results = CiInACan::Run.all
       results.count.must_equal 1
 
       results.first.created_at.must_equal test_result.created_at 
@@ -24,9 +24,9 @@ describe CiInACan::LastRunList do
 
       test_result.id = UUID.new.generate
 
-      CiInACan::LastRunList.add build, test_result
+      CiInACan::Run.add build, test_result
 
-      results = CiInACan::LastRunList.all
+      results = CiInACan::Run.all
 
       results.first.test_result_id.must_equal test_result.id
         
@@ -34,32 +34,32 @@ describe CiInACan::LastRunList do
 
     it "should track the build id" do
       build.id = UUID.new.generate
-      CiInACan::LastRunList.add build, test_result
-      CiInACan::LastRunList.all.first.build_id.must_equal build.id
+      CiInACan::Run.add build, test_result
+      CiInACan::Run.all.first.build_id.must_equal build.id
     end
 
     it "should track the build id" do
       build.id = UUID.new.generate
-      CiInACan::LastRunList.add build, test_result
-      CiInACan::LastRunList.all.first.build_id.must_equal build.id
+      CiInACan::Run.add build, test_result
+      CiInACan::Run.all.first.build_id.must_equal build.id
     end
 
     it "should track the build sha" do
       build.sha = UUID.new.generate
-      CiInACan::LastRunList.add build, test_result
-      CiInACan::LastRunList.all.first.sha.must_equal build.sha
+      CiInACan::Run.add build, test_result
+      CiInACan::Run.all.first.sha.must_equal build.sha
     end
 
     it "should track the build repo" do
       build.repo = UUID.new.generate
-      CiInACan::LastRunList.add build, test_result
-      CiInACan::LastRunList.all.first.repo.must_equal build.repo
+      CiInACan::Run.add build, test_result
+      CiInACan::Run.all.first.repo.must_equal build.repo
     end
 
     it "should track the branch" do
       build.branch = UUID.new.generate
-      CiInACan::LastRunList.add build, test_result
-      CiInACan::LastRunList.all.first.branch.must_equal build.branch
+      CiInACan::Run.add build, test_result
+      CiInACan::Run.all.first.branch.must_equal build.branch
     end
 
     [true, false].each do |passed|
@@ -71,9 +71,9 @@ describe CiInACan::LastRunList do
         it "should track the test_result passed outcome" do
           test_result.passed = passed
 
-          CiInACan::LastRunList.add build, test_result
+          CiInACan::Run.add build, test_result
 
-          results = CiInACan::LastRunList.all
+          results = CiInACan::Run.all
 
           results.first.passed.must_equal passed
         end
@@ -94,9 +94,9 @@ describe CiInACan::LastRunList do
                         CiInACan::TestResult.new(id: 'a', created_at: Time.parse('1/1/2014')),
                         CiInACan::TestResult.new(id: 'b', created_at: Time.parse('1/2/2014'))]
 
-        test_results.each { |t| CiInACan::LastRunList.add build, t }
+        test_results.each { |t| CiInACan::Run.add build, t }
 
-        results = CiInACan::LastRunList.all
+        results = CiInACan::Run.all
         results[0].test_result_id.must_equal 'c'
         results[1].test_result_id.must_equal 'b'
         results[2].test_result_id.must_equal 'a'
@@ -106,4 +106,22 @@ describe CiInACan::LastRunList do
 
   end
 
+  describe "to html" do
+    it "should show the results of run_view_model" do
+      html       = Object.new
+      view_model = Object.new
+
+      run = CiInACan::Run.new
+
+      view_model.stubs(:to_html).returns html
+
+      CiInACan::ViewModels::RunViewModel
+        .stubs(:new)
+        .with(run)
+        .returns view_model
+
+      run.to_html.must_be_same_as html
+    end
+  end
+
 end

data/spec/ci_in_a_can/test_result_notifier_spec.rb

@@ -8,7 +8,7 @@ describe CiInACan::TestResultNotifier do
     let(:test_result) { CiInACan::TestResult.new }
 
     before do
-      CiInACan::LastRunList.stubs(:add)
+      CiInACan::Run.stubs(:add)
       CiInACan::Github.stubs(:report_complete_status_for)
     end
 
@@ -25,7 +25,7 @@ describe CiInACan::TestResultNotifier do
     end
 
     it "should add the build and test result to the last run list" do
-      CiInACan::LastRunList.expects(:add).with build, test_result
+      CiInACan::Run.expects(:add).with build, test_result
       CiInACan::TestResultNotifier.send_for build, test_result
     end
 

data/spec/ci_in_a_can/test_result_spec.rb

@@ -95,4 +95,22 @@ describe CiInACan::TestRunner do
 
   end
 
+  describe "to html" do
+    it "should show the results of test_result_view_model" do
+      html       = Object.new
+      view_model = Object.new
+
+      test_result = CiInACan::TestResult.new
+
+      view_model.stubs(:to_html).returns html
+
+      CiInACan::ViewModels::TestResultViewModel
+        .stubs(:new)
+        .with(test_result)
+        .returns view_model
+
+      test_result.to_html.must_be_same_as html
+    end
+  end
+
 end

data/spec/ci_in_a_can/view_models/view_model_spec.rb

@@ -0,0 +1,20 @@
+require_relative '../../spec_helper'
+
+describe CiInACan::ViewModels::ViewModel do
+
+  it "should serve as a wrapper around data provided to it" do
+
+    stub = Object.new
+    stub.stubs(:one).returns Object.new
+    stub.stubs(:two).returns Object.new
+    stub.stubs(:three).returns Object.new
+
+    view_model = CiInACan::ViewModels::ViewModel.new stub
+
+    view_model.one.must_be_same_as stub.one
+    view_model.two.must_be_same_as stub.two
+    view_model.three.must_be_same_as stub.three
+      
+  end
+
+end

data/spec/ci_in_a_can/watcher_spec.rb

@@ -2,6 +2,10 @@ require_relative '../spec_helper'
 
 describe CiInACan::Watcher do
 
+  before do
+    File.stubs(:delete)
+  end
+
   describe "watch" do
 
     let(:watching_location) { Object.new }
@@ -103,6 +107,61 @@ describe CiInACan::Watcher do
             CiInACan::Watcher.send(:build_callback, test.working_location).call [], [added_file], []
           end
 
+          it "should delete the file" do
+
+            CiInACan::Runner.stubs(:wl).returns test.working_location
+
+            uuid = Object.new
+            uuid.stubs(:generate).returns test.random_string
+            UUID.stubs(:new).returns uuid
+            CiInACan::Runner.stubs(:run)
+
+            File.expects(:delete).with added_file
+
+            CiInACan::Watcher.send(:build_callback, test.working_location).call [], [added_file], []
+          end
+
+          it "should not delete the file before it is read" do
+
+            content = Object.new
+            CiInACan::Runner.stubs(:wl).returns test.working_location
+
+            uuid = Object.new
+            uuid.stubs(:generate).returns test.random_string
+            UUID.stubs(:new).returns uuid
+            CiInACan::Runner.stubs(:run)
+
+            CiInACan::Build.stubs(:parse).with(content).returns build
+
+            delete_called = false
+            File.expects(:read).with do |added_file|
+              delete_called.must_equal false
+              true
+            end.returns content
+
+            File.expects(:delete).with do |added_file|
+              delete_called = true
+              true
+            end
+
+            CiInACan::Watcher.send(:build_callback, test.working_location).call [], [added_file], []
+          end
+
+          it "should not let an error in deleting a file bubble up" do
+
+            CiInACan::Runner.stubs(:wl).returns test.working_location
+
+            uuid = Object.new
+            uuid.stubs(:generate).returns test.random_string
+            UUID.stubs(:new).returns uuid
+            CiInACan::Runner.stubs(:run)
+
+            File.stubs(:delete).raises 'k'
+
+            # this should not throw
+            CiInACan::Watcher.send(:build_callback, test.working_location).call [], [added_file], []
+          end
+
         end
 
       end

data/spec/spec_helper.rb

@@ -1,7 +1,6 @@
 require_relative '../lib/ci_in_a_can'
 require 'minitest/autorun'
 require 'minitest/spec'
-require 'minitest/pride'
 require 'subtle'
 require 'contrast'
 require 'timecop'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ci_in_a_can
 version: !ruby/object:Gem::Version
-  version: 0.0.25
+  version: 0.1.0
 platform: ruby
 authors:
 - Darren Cauthon
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-16 00:00:00.000000000 Z
+date: 2014-02-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -231,13 +231,16 @@ files:
 - lib/ci_in_a_can/daemon.rb
 - lib/ci_in_a_can/github.rb
 - lib/ci_in_a_can/github_build_parser.rb
-- lib/ci_in_a_can/last_run_list.rb
 - lib/ci_in_a_can/persistence.rb
+- lib/ci_in_a_can/run.rb
 - lib/ci_in_a_can/runner.rb
 - lib/ci_in_a_can/test_result.rb
 - lib/ci_in_a_can/test_result_notifier.rb
 - lib/ci_in_a_can/test_runner.rb
 - lib/ci_in_a_can/version.rb
+- lib/ci_in_a_can/view_models/run_view_model.rb
+- lib/ci_in_a_can/view_models/test_result_view_model.rb
+- lib/ci_in_a_can/view_models/view_model.rb
 - lib/ci_in_a_can/watcher.rb
 - lib/ci_in_a_can/web_content.rb
 - sample.json
@@ -248,11 +251,12 @@ files:
 - spec/ci_in_a_can/cloner_spec.rb
 - spec/ci_in_a_can/github_build_parser_spec.rb
 - spec/ci_in_a_can/github_spec.rb
-- spec/ci_in_a_can/last_run_list_spec.rb
+- spec/ci_in_a_can/run_spec.rb
 - spec/ci_in_a_can/runner_spec.rb
 - spec/ci_in_a_can/test_result_notifier_spec.rb
 - spec/ci_in_a_can/test_result_spec.rb
 - spec/ci_in_a_can/test_runner_spec.rb
+- spec/ci_in_a_can/view_models/view_model_spec.rb
 - spec/ci_in_a_can/watcher_spec.rb
 - spec/spec_helper.rb
 - spec/temp/.gitkeep
@@ -288,11 +292,12 @@ test_files:
 - spec/ci_in_a_can/cloner_spec.rb
 - spec/ci_in_a_can/github_build_parser_spec.rb
 - spec/ci_in_a_can/github_spec.rb
-- spec/ci_in_a_can/last_run_list_spec.rb
+- spec/ci_in_a_can/run_spec.rb
 - spec/ci_in_a_can/runner_spec.rb
 - spec/ci_in_a_can/test_result_notifier_spec.rb
 - spec/ci_in_a_can/test_result_spec.rb
 - spec/ci_in_a_can/test_runner_spec.rb
+- spec/ci_in_a_can/view_models/view_model_spec.rb
 - spec/ci_in_a_can/watcher_spec.rb
 - spec/spec_helper.rb
 - spec/temp/.gitkeep