consul/grecia
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove unneeded provider key when mocking OmniAuth

Browse Source 
OmniAuth already adds the provider to the mock because it's passed as
the first parameter of the `add_mock` method.
This commit is contained in:
Javi Martín
2022-10-10 16:06:22 +02:00
parent 12e41ae0f3
commit a86792c9fa
1 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
spec/system/users_auth_spec.rb
View File

@@ -223,11 +223,10 @@ describe "Users" do
end end
context "Twitter" do context "Twitter" do
let(:twitter_hash) { { provider: "twitter", uid: "12345", info: { name: "manuela" }} } let(:twitter_hash) { { uid: "12345", info: { name: "manuela" }} }
let(:twitter_hash_with_email) { { provider: "twitter", uid: "12345", info: { name: "manuela", email: "manuelacarmena@example.com" }} } let(:twitter_hash_with_email) { { uid: "12345", info: { name: "manuela", email: "manuelacarmena@example.com" }} }
let(:twitter_hash_with_verified_email) do let(:twitter_hash_with_verified_email) do
{ {
provider: "twitter",
uid: "12345", uid: "12345",
info: { info: {
name: "manuela", name: "manuela",
@@ -482,11 +481,13 @@ describe "Users" do
context "Wordpress" do context "Wordpress" do
let(:wordpress_hash) do let(:wordpress_hash) do
{ provider: "wordpress", {
uid: "12345", uid: "12345",
info: { info: {
name: "manuela", name: "manuela",
email: "manuelacarmena@example.com" }} email: "manuelacarmena@example.com"
}
}
end end
before { Setting["feature.wordpress_login"] = true } before { Setting["feature.wordpress_login"] = true }