ci_in_a_can 0.2.4 → 0.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8cc7b9a3c96269cab17933b73e3c25f8bf640fb9
-  data.tar.gz: a0f059a4b8ffeb8d3c7adbd5e4ee8caf5c8ac5ba
+  metadata.gz: ec408de18d9c47e09339a8c2ed0bc8453a42a039
+  data.tar.gz: cb23d747865712a372931f1a558fee78f618bf95
 SHA512:
-  metadata.gz: d66137bb18e8252c226c3ba61bf5f437d62b701ea8a8a4c98d8b59d67a4f85aa676621ca9185db49653a8ec6d7d4d6bf179a361cde2778c4fba0847eb71885bc
-  data.tar.gz: 5a44a7b4a1f3e92452c0aa410b9f84829014cf72708dcd1cfed779ea1bdee29925946a79211ca03652e53dc222c790d034dc5718f3b9c036f3b75c9250d7cbe3
+  metadata.gz: 8475a583069b024c3460c151fdbc30b0f404072936784378a9be2076eb19a25b6acfbf03601b551497914adc154085c00afbbf2744140113beaf5304c31d2d4d
+  data.tar.gz: fe8538c757f0d30cb4da661c083e06285a031d0a5a9563c891c63f0060d7dc582797fe85fcc800f921676b7190b9f24bb2c0d6e90f5c28950aa8f9090fdebe7f

data/lib/ci_in_a_can/app.rb

@@ -15,93 +15,51 @@ module CiInACan
     end
 
     get '/login' do
-      CiInACan::WebContent.full_page_of(
-<<EOF
-<form action="/login" method="post">
-Passphrase
-<input type="password" name="passphrase">
-<button type="submit">Submit</button>
-</form>
-EOF
-)
+      CiInACan::Web.new.login_page
     end
 
     post '/login' do
-      session[:passphrase] = params[:passphrase]
+      web = CiInACan::Web.new(params: params, session: session)
+      web.submit_a_passphrase
       redirect '/'
     end
 
     get '/test_result/:id.json' do
-      CiInACan::TestResult.find(params[:id]).to_json
+      CiInACan::Web.new(params: params).show_the_test_result_in_json
     end
 
     post %r{/repo/(.+)} do
 
-      unless session[:authenticated]
+      web = CiInACan::Web.new(params: params, session: session)
+      unless web.logged_in?
         redirect '/login'
         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 != '' }
-      repo = CiInACan::Repo.find params[:id]
-      repo = CiInACan::Repo.create(id: params[:id]) unless repo
-      repo.build_commands = commands
-      repo.save
-      redirect "/repo/#{params[:id]}"
+      repo = web.update_repo_details
+      redirect "/repo/#{repo.id}"
     end
 
     get %r{/repo/(.+)} do
+      web = CiInACan::Web.new(params: params, session: session)
 
-      unless session[:authenticated]
+      unless web.logged_in?
         redirect '/login'
         return
       end
 
-      params[:id] = params[:captures].first
-      repo = CiInACan::Repo.find(params[:id])
-      url      = repo ? repo.url : nil
-      commands = repo ? repo.build_commands.join("\n") : ''
-      CiInACan::WebContent.full_page_of(
-<<EOF
-<form action="/repo/#{params[:id]}" method="post">
-<div>#{url}</div>
-<textarea name="commands">
-#{commands}
-</textarea>
-<input type="submit">Submit</input>
-</form>
-EOF
-)
+      web.show_the_repo_edit_form
     end
 
     get '/test_result/:id' do
-      test_result = CiInACan::TestResult.find(params[:id])
-      CiInACan::WebContent.full_page_of test_result.to_html
+      CiInACan::Web.new(params: params).show_the_test_result
     end
 
     get '/' do
-      runs = CiInACan::Run.all.to_a
-      ::CiInACan::ViewModels::AListOfRuns.new(runs).to_html
+      CiInACan::Web.new.show_a_list_of_the_runs
     end
 
     post %r{/push/(.+)} do
-      capture = params[:captures].first.split('/')
-      api_key = capture.pop
-      id      = capture.join('/')
-
-      repo = CiInACan::Repo.find id
-      raise 'Could not find this repo' unless repo
-      raise 'Invalid API Key' unless repo.api_key == api_key
-
-      write_a_file_with params
-    end
-
-    def write_a_file_with params
-      data = params.to_json
-      File.open("#{self.class.jobs_location}/#{UUID.new.generate}.json", 'w') { |f| f.write data }
-      data
+      CiInACan::Web.new(params: params).start_a_new_build
     end
 
   end

data/lib/ci_in_a_can/version.rb

@@ -1,3 +1,3 @@
 module CiInACan
-  VERSION = "0.2.4"
+  VERSION = "0.2.5"
 end

data/lib/ci_in_a_can/view_models/login_form.rb

@@ -0,0 +1,24 @@
+module CiInACan
+
+  module ViewModels
+
+    class LoginForm < ViewModel
+
+      def to_html
+        CiInACan::WebContent.full_page_of(
+<<EOF
+<form action="/login" method="post">
+Passphrase
+<input type="password" name="passphrase">
+<button type="submit">Submit</button>
+</form>
+EOF
+)
+      end
+
+    end
+
+  end
+
+end
+

data/lib/ci_in_a_can/view_models/repo_form.rb

@@ -0,0 +1,25 @@
+module CiInACan
+
+  module ViewModels
+
+    class RepoForm < ViewModel
+
+      def to_html
+        CiInACan::WebContent.full_page_of(
+<<EOF
+<form action="/repo/#{id}" method="post">
+<div>#{url}</div>
+<textarea name="commands">
+#{build_commands.join("\n")}
+</textarea>
+<input type="submit">Submit</input>
+</form>
+EOF
+)
+      end
+
+    end
+
+  end
+
+end

data/lib/ci_in_a_can/view_models/view_model.rb

@@ -4,7 +4,7 @@ module CiInACan
 
     class ViewModel
 
-      def initialize value
+      def initialize value = nil
         @value = value
       end
 

data/lib/ci_in_a_can/web.rb

@@ -0,0 +1,78 @@
+module CiInACan
+
+  class Web
+
+    params_constructor
+
+    attr_accessor :params, :session
+
+    class << self
+      attr_accessor :jobs_location
+    end
+
+    def login_page
+      CiInACan::ViewModels::LoginForm.new.to_html
+    end
+
+    def submit_a_passphrase
+      session[:passphrase] = params[:passphrase]
+    end
+
+    def show_the_test_result_in_json
+      CiInACan::TestResult.find(params[:id]).to_json
+    end
+
+    def logged_in?
+      return false if session[:passphrase].to_s == ''
+      session[:passphrase] == ENV['PASSPHRASE']
+    end
+
+    def show_the_repo_edit_form
+      id = params[:captures].first
+      repo = CiInACan::Repo.find(id) || CiInACan::Repo.new(id: id)
+      CiInACan::ViewModels::RepoForm.new(repo).to_html
+    end
+
+    def start_a_new_build
+      assert_that_the_new_build_is_valid
+
+      write_a_file_with params
+      params.to_json
+    end
+
+    def show_a_list_of_the_runs
+      CiInACan::ViewModels::AListOfRuns.new(CiInACan::Run.all.to_a).to_html
+    end
+
+    def show_the_test_result
+      test_result = CiInACan::TestResult.find(params[:id])
+      CiInACan::WebContent.full_page_of test_result.to_html
+    end
+
+    def update_repo_details
+      CiInACan::Repo.create(id:             params[:captures].first, 
+                            build_commands: params[:commands].gsub("\r\n", "\n")
+                                                             .split("\n")
+                                                             .map    { |x| x.strip }
+                                                             .select { |x| x != '' } )
+    end
+
+    def write_a_file_with params
+      data = params.to_json
+      file = "#{self.class.jobs_location}/#{UUID.new.generate}.json"
+      CiInACan::FileSystem.create_file file, data
+    end
+
+    def assert_that_the_new_build_is_valid
+      capture = params[:captures].first.split('/')
+      api_key = capture.pop
+      id      = capture.join('/')
+
+      repo = CiInACan::Repo.find id
+      raise 'Could not find this repo' unless repo
+      raise 'Invalid API Key' unless repo.api_key == api_key
+    end
+
+  end
+
+end

data/spec/ci_in_a_can/web_spec.rb

@@ -0,0 +1,426 @@
+require_relative '../spec_helper'
+
+describe CiInACan::Web do
+
+  let(:params)  { {} }
+  let(:session) { {} }
+
+  let(:web) do
+    CiInACan::Web.new(params:  params,
+                      session: session)
+  end
+
+  describe "login page" do
+
+    it "should return the login form html" do
+
+      html       = Object.new
+      login_form = Object.new
+
+      login_form.stubs(:to_html).returns html
+      CiInACan::ViewModels::LoginForm.stubs(:new).returns login_form
+
+      web.login_page.must_be_same_as html
+
+    end
+
+  end
+
+  describe "submit a passphrase" do
+
+    it "should set the passphrase in session" do
+
+      passphrase = Object.new
+      params[:passphrase] = passphrase
+
+      web.submit_a_passphrase
+
+      session[:passphrase].must_be_same_as passphrase
+
+    end
+
+  end
+
+  describe "showing the test result in html" do
+
+    it "should return the html results" do
+
+      params[:id] = Object.new
+      test_result = Object.new
+      web_content = Object.new
+      html        = Object.new
+
+      test_result.stubs(:to_html).returns html
+
+      CiInACan::TestResult.stubs(:find)
+                          .with(params[:id])
+                          .returns test_result
+
+      CiInACan::WebContent.stubs(:full_page_of)
+                          .with(html)
+                          .returns web_content
+
+      result = web.show_the_test_result
+
+      result.must_be_same_as web_content
+
+    end
+
+  end
+
+  describe "showing the test result in json" do
+
+    it "should return the json results" do
+
+      params[:id] = Object.new
+      test_result = Object.new
+      json        = Object.new
+
+      test_result.stubs(:to_json).returns json
+
+      CiInACan::TestResult.stubs(:find)
+                          .with(params[:id])
+                          .returns test_result
+
+      result = web.show_the_test_result_in_json
+
+      result.must_be_same_as json
+
+    end
+
+  end
+
+  describe "determining if the user is logged in" do
+
+    it "should return true if the passphrase in session and ENV match" do
+
+      passphrase = Object.new
+
+      ENV.stubs(:[]).with('PASSPHRASE').returns passphrase
+      session[:passphrase] = passphrase
+
+      web.logged_in?.must_equal true
+
+    end
+
+    it "should return false if the passphrases do not match" do
+
+      ENV.stubs(:[]).with('PASSPHRASE').returns Object.new
+      session[:passphrase] = Object.new
+
+      web.logged_in?.must_equal false
+
+    end
+
+    it "should return false if the passphrases are nil" do
+
+      ENV.stubs(:[]).with('PASSPHRASE').returns nil
+      session[:passphrase] = nil
+
+      web.logged_in?.must_equal false
+
+    end
+
+    it "should return false if the passphrases are empty strings" do
+
+      ENV.stubs(:[]).with('PASSPHRASE').returns ""
+      session[:passphrase] = ""
+
+      web.logged_in?.must_equal false
+
+    end
+
+  end
+
+  describe "showing the repo edit form" do
+
+    describe "when the id matches an existing repo" do
+
+      it "should return the html for the repo form" do
+
+        id                = Object.new
+        params[:captures] = [id]
+        repo              = Object.new
+        repo_form         = Object.new
+        html              = Object.new
+
+        repo_form.stubs(:to_html).returns html
+
+        CiInACan::Repo.stubs(:find).with(id).returns repo
+        CiInACan::ViewModels::RepoForm.stubs(:new).with(repo).returns repo_form
+
+        web.show_the_repo_edit_form.must_be_same_as html
+          
+      end
+
+    end
+
+    describe "when the id has never been used" do
+
+      it "should return the html for a new repo form" do
+
+        id                = Object.new
+        params[:captures] = [id]
+        repo              = Object.new
+        repo_form         = Object.new
+        html              = Object.new
+
+        repo_form.stubs(:to_html).returns html
+
+        CiInACan::Repo.stubs(:find).with(id).returns nil
+        CiInACan::Repo.stubs(:new).with(id: id).returns repo
+        CiInACan::ViewModels::RepoForm.stubs(:new).with(repo).returns repo_form
+
+        web.show_the_repo_edit_form.must_be_same_as html
+          
+      end
+
+    end
+
+  end
+
+  [:captures, :commands, :expected_id, :expected_commands].to_objects {[
+    [['one/two'],   "\na\r\nb\nc\r\nd\n\n", 'one/two',   ['a', 'b', 'c', 'd']],
+    [['two/three'], "w\nx\ny\r\nz",         'two/three', ['w', 'x', 'y', 'z']]
+  ]}.each do |test|
+
+    describe "updating the repo details" do
+
+      before do
+        clear_all_persisted_data
+      end
+
+      describe "a repo that has never been saved before" do
+
+        before do
+          params[:captures] = test.captures
+          params[:commands] = test.commands
+        end
+
+        it "should create a repo" do
+          web.update_repo_details
+          CiInACan::Repo.all.count.must_equal 1
+        end
+
+        it "should set the id" do
+          web.update_repo_details
+          CiInACan::Repo.all.first.id.must_equal test.expected_id
+        end
+
+        it "should set the build commands" do
+          web.update_repo_details
+          CiInACan::Repo.all.first.build_commands.must_equal test.expected_commands
+        end
+
+      end
+
+      describe "a repo that been saved before" do
+
+        before do
+          CiInACan::Repo.create(id: test.expected_id)
+          params[:captures] = test.captures
+          params[:commands] = test.commands
+        end
+
+        it "should create a repo" do
+          web.update_repo_details
+          CiInACan::Repo.all.count.must_equal 1
+        end
+
+        it "should set the build commands" do
+          web.update_repo_details
+          CiInACan::Repo.all.first.build_commands.must_equal test.expected_commands
+        end
+
+        it "should return the repo" do
+          result = web.update_repo_details
+          result.is_a? CiInACan::Repo
+          result.id.must_equal test.expected_id
+        end
+
+      end
+
+    end
+
+  end
+
+  describe "showing a list of the runs" do
+
+    it "should return the html for all of the runs" do
+
+      runs          = Object.new
+      array_of_runs = Object.new
+      html          = Object.new
+      list_of_runs  = Object.new
+
+
+      CiInACan::Run.stubs(:all).returns runs
+      runs.stubs(:to_a).returns array_of_runs
+      CiInACan::ViewModels::AListOfRuns.stubs(:new)
+                                       .with(array_of_runs)
+                                       .returns list_of_runs
+      list_of_runs.stubs(:to_html).returns html
+
+      result = web.show_a_list_of_the_runs
+
+      result.must_be_same_as html
+
+    end
+
+  end
+
+  describe "starting a new build" do
+
+    [:url, :repo, :api_key, :jobs_location, :uuid].to_objects {[
+      ['account/name/api_key', 'account/name', 'api_key', 'the job', 'the uuid'],
+      ['one/two/three',        'one/two',      'three',   'A',       'B']
+    ]}.each do |test|
+
+      describe "when the api key is valid" do
+
+        let(:repo) do
+          r = Object.new
+          r.stubs(:api_key).returns test.api_key
+          r
+        end
+
+        let(:json) { Object.new }
+
+        before do
+          params[:captures] = [test.url]
+
+          params.stubs(:to_json).returns json
+
+          CiInACan::Repo.stubs(:find)
+                        .with(test.repo)
+                        .returns repo
+
+          CiInACan::Web.stubs(:jobs_location).returns test.jobs_location
+
+          CiInACan::FileSystem.stubs(:create_file)
+
+          uuid = Object.new
+          uuid.stubs(:generate).returns test.uuid
+          UUID.stubs(:new).returns uuid
+        end
+
+        it "should not fail when we call it" do
+          web.start_a_new_build
+        end
+
+        it "should write a file" do
+          CiInACan::FileSystem.expects(:create_file)
+                              .with("#{test.jobs_location}/#{test.uuid}.json", json)
+          web.start_a_new_build
+        end
+
+        it "should return the json back" do
+          web.start_a_new_build.must_be_same_as json
+        end
+
+      end
+
+    end
+
+    [:url, :repo, :api_key, :jobs_location, :uuid].to_objects {[
+      ['account/name/api_key', 'account/name', 'api_key', 'the job', 'the uuid'],
+    ]}.each do |test|
+
+      describe "when the repo does not exist" do
+
+        let(:repo) do
+          r = Object.new
+          r.stubs(:api_key).returns test.api_key
+          r
+        end
+
+        before do
+          params[:captures] = [test.url]
+
+          CiInACan::Repo.stubs(:find)
+                        .with(test.repo)
+                        .returns nil
+
+          CiInACan::FileSystem.stubs(:create_file)
+
+          uuid = Object.new
+          uuid.stubs(:generate).returns test.uuid
+          UUID.stubs(:new).returns uuid
+        end
+
+        it "should throw an error" do
+          error_hit = false
+          begin
+            web.start_a_new_build
+          rescue
+            error_hit = true
+          end
+
+          error_hit.must_equal true
+        end
+
+        it "should not write any files" do
+          called = false
+          # having some issue with never?
+          CiInACan::FileSystem.stubs(:create_file).with { |a, b| [a, b].inspect; called = true; true }
+          -> { web.start_a_new_build }.call_safely
+          called.must_equal false
+        end
+
+      end
+
+    end
+
+    [:url, :repo, :api_key, :jobs_location, :uuid, :the_other_api_key].to_objects {[
+      ['account/name/api_key', 'account/name', 'api_key', 'the job', 'the uuid', 'sdf'],
+      ['account/name/api_key', 'account/name', 'api_key', 'the job', 'the uuid', 'abc'],
+    ]}.each do |test|
+
+      describe "when the repo api key does not match the request" do
+
+        let(:repo) do
+          r = Object.new
+          r.stubs(:api_key).returns test.the_other_api_key
+          r
+        end
+
+        before do
+          params[:captures] = [test.url]
+
+          CiInACan::Repo.stubs(:find)
+                        .with(test.repo)
+                        .returns repo
+
+          CiInACan::FileSystem.stubs(:create_file)
+
+          uuid = Object.new
+          uuid.stubs(:generate).returns test.uuid
+          UUID.stubs(:new).returns uuid
+        end
+
+        it "should throw an error" do
+          error_hit = false
+          begin
+            web.start_a_new_build
+          rescue
+            error_hit = true
+          end
+
+          error_hit.must_equal true
+        end
+
+        it "should not write any files" do
+          called = false
+          # having some issue with never?
+          CiInACan::FileSystem.stubs(:create_file).with { |a, b| [a, b].inspect; called = true; true }
+          -> { web.start_a_new_build }.call_safely
+          called.must_equal false
+        end
+
+      end
+
+    end
+
+  end
+
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ci_in_a_can
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.2.5
 platform: ruby
 authors:
 - Darren Cauthon
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-03-08 00:00:00.000000000 Z
+date: 2014-03-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -248,10 +248,13 @@ files:
 - lib/ci_in_a_can/test_runner.rb
 - lib/ci_in_a_can/version.rb
 - lib/ci_in_a_can/view_models/a_list_of_runs.rb
+- lib/ci_in_a_can/view_models/login_form.rb
+- lib/ci_in_a_can/view_models/repo_form.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.rb
 - lib/ci_in_a_can/web_content.rb
 - sample.json
 - spec/ci_in_a_can/bash_result_spec.rb
@@ -270,6 +273,7 @@ files:
 - 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/ci_in_a_can/web_spec.rb
 - spec/spec_helper.rb
 - spec/temp/.gitkeep
 homepage: ''
@@ -313,5 +317,6 @@ test_files:
 - 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/ci_in_a_can/web_spec.rb
 - spec/spec_helper.rb
 - spec/temp/.gitkeep