diff options
| author | jessib <jessib@riseup.net> | 2013-09-02 09:59:49 -0700 |
|---|---|---|
| committer | jessib <jessib@riseup.net> | 2013-09-02 09:59:49 -0700 |
| commit | 6110093e939db07fd27fac7c28ddcd09a49e70ed (patch) | |
| tree | 4393c3e8fcb2969b0bd21800bff7dd5a1fc9c3d7 /users/test/unit/warden_strategy_secure_remote_password_test.rb | |
| parent | 060e06daa065f02b811dfe12850b101a62c12c8d (diff) | |
| parent | 77af7ac953fb22c181056e8c40c8749d12a63922 (diff) | |
Merge pull request #74 from azul/refactor/finding-users
there's no need for User#find_by_param. clean it up
Diffstat (limited to 'users/test/unit/warden_strategy_secure_remote_password_test.rb')
| -rw-r--r-- | users/test/unit/warden_strategy_secure_remote_password_test.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/users/test/unit/warden_strategy_secure_remote_password_test.rb b/users/test/unit/warden_strategy_secure_remote_password_test.rb index 319809a..e6fcfbe 100644 --- a/users/test/unit/warden_strategy_secure_remote_password_test.rb +++ b/users/test/unit/warden_strategy_secure_remote_password_test.rb @@ -21,7 +21,7 @@ class WardenStrategySecureRemotePasswordTest < ActiveSupport::TestCase returns(@server_handshake) @server_handshake.expects(:to_json). returns({'B' => @server_hex, 'salt' => @salt}.to_json) - User.expects(:find_by_param).with(@user.login).returns(@user) + User.expects(:find).with(@user.login).returns(@user) assert_equal @server_handshake, session[:handshake] assert_response :success assert_json_response :B => @server_hex, :salt => @salt @@ -29,7 +29,7 @@ class WardenStrategySecureRemotePasswordTest < ActiveSupport::TestCase test "should report user not found" do unknown = "login_that_does_not_exist" - User.expects(:find_by_param).with(unknown).raises(RECORD_NOT_FOUND) + User.expects(:find).with(unknown).raises(RECORD_NOT_FOUND) post :create, :login => unknown assert_response :success assert_json_error "login" => ["unknown user"] |