sinatra-bouncer 1.1.2 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 8c2cecc7324f61545be2136b73b66a0e3965ac64
-  data.tar.gz: e83e24a0f44fedf13b0caab14378007f80a16d8a
+SHA256:
+  metadata.gz: 6b8592539c3824fa1985a4c02b877bd7d53e6ab2635f4227b173325f16484465
+  data.tar.gz: 1aac0ef35f8584e83cbafd5843172798f8967ad217235f9048e476cf3cc8945f
 SHA512:
-  metadata.gz: 2a9cf4365f81e73630bc845767d29bad3ed3e08f1c033c961eb28d5f52e10235b1611a1634626ffc1e581a833504ddd4987678677e157d28a1b110931aa756b5
-  data.tar.gz: 7eb46c0a4567a20d9c20952bb6da6eba78fb18fd45637b444c64c1a2e5cf61866b835b1e5952be7ddd08d4e9424db266b598715121c03767456205924ac71224
+  metadata.gz: 83ed25a0acff1d864f978dbbd3667d20f53be0bea36fa5db5abd55eeb4353cad53e29ebeb424f1c462e4a8790444f4a5850cad33c579729aab684ff4f0d482aa
+  data.tar.gz: a3caa0c02709faa60e5512f73f2b4c4f21066f519999e014ffac2545271198d8a98a7a73d430494a185a00e0ea255170e1693274c7fb8bca51b56c6108fcd8a6

data/.gitignore

@@ -0,0 +1,23 @@
+# See http://help.github.com/ignore-files/ for more about ignoring files.
+#
+# If you find yourself ignoring temporary files generated by your text editor
+# or operating system, you probably want to add a global ignore instead:
+#   git config --global core.excludesfile ~/.gitignore_global
+
+# Ignore bundler config
+/.bundle
+
+# Ignore all logfiles and tempfiles.
+*.log
+tmp/*
+
+# Rubymine stuff
+.idea
+
+# Ignore all gem files.
+*.gem
+
+core/config.yml
+persist/database.yml
+
+Gemfile.lock

data/.rubocop.yml

@@ -0,0 +1,22 @@
+inherit_from: ~/.config/rubocop/config.yml
+
+AllCops:
+  Exclude:
+    - 'bin/*'
+
+Layout/LineLength:
+  Exclude:
+    - 'spec/**/*.rb'
+
+# setting to 6 to match RubyMine autoformat
+Layout/FirstArrayElementIndentation:
+  IndentationWidth: 6
+
+# rspec blocks are huge by design
+Metrics/BlockLength:
+  Exclude:
+    - 'tests/spec/**/*.rb'
+
+Metrics/ModuleLength:
+  Exclude:
+    - 'tests/spec/**/*.rb'

data/.ruby-version

@@ -0,0 +1 @@
+ruby-3.1.4

data/.simplecov

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+SimpleCov.start do
+   coverage_dir '.coverage'
+   enable_coverage :branch
+
+   root __dir__
+
+   add_filter 'tests/'
+   # add_filter 'spec/'
+end

data/Gemfile

@@ -1,4 +1,19 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
-# Specify your gem's dependencies in dirt_core.gemspec
-gemspec
+# Gem dependencies in dirt_core.gemspec
+gemspec
+
+group :development do
+   gem 'bundler', '~> 2.4'
+   gem 'rake', '~> 13.0'
+   gem 'yard', '~> 0.9'
+end
+
+group :test do
+   gem 'capybara', '~> 3.39'
+   gem 'cucumber', '~> 8.0'
+   gem 'rspec', '~> 3.12'
+   gem 'simplecov', '~> 0.22'
+end

data/Rakefile

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
+require 'yard'
+
+RSpec::Core::RakeTask.new(:spec)
+
+task default: :spec
+
+YARD::Rake::YardocTask.new do |t|
+   t.files = %w[lib/**/*.rb]
+   # t.options       = %w[--some-option]
+   t.stats_options = ['--list-undoc']
+end

data/cucumber.yml

@@ -0,0 +1 @@
+default: --publish-quiet

data/lib/sinatra/{basic_bouncer.rb → bouncer/basic_bouncer.rb}

@@ -1,10 +1,11 @@
+# frozen_string_literal: true
+
 require_relative 'rule'
 
 module Sinatra
    module Bouncer
       class BasicBouncer
-         attr_accessor :bounce_with
-         attr_accessor :rules_initializer
+         attr_accessor :bounce_with, :rules_initializer
 
          def initialize
             # @rules = Hash.new do |method_to_paths, method|
@@ -13,10 +14,11 @@ module Sinatra
             #    end
             # end
 
-            @ruleset           = Hash.new do
+            @ruleset = Hash.new do
                []
             end
-            @rules_initializer = Proc.new {}
+
+            @rules_initializer = proc {}
          end
 
          def reset!
@@ -25,7 +27,8 @@ module Sinatra
 
          def can(method, *paths)
             if block_given?
-               raise BouncerError.new('You cannot provide a block to #can. If you wish to conditionally allow, use #can_sometimes instead.')
+               hint = 'If you wish to conditionally allow, use #can_sometimes instead.'
+               raise BouncerError, "You cannot provide a block to #can. #{ hint }"
             end
 
             can_sometimes(method, *paths) do
@@ -34,8 +37,9 @@ module Sinatra
          end
 
          def can_sometimes(method, *paths, &block)
-            unless block_given?
-               raise BouncerError.new('You must provide a block to #can_sometimes. If you wish to always allow, use #can instead.')
+            unless block
+               hint = 'If you wish to always allow, use #can instead.'
+               raise BouncerError, "You must provide a block to #can_sometimes. #{ hint }"
             end
 
             paths.each do |path|
@@ -55,7 +59,7 @@ module Sinatra
 
          def bounce(instance)
             if bounce_with
-               instance.instance_exec &bounce_with
+               instance.instance_exec(&bounce_with)
             else
                instance.halt 403
             end
@@ -63,7 +67,6 @@ module Sinatra
       end
 
       class BouncerError < StandardError
-
       end
    end
-end
+end

data/lib/sinatra/{rule.rb → bouncer/rule.rb}

@@ -1,22 +1,24 @@
+# frozen_string_literal: true
+
 module Sinatra
    module Bouncer
       class Rule
-         def initialize(path, &ruleblock)
+         def initialize(path, &rule_block)
             if path == :all
                @path = :all
             else
-               path = '/' + path unless path.start_with?('/')
+               path = "/#{ path }" unless path.start_with?('/')
 
                @path = path.split('/')
             end
 
-            @rule = ruleblock
+            @rule = rule_block
          end
 
          def match_path?(path)
             return true if @path == :all
 
-            path = '/' + path unless path.start_with?('/')
+            path = "/#{ path }" unless path.start_with?('/')
 
             split_path = path.split('/')
             matches    = @path.length == split_path.length
@@ -34,15 +36,19 @@ module Sinatra
          def rule_passes?
             ruling = @rule.call
 
-            unless ruling.is_a?(TrueClass)|| ruling.is_a?(FalseClass)
+            unless ruling.is_a?(TrueClass) || ruling.is_a?(FalseClass)
                source = @rule.source_location.join(':')
-               raise BouncerError.new("Rule block at does not return explicit true/false.\n\n"+
-                                            "Rules must return explicit true or false to prevent accidental truthy values.\n\n"+
-                                            "Source: #{source}\n")
+               msg    = <<~ERR
+                  Rule block at does not return explicit true/false.
+                  Rules must return explicit true or false to prevent accidental truthy values.
+                  Source: #{ source }
+               ERR
+
+               raise BouncerError, msg
             end
 
             ruling
          end
       end
    end
-end
+end

data/lib/sinatra/bouncer/version.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+module Sinatra
+   module Bouncer
+      # Current version of the gem
+      VERSION = '1.3.0'
+   end
+end

data/lib/sinatra/bouncer.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 #--
 # Copyright (c) 2014 Tenjin Inc.
 #
@@ -21,7 +23,7 @@
 # WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 #++
 
-require_relative 'basic_bouncer'
+require_relative 'bouncer/basic_bouncer'
 
 module Sinatra
    module Bouncer
@@ -36,14 +38,12 @@ module Sinatra
          base_class.before do
             bouncer.reset! # must clear all rules otherwise will leave doors open
 
-            self.instance_exec &bouncer.rules_initializer
+            instance_exec(&bouncer.rules_initializer)
 
             http_method = request.request_method.downcase.to_sym
             path        = request.path.downcase
 
-            unless bouncer.can?(http_method, path)
-               bouncer.bounce(self)
-            end
+            bouncer.bounce(self) unless bouncer.can?(http_method, path)
          end
       end
 
@@ -55,7 +55,6 @@ module Sinatra
       def rules(&block)
          bouncer.rules_initializer = block
       end
-
       # End ExtensionMethods
 
       module HelperMethods
@@ -63,8 +62,8 @@ module Sinatra
             settings.bouncer.can(*args)
          end
 
-         def can_sometimes(*args, &block)
-            settings.bouncer.can_sometimes(*args, &block)
+         def can_sometimes(...)
+            settings.bouncer.can_sometimes(...)
          end
       end
    end
@@ -72,4 +71,4 @@ module Sinatra
    if defined? register
       register Bouncer
    end
-end
+end

data/sinatra_bouncer.gemspec

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+lib = File.expand_path('lib', __dir__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+
+require 'sinatra/bouncer/version'
+
+Gem::Specification.new do |spec|
+   spec.name    = 'sinatra-bouncer'
+   spec.version = Sinatra::Bouncer::VERSION
+   spec.authors = ['Tenjin Inc', 'Robin Miller']
+   spec.email   = %w[contact@tenjin.ca robin@tenjin.ca]
+
+   spec.summary     = 'Sinatra permissions plugin'
+   spec.description = 'Bouncer brings simple authorization to Sinatra.'
+   spec.homepage    = 'https://github.com/TenjinInc/Sinatra-Bouncer'
+   spec.license     = 'MIT'
+   spec.metadata    = {
+         'rubygems_mfa_required' => 'true'
+   }
+
+   spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features|integrations)/}) }
+   spec.require_paths = ['lib']
+
+   spec.required_ruby_version = '>= 3.1'
+
+   spec.add_dependency 'sinatra', '>= 2.2'
+end

data/tests/integrations/dev_defines_legal_routes.feature

@@ -0,0 +1,57 @@
+Feature: Developer defines legal routes
+   As a developer
+   So that clients can safely access my server
+   I will allow specific routes
+   
+   Scenario Outline: it should allows access to whitelist routes
+      Given a sinatra server with bouncer and routes:
+         | method | path   | allowed |
+         | get    | <path> | yes     |
+      When I visit /<path>
+      Then it should be at /<path>
+      And it should have status code 200
+      Examples:
+         | path         |
+         | some_path    |
+         | another_path |
+   
+   Scenario: it should NOT allow access to other routes
+      Given a sinatra server with bouncer and routes:
+         | method | path         | allowed |
+         | get    | some_path    | yes     |
+         | get    | illegal_path | no      |
+      When I visit /illegal_path
+      Then it should have status code 403
+   
+   Scenario Outline: it should allow multiple routes with a splat
+      Given a sinatra server with bouncer and routes:
+         | method | path    | allowed |
+         | get    | admin/* | yes     |
+      When I visit /admin/<sub_path>
+      Then it should be at /admin/<sub_path>
+      And it should have status code 200
+      Examples:
+         | sub_path  |
+         | dashboard |
+         | users     |
+   
+   Scenario Outline: it should allow splat to be in the middle of the route
+      Given a sinatra server with bouncer and routes:
+         | method | path           | allowed |
+         | get    | admin/*/create | yes     |
+      When I visit /admin/<sub_path>/create
+      Then it should be at /admin/<sub_path>/create
+      And it should have status code 200
+      Examples:
+         | sub_path |
+         | tasks    |
+         | users    |
+   
+   Scenario: it should forget rules between requests
+      Given a sinatra server with bouncer and routes:
+         | method | path      | allowed |
+         | get    | some_path | once    |
+      When I double visit /some_path
+      Then it should be at /some_path
+      And it should have status code 403
+      

data/tests/integrations/dev_installs_bouncer.feature

@@ -0,0 +1,12 @@
+Feature: Developer installs Bouncer
+   As a developer
+   So that I can secure my sinatra server
+   I will install bouncer
+   
+   Scenario: it should auto protect all routes
+      Given a sinatra server with bouncer and routes:
+         | method | path      |
+         | get    | some_path |
+      When I visit /some_path
+      Then it should have status code 403
+      

data/tests/integrations/step_definitions/given.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+Given 'a sinatra server with bouncer and routes:' do |table|
+   app = Capybara.app
+
+   allowed_paths = []
+
+   table.hashes.each do |row|
+      path = row[:path]
+      path = "/#{ path }" if path[0] != '/' # ensure leading slash
+
+      method = row[:method].to_sym
+      # build the routes
+      app.send(method, path) do
+         'The result of path'
+      end
+
+      is_once = row[:allowed] =~ /once/i
+
+      next unless is_once || parse_bool(row[:allowed])
+
+      allowed_paths << path
+
+      @allowed_once_paths << path if is_once
+   end
+
+   onces = @allowed_once_paths
+
+   app.rules do
+      allowed_paths.each do |path|
+         can_sometimes(:any_method, path) do
+            !onces.include?(path)
+         end
+      end
+   end
+end

data/tests/integrations/step_definitions/then.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+Then 'it should have status code {int}' do |status|
+   expect(page.driver.status_code).to eq status
+end
+
+Then 'it should be at {path}' do |path|
+   expect(page).to have_current_path path.to_s
+end

data/tests/integrations/step_definitions/when.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+When 'he/she/they/I double visit(s) {path}' do |path|
+   visit path
+   visit '/'
+   visit path
+end
+
+When 'he/she/they/I visit(s) {path}' do |path|
+   visit path
+end

data/tests/integrations/support/env.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+src_dir = File.expand_path('../../..', __dir__)
+$LOAD_PATH.unshift(src_dir) unless $LOAD_PATH.include?(src_dir)
+
+require 'simplecov'
+
+SimpleCov.command_name 'spec'
+
+require 'capybara/cucumber'
+require 'rspec/expectations'
+
+require 'tests/test_app'
+
+# == CAPYBARA ==
+Capybara.app = Sinatra::Application
+
+# Set this to whatever the server's normal port is for you. Sinatra is 4567; rack 9292 by default.
+# Also note: you have to be running the server for this to work.
+Capybara.asset_host = 'http://localhost:4567'
+
+# == REGULAR SETTINGS ==
+Before do
+   Capybara.reset_sessions!
+   Capybara.app.settings.bouncer.instance_variable_get(:@ruleset).clear
+
+   @allowed_once_paths = []
+end
+
+World(RSpec::Matchers)

data/tests/integrations/support/helpers.rb

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+
+# Helper methods for Cucumber steps
+module HelperMethods
+   # TEST_TMP_ROOT = Pathname.new(Dir.mktmpdir('bouncer_test_')).expand_path.freeze
+   # TEST_TMP_LOG  = (TEST_TMP_ROOT / 'log').expand_path.freeze
+
+   # Data conversion helpers for Cucumber steps
+   module Conversion
+      def extract_list(list_string)
+         (list_string || '').split(',').map(&:strip)
+      end
+
+      def symrow(table)
+         table.symbolic_hashes.first
+      end
+
+      def symtable(table)
+         table.map_headers do |header|
+            header.tr(' ', '_').downcase.to_sym
+         end
+      end
+
+      def parse_bool(string)
+         !(string =~ /t|y/i).nil?
+      end
+
+      def parse_phone(string)
+         string.to_s.split(/:\s+/)
+      end
+
+      def parse_duration(string)
+         scalar, unit = string.split(/\s/)
+
+         return nil if unit.nil? || unit.empty?
+
+         unit = "#{ unit }s" unless unit.end_with?('s')
+
+         unit_map = {
+               years:   365.25 * 86400,
+               months:  30 * 86400,
+               weeks:   7 * 86400,
+               days:    86400,
+               hours:   3600,
+               minutes: 60,
+               seconds: 1
+         }
+
+         scalar.to_i * unit_map[unit.downcase.to_sym]
+      end
+   end
+end
+
+# Inject the HelperMethods into the Cucumber test context
+World(HelperMethods, HelperMethods::Conversion)

data/tests/integrations/support/types.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+ParameterType(name:        'path',
+              regexp:      %r{/(?:\S+/?)*},
+              type:        Pathname,
+              transformer: lambda do |str|
+                 Pathname.new(str)
+              end)
+
+ParameterType(name:        'boolean',
+              regexp:      /(enabled|disabled|true|false|on|off|yes|no)/,
+              transformer: lambda do |str|
+                 %w[enabled true on yes].include? str.downcase
+              end)
+
+ParameterType(name:        'html element',
+              regexp:      /<(\S+)>/,
+              type:        String,
+              transformer: lambda do |str|
+                 str
+              end)

data/tests/spec/basic_bouncer_spec.rb

@@ -0,0 +1,148 @@
+# frozen_string_literal: true
+
+require_relative 'spec_helper'
+
+describe Sinatra::Bouncer::BasicBouncer do
+   let(:bouncer) { Sinatra::Bouncer::BasicBouncer.new }
+
+   describe '#can' do
+      it 'should raise an error if provided a block' do
+         msg = <<~ERR
+            You cannot provide a block to #can. If you wish to conditionally allow, use #can_sometimes instead.
+         ERR
+
+         expect do
+            bouncer.can(:post, 'some_path') do
+               # stub
+            end
+         end.to raise_error(Sinatra::Bouncer::BouncerError, msg.chomp)
+      end
+
+      it 'should handle a list of paths' do
+         bouncer.can(:post, 'some_path', 'other_path')
+
+         expect(bouncer.can?(:post, 'some_path')).to be true
+         expect(bouncer.can?(:post, 'other_path')).to be true
+      end
+
+      it 'should accept a splat' do
+         bouncer.can(:post, 'directory/*')
+
+         expect(bouncer.can?(:post, 'directory/some_path')).to be true
+      end
+   end
+
+   describe '#can_sometimes' do
+      it 'should accept :any_method to mean all http methods' do
+         bouncer.can_sometimes(:any_method, 'some_path') do
+            true
+         end
+
+         expect(bouncer.can?(:get, 'some_path')).to be true
+         expect(bouncer.can?(:post, 'some_path')).to be true
+         expect(bouncer.can?(:put, 'some_path')).to be true
+         expect(bouncer.can?(:delete, 'some_path')).to be true
+         expect(bouncer.can?(:options, 'some_path')).to be true
+         expect(bouncer.can?(:link, 'some_path')).to be true
+         expect(bouncer.can?(:unlink, 'some_path')).to be true
+         expect(bouncer.can?(:head, 'some_path')).to be true
+         expect(bouncer.can?(:trace, 'some_path')).to be true
+         expect(bouncer.can?(:connect, 'some_path')).to be true
+         expect(bouncer.can?(:patch, 'some_path')).to be true
+      end
+
+      it 'should accept :all to mean all paths' do
+         bouncer.can_sometimes(:get, :all) do
+            true
+         end
+
+         expect(bouncer.can?(:get, 'some_path')).to be true
+      end
+
+      it 'should accept a list of paths' do
+         bouncer.can_sometimes(:post, 'some_path', 'other_path') do
+            true
+         end
+
+         expect(bouncer.can?(:post, 'some_path')).to be true
+         expect(bouncer.can?(:post, 'other_path')).to be true
+      end
+
+      it 'should accept a splat' do
+         bouncer.can_sometimes(:post, 'directory/*') do
+            true
+         end
+
+         expect(bouncer.can?(:post, 'directory/some_path')).to be true
+      end
+
+      it 'should not raise an error if provided a block' do
+         expect do
+            bouncer.can_sometimes(:any_method, 'some_path') do
+               true
+            end
+         end.to_not raise_error
+      end
+
+      it 'should raise an error if not provided a block' do
+         msg = <<~ERR
+            You must provide a block to #can_sometimes. If you wish to always allow, use #can instead.
+         ERR
+
+         expect do
+            bouncer.can_sometimes(:any_method, 'some_path')
+         end.to raise_error(Sinatra::Bouncer::BouncerError, msg.chomp)
+      end
+   end
+
+   describe '#can?' do
+      it 'should pass when declared allowed' do
+         bouncer.can(:any_method, 'some_path')
+
+         expect(bouncer.can?(:post, 'some_path')).to be true
+      end
+
+      it 'should fail when not declared allowed' do
+         expect(bouncer.can?(:post, 'some_path')).to be false
+      end
+
+      it 'should pass if the rule block passes' do
+         bouncer.can_sometimes(:any_method, 'some_path') do
+            true
+         end
+
+         expect(bouncer.can?(:post, 'some_path')).to be true
+      end
+
+      it 'should fail if the rule block fails' do
+         bouncer.can_sometimes(:any_method, 'some_path') do
+            false
+         end
+
+         expect(bouncer.can?(:post, 'some_path')).to be false
+      end
+   end
+
+   describe '#bounce' do
+      it 'should run the bounce_with block on sinatra instance' do
+         runner  = nil
+         sinatra = double('sinatra')
+
+         bouncer.bounce_with = proc do
+            runner = self # self should be the sinatra double
+         end
+
+         bouncer.bounce(sinatra)
+
+         expect(runner).to be sinatra
+      end
+
+      it 'should halt 403 if no block provided' do
+         app = double('sinatra')
+
+         expect(app).to receive(:halt).with(403)
+
+         bouncer.bounce(app)
+      end
+   end
+end

data/tests/spec/rule_spec.rb

@@ -0,0 +1,67 @@
+# frozen_string_literal: true
+
+require_relative 'spec_helper'
+
+describe Sinatra::Bouncer::Rule do
+   describe '#match_path?' do
+      it 'should match simple paths' do
+         rule = Sinatra::Bouncer::Rule.new('/some_path') { true }
+
+         expect(rule.match_path?('/some_path')).to be true
+      end
+
+      it 'should append leading slashes to the given path' do
+         rule = Sinatra::Bouncer::Rule.new('some_path') { true }
+
+         expect(rule.match_path?('/some_path')).to be true
+      end
+
+      it 'should append leading slashes to the tested path' do
+         rule = Sinatra::Bouncer::Rule.new('/other_path') { true }
+
+         expect(rule.match_path?('other_path')).to be true
+      end
+
+      it 'should match splats' do
+         rule = Sinatra::Bouncer::Rule.new('/directory/*') { true }
+
+         %w[/directory/one /directory/two /directory/three].each do |path|
+            expect(rule.match_path?(path)).to be true
+         end
+      end
+
+      it 'should NOT match empty string to a splat' do
+         rule = Sinatra::Bouncer::Rule.new('/directory/*') { true }
+
+         expect(rule.match_path?('/directory/')).to be false
+      end
+
+      it 'should require that both paths are same length' do
+         rule = Sinatra::Bouncer::Rule.new('/directory/*') { true }
+
+         %w[/directory /directory/extra/length].each do |path|
+            expect(rule.match_path?(path)).to be false
+         end
+      end
+   end
+
+   describe '#rule_passes?' do
+      it 'should raise an error if rule returns nonbool' do
+         rule = Sinatra::Bouncer::Rule.new('/something') { nil }
+
+         expect { rule.rule_passes? }.to raise_error Sinatra::Bouncer::BouncerError
+      end
+
+      it 'should return true when the block is true' do
+         rule = Sinatra::Bouncer::Rule.new('/something') { true }
+
+         expect(rule.rule_passes?).to be true
+      end
+
+      it 'should return true when the block is false' do
+         rule = Sinatra::Bouncer::Rule.new('/something') { false }
+
+         expect(rule.rule_passes?).to be false
+      end
+   end
+end

data/tests/spec/spec_helper.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+src_dir = File.expand_path('../..', __dir__)
+$LOAD_PATH.unshift(src_dir) unless $LOAD_PATH.include?(src_dir)
+
+require 'simplecov'
+
+SimpleCov.command_name 'spec'
+
+require 'lib/sinatra/bouncer'
+require 'rspec/matchers'

data/tests/test_app.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+require 'sinatra'
+require 'lib/sinatra/bouncer'
+
+# class TestApp < Sinatra::Base
+#   register Sinatra::Bouncer
+
+# end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: sinatra-bouncer
 version: !ruby/object:Gem::Version
-  version: 1.1.2
+  version: 1.3.0
 platform: ruby
 authors:
-- Tenjin
+- Tenjin Inc
 - Robin Miller
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-10-02 00:00:00.000000000 Z
+date: 2023-09-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sinatra
@@ -17,113 +17,53 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: simplecov
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 2.14.1
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 2.14.1
-- !ruby/object:Gem::Dependency
-  name: cucumber
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 1.3.19
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 1.3.19
-- !ruby/object:Gem::Dependency
-  name: capybara
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: launchy
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: parallel_tests
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.2'
 description: Bouncer brings simple authorization to Sinatra.
-email: contact@tenjin.ca
+email:
+- contact@tenjin.ca
+- robin@tenjin.ca
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".gitignore"
+- ".rubocop.yml"
+- ".ruby-version"
+- ".simplecov"
 - Gemfile
 - MIT-LICENSE
 - README.md
-- lib/sinatra/basic_bouncer.rb
+- Rakefile
+- cucumber.yml
 - lib/sinatra/bouncer.rb
-- lib/sinatra/rule.rb
-homepage: http://www.tenjin.ca
-licenses: []
-metadata: {}
+- lib/sinatra/bouncer/basic_bouncer.rb
+- lib/sinatra/bouncer/rule.rb
+- lib/sinatra/bouncer/version.rb
+- sinatra_bouncer.gemspec
+- tests/integrations/dev_defines_legal_routes.feature
+- tests/integrations/dev_installs_bouncer.feature
+- tests/integrations/step_definitions/given.rb
+- tests/integrations/step_definitions/then.rb
+- tests/integrations/step_definitions/when.rb
+- tests/integrations/support/env.rb
+- tests/integrations/support/helpers.rb
+- tests/integrations/support/types.rb
+- tests/spec/basic_bouncer_spec.rb
+- tests/spec/rule_spec.rb
+- tests/spec/spec_helper.rb
+- tests/test_app.rb
+homepage: https://github.com/TenjinInc/Sinatra-Bouncer
+licenses:
+- MIT
+metadata:
+  rubygems_mfa_required: 'true'
 post_install_message: 
 rdoc_options: []
 require_paths:
@@ -132,15 +72,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 1.9.3
+      version: '3.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 3.4.18
 signing_key: 
 specification_version: 4
 summary: Sinatra permissions plugin