shoulda-context 1.2.2 → 2.0.0

data/gemfiles/rails_6_0.gemfile

@@ -0,0 +1,38 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "appraisal"
+gem "bundler", "~> 1.0"
+gem "byebug"
+gem "m"
+gem "minitest"
+gem "mocha"
+gem "pry", "~> 0.12.0"
+gem "pry-byebug", "~> 3.6.0"
+gem "rake"
+gem "rubocop", "0.71.0"
+gem "snowglobe", ">= 0.3.0"
+gem "test-unit"
+gem "warnings_logger"
+gem "sqlite3", "~> 1.4.0"
+gem "spring"
+gem "spring-commands-rspec"
+gem "minitest-reporters"
+gem "rails", "~> 6.0.2"
+gem "puma", "~> 4.1"
+gem "sass-rails", ">= 6"
+gem "webpacker", "~> 4.0"
+gem "turbolinks", "~> 5"
+gem "jbuilder", "~> 2.7"
+gem "bcrypt", "~> 3.1.7"
+gem "bootsnap", ">= 1.4.2", require: false
+gem "listen", ">= 3.0.5", "< 3.2"
+gem "spring-watcher-listen", "~> 2.0.0"
+gem "capybara", ">= 2.15"
+gem "selenium-webdriver"
+gem "webdrivers"
+gem "rails-controller-testing", ">= 1.0.4"
+gem "pg", "~> 1.1", platform: :ruby
+
+gemspec path: "../"

data/gemfiles/rails_6_0.gemfile.lock

@@ -0,0 +1,286 @@
+PATH
+  remote: ..
+  specs:
+    shoulda-context (2.0.0)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    actioncable (6.0.2.2)
+      actionpack (= 6.0.2.2)
+      nio4r (~> 2.0)
+      websocket-driver (>= 0.6.1)
+    actionmailbox (6.0.2.2)
+      actionpack (= 6.0.2.2)
+      activejob (= 6.0.2.2)
+      activerecord (= 6.0.2.2)
+      activestorage (= 6.0.2.2)
+      activesupport (= 6.0.2.2)
+      mail (>= 2.7.1)
+    actionmailer (6.0.2.2)
+      actionpack (= 6.0.2.2)
+      actionview (= 6.0.2.2)
+      activejob (= 6.0.2.2)
+      mail (~> 2.5, >= 2.5.4)
+      rails-dom-testing (~> 2.0)
+    actionpack (6.0.2.2)
+      actionview (= 6.0.2.2)
+      activesupport (= 6.0.2.2)
+      rack (~> 2.0, >= 2.0.8)
+      rack-test (>= 0.6.3)
+      rails-dom-testing (~> 2.0)
+      rails-html-sanitizer (~> 1.0, >= 1.2.0)
+    actiontext (6.0.2.2)
+      actionpack (= 6.0.2.2)
+      activerecord (= 6.0.2.2)
+      activestorage (= 6.0.2.2)
+      activesupport (= 6.0.2.2)
+      nokogiri (>= 1.8.5)
+    actionview (6.0.2.2)
+      activesupport (= 6.0.2.2)
+      builder (~> 3.1)
+      erubi (~> 1.4)
+      rails-dom-testing (~> 2.0)
+      rails-html-sanitizer (~> 1.1, >= 1.2.0)
+    activejob (6.0.2.2)
+      activesupport (= 6.0.2.2)
+      globalid (>= 0.3.6)
+    activemodel (6.0.2.2)
+      activesupport (= 6.0.2.2)
+    activerecord (6.0.2.2)
+      activemodel (= 6.0.2.2)
+      activesupport (= 6.0.2.2)
+    activestorage (6.0.2.2)
+      actionpack (= 6.0.2.2)
+      activejob (= 6.0.2.2)
+      activerecord (= 6.0.2.2)
+      marcel (~> 0.3.1)
+    activesupport (6.0.2.2)
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      i18n (>= 0.7, < 2)
+      minitest (~> 5.1)
+      tzinfo (~> 1.1)
+      zeitwerk (~> 2.2)
+    addressable (2.7.0)
+      public_suffix (>= 2.0.2, < 5.0)
+    ansi (1.5.0)
+    appraisal (2.2.0)
+      bundler
+      rake
+      thor (>= 0.14.0)
+    ast (2.4.0)
+    bcrypt (3.1.13)
+    bootsnap (1.4.6)
+      msgpack (~> 1.0)
+    builder (3.2.4)
+    byebug (10.0.2)
+    capybara (3.32.0)
+      addressable
+      mini_mime (>= 0.1.3)
+      nokogiri (~> 1.8)
+      rack (>= 1.6.0)
+      rack-test (>= 0.6.3)
+      regexp_parser (~> 1.5)
+      xpath (~> 3.2)
+    childprocess (3.0.0)
+    coderay (1.1.2)
+    concurrent-ruby (1.1.6)
+    crass (1.0.6)
+    erubi (1.9.0)
+    ffi (1.12.2)
+    globalid (0.4.2)
+      activesupport (>= 4.2.0)
+    i18n (1.8.2)
+      concurrent-ruby (~> 1.0)
+    jaro_winkler (1.5.4)
+    jbuilder (2.10.0)
+      activesupport (>= 5.0.0)
+    listen (3.1.5)
+      rb-fsevent (~> 0.9, >= 0.9.4)
+      rb-inotify (~> 0.9, >= 0.9.7)
+      ruby_dep (~> 1.2)
+    loofah (2.4.0)
+      crass (~> 1.0.2)
+      nokogiri (>= 1.5.9)
+    m (1.5.1)
+      method_source (>= 0.6.7)
+      rake (>= 0.9.2.2)
+    mail (2.7.1)
+      mini_mime (>= 0.1.1)
+    marcel (0.3.3)
+      mimemagic (~> 0.3.2)
+    method_source (0.9.2)
+    mimemagic (0.3.4)
+    mini_mime (1.0.2)
+    mini_portile2 (2.4.0)
+    minitest (5.14.0)
+    minitest-reporters (1.4.2)
+      ansi
+      builder
+      minitest (>= 5.0)
+      ruby-progressbar
+    mocha (1.11.2)
+    msgpack (1.3.3)
+    nio4r (2.5.2)
+    nokogiri (1.10.9)
+      mini_portile2 (~> 2.4.0)
+    parallel (1.19.1)
+    parser (2.7.1.0)
+      ast (~> 2.4.0)
+    pg (1.2.3)
+    power_assert (1.1.7)
+    pry (0.12.2)
+      coderay (~> 1.1.0)
+      method_source (~> 0.9.0)
+    pry-byebug (3.6.0)
+      byebug (~> 10.0)
+      pry (~> 0.10)
+    public_suffix (4.0.3)
+    puma (4.3.3)
+      nio4r (~> 2.0)
+    rack (2.2.2)
+    rack-proxy (0.6.5)
+      rack
+    rack-test (1.1.0)
+      rack (>= 1.0, < 3)
+    rails (6.0.2.2)
+      actioncable (= 6.0.2.2)
+      actionmailbox (= 6.0.2.2)
+      actionmailer (= 6.0.2.2)
+      actionpack (= 6.0.2.2)
+      actiontext (= 6.0.2.2)
+      actionview (= 6.0.2.2)
+      activejob (= 6.0.2.2)
+      activemodel (= 6.0.2.2)
+      activerecord (= 6.0.2.2)
+      activestorage (= 6.0.2.2)
+      activesupport (= 6.0.2.2)
+      bundler (>= 1.3.0)
+      railties (= 6.0.2.2)
+      sprockets-rails (>= 2.0.0)
+    rails-controller-testing (1.0.4)
+      actionpack (>= 5.0.1.x)
+      actionview (>= 5.0.1.x)
+      activesupport (>= 5.0.1.x)
+    rails-dom-testing (2.0.3)
+      activesupport (>= 4.2.0)
+      nokogiri (>= 1.6)
+    rails-html-sanitizer (1.3.0)
+      loofah (~> 2.3)
+    railties (6.0.2.2)
+      actionpack (= 6.0.2.2)
+      activesupport (= 6.0.2.2)
+      method_source
+      rake (>= 0.8.7)
+      thor (>= 0.20.3, < 2.0)
+    rainbow (3.0.0)
+    rake (13.0.1)
+    rb-fsevent (0.10.3)
+    rb-inotify (0.10.1)
+      ffi (~> 1.0)
+    regexp_parser (1.7.0)
+    rubocop (0.71.0)
+      jaro_winkler (~> 1.5.1)
+      parallel (~> 1.10)
+      parser (>= 2.6)
+      rainbow (>= 2.2.2, < 4.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (>= 1.4.0, < 1.7)
+    ruby-progressbar (1.10.1)
+    ruby_dep (1.5.0)
+    rubyzip (2.3.0)
+    sass-rails (6.0.0)
+      sassc-rails (~> 2.1, >= 2.1.1)
+    sassc (2.2.1)
+      ffi (~> 1.9)
+    sassc-rails (2.1.2)
+      railties (>= 4.0.0)
+      sassc (>= 2.0)
+      sprockets (> 3.0)
+      sprockets-rails
+      tilt
+    selenium-webdriver (3.142.7)
+      childprocess (>= 0.5, < 4.0)
+      rubyzip (>= 1.2.2)
+    snowglobe (0.3.0)
+    spring (2.1.0)
+    spring-commands-rspec (1.0.4)
+      spring (>= 0.9.1)
+    spring-watcher-listen (2.0.1)
+      listen (>= 2.7, < 4.0)
+      spring (>= 1.2, < 3.0)
+    sprockets (4.0.0)
+      concurrent-ruby (~> 1.0)
+      rack (> 1, < 3)
+    sprockets-rails (3.2.1)
+      actionpack (>= 4.0)
+      activesupport (>= 4.0)
+      sprockets (>= 3.0.0)
+    sqlite3 (1.4.2)
+    test-unit (3.3.5)
+      power_assert
+    thor (1.0.1)
+    thread_safe (0.3.6)
+    tilt (2.0.10)
+    turbolinks (5.2.1)
+      turbolinks-source (~> 5.2)
+    turbolinks-source (5.2.0)
+    tzinfo (1.2.7)
+      thread_safe (~> 0.1)
+    unicode-display_width (1.6.1)
+    warnings_logger (0.1.0)
+    webdrivers (4.2.0)
+      nokogiri (~> 1.6)
+      rubyzip (>= 1.3.0)
+      selenium-webdriver (>= 3.0, < 4.0)
+    webpacker (4.2.2)
+      activesupport (>= 4.2)
+      rack-proxy (>= 0.6.1)
+      railties (>= 4.2)
+    websocket-driver (0.7.1)
+      websocket-extensions (>= 0.1.0)
+    websocket-extensions (0.1.4)
+    xpath (3.2.0)
+      nokogiri (~> 1.8)
+    zeitwerk (2.3.0)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  appraisal
+  bcrypt (~> 3.1.7)
+  bootsnap (>= 1.4.2)
+  bundler (~> 1.0)
+  byebug
+  capybara (>= 2.15)
+  jbuilder (~> 2.7)
+  listen (>= 3.0.5, < 3.2)
+  m
+  minitest
+  minitest-reporters
+  mocha
+  pg (~> 1.1)
+  pry (~> 0.12.0)
+  pry-byebug (~> 3.6.0)
+  puma (~> 4.1)
+  rails (~> 6.0.2)
+  rails-controller-testing (>= 1.0.4)
+  rake
+  rubocop (= 0.71.0)
+  sass-rails (>= 6)
+  selenium-webdriver
+  shoulda-context!
+  snowglobe (>= 0.3.0)
+  spring
+  spring-commands-rspec
+  spring-watcher-listen (~> 2.0.0)
+  sqlite3 (~> 1.4.0)
+  test-unit
+  turbolinks (~> 5)
+  warnings_logger
+  webdrivers
+  webpacker (~> 4.0)
+
+BUNDLED WITH
+   1.17.3

data/lib/shoulda/context.rb

@@ -1,20 +1,17 @@
-require 'shoulda/context/test_framework_detection'
-require 'shoulda/context/version'
-require 'shoulda/context/proc_extensions'
-require 'shoulda/context/assertions'
-require 'shoulda/context/context'
-require 'shoulda/context/autoload_macros'
+require "shoulda/context/autoload_macros"
+require "shoulda/context/configuration"
+require "shoulda/context/context"
+require "shoulda/context/dsl"
+require "shoulda/context/proc_extensions"
+require "shoulda/context/test_framework_detection"
+require "shoulda/context/version"
+require "shoulda/context/world"
 
-module ShouldaContextLoadable
-  def self.included(base)
-    base.class_eval do
-      include Shoulda::Context::Assertions
-      include Shoulda::Context::InstanceMethods
-    end
-    base.extend(Shoulda::Context::ClassMethods)
-  end
+if defined?(Rails)
+  require "shoulda/context/railtie"
+  require "shoulda/context/rails_test_unit_reporter_patch"
 end
 
-Shoulda::Context.test_framework_test_cases.each do |test_case|
-  test_case.class_eval { include ShouldaContextLoadable }
+Shoulda::Context.configure do |config|
+  config.include(Shoulda::Context::DSL)
 end

data/lib/shoulda/context/assertions.rb

@@ -3,7 +3,7 @@ module Shoulda # :nodoc:
     module Assertions
       # Asserts that two arrays contain the same elements, the same number of times.  Essentially ==, but unordered.
       #
-      #   assert_same_elements([:a, :b, :c], [:c, :a, :b]) => passes
+      #   assert_same_elements([:a, :b, :c], [:c, :a, :b]) => passes)
       def assert_same_elements(a1, a2, msg = nil)
         [:select, :inject, :size].each do |m|
           [a1, a2].each {|a| assert_respond_to(a, m, "Are you sure that #{a.inspect} is an array?  It doesn't respond to #{m}.") }
@@ -45,7 +45,8 @@ module Shoulda # :nodoc:
         end
       end
 
-      # Asserts that the given matcher returns true when +target+ is passed to #matches?
+      # Asserts that the given matcher returns true when +target+ is passed to
+      # #matches?
       def assert_accepts(matcher, target, options = {})
         if matcher.respond_to?(:in_context)
           matcher.in_context(self)
@@ -54,33 +55,35 @@ module Shoulda # :nodoc:
         if matcher.matches?(target)
           safe_assert_block { true }
           if options[:message]
-            message = matcher.respond_to?(:failure_message_for_should_not) ? matcher.failure_message_for_should_not : matcher.negative_failure_message
-            assert_match options[:message], message
+            assert_match options[:message], matcher.failure_message_when_negated
           end
         else
-          message = matcher.respond_to?(:failure_message_for_should) ? matcher.failure_message_for_should : matcher.failure_message
-          safe_assert_block(message) { false }
+          safe_assert_block(matcher.failure_message) { false }
         end
       end
 
-      # Asserts that the given matcher returns true when +target+ is passed to #does_not_match?
-      # or false when +target+ is passed to #matches? if #does_not_match? is not implemented
+      # Asserts that the given matcher returns true when +target+ is passed to
+      # #does_not_match? or false when +target+ is passed to #matches? if
+      # #does_not_match? is not implemented
       def assert_rejects(matcher, target, options = {})
         if matcher.respond_to?(:in_context)
           matcher.in_context(self)
         end
 
-        not_match = matcher.respond_to?(:does_not_match?) ? matcher.does_not_match?(target) : !matcher.matches?(target)
+        not_match =
+          if matcher.respond_to?(:does_not_match?)
+            matcher.does_not_match?(target)
+          else
+            !matcher.matches?(target)
+          end
 
         if not_match
           safe_assert_block { true }
           if options[:message]
-            message = matcher.respond_to?(:failure_message_for_should) ? matcher.failure_message_for_should : matcher.failure_message
-            assert_match options[:message], message
+            assert_match options[:message], matcher.failure_message
           end
         else
-          message = matcher.respond_to?(:failure_message_for_should_not) ? matcher.failure_message_for_should_not : matcher.negative_failure_message
-          safe_assert_block(message) { false }
+          safe_assert_block(matcher.failure_message_when_negated) { false }
         end
       end
 

data/lib/shoulda/context/configuration.rb

@@ -0,0 +1,19 @@
+module Shoulda
+  module Context
+    def self.configure
+      yield self
+    end
+
+    def self.include(mod)
+      test_framework_test_cases.each do |test_case|
+        test_case.class_eval { include mod }
+      end
+    end
+
+    def self.extend(mod)
+      test_framework_test_cases.each do |test_case|
+        test_case.extend(mod)
+      end
+    end
+  end
+end

data/lib/shoulda/context/context.rb

@@ -1,290 +1,6 @@
 module Shoulda
   module Context
-    class << self
-      def contexts # :nodoc:
-        @contexts ||= []
-      end
-      attr_writer :contexts
-
-      def current_context # :nodoc:
-        self.contexts.last
-      end
-
-      def add_context(context) # :nodoc:
-        self.contexts.push(context)
-      end
-
-      def remove_context # :nodoc:
-        self.contexts.pop
-      end
-    end
-
-    module ClassMethods
-      # == Should statements
-      #
-      # Should statements are just syntactic sugar over normal Test::Unit test
-      # methods.  A should block contains all the normal code and assertions
-      # you're used to seeing, with the added benefit that they can be wrapped
-      # inside context blocks (see below).
-      #
-      # === Example:
-      #
-      #  class UserTest < Test::Unit::TestCase
-      #
-      #    def setup
-      #      @user = User.new("John", "Doe")
-      #    end
-      #
-      #    should "return its full name"
-      #      assert_equal 'John Doe', @user.full_name
-      #    end
-      #
-      #  end
-      #
-      # ...will produce the following test:
-      # * <tt>"test: User should return its full name. "</tt>
-      #
-      # Note: The part before <tt>should</tt> in the test name is gleamed from the name of the Test::Unit class.
-      #
-      # Should statements can also take a Proc as a <tt>:before </tt>option.  This proc runs after any
-      # parent context's setups but before the current context's setup.
-      #
-      # === Example:
-      #
-      #  context "Some context" do
-      #    setup { puts("I run after the :before proc") }
-      #
-      #    should "run a :before proc", :before => lambda { puts("I run before the setup") }  do
-      #      assert true
-      #    end
-      #  end
-      #
-      # Should statements can also wrap matchers, making virtually any matcher
-      # usable in a macro style. The matcher's description is used to generate a
-      # test name and failure message, and the test will pass if the matcher
-      # matches the subject.
-      #
-      # === Example:
-      #
-      #   should validate_presence_of(:first_name).with_message(/gotta be there/)
-      #
-
-      def should(name_or_matcher, options = {}, &blk)
-        if Shoulda::Context.current_context
-          Shoulda::Context.current_context.should(name_or_matcher, options, &blk)
-        else
-          context_name = self.name.gsub(/Test/, "") if self.name
-          context = Shoulda::Context::Context.new(context_name, self) do
-            should(name_or_matcher, options, &blk)
-          end
-          context.build
-        end
-      end
-
-      # Allows negative tests using matchers. The matcher's description is used
-      # to generate a test name and negative failure message, and the test will
-      # pass unless the matcher matches the subject.
-      #
-      # === Example:
-      #
-      #   should_not set_the_flash
-      def should_not(matcher)
-        if Shoulda::Context.current_context
-          Shoulda::Context.current_context.should_not(matcher)
-        else
-          context_name = self.name.gsub(/Test/, "") if self.name
-          context = Shoulda::Context::Context.new(context_name, self) do
-            should_not(matcher)
-          end
-          context.build
-        end
-      end
-
-      # == Before statements
-      #
-      # Before statements are should statements that run before the current
-      # context's setup. These are especially useful when setting expectations.
-      #
-      # === Example:
-      #
-      #  class UserControllerTest < Test::Unit::TestCase
-      #    context "the index action" do
-      #      setup do
-      #        @users = [Factory(:user)]
-      #        User.stubs(:find).returns(@users)
-      #      end
-      #
-      #      context "on GET" do
-      #        setup { get :index }
-      #
-      #        should respond_with(:success)
-      #
-      #        # runs before "get :index"
-      #        before_should "find all users" do
-      #          User.expects(:find).with(:all).returns(@users)
-      #        end
-      #      end
-      #    end
-      #  end
-      def before_should(name, &blk)
-        should(name, :before => blk) { assert true }
-      end
-
-      # Just like should, but never runs, and instead prints an 'X' in the Test::Unit output.
-      def should_eventually(name, options = {}, &blk)
-        context_name = self.name.gsub(/Test/, "")
-        context = Shoulda::Context::Context.new(context_name, self) do
-          should_eventually(name, &blk)
-        end
-        context.build
-      end
-
-      # == Contexts
-      #
-      # A context block groups should statements under a common set of setup/teardown methods.
-      # Context blocks can be arbitrarily nested, and can do wonders for improving the maintainability
-      # and readability of your test code.
-      #
-      # A context block can contain setup, should, should_eventually, and teardown blocks.
-      #
-      #  class UserTest < Test::Unit::TestCase
-      #    context "A User instance" do
-      #      setup do
-      #        @user = User.find(:first)
-      #      end
-      #
-      #      should "return its full name"
-      #        assert_equal 'John Doe', @user.full_name
-      #      end
-      #    end
-      #  end
-      #
-      # This code will produce the method <tt>"test: A User instance should return its full name. "</tt>.
-      #
-      # Contexts may be nested.  Nested contexts run their setup blocks from out to in before each
-      # should statement.  They then run their teardown blocks from in to out after each should statement.
-      #
-      #  class UserTest < Test::Unit::TestCase
-      #    context "A User instance" do
-      #      setup do
-      #        @user = User.find(:first)
-      #      end
-      #
-      #      should "return its full name"
-      #        assert_equal 'John Doe', @user.full_name
-      #      end
-      #
-      #      context "with a profile" do
-      #        setup do
-      #          @user.profile = Profile.find(:first)
-      #        end
-      #
-      #        should "return true when sent :has_profile?"
-      #          assert @user.has_profile?
-      #        end
-      #      end
-      #    end
-      #  end
-      #
-      # This code will produce the following methods
-      # * <tt>"test: A User instance should return its full name. "</tt>
-      # * <tt>"test: A User instance with a profile should return true when sent :has_profile?. "</tt>
-      #
-      # <b>Just like should statements, a context block can exist next to normal <tt>def test_the_old_way; end</tt>
-      # tests</b>.  This means you do not have to fully commit to the context/should syntax in a test file.
-
-      def context(name, &blk)
-        if Shoulda::Context.current_context
-          Shoulda::Context.current_context.context(name, &blk)
-        else
-          context = Shoulda::Context::Context.new(name, self, &blk)
-          context.build
-        end
-      end
-
-      # Returns the class being tested, as determined by the test class name.
-      #
-      #   class UserTest; described_type; end
-      #   # => User
-      def described_type
-        @described_type ||= self.name.
-          gsub(/Test$/, '').
-          split('::').
-          inject(Object) do |parent, local_name|
-            parent.const_get(local_name, false)
-          end
-      end
-
-      # Sets the return value of the subject instance method:
-      #
-      #   class UserTest < Test::Unit::TestCase
-      #     subject { User.first }
-      #
-      #     # uses the existing user
-      #     should validate_uniqueness_of(:email)
-      #   end
-      def subject(&block)
-        @subject_block = block
-      end
-
-      def subject_block # :nodoc:
-        @subject_block ||= nil
-      end
-    end
-
-    module InstanceMethods
-      # Returns an instance of the class under test.
-      #
-      #   class UserTest
-      #     should "be a user" do
-      #       assert_kind_of User, subject # passes
-      #     end
-      #   end
-      #
-      # The subject can be explicitly set using the subject class method:
-      #
-      #   class UserTest
-      #     subject { User.first }
-      #     should "be an existing user" do
-      #       assert !subject.new_record? # uses the first user
-      #     end
-      #   end
-      #
-      # The subject is used by all macros that require an instance of the class
-      # being tested.
-      def subject
-        @shoulda_subject ||= construct_subject
-      end
-
-      def subject_block # :nodoc:
-        (@shoulda_context && @shoulda_context.subject_block) || self.class.subject_block
-      end
-
-      def get_instance_of(object_or_klass) # :nodoc:
-        if object_or_klass.is_a?(Class)
-          object_or_klass.new
-        else
-          object_or_klass
-        end
-      end
-
-      def instance_variable_name_for(klass) # :nodoc:
-        klass.to_s.split('::').last.underscore
-      end
-
-      private
-
-      def construct_subject
-        if subject_block
-          instance_eval(&subject_block)
-        else
-          get_instance_of(self.class.described_type)
-        end
-      end
-    end
-
     class Context # :nodoc:
-
       attr_accessor :name               # my name
       attr_accessor :parent             # may be another context, or the original test::unit class.
       attr_accessor :subcontexts        # array of contexts nested under myself
@@ -389,40 +105,49 @@ module Shoulda
       end
 
       def create_test_from_should_hash(should)
-        test_name = [test_name_prefix, full_name, "should", "#{should[:name]}. "].flatten.join(' ').to_sym
+        test_name = build_test_name_from(should)
 
-        if test_methods[test_unit_class][test_name.to_s] then
-          raise DuplicateTestError, "'#{test_name}' is defined more than once."
+        if test_methods[test_unit_class][test_name.to_s]
+          raise Shoulda::Context::DuplicateTestError.new(
+            "'#{test_name}' is defined more than once."
+          )
         end
 
         test_methods[test_unit_class][test_name.to_s] = true
         file, line_no = should[:block].source_location
-        context = self
+
+        # Ruby doesn't know that we are referring to this variable inside of the
+        # eval, so it will emit a warning that it's "assigned but unused".
+        # However, making a double assignment places `context` on the right hand
+        # side of the assignment, thereby putting it into use.
+        context = context = self
+
         test_unit_class.class_eval <<-end_eval, file, line_no
           define_method test_name do
-          @shoulda_context = context
-          begin
-            context.run_parent_setup_blocks(self)
-            if should[:before]
-              if self.respond_to?(:instance_exec)
-                self.instance_exec(&should[:before])
-              else
-                should[:before].bind(self).call
+            @shoulda_context = context
+            begin
+              context.run_parent_setup_blocks(self)
+              if should[:before]
+                instance_exec(&should[:before])
               end
+              context.run_current_setup_blocks(self)
+              instance_exec(&should[:block])
+            ensure
+              context.run_all_teardown_blocks(self)
             end
-            context.run_current_setup_blocks(self)
-            if self.respond_to?(:instance_exec)
-              self.instance_exec(&should[:block])
-            else
-              should[:block].bind(self).call
-            end
-          ensure
-            context.run_all_teardown_blocks(self)
-          end
           end
         end_eval
       end
 
+      def build_test_name_from(should)
+        [
+          test_name_prefix,
+          full_name,
+          "should",
+          "#{should[:name]}. "
+        ].flatten.join(' ').to_sym
+      end
+
       def run_all_setup_blocks(binding)
         run_parent_setup_blocks(binding)
         run_current_setup_blocks(binding)
@@ -473,7 +198,7 @@ module Shoulda
       end
 
       def test_name_prefix
-        if defined?(Minitest) || defined?(MiniTest)
+        if defined?(Minitest)
           'test_:'
         else
           'test:'
@@ -484,7 +209,8 @@ module Shoulda
         test_unit_class.send(method, *args, &blk)
       end
     end
+
+    class DuplicateTestError < RuntimeError; end
   end
 end
 
-class DuplicateTestError < RuntimeError; end