From a86792c9fa41d48f7ebe038be422f479e3bbc623 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Javi=20Mart=C3=ADn?= Date: Mon, 10 Oct 2022 16:06:22 +0200 Subject: [PATCH] Remove unneeded provider key when mocking OmniAuth OmniAuth already adds the provider to the mock because it's passed as the first parameter of the `add_mock` method. --- spec/system/users_auth_spec.rb | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/spec/system/users_auth_spec.rb b/spec/system/users_auth_spec.rb index c41c23e77..3180c6e57 100644 --- a/spec/system/users_auth_spec.rb +++ b/spec/system/users_auth_spec.rb @@ -223,11 +223,10 @@ describe "Users" do end context "Twitter" do - let(:twitter_hash) { { provider: "twitter", uid: "12345", info: { name: "manuela" }} } - let(:twitter_hash_with_email) { { provider: "twitter", uid: "12345", info: { name: "manuela", email: "manuelacarmena@example.com" }} } + let(:twitter_hash) { { uid: "12345", info: { name: "manuela" }} } + let(:twitter_hash_with_email) { { uid: "12345", info: { name: "manuela", email: "manuelacarmena@example.com" }} } let(:twitter_hash_with_verified_email) do { - provider: "twitter", uid: "12345", info: { name: "manuela", @@ -482,11 +481,13 @@ describe "Users" do context "Wordpress" do let(:wordpress_hash) do - { provider: "wordpress", + { uid: "12345", info: { name: "manuela", - email: "manuelacarmena@example.com" }} + email: "manuelacarmena@example.com" + } + } end before { Setting["feature.wordpress_login"] = true }