override_kernel 0.0.1 → 1.0.0

data/README

@@ -23,11 +23,11 @@ Currently, only the exit function is dealt with in such a manner.
 Simply use the following call to start brushing the exit function off like it's
 a harmless pest:
 
- start_override_exit
+ OverrideKernel.start_override_exit
 
 This line will gladly put it back in your way:
 
- stop_override_exit
+ OverrideKernel.stop_override_exit
 
 You can check the state of the application by
 checking OverrideKernel.override_exit.

data/lib/override_kernel/exit.rb

@@ -38,8 +38,8 @@ module Kernel
   # (A call to exit will cause testing to stop otherwise).
   alias :old_exit :exit
   def exit(status = true)
-    old_exit(status) unless OverrideKernel.override_exit
     OverrideKernel::AppState.state = :dead
+    old_exit(status) unless OverrideKernel.override_exit
     return status
   end
 end

data/lib/override_kernel/override_kernel.rb

@@ -35,26 +35,12 @@
 
 module OverrideKernel
 
-  # Contains a list of methods that will be dynamically handled by the module.
-  # The following methods will be created:
-  #
-  #  :start_#{method_name} #=> @method_name = true
-  #  :stop_#{method_name}  #=> @method_name = false
-  #  :#{method_name}       #=> Returns the value of @method_name
-  METHOD_LIST = [
-    'override_exit',
-  ]
-
   # This constant is used to indicate whether the application would have
   # been halted by Kernel functions.
   AppState = StateManager.new(:alive, [:alive, :dead])
 
-  ############################################################################
-  module_function
-  ############################################################################
-
-  def method_missing(method, *args, &block)
-    if METHOD_LIST.include?(method.to_s.sub(/^(start|stop)_/, ''))
+  def self.method_missing(method, *args, &block)
+    if method_list.include?(method.to_s.sub(/^(start|stop)_/, '').to_sym)
       var = '@' + method.to_s.sub(/^(start|stop)_/, '')
       case method.to_s
         when /^start/
@@ -62,6 +48,7 @@ module OverrideKernel
         when /^stop/
           send :instance_variable_set, var.to_sym, false
         when var[1..-1]
+          return nil unless instance_variables.include?(var.to_sym)
           send :instance_variable_get, var.to_sym
       end
     else
@@ -69,11 +56,28 @@ module OverrideKernel
     end
   end
 
-  def respond_to_missing?(method, include_private)
-    if METHOD_LIST.include?(method.to_s.sub(/^(start|stop)_/, ''))
+  def self.respond_to_missing?(method, include_private)
+    if method_list.include?(method.to_s.sub(/^(start|stop)_/, '').to_sym)
       return true
     else
       super
     end
   end
+
+  class << self
+    private
+    # Returns a list of methods that will be dynamically handled by the module.
+    # The following methods will be created:
+    #
+    #  :start_#{method_name} #=> @method_name = true
+    #  :stop_#{method_name}  #=> @method_name = false
+    #  :#{method_name}       #=> Returns the value of @method_name
+    # ==== Output
+    # [Array] An array of base method names.
+    def method_list
+      [
+        :override_exit,
+      ]
+    end
+  end
 end

data/override_kernel.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name = 'override_kernel'
-  s.version = '0.0.1'
+  s.version = '1.0.0'
 
   s.summary = 'Overrides specific Kernel functions.'
   s.description = %Q{

data/test/exit_test.rb

@@ -46,6 +46,7 @@ class ExitTest < Test::Unit::TestCase
   def test_without_override
     override_exit(false)
     assert_raise(SystemExit) { exit }
+    assert_dead
   end
 
   def test_basic_exit_call

data/test/lib/test_class.rb

@@ -0,0 +1,3 @@
+class TestClass
+  include OverrideKernel
+end

data/test/lib/test_module.rb

@@ -0,0 +1,3 @@
+module TestModule
+  include OverrideKernel
+end

data/test/override_kernel_test.rb

@@ -33,21 +33,110 @@
 require_relative 'require'
 
 class OverrideKernelTest < Test::Unit::TestCase
-  METHOD_LIST = [
-    :override_exit,
-  ]
+  ITEMS = {
+    :method_list => [:override_exit,],
+    :constants => {
+      :AppState => StateManager.new(:alive, [:alive, :dead]),
+    },
+  }
+
+  def setup
+    @module = OverrideKernel
+    @module.instance_eval do
+      instance_variables.each do |var|
+        remove_instance_variable var
+      end
+    end
+    super
+  end
+
+  def test_constants
+    ITEMS[:constants].each_key do |constant|
+      assert_equal strip_id(ITEMS[:constants][constant]),
+        strip_id(@module.const_get(constant))
+    end
+  end
+
+  def test_method_list
+    assert_equal ITEMS[:method_list], @module.send(:method_list)
+  end
 
   def test_all_methods
-    METHOD_LIST.each do |method|
+    ITEMS[:method_list].each do |method|
+      assert_equal nil, @module.send(method)
+
       {
         "start_#{method}".to_sym => true,
         "stop_#{method}".to_sym => false,
       }.each do |method_name, value|
-        assert_respond_to OverrideKernel, method_name
-        assert_nothing_raised { OverrideKernel.send method_name }
-        assert_respond_to OverrideKernel, method
-        assert_equal value, OverrideKernel.send(method)
+        assert_respond_to @module, method_name
+        assert_nothing_raised { @module.send method_name }
+        assert_respond_to @module, method
+        assert_equal value, @module.send(method)
+      end
+    end
+
+    assert_no_response @module, :method_list, false
+  end
+
+  def test_module_include
+    ITEMS[:method_list].each do |method|
+      assert_no_response TestModule, method
+
+      [
+        "start_#{method}".to_sym,
+        "stop_#{method}".to_sym,
+      ].each do |meth|
+        assert_no_response TestModule, meth
       end
     end
+
+    ITEMS[:constants].each_key do |constant|
+      assert_equal strip_id(ITEMS[:constants][constant]),
+        strip_id(TestModule.const_get(constant))
+    end
+
+    assert_no_response TestModule, :method_list
+  end
+
+  def test_class_include
+    obj = TestClass.new
+    ITEMS[:method_list].each do |method|
+      assert_no_response TestClass, method
+      assert_no_response obj, method
+
+      [
+        "start_#{method}".to_sym,
+        "stop_#{method}".to_sym,
+      ].each do |meth|
+        assert_no_response TestClass, meth
+        assert_no_response obj, meth
+      end
+    end
+
+    ITEMS[:constants].each_key do |constant|
+      assert_equal strip_id(ITEMS[:constants][constant]),
+        strip_id(TestClass.const_get(constant))
+    end
+
+    assert_no_response TestClass, :method_list
+    assert_no_response obj, :method_list
+  end
+
+  ############################################################################
+  private
+  ############################################################################
+
+  def strip_id(object)
+    return object unless object.respond_to?(:pretty_inspect) &&
+      object.pretty_inspect.match(/^#</)
+    object.pretty_inspect.sub(/:\w+? /, ' ')
+  end
+
+  def assert_no_response(obj, method, include_private = true, msg = nil)
+    msg = message(msg) {
+      "Did not expect #{mu_pp(obj)} (#{obj.class}) to respond to ##{method}"
+    }
+    assert !obj.respond_to?(method, include_private), msg
   end
 end

data/test/require.rb

@@ -3,3 +3,6 @@ Bundler.require :test
 require 'test/unit'
 
 require_relative '../lib/override_kernel'
+
+require_relative File.join('lib', 'test_module')
+require_relative File.join('lib', 'test_class')

metadata

@@ -3,10 +3,10 @@ name: override_kernel
 version: !ruby/object:Gem::Version 
   prerelease: false
   segments: 
+  - 1
   - 0
   - 0
-  - 1
-  version: 0.0.1
+  version: 1.0.0
 platform: ruby
 authors: 
 - Travis Herrick
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-08-10 00:00:00 -04:00
+date: 2011-08-15 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -68,6 +68,8 @@ files:
 - rakefile
 - README
 - test/override_kernel_test.rb
+- test/lib/test_module.rb
+- test/lib/test_class.rb
 - test/require.rb
 - test/exit_test.rb
 has_rdoc: true
@@ -84,7 +86,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: -3169159202290470423
+      hash: -1613111938212231117
       segments: 
       - 0
       version: "0"
@@ -105,5 +107,7 @@ specification_version: 3
 summary: Overrides specific Kernel functions.
 test_files: 
 - test/override_kernel_test.rb
+- test/lib/test_module.rb
+- test/lib/test_class.rb
 - test/require.rb
 - test/exit_test.rb