Merge pull request #86 from tobiashm/test-require

Fixing tests
openid · Mar 5, 2015 · 624f282 · 624f282
2 parents 56a5290 + b6402eb
commit 624f282
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 42 deletions.
diff --git a/test/test_association.rb b/test/test_association.rb
@@ -1,5 +1,6 @@
 require "minitest/autorun"
 require "openid/association"
+require "openid/protocolerror"
 
 module OpenID
   class AssociationTestCase < Minitest::Test

diff --git a/test/test_associationmanager.rb b/test/test_associationmanager.rb
@@ -1,10 +1,12 @@
 require "minitest/autorun"
+require "testutil"
 require "openid/consumer/associationmanager"
 require "openid/association"
 require "openid/dh"
 require "openid/util"
 require "openid/cryptutil"
 require "openid/message"
+require "openid/protocolerror"
 require "openid/store/memory"
 require "util"
 require "time"

diff --git a/test/test_discover.rb b/test/test_discover.rb
@@ -746,13 +746,9 @@ def test_discover_function
       # XXX these were all different tests in python, but they're
       # combined here so I only have to use with_method_overridden
       # once.
-      discoverXRI = Proc.new { |identifier|
-        return 'XRI'
-      }
+      discoverXRI = Proc.new { |identifier| 'XRI' }
 
-      discoverURI = Proc.new { |identifier|
-        return 'URI'
-      }
+      discoverURI = Proc.new { |identifier| 'URI' }
 
       OpenID.extend(OverrideMethodMixin)
 

diff --git a/test/test_discovery_manager.rb b/test/test_discovery_manager.rb
@@ -1,5 +1,5 @@
 require 'minitest/autorun'
-require 'openid/consumer/discovery_manager'
+require 'openid/consumer'
 require 'testutil'
 
 module OpenID

diff --git a/test/test_extras.rb b/test/test_extras.rb
diff --git a/test/test_pape.rb b/test/test_pape.rb
@@ -2,7 +2,7 @@
 require 'openid/extensions/pape'
 require 'openid/message'
 require 'openid/server'
-require 'openid/consumer/responses'
+require 'openid/consumer'
 
 module OpenID
   module PAPETest