looksee 1.0.3-universal-java-1.6 → 2.0.0-universal-java-1.6

data/spec/editor_spec.rb

@@ -1,128 +1,110 @@
 require 'spec_helper'
 
 describe Looksee::Editor do
-  def editor_command(command)
-    Looksee::Editor.new(command).command_for('FILE', 'LINE')
-  end
+  describe "#command_for" do
+    def editor_command(command)
+      Looksee::Editor.new(command).command_for('FILE', 'LINE')
+    end
 
-  it "should infer the file and line arguments for 'vi'" do
-    editor_command('vi').should == ['vi', '+LINE', 'FILE']
-  end
+    it "should infer the file and line arguments for 'vi'" do
+      editor_command('vi').should == ['vi', '+LINE', 'FILE']
+    end
 
-  it "should infer the file and line arguments for 'vim'" do
-    editor_command('vim').should == ['vim', '+LINE', 'FILE']
-  end
+    it "should infer the file and line arguments for 'vim'" do
+      editor_command('vim').should == ['vim', '+LINE', 'FILE']
+    end
 
-  it "should infer the file and line arguments for 'gvim'" do
-    editor_command('gvim').should == ['gvim', '+LINE', 'FILE']
-  end
+    it "should infer the file and line arguments for 'gvim'" do
+      editor_command('gvim').should == ['gvim', '+LINE', 'FILE']
+    end
 
-  it "should infer the file and line arguments for 'emacs'" do
-    editor_command('emacs').should == ['emacs', '+LINE', 'FILE']
-  end
+    it "should infer the file and line arguments for 'emacs'" do
+      editor_command('emacs').should == ['emacs', '+LINE', 'FILE']
+    end
 
-  it "should infer the file and line arguments for 'xemacs'" do
-    editor_command('xemacs').should == ['xemacs', '+LINE', 'FILE']
-  end
+    it "should infer the file and line arguments for 'xemacs'" do
+      editor_command('xemacs').should == ['xemacs', '+LINE', 'FILE']
+    end
 
-  it "should infer the file and line arguments for 'aquamacs'" do
-    editor_command('aquamacs').should == ['aquamacs', '+LINE', 'FILE']
-  end
+    it "should infer the file and line arguments for 'aquamacs'" do
+      editor_command('aquamacs').should == ['aquamacs', '+LINE', 'FILE']
+    end
 
-  it "should infer the file and line arguments for 'pico'" do
-    editor_command('pico').should == ['pico', '+LINE', 'FILE']
-  end
+    it "should infer the file and line arguments for 'pico'" do
+      editor_command('pico').should == ['pico', '+LINE', 'FILE']
+    end
 
-  it "should infer the file and line arguments for 'nano'" do
-    editor_command('nano').should == ['nano', '+LINE', 'FILE']
-  end
+    it "should infer the file and line arguments for 'nano'" do
+      editor_command('nano').should == ['nano', '+LINE', 'FILE']
+    end
 
-  it "should infer the file and line arguments for 'mate'" do
-    editor_command('mate').should == ['mate', '-lLINE', 'FILE']
-  end
+    it "should infer the file and line arguments for 'mate'" do
+      editor_command('mate').should == ['mate', '-lLINE', 'FILE']
+    end
 
-  it "should support escaped '%'-signs" do
-    editor_command('%% %f %l').should == ['%', 'FILE', 'LINE']
-  end
+    it "should support escaped '%'-signs" do
+      editor_command('%% %f %l').should == ['%', 'FILE', 'LINE']
+    end
 
-  it "should not infer file and line arguments for unknown editors" do
-    editor_command('wtfbbq').should == ['wtfbbq']
+    it "should not infer file and line arguments for unknown editors" do
+      editor_command('wtfbbq').should == ['wtfbbq']
+    end
   end
 
   describe "#edit" do
-    TMP = "#{ROOT}/spec/tmp"
+    let(:tmp) { "#{ROOT}/spec/tmp" }
+    let(:editor_path) { "#{tmp}/edit" }
+    let(:editor_output) { "#{tmp}/edit.out" }
+    let(:editor) { Looksee::Editor.new("#{editor_path} %f %l") }
+    let(:editor_invocation) { File.exist?(editor_output) ? File.read(editor_output) : nil }
+    let(:source_location) { ["#{tmp}/c.rb", 2] }
+    let(:object) { C.new }
 
     before do
-      FileUtils.mkdir_p TMP
-      make_editor "#{TMP}/edit"
-      @editor = Looksee::Editor.new("#{TMP}/edit %f %l")
+      FileUtils.mkdir_p tmp
+      set_up_editor
+
+      file, line = *source_location
+      FileUtils.touch file
+      Object.const_set(:C, Class.new { def f; end })
+      Looksee.adapter.set_methods(C, [:f], [], [], [])
+      Looksee.adapter.set_source_location(C, :f, source_location)
+      Looksee.adapter.ancestors[object] = [C, Object]
     end
 
     after do
-      FileUtils.rm_rf TMP
+      Object.send(:remove_const, :C)
+      FileUtils.rm_rf tmp
     end
 
-    def make_editor(path)
-      open(path, 'w') { |f| f.puts <<-EOS.demargin }
+    def set_up_editor
+      open(editor_path, 'w') { |f| f.puts <<-EOS.demargin }
         |#!/bin/sh
-        |echo $# $1 $2 > "#{TMP}/editor.out"
+        |echo $# $1 $2 > "#{editor_output}"
       EOS
-      File.chmod 0755, path
+      File.chmod 0755, editor_path
     end
 
-    def with_source_file
-      path = "#{TMP}/c.rb"
-      open(path, 'w') { |f| f.puts <<-EOS.demargin }
-        |class C
-        |  def f
-        |  end
-        |end
-      EOS
-      begin
-        load path
-        Looksee.adapter.set_methods(C, [:f], [], [], [])
-        yield path
-      ensure
-        Object.send(:remove_const, :C)
-      end
+    it "should run the editor on the method's source location if available" do
+      editor.edit(object, :f)
+      editor_invocation.should == "2 #{source_location.join(' ')}\n"
     end
 
-    it "should run the editor on the method's source location if available" do
-      with_source_file do |path|
-        c = C.new
-        Looksee.adapter.ancestors[c] = [C, Object]
-        Looksee.adapter.set_source_location(C, :f, [path, 2])
-        @editor.edit(c, :f)
-        File.read("#{TMP}/editor.out").should == "2 #{path} 2\n"
-      end
-    end
-
-    it "should not run the editor if the source file does not exist" do
-      with_source_file do |path|
-        FileUtils.rm_f path
-        @editor.edit(C.new, :f)
-        File.should_not exist("#{TMP}/editor.out")
-      end
-    end
-
-    it "should not run the editor for methods defined with eval where no source file specified" do
-      eval <<-EOS.demargin
-        |class ::C
-        |  def f
-        |  end
-        |end
-      EOS
-      begin
-        @editor.edit(C.new, :f)
-        File.should_not exist("#{TMP}/editor.out")
-      ensure
-        Object.send(:remove_const, :C)
-      end
+    it "should raise NoMethodError and not run the editor if the method does not exist" do
+      expect { editor.edit(object, :x) }.to raise_error(Looksee::NoMethodError)
+      editor_invocation.should be_nil
+    end
+
+    it "should raise NoSourceFileError and not run the editor if the source file does not exist" do
+      FileUtils.rm_f source_location.first
+      expect { editor.edit(object, :f) }.to raise_error(Looksee::NoSourceFileError)
+      editor_invocation.should be_nil
     end
 
-    it "should not run the editor for primitives" do
-      @editor.edit('', :size)
-      File.should_not exist("#{TMP}/editor.out")
+    it "should raise NoSourceLocationError and not run the editor if no source location is available" do
+      Looksee.adapter.set_source_location(C, :f, nil)
+      expect { editor.edit(object, :f) }.to raise_error(Looksee::NoSourceLocationError)
+      editor_invocation.should be_nil
     end
   end
 end

data/spec/spec_helper.rb

@@ -1,4 +1,5 @@
-ENV['LOOKSEE_METHODS'] = nil
+$:.unshift File.expand_path('..', File.dirname(__FILE__))
+ENV['LOOKSEE_METHOD'] = nil
 
 require 'rspec'
 require 'looksee'
@@ -7,7 +8,7 @@ require 'rbconfig'
 require 'set'
 require 'fileutils'
 
-ROOT = File.dirname(File.dirname(__FILE__))
+ROOT = File.expand_path('..', File.dirname(__FILE__))
 
 Dir['spec/support/*.rb'].each do |path|
   require path
@@ -18,8 +19,3 @@ NATIVE_ADAPTER = Looksee.adapter
 RSpec.configure do |config|
   config.before { Looksee.adapter = TestAdapter.new }
 end
-
-if Looksee.ruby_engine == 'jruby'
-  require 'jruby'
-  JRuby.objectspace = true
-end

data/spec/support/temporary_classes.rb

@@ -42,29 +42,6 @@ module TemporaryClasses
     mod
   end
 
-  #
-  # Remove all methods defined exactly on the given module.
-  #
-  # As Ruby's reflection on singleton classes of classes isn't quite
-  # adequate, you need to provide a :class_singleton option when such
-  # a class is given.
-  #
-  def remove_methods(mod, opts={})
-    names = all_instance_methods(mod)
-
-    # all_instance_methods can't get just the methods on a class
-    # singleton class.  Filter out superclass methods here.
-    if opts[:class_singleton]
-      klass = ObjectSpace.each_object(mod){|klass| break klass}
-      names -= all_instance_methods(klass.superclass.singleton_class)
-    end
-
-    names.sort_by{|name| name.in?([:remove_method, :send]) ? 1 : 0}.flatten
-    names.each do |name|
-      mod.send :remove_method, name
-    end
-  end
-
   #
   # Replace the methods of the given module with those named.
   #
@@ -78,8 +55,7 @@ module TemporaryClasses
   #
   #   replace_methods MyClass, :public => [:a, :b]
   #
-  def replace_methods(mod, options={})
-    remove_methods(mod, options)
+  def add_methods(mod, options={})
     mod.module_eval do
       [:public, :protected, :private].each do |visibility|
         Array(options[visibility]).each do |name|

data/spec/wirble_compatibility_spec.rb

@@ -12,10 +12,9 @@ describe Looksee::WirbleCompatibility do
         |c.ls
       EOS
       code = code.chomp.gsub(/\n/, ';') # only print value of last line
-      irb = File.join Looksee::Config::CONFIG['bindir'], 'irb'
-      lib_dir = File.expand_path('lib')
+      lib = File.expand_path('lib')
       # irb hangs when using readline without a tty
-      output = IO.popen("bundle exec #{irb} -f --noreadline --noprompt --noverbose -I#{lib_dir} 2>&1", 'r+') do |io|
+      output = IO.popen("bundle exec irb -f --noreadline --noprompt --noverbose -I#{lib}", 'r+') do |io|
         io.puts code
         io.flush
         io.close_write

metadata

@@ -1,152 +1,115 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: looksee
-version: !ruby/object:Gem::Version 
-  prerelease: 
-  version: 1.0.3
+version: !ruby/object:Gem::Version
+  version: 2.0.0
 platform: universal-java-1.6
-authors: 
-  - George Ogata
-autorequire: 
+authors:
+- George Ogata
+autorequire:
 bindir: bin
 cert_chain: []
-
-date: 2011-09-05 00:00:00 -04:00
-default_executable: 
-dependencies: 
-  - !ruby/object:Gem::Dependency 
-    name: ritual
-    prerelease: false
-    requirement: &id001 !ruby/object:Gem::Requirement 
-      none: false
-      requirements: 
-        - - "="
-          - !ruby/object:Gem::Version 
-            version: 0.3.0
-    type: :development
-    version_requirements: *id001
-  - !ruby/object:Gem::Dependency 
-    name: rspec
-    prerelease: false
-    requirement: &id002 !ruby/object:Gem::Requirement 
-      none: false
-      requirements: 
-        - - "="
-          - !ruby/object:Gem::Version 
-            version: 2.5.0
-    type: :development
-    version_requirements: *id002
-  - !ruby/object:Gem::Dependency 
-    name: wirble
-    prerelease: false
-    requirement: &id003 !ruby/object:Gem::Requirement 
-      none: false
-      requirements: 
-        - - ">="
-          - !ruby/object:Gem::Version 
-            version: "0"
-    type: :development
-    version_requirements: *id003
-description: 
-email: 
-  - george.ogata@gmail.com
+date: 2014-01-01 00:00:00.000000000 Z
+dependencies: []
+description:
+email:
+- george.ogata@gmail.com
 executables: []
-
 extensions: []
-
-extra_rdoc_files: 
-  - CHANGELOG
-  - LICENSE
-  - README.markdown
-files: 
-  - lib/looksee.rb
-  - lib/looksee/adapter.rb
-  - lib/looksee/clean.rb
-  - lib/looksee/columnizer.rb
-  - lib/looksee/core_ext.rb
-  - lib/looksee/editor.rb
-  - lib/looksee/help.rb
-  - lib/looksee/inspector.rb
-  - lib/looksee/JRuby.jar
-  - lib/looksee/lookup_path.rb
-  - lib/looksee/shortcuts.rb
-  - lib/looksee/version.rb
-  - lib/looksee/wirble_compatibility.rb
-  - lib/looksee/adapter/base.rb
-  - lib/looksee/adapter/rubinius.rb
-  - ext/mri/mri.c
-  - ext/rbx/rbx.c
-  - ext/mri/env-1.8.h
-  - ext/mri/eval_c-1.8.h
-  - ext/mri/node-1.9.h
-  - ext/mri/1.9.2/debug.h
-  - ext/mri/1.9.2/id.h
-  - ext/mri/1.9.2/method.h
-  - ext/mri/1.9.2/node.h
-  - ext/mri/1.9.2/thread_pthread.h
-  - ext/mri/1.9.2/vm_core.h
-  - ext/mri/1.9.2/vm_opts.h
-  - ext/mri/1.9.3/atomic.h
-  - ext/mri/1.9.3/debug.h
-  - ext/mri/1.9.3/id.h
-  - ext/mri/1.9.3/internal.h
-  - ext/mri/1.9.3/method.h
-  - ext/mri/1.9.3/node.h
-  - ext/mri/1.9.3/thread_pthread.h
-  - ext/mri/1.9.3/vm_core.h
-  - ext/mri/1.9.3/vm_opts.h
-  - ext/extconf.rb
-  - CHANGELOG
-  - LICENSE
-  - Rakefile
-  - README.markdown
-  - spec/adapter_spec.rb
-  - spec/columnizer_spec.rb
-  - spec/core_ext_spec.rb
-  - spec/editor_spec.rb
-  - spec/inspector_spec.rb
-  - spec/lookup_path_spec.rb
-  - spec/spec_helper.rb
-  - spec/wirble_compatibility_spec.rb
-  - spec/support/core_ext.rb
-  - spec/support/temporary_classes.rb
-  - spec/support/test_adapter.rb
-has_rdoc: true
+extra_rdoc_files:
+- CHANGELOG
+- LICENSE
+- README.markdown
+files:
+- CHANGELOG
+- LICENSE
+- README.markdown
+- Rakefile
+- ext/extconf.rb
+- ext/mri/1.9.2/debug.h
+- ext/mri/1.9.2/id.h
+- ext/mri/1.9.2/method.h
+- ext/mri/1.9.2/node.h
+- ext/mri/1.9.2/thread_pthread.h
+- ext/mri/1.9.2/vm_core.h
+- ext/mri/1.9.2/vm_opts.h
+- ext/mri/1.9.3/atomic.h
+- ext/mri/1.9.3/debug.h
+- ext/mri/1.9.3/id.h
+- ext/mri/1.9.3/internal.h
+- ext/mri/1.9.3/internal_falcon.h
+- ext/mri/1.9.3/method.h
+- ext/mri/1.9.3/node.h
+- ext/mri/1.9.3/thread_pthread.h
+- ext/mri/1.9.3/vm_core.h
+- ext/mri/1.9.3/vm_opts.h
+- ext/mri/2.0.0/internal.h
+- ext/mri/2.0.0/method.h
+- ext/mri/2.1.0/internal.h
+- ext/mri/2.1.0/method.h
+- ext/mri/env-1.8.h
+- ext/mri/eval_c-1.8.h
+- ext/mri/mri.c
+- ext/mri/node-1.9.h
+- ext/rbx/rbx.c
+- lib/looksee.rb
+- lib/looksee/JRuby.jar
+- lib/looksee/adapter.rb
+- lib/looksee/adapter/base.rb
+- lib/looksee/adapter/rubinius.rb
+- lib/looksee/clean.rb
+- lib/looksee/columnizer.rb
+- lib/looksee/core_ext.rb
+- lib/looksee/editor.rb
+- lib/looksee/help.rb
+- lib/looksee/inspector.rb
+- lib/looksee/lookup_path.rb
+- lib/looksee/shortcuts.rb
+- lib/looksee/version.rb
+- lib/looksee/wirble_compatibility.rb
+- spec/adapter_spec.rb
+- spec/columnizer_spec.rb
+- spec/core_ext_spec.rb
+- spec/editor_spec.rb
+- spec/inspector_spec.rb
+- spec/lookup_path_spec.rb
+- spec/spec_helper.rb
+- spec/support/core_ext.rb
+- spec/support/temporary_classes.rb
+- spec/support/test_adapter.rb
+- spec/wirble_compatibility_spec.rb
 homepage: http://github.com/oggy/looksee
-licenses: []
-
-post_install_message: 
+licenses:
+- MIT
+metadata: {}
+post_install_message:
 rdoc_options: []
-
-require_paths: 
-  - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
-  none: false
-  requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
-  none: false
-  requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        version: "0"
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - '>='
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
-rubyforge_project: 
-rubygems_version: 1.5.1
-signing_key: 
+rubyforge_project:
+rubygems_version: 2.2.0
+signing_key:
 specification_version: 3
 summary: Supercharged method introspection in IRB.
-test_files: 
-  - spec/adapter_spec.rb
-  - spec/columnizer_spec.rb
-  - spec/core_ext_spec.rb
-  - spec/editor_spec.rb
-  - spec/inspector_spec.rb
-  - spec/lookup_path_spec.rb
-  - spec/spec_helper.rb
-  - spec/wirble_compatibility_spec.rb
-  - spec/support/core_ext.rb
-  - spec/support/temporary_classes.rb
-  - spec/support/test_adapter.rb
+test_files:
+- spec/adapter_spec.rb
+- spec/columnizer_spec.rb
+- spec/core_ext_spec.rb
+- spec/editor_spec.rb
+- spec/inspector_spec.rb
+- spec/lookup_path_spec.rb
+- spec/spec_helper.rb
+- spec/wirble_compatibility_spec.rb
+- spec/support/core_ext.rb
+- spec/support/temporary_classes.rb
+- spec/support/test_adapter.rb