proxy_pac_rb 0.3.8 → 0.4.0

data/spec/rack/proxy_pac_linter_spec.rb

@@ -0,0 +1,65 @@
+require 'spec_helper'
+require 'proxy_pac_rb/rack/proxy_pac_linter'
+
+RSpec.describe ProxyPacRb::Rack::ProxyPacLinter, type: :rack_test do
+  let(:content) do
+    <<-EOS.strip_heredoc.chomp
+    function FindProxyForURL(url, host) {
+      return "DIRECT";
+    }
+    EOS
+  end
+
+  before(:each) { get '/' }
+  subject(:body) { last_response.body }
+
+  context 'when valid proxy pac is given' do
+    let(:app) do
+      a = Class.new(Sinatra::Base) do
+        before do
+          content_type 'application/x-ns-proxy-autoconfig'
+        end
+
+        get '/' do
+          <<-EOS.strip_heredoc.chomp
+          function FindProxyForURL(url, host) {
+            return "DIRECT";
+          }
+          EOS
+        end
+      end
+
+      a.use ProxyPacRb::Rack::ProxyPacLinter
+
+      a.new
+    end
+
+    it { expect(body).to include content }
+  end
+
+  context 'when invalid proxy pac is given' do
+    let(:content) { 'Unexpected string' }
+
+    let(:app) do
+      a = Class.new(Sinatra::Base) do
+        before do
+          content_type 'application/x-ns-proxy-autoconfig'
+        end
+
+        get '/' do
+          <<-EOS.strip_heredoc.chomp
+          function FindProxyForURL(url, host) {
+            return $"§$ "DIRECT";
+          }
+          EOS
+        end
+      end
+
+      a.use ProxyPacRb::Rack::ProxyPacLinter
+
+      a.new
+    end
+
+    it { expect(body).to include content }
+  end
+end

data/spec/spec_helper.rb

@@ -12,6 +12,7 @@ Bundler.require :default, :test, :development
 
 # Loading support files
 Dir.glob(::File.expand_path('../support/*.rb', __FILE__)).each { |f| require_relative f }
+Dir.glob(::File.expand_path('../support/**/*.rb', __FILE__)).each { |f| require_relative f }
 
 # Avoid writing "describe LocalPac::MyClass do [..]" but "describe MyClass do [..]"
 include ProxyPacRb

data/spec/support/matchers/file.rb

@@ -0,0 +1,23 @@
+RSpec::Matchers.define :be_existing_file do |_|
+  match { |actual| File.file?(actual) }
+
+  failure_message do |actual|
+    format("expected that file \"%s\" exists", actual)
+  end
+
+  failure_message_when_negated do |actual|
+    format("expected that file \"%s\" does not exist", actual)
+  end
+end
+
+RSpec::Matchers.define :have_content do |expected|
+  match { |actual| File.read(actual).chomp == expected.chomp }
+
+  failure_message do |actual|
+    format("expected that file \"%s\" contains:\n%s", actual, expected)
+  end
+
+  failure_message_when_negated do |actual|
+    format("expected that file \"%s\" does not contain:\n%s", actual, expected)
+  end
+end

data/spec/support/rack_test.rb

@@ -0,0 +1 @@
+require 'rack/test'

data/spec/support/rspec.rb

@@ -5,7 +5,7 @@ RSpec.configure do |config|
 
   config.default_formatter = 'doc' if config.files_to_run.one?
 
-  config.profile_examples = 10
+  #  config.profile_examples = 10
   config.order = :random
   Kernel.srand config.seed
 
@@ -17,4 +17,6 @@ RSpec.configure do |config|
     mocks.syntax = :expect
     mocks.verify_partial_doubles = true
   end
+
+  config.include Rack::Test::Methods, type: :rack_test
 end

data/spec/support/webmock.rb

@@ -0,0 +1 @@
+require 'webmock/rspec'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: proxy_pac_rb
 version: !ruby/object:Gem::Version
-  version: 0.3.8
+  version: 0.4.0
 platform: ruby
 authors:
 - Dennis Günnewig
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-02-02 00:00:00.000000000 Z
+date: 2015-03-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable
@@ -80,37 +80,45 @@ files:
 - cucumber.yml
 - doc/dependencies/dependencies.db
 - features/.keep
+- features/check_proxy_pac.feature
 - features/compress_proxy_pac.feature
 - features/resolve_proxy.feature
 - features/step_definitions.rb
 - features/support/aruba.rb
 - features/support/env.rb
-- features/support/fixtures.rb
 - features/support/reporting.rb
 - files/sample.pac
 - files/sample2.pac
 - files/sample3.pac
 - lib/proxy_pac_rb.rb
-- lib/proxy_pac_rb/cli.rb
 - lib/proxy_pac_rb/cli/compress.rb
 - lib/proxy_pac_rb/cli/compress_proxy_pac.rb
 - lib/proxy_pac_rb/cli/find.rb
 - lib/proxy_pac_rb/cli/find_proxy.rb
+- lib/proxy_pac_rb/cli/lint.rb
+- lib/proxy_pac_rb/cli/lint_proxy_pac.rb
 - lib/proxy_pac_rb/cli/runner.rb
 - lib/proxy_pac_rb/cli/shared.rb
 - lib/proxy_pac_rb/cli/show.rb
 - lib/proxy_pac_rb/cli_validator.rb
 - lib/proxy_pac_rb/encoding.rb
 - lib/proxy_pac_rb/environment.rb
+- lib/proxy_pac_rb/errors.rb
 - lib/proxy_pac_rb/exceptions.rb
-- lib/proxy_pac_rb/file.rb
-- lib/proxy_pac_rb/javascript_compressor.rb
+- lib/proxy_pac_rb/javascript.rb
+- lib/proxy_pac_rb/main.rb
 - lib/proxy_pac_rb/parser.rb
 - lib/proxy_pac_rb/proxy_pac.rb
+- lib/proxy_pac_rb/proxy_pac_compressor.rb
+- lib/proxy_pac_rb/proxy_pac_dumper.rb
 - lib/proxy_pac_rb/proxy_pac_file.rb
 - lib/proxy_pac_rb/proxy_pac_js.rb
-- lib/proxy_pac_rb/proxy_pac_template
+- lib/proxy_pac_rb/proxy_pac_linter.rb
+- lib/proxy_pac_rb/proxy_pac_loader.rb
+- lib/proxy_pac_rb/proxy_pac_parser.rb
 - lib/proxy_pac_rb/proxy_pac_template.rb
+- lib/proxy_pac_rb/rack/proxy_pac_compressor.rb
+- lib/proxy_pac_rb/rack/proxy_pac_linter.rb
 - lib/proxy_pac_rb/runtime.rb
 - lib/proxy_pac_rb/runtimes.rb
 - lib/proxy_pac_rb/runtimes/rubyracer.rb
@@ -124,14 +132,19 @@ files:
 - script/release
 - script/test_web
 - script/unit_test
+- spec/api_spec.rb
 - spec/environment_spec.rb
-- spec/file_spec.rb
 - spec/parser_spec.rb
+- spec/rack/proxy_pac_compressor_spec.rb
+- spec/rack/proxy_pac_linter_spec.rb
 - spec/spec_helper.rb
 - spec/support/aruba.rb
+- spec/support/matchers/file.rb
+- spec/support/rack_test.rb
 - spec/support/reporting.rb
 - spec/support/rspec.rb
 - spec/support/strip.rb
+- spec/support/webmock.rb
 - tmp/functions.js
 - tmp/functions.rb
 - tmp/sample.pac
@@ -146,9 +159,9 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - "~>"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -162,19 +175,24 @@ specification_version: 4
 summary: gem to parse proxy auto-config files.
 test_files:
 - features/.keep
+- features/check_proxy_pac.feature
 - features/compress_proxy_pac.feature
 - features/resolve_proxy.feature
 - features/step_definitions.rb
 - features/support/aruba.rb
 - features/support/env.rb
-- features/support/fixtures.rb
 - features/support/reporting.rb
+- spec/api_spec.rb
 - spec/environment_spec.rb
-- spec/file_spec.rb
 - spec/parser_spec.rb
+- spec/rack/proxy_pac_compressor_spec.rb
+- spec/rack/proxy_pac_linter_spec.rb
 - spec/spec_helper.rb
 - spec/support/aruba.rb
+- spec/support/matchers/file.rb
+- spec/support/rack_test.rb
 - spec/support/reporting.rb
 - spec/support/rspec.rb
 - spec/support/strip.rb
+- spec/support/webmock.rb
 has_rdoc: 

data/features/support/fixtures.rb

@@ -1,15 +0,0 @@
-# encoding: utf-8
-module FeatureHelper
-  # Helper for fixtures
-  module Fixtures
-    def fixtures_manager
-      @fixtures_manager ||= FeduxOrgStdlib::FixturesManagement::FixturesManager.new
-
-      @fixtures_manager.load_fixtures(File.expand_path('../../../fixtures', __FILE__))
-
-      @fixtures_manager
-    end
-  end
-end
-
-World(FeatureHelper::Fixtures)

data/lib/proxy_pac_rb/cli.rb

@@ -1,75 +0,0 @@
-# encoding: utf-8
-module ProxyPacRb
-  # Command line interface
-  class Cli
-    attr_accessor :options
-
-    def self.start(argv = ARGV)
-      cli = new
-      cli.parse(argv)
-
-      CliValidator.new(cli.options).validate
-
-      environment = ProxyPacRb::Environment.new(client_ip: cli.options.client_ip, time: cli.options.time)
-      file = ProxyPacRb::Parser.new(environment).source(cli.options.proxy_pac_file)
-
-      $stderr.printf("%30s: %-s\n", 'url', 'result')
-      cli.options.urls.each do |u|
-        begin
-          $stderr.printf("%30s: %-s\n", u, file.find(u))
-        rescue Exceptions::UrlInvalid
-          $stderr.puts "You provide an invalid url \"#{u}\". Please use a correct one."
-        end
-      end
-    end
-
-    def initialize
-      @options = OpenStruct.new
-
-      @options.client_ip = IPAddr.new('127.0.0.1')
-      @options.time      = Time.now
-    end
-
-    def parse(argv)
-      parser = OptionParser.new do |opts|
-        opts.banner = 'Usage: pprb [options] url1 [url2..urln]'
-
-        opts.on('-c', '--client-ip IP', 'Client ip address') do |i|
-          options.client_ip = IPAddr.new(i)
-        end
-
-        opts.on('-t', '--time YYYY-MM-DD HH:MM:SS', 'Time to use during lookup url') do |t|
-          options.time = Time.parse(t)
-        end
-
-        opts.on('-p', '--proxy-pac FILE|URL', 'Proxy.pac-file') do |f|
-          uri = Addressable::URI.parse(f)
-
-          uri.path = ::File.expand_path(uri.path) if uri.host.nil?
-
-          ENV.delete 'HTTP_PROXY'
-          ENV.delete 'HTTPS_PROXY'
-          ENV.delete 'http_proxy'
-          ENV.delete 'https_proxy'
-
-          options.proxy_pac_file = open(uri, proxy: false).read
-        end
-
-        opts.on_tail('-h', '--help', 'Show this message') do
-          $stderr.puts opts
-          exit
-        end
-
-        opts.on_tail('--version', 'Show version') do
-          $stderr.puts ProxyPacRb::VERSION
-          exit
-        end
-      end
-
-      $stderr.puts parser.help if argv.blank?
-
-      parser.parse!(argv)
-      options.urls = argv
-    end
-  end
-end

data/lib/proxy_pac_rb/file.rb

@@ -1,21 +0,0 @@
-module ProxyPacRb
-  # Proxy pac file
-  class File
-    private
-
-    attr_reader :javascript
-
-    public
-
-    def initialize(javascript)
-      @javascript  = javascript
-    end
-
-    def find(url)
-      uri = Addressable::URI.heuristic_parse(url)
-      fail Exceptions::UrlInvalid, 'url is missing host' unless uri.host
-
-      javascript.call('FindProxyForURL', url, uri.host)
-    end
-  end
-end

data/lib/proxy_pac_rb/javascript_compressor.rb

@@ -1,9 +0,0 @@
-# encoding: utf-8
-module ProxyPacRb
-  # Compress javascript files
-  class JavascriptCompressor
-    def compress(proxy_pac_template)
-      proxy_pac_template.compressed_content = Uglifier.new.compile(proxy_pac_template.raw_content)
-    end
-  end
-end

data/lib/proxy_pac_rb/proxy_pac_template

@@ -1,39 +0,0 @@
-# encoding: utf-8
-module ProxyPacRb
-  class ProxyPacTemplate
-
-    private
-
-    attr_reader :path
-
-    public
-
-    def initialize(path)
-      @path = path
-    end
-
-    def path
-      @path + '.in'
-    end
-
-    def content
-      read_proxy_pac(path)
-    end
-
-    private
-
-    def read_proxy_pac(path)
-      uri = Addressable::URI.parse(path)
-
-      uri.path = ::File.expand_path(uri.path) if uri.host.nil?
-
-      ENV.delete 'HTTP_PROXY'
-      ENV.delete 'HTTPS_PROXY'
-      ENV.delete 'http_proxy'
-      ENV.delete 'https_proxy'
-
-      open(uri, proxy: false).read
-    end
-
-  end
-end

data/spec/file_spec.rb

@@ -1,45 +0,0 @@
-# encoding: utf-8
-require 'spec_helper'
-
-describe File do
-  let(:simple) do
-    <<-EOS.strip_heredoc
-      function FindProxyForURL(url, host) {
-        return "DIRECT";
-      }
-    EOS
-  end
-
-  let(:client_ip_pac) do
-    <<-EOS.strip_heredoc
-      function FindProxyForURL(url, host) {
-        if ( myIpAddress() == '127.0.0.2' ) {
-          return "DIRECT";
-        } else {
-          return "PROXY localhost:8080";
-        }
-      }
-    EOS
-  end
-
-  let(:time_pac) do
-    <<-EOS.strip_heredoc
-      function FindProxyForURL(url, host) {
-        if ( timeRange(8, 18) ) {
-          return "PROXY localhost:8080";
-        } else {
-          return "DIRECT";
-        }
-      }
-    EOS
-  end
-
-  context '#find' do
-    it 'returns result of proxy.pac' do
-      javascript = double('javascript')
-      expect(javascript).to receive(:call).with('FindProxyForURL', 'http://localhost', 'localhost')
-
-      ProxyPacRb::File.new(javascript).find('http://localhost')
-    end
-  end
-end