Remove some more stub files now that opal parses them correctly

opal · Nov 2, 2013 · a8d4093 · a8d4093
1 parent 3d40ed9
commit a8d4093
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 12 deletions.
diff --git a/README.md b/README.md
@@ -46,18 +46,6 @@ opal has to include all dependencies into build.
 
 * **line 171**: `&::Time.method(:now)` doesnt work so wrong method is set
 
-### rspec/core/example_group.rb
-
-* **line 547**: opal cannot use mutable strings (see opal/rspec/fixes.rb)
-
-* **line 564**: opal cannot use mutable strings (see opal/rspec/fixes.rb). Also, opal
-does not support 2 regexp special characters yet (`\A` and `\z`).
-
-### rspec/matchers/built_in/have.rb
-
-* **line 1**: this is an error in rspec. This autoload does not exist so we must
-stub the file.
-
 ## License
 
 (The MIT License)

diff --git a/Rakefile b/Rakefile
@@ -31,6 +31,9 @@ def build_rspec
     Opal::Processor.stub_file asset
   end
 
+  # bug in rspec? this autoload doesnt exist so we must stub it
+  Opal::Processor.stub_file 'rspec/matchers/built_in/have'
+
   Opal.process('rspec-builder')
 end
 

diff --git a/app/rspec/matchers/built_in/have.rb b/app/rspec/matchers/built_in/have.rb
diff --git a/opal/opal/rspec/fixes.rb b/opal/opal/rspec/fixes.rb
@@ -31,6 +31,7 @@ def prefix_and_expected(symbol)
 end
 
 module RSpec::ExampleGroups
+  # opal cannot use mutable strings AND opal doesnt support `\A` or `\z` anchors
   def self.base_name_for(group)
     return "Anonymous" if group.description.empty?
 
@@ -48,6 +49,7 @@ def self.base_name_for(group)
     name
   end
 
+  # opal cannot use mutable strings
   def self.disambiguate(name, const_scope)
     return name unless const_scope.const_defined?(name)