search for index.js #2

[simi]

followup to #17

I try to add specs soon.

/cc @cowboyd @ftiasch

[cowboyd]

Ok, this looks good. Can we add some specs for the new behavior?

[simi]

sure

[Wardrop]

Can this be merged already?

[cowboyd]

@Wardrop As you can see from the last comment this PR is waiting on specs for the added behavior so the answer to your question is no. However, if you would like to take a crack at writing those specs, I would be happy to assist in any way I can.

[ojab]

At least this should be added to correctly require files in modules that was required by dirname:

diff --git a/lib/commonjs/module.rb b/lib/commonjs/module.rb
index 27b318d..4529ee5 100644
--- a/lib/commonjs/module.rb
+++ b/lib/commonjs/module.rb
@@ -5,6 +5,9 @@ module CommonJS
     attr_accessor :exports

     def initialize(id, env, append = false)
+      id.sub!(%r{/index.js$}, '') if append
       @id = id
       @env = env
       @exports = env.new_object