local_pac 0.1.13 → 0.2.0

data/spec/actions/print_title_spec.rb

@@ -0,0 +1,50 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe Actions::PrintTitle do
+  context '#initialize' do
+    it 'requires a title' do
+      expect {
+        Actions::PrintTitle.new('title')
+      }.not_to raise_error
+    end
+
+    it 'accepts a title width as well' do
+      expect {
+        Actions::PrintTitle.new('title', 25)
+      }.not_to raise_error
+    end
+  end
+
+  context '#run' do
+    it 'prints a title' do
+      action = Actions::PrintTitle.new('Hello World')
+
+      result = capture(:stdout) do
+        action.run
+      end
+
+      expect(result).to include('Hello World')
+    end
+
+    it 'prints a title with #' do
+      action = Actions::PrintTitle.new('Hello World' )
+
+      result = capture(:stdout) do
+        action.run
+      end
+
+      expect(result).to eq("###############\n# Hello World #\n###############\n\n")
+    end
+
+    it 'prints a title with 25 #' do
+      action = Actions::PrintTitle.new('Hello World', 25 )
+
+      result = capture(:stdout) do
+        action.run
+      end
+
+      expect(result).to eq("###########################\n####### Hello World #######\n###########################\n\n")
+    end
+  end
+end

data/spec/actions/reload_configuration_spec.rb

@@ -0,0 +1,38 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe Actions::ReloadConfiguration do
+  context '#initialize' do
+    it 'accepts a config instance' do
+      config = double(config)
+
+      expect {
+        Actions::ReloadConfiguration.new(config)
+      }.not_to raise_error
+    end
+  end
+
+  context '#run' do
+    it 'reloads configuration from known places' do
+      config_file = create_file 'config.yaml', <<-EOS.strip_heredoc
+      ---
+      :gem_path: '/asdf/bin/local_pac'
+      EOS
+
+      LocalPac.config = LocalPac::Config.new(config_file)
+
+      config_file = create_file 'config.yaml', <<-EOS.strip_heredoc
+      ---
+      :gem_path: 'asdf'
+      EOS
+
+      config = LocalPac::Config.new(config_file)
+
+      with_environment 'HOME' => working_directory do
+        Actions::ReloadConfiguration.new(config).run
+      end
+
+      expect(LocalPac.config.gem_path).to eq('asdf')
+    end
+  end
+end

data/spec/actions/reload_repository_spec.rb

@@ -0,0 +1,35 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe Actions::ReloadLocalStorage do
+  context '#initialize' do
+    it 'requires a sinatra app' do
+      expect {
+        Actions::ReloadLocalStorage.new(sintra_app)
+      }
+    end
+  end
+
+  context '#run' do
+    it 'reloads local storage' do
+      storage = double('local_storage')
+
+      sintra_app = double('sintra_app')
+      expect(sintra_app).to receive(:set).with(:local_storage, storage)
+
+      Actions::ReloadLocalStorage.new(sintra_app, storage).run
+    end
+
+    it 'reloads local storage for multiple apps' do
+      storage = double('local_storage')
+
+      sintra_app1 = double('sintra_app1')
+      expect(sintra_app1).to receive(:set).with(:local_storage, storage)
+
+      sintra_app2 = double('sintra_app2')
+      expect(sintra_app2).to receive(:set).with(:local_storage, storage)
+
+      Actions::ReloadLocalStorage.new([sintra_app1, sintra_app2], storage).run
+    end
+  end
+end

data/spec/actions/send_signal_spec.rb

@@ -0,0 +1,58 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe Actions::SendSignal do
+  context '#initialize' do
+    it 'requires a signal' do
+      expect { 
+        Actions::SendSignal.new
+      }.to raise_error ArgumentError
+    end
+
+    it 'accepts a pid_file' do
+      expect { 
+        Actions::SendSignal.new(:USR1, 'asdf')
+      }.not_to raise_error
+    end
+  end
+
+  context '#run' do
+    it 'sends a signal to process' do
+      read_pipe, write_pipe = IO.pipe
+      
+      unless child_pid = Kernel.fork # Child process
+        read_pipe.close
+
+        trap :USR1 do
+          write_pipe.print "Received signal"
+          write_pipe.close
+          Kernel.exit!
+        end
+
+        loop { sleep 1 }
+      end
+
+      write_pipe.close
+
+      pid_file = create_file 'pid_file', child_pid
+
+      sleep 1
+
+      Actions::SendSignal.new(:USR1, pid_file).run
+
+      expect(read_pipe.read).to eq('Received signal')
+      read_pipe.close
+    end
+
+    it 'logs a warning if pid file does not exist' do
+      LocalPac.ui_logger.level = ::Logger::WARN
+
+      result = capture(:stderr) do
+        Actions::SendSignal.new(:USR1, 'pid_file').run
+      end
+
+      expect(result).to include("does not exist. I'm not able to send daemon signal")
+    end
+
+  end
+end

data/spec/actions/show_config_spec.rb

@@ -0,0 +1,22 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe Actions::ShowConfig do
+  context '#initialize' do
+    it 'has no special requirements' do
+      expect {
+        Actions::ShowConfig.new
+      }.not_to raise_error
+    end
+  end
+
+  context '#run' do
+    it 'gathers all relevant information and prints them to stdout' do
+      result = capture(:stdout) do
+        Actions::ShowConfig.new.run
+      end
+
+      expect(result).to include('pid_file')
+    end
+  end
+end

data/spec/actions/show_process_information_spec.rb

@@ -0,0 +1,45 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe Actions::ShowProcessInformation do
+  context '#initialize' do
+    it 'requires a pid or a pid file' do
+      expect {
+        Actions::ShowProcessInformation.new($$)
+      }.not_to raise_error
+
+      file = create_file 'pid_file', '1'
+
+      expect {
+        Actions::ShowProcessInformation.new(::File.new(file))
+      }.not_to raise_error
+    end
+  end
+
+  context '#run' do
+    it 'gathers information about a running process' do
+      result = capture(:stdout) do
+        Actions::ShowProcessInformation.new($$).run
+      end
+
+      expect(result).to include('pid')
+      expect(result).to include($$.to_s)
+    end
+
+    it 'reports an error if process cannot be found' do
+      result = capture(:stdout) do
+        Actions::ShowProcessInformation.new(99999999999).run
+      end
+
+      expect(result).to include('Process "99999999999" cannot be found. It is not a running process.')
+    end
+
+    it 'reports an error if process cannot be found and gives a hint if pid file was used.' do
+      result = capture(:stdout) do
+        Actions::ShowProcessInformation.new('/path/not/exist').run
+      end
+
+      expect(result).to include('Pid-file /path/not/exist cannot be found. Please choose a correct path and try again.')
+    end
+  end
+end

data/spec/application_status_spec.rb

@@ -0,0 +1,40 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe ApplicationStatus do
+  context '#initialize' do
+    it 'has no special requirements' do
+      expect {
+        ApplicationStatus.new
+      }.not_to raise_error
+    end
+
+    it 'let you deactivate paging' do
+      expect {
+        ApplicationStatus.new(pager: false)
+      }.not_to raise_error
+    end
+  end
+
+  context '#show' do
+    it 'show application status' do
+      config_string = <<-EOS.strip_heredoc
+        :pid_file: #{::File.join(working_directory, 'run', 'pid')}
+      EOS
+
+      config_file = create_file('config.yaml', config_string)
+      pid_file = create_file('run/pid', $$)
+
+      config = LocalPac::Config.new(config_file)
+
+      result = capture(:stdout) do
+        ApplicationStatus.new(pager: false, config: config).show
+      end
+
+      expect(result).to include('osfamily')
+      expect(result).to include('pid_file')
+      expect(result).to include('pid_file')
+      expect(result).to include('cmdline')
+    end
+  end
+end

data/spec/config_spec.rb

@@ -23,7 +23,7 @@ describe LocalPac::Config do
 
     it 'returns default value if no config file is available' do
       config = LocalPac::Config.new(config_file)
-      file = File.join(ENV['HOME'], '.local', 'share', 'local_pac', 'run', 'pid')
+      file = ::File.join(ENV['HOME'], '.local', 'share', 'local_pac', 'run', 'pid')
 
       expect(config.pid_file).to eq(file)
     end
@@ -33,37 +33,37 @@ describe LocalPac::Config do
     it 'returns value of config file' do
       config_file = create_file 'config.yaml', <<-EOS.strip_heredoc
       ---
-      :local_storage: '/asdf/pid'
+      :local_storage: '/asdf/storage.git'
       EOS
 
       config = LocalPac::Config.new(config_file)
-      expect(config.local_storage).to eq('/asdf/pid')
+      expect(config.local_storage).to eq('/asdf/storage.git')
     end
 
     it 'returns default value if no config file is available' do
       config = LocalPac::Config.new(config_file)
-      file = File.join(ENV['HOME'], '.local', 'share', 'local_pac', 'data')
+      file = ::File.join(ENV['HOME'], '.local', 'share', 'local_pac', 'data', 'storage.git')
 
       expect(config.local_storage).to eq(file)
     end
   end
 
-  context '#log_sink' do
+  context '#access_log' do
     it 'returns value of config file' do
       config_file = create_file 'config.yaml', <<-EOS.strip_heredoc
       ---
-      :log_sink: '/asdf/log'
+      :access_log: '/asdf/log/access.log'
       EOS
 
       config = LocalPac::Config.new(config_file)
-      expect(config.log_sink).to eq('/asdf/log')
+      expect(config.access_log).to eq('/asdf/log/access.log')
     end
 
     it 'returns default value if no config file is available' do
       config = LocalPac::Config.new(config_file)
-      file = File.join(ENV['HOME'], '.local', 'share', 'local_pac', 'log')
+      file = ::File.join(ENV['HOME'], '.local', 'share', 'local_pac', 'log', 'access.log')
 
-      expect(config.log_sink).to eq(file)
+      expect(config.access_log).to eq(file)
     end
   end
 
@@ -80,7 +80,7 @@ describe LocalPac::Config do
 
     it 'returns default value if no config file is available' do
       config = LocalPac::Config.new(config_file)
-      file = File.expand_path('../../bin/local_pac', __FILE__)
+      file = ::File.expand_path('../../bin/local_pac', __FILE__)
 
       expect(config.executable).to eq(file)
     end
@@ -107,7 +107,7 @@ describe LocalPac::Config do
   context '#to_s' do
     it 'outputs a nice config overview' do
       config = LocalPac::Config.new(config_file)
-      expect(config.to_s).to include File.join('.local', 'share', 'local_pac', 'run', 'pid')
+      expect(config.to_s).to include ::File.join('.local', 'share', 'local_pac', 'run', 'pid')
     end
   end
 

data/spec/features/fetch_proxy_pac_spec.rb

@@ -16,7 +16,8 @@ describe 'Fetch proxy pac', :type => :feature do
 
     before :each do
       git_init(git_repo)
-      create_file(File.join(git_repo, 'file.pac'), valid_pac_file)
+      git_set_author(git_repo)
+      create_file(::File.join(git_repo, 'file.pac'), valid_pac_file)
       git_add(git_repo, 'file.pac')
       git_commit(git_repo)
     end
@@ -26,11 +27,11 @@ describe 'Fetch proxy pac', :type => :feature do
         include FeduxOrg::Stdlib::Filesystem
 
         def root_directory
-          File.expand_path('../../../', __FILE__)
+          ::File.expand_path('../../../', __FILE__)
         end
 
         def local_storage
-          File.join(working_directory, 'git_repo', '.git')
+          ::File.join(working_directory, 'git_repo', '.git')
         end
       end.new
 
@@ -38,27 +39,27 @@ describe 'Fetch proxy pac', :type => :feature do
       Capybara.app = LocalPac::App::FileServeController
     end
 
-    it 'finds an existing proxy pac', :focus do
+    it 'finds an existing proxy pac' do
       visit('/file.pac')
       expect(page).to have_content('function FindProxyForURL(){return"DIRECT"}')
     end
 
-    it 'exits with 404 if file does not exist', :focus do
+    it 'exits with 404 if file does not exist' do
       visit('/does_not_exist.pac')
       expect(page.status_code).to eq(404)
     end
 
-    it 'compresses data for transit if requested', :focus do
+    it 'compresses data for transit if requested' do
       def app
         config = Class.new do
           include FeduxOrg::Stdlib::Filesystem
 
           def root_directory
-            File.expand_path('../../../', __FILE__)
+            ::File.expand_path('../../../', __FILE__)
           end
 
           def local_storage
-            File.join(working_directory, 'git_repo', '.git')
+            ::File.join(working_directory, 'git_repo', '.git')
           end
         end.new
 
@@ -78,11 +79,11 @@ describe 'Fetch proxy pac', :type => :feature do
           include FeduxOrg::Stdlib::Filesystem
 
           def root_directory
-            File.expand_path('../../../', __FILE__)
+            ::File.expand_path('../../../', __FILE__)
           end
 
           def local_storage
-            File.join(working_directory, 'git_repo', '.git')
+            ::File.join(working_directory, 'git_repo', '.git')
           end
         end.new
 

data/spec/features/lookup_proxy_spec.rb

@@ -2,8 +2,8 @@
 require 'spec_helper'
 require 'spec_helper_features'
 
-describe 'Fetch proxy pac' do
-  context '/v1/pac' do
+describe 'Lookup proxy for url' do
+  context '/v1/lookup' do
     let(:valid_pac_file) do <<-EOS.strip_heredoc.chomp
                            function FindProxyForURL(url, host) {
                              return "DIRECT";
@@ -29,7 +29,8 @@ describe 'Fetch proxy pac' do
 
     before(:each) do
       git_init(git_repo)
-      create_file(File.join(git_repo, 'file.pac'), valid_pac_file)
+      git_set_author(git_repo)
+      create_file(::File.join(git_repo, 'file.pac'), valid_pac_file)
       git_add(git_repo, 'file.pac')
       git_commit(git_repo)
     end
@@ -39,11 +40,11 @@ describe 'Fetch proxy pac' do
         include FeduxOrg::Stdlib::Filesystem
 
         def root_directory
-          File.expand_path('../../../', __FILE__)
+          ::File.expand_path('../../../', __FILE__)
         end
 
         def local_storage
-          File.join(working_directory, 'git_repo', '.git')
+          ::File.join(working_directory, 'git_repo', '.git')
         end
       end.new
 
@@ -51,52 +52,43 @@ describe 'Fetch proxy pac' do
       Capybara.app = LocalPac::App::LookupController
     end
 
-    it 'looks up proxy', :focus do
+    it 'looks up proxy for valid url' do
+      search_for 'http://www.example.org', 'You asked me to look up'
+    end
+
+    it 'returns a error message for a invalid url' do
       visit('/file.pac')
       within('#search') do
-        fill_in 'url', :with => 'http://www.example.org'
+        fill_in 'url', :with => '§ASDF$$'
       end
-      expect(page).to have_content('asdf')
-    end
-
-    it 'shows a result output' do
-      rendered_view = <<-EOS.strip_heredoc
-      EOS
-      response = post('/file.pac', {:url => 'http://example.org'})
-      expect(last_response).to be_ok
-      expect(response.body).to eq('Please enter fully quallified url')
+      click_on('Search')
+      expect(page).to have_content('Invalid URL...')
     end
 
-    it 'handles a nil file as well' do
-      create_file(File.join(git_repo, 'file_nil.pac'), pac_file_return_nil)
-      git_add(git_repo, 'file_nil.pac')
-      git_commit(git_repo)
-
-      rendered_view = <<-EOS.strip_heredoc
-      EOS
-
-      response = post('/file_nil.pac', {:url => 'http://example.org'})
+    it 'returns a error message for a invalid url on the second page' do
+      visit('/file.pac')
+      within('#search') do
+        fill_in 'url', :with => 'http://www.example.org'
+      end
+      click_on('Search')
 
-      expect(last_response).to be_ok
-      expect(response.body).to eq(rendered_view)
-    end
+      within('#search') do
+        fill_in 'url', :with => '§ASDF$$'
+      end
 
-    it 'raises an exception on invalid pac file' do
-      create_file(File.join(git_repo, 'invalid.pac'), invalid_pac_file)
-      git_add(git_repo, 'invalid.pac')
-      git_commit(git_repo)
+      click_on('Search')
 
-      expect {
-        response = post('/invalid.pac', {:url => 'http://example.org'})
-      }.to raise_error RuntimeError
+      expect(page).to have_content('Invalid URL...')
     end
 
-    it 'returns a result for an existing proxy pac' do
-      pending
+    it 'returns a error message for a invalid url' do
+      visit('/file.pac')
+      within('#search') do
+        fill_in 'url', :with => ''
+      end
+      click_on('Search')
+      expect(page).to have_content('Invalid URL...')
     end
 
-    it 'returns a error if file does not exist' do
-      pending
-    end
   end
 end