Merge pull request #1813 from opal/elia/builder-relative-path-fix

Use relative module names in Builder#build
opal · May 3, 2018 · e59c4d0 · e59c4d0
2 parents f11374d + 6fdea2d
commit e59c4d0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/lib/opal/builder.rb b/lib/opal/builder.rb
@@ -80,7 +80,6 @@ def self.build(*args, &block)
 
     def build(path, options = {})
       source = read(path)
-      path = path_reader.expand(path)
       build_str(source, path, options)
     end
 

diff --git a/spec/lib/builder_spec.rb b/spec/lib/builder_spec.rb
@@ -118,4 +118,10 @@
       end
     end
   end
+
+  describe ':requirable' do
+    it 'it uses relative paths as module names' do
+      expect(builder.build('stringio', requirable: true).to_s).to include(%{Opal.modules["stringio"]})
+    end
+  end
 end