auth0 5.4.0 → 5.7.0

data/spec/lib/auth0/api/v2/clients_spec.rb

@@ -12,23 +12,23 @@ describe Auth0::Api::V2::Clients do
 
     it 'is expected to send get request to the Clients endpoint' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/clients',
+        '/api/v2/clients', {
         fields: nil,
         include_fields: nil,
         page: nil,
         per_page: nil
-      )
+      })
       expect { @instance.clients }.not_to raise_error
     end
 
     it 'is expected to send get request to the Clients endpoint with a name parameter' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/clients',
+        '/api/v2/clients', {
         include_fields: true,
         fields: 'name',
         page: nil,
         per_page: nil
-      )
+      })
       expect {
         @instance.clients(fields: 'name', include_fields: true)
       }.not_to raise_error
@@ -36,12 +36,12 @@ describe Auth0::Api::V2::Clients do
 
     it 'is expected to send get request to Clients endpoint using an array of fields' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/clients',
+        '/api/v2/clients', {
         include_fields: true,
         fields: 'name,app_type',
         page: nil,
         per_page: nil
-      )
+      })
       expect {
         @instance.clients(fields: ['name','app_type'], include_fields: true)
       }.not_to raise_error
@@ -49,12 +49,12 @@ describe Auth0::Api::V2::Clients do
 
     it 'is expected to send get request to Clients endpoint with pagination' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/clients',
+        '/api/v2/clients', {
         page: 1,
         per_page: 10,
         fields: nil,
         include_fields: nil
-      )
+      })
       expect {
         @instance.clients(page: 1, per_page: 10)
       }.not_to raise_error
@@ -64,11 +64,11 @@ describe Auth0::Api::V2::Clients do
   context '.client' do
     it { expect(@instance).to respond_to(:client) }
     it 'is expected to send get request to /api/v2/clients/1' do
-      expect(@instance).to receive(:get).with('/api/v2/clients/1', fields: nil, include_fields: nil)
+      expect(@instance).to receive(:get).with('/api/v2/clients/1', { fields: nil, include_fields: nil })
       expect { @instance.client(1) }.not_to raise_error
     end
     it 'is expected to send get request to /api/v2/clients?fields=name&include_fields=true' do
-      expect(@instance).to receive(:get).with('/api/v2/clients/1', include_fields: true, fields: [:name])
+      expect(@instance).to receive(:get).with('/api/v2/clients/1', { include_fields: true, fields: [:name] })
       expect { @instance.client(1, include_fields: true, fields: [:name]) }.not_to raise_error
     end
     it { expect { @instance.client('') }.to raise_error 'Must specify a client id' }
@@ -77,7 +77,7 @@ describe Auth0::Api::V2::Clients do
   context '.create_client' do
     it { expect(@instance).to respond_to(:create_client) }
     it 'is expected to send post to /api/v2/clients' do
-      expect(@instance).to receive(:post).with('/api/v2/clients', name: 'name', opt: 'opt')
+      expect(@instance).to receive(:post).with('/api/v2/clients', { name: 'name', opt: 'opt' })
       expect { @instance.create_client('name', name: '/name1', opt: 'opt') }.not_to raise_error
     end
     it { expect { @instance.create_client('') }.to raise_error 'Must specify a valid client name' }
@@ -95,7 +95,7 @@ describe Auth0::Api::V2::Clients do
   context '.patch_client' do
     it { expect(@instance).to respond_to(:patch_client) }
     it 'is expected to send patch to /api/v2/clients/1' do
-      expect(@instance).to receive(:patch).with('/api/v2/clients/1', fields: 'fields')
+      expect(@instance).to receive(:patch).with('/api/v2/clients/1', { fields: 'fields' })
       expect { @instance.patch_client('1', fields: 'fields') }.not_to raise_error
     end
     it { expect { @instance.patch_client('', nil) }.to raise_error 'Must specify a client id' }

data/spec/lib/auth0/api/v2/connections_spec.rb

@@ -12,27 +12,27 @@ describe Auth0::Api::V2::Connections do
 
     it 'is expected to call /api/v2/connections' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/connections',
+        '/api/v2/connections', {
         strategy: nil,
         name: nil,
         fields: nil,
         include_fields: nil,
         page: nil,
         per_page: nil
-      )
+      })
       expect { @instance.connections }.not_to raise_error
     end
 
     it 'is expected to send get request to /api/v2/connections?fields=name' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/connections',
+        '/api/v2/connections', {
         include_fields: true,
         fields: 'name',
         strategy: nil,
         name: nil,
         page: nil,
         per_page: nil
-      )
+      })
       expect {
         @instance.connections(fields: 'name', include_fields: true)
       }.not_to raise_error
@@ -40,14 +40,14 @@ describe Auth0::Api::V2::Connections do
 
     it 'is expected to convert fields param from Array to string' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/connections',
+        '/api/v2/connections', {
         include_fields: true,
         fields: 'name,strategy',
         strategy: nil,
         name: nil,
         page: nil,
         per_page: nil
-      )
+      })
       expect {
         @instance.connections(fields: ['name','strategy'], include_fields: true)
       }.not_to raise_error
@@ -55,14 +55,14 @@ describe Auth0::Api::V2::Connections do
 
     it 'is expected to add pagination' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/connections',
+        '/api/v2/connections', {
         page: 1,
         per_page: 10,
         strategy: nil,
         name: nil,
         fields: nil,
         include_fields: nil
-      )
+      })
       expect {
         @instance.connections(page: 1, per_page: 10)
       }.not_to raise_error
@@ -88,7 +88,7 @@ describe Auth0::Api::V2::Connections do
   context '.connection' do
     it { expect(@instance).to respond_to(:connection) }
     it 'is expected to call get request to /api/v2/connection/connectionId' do
-      expect(@instance).to receive(:get).with('/api/v2/connections/connectionId', fields: nil, include_fields: true)
+      expect(@instance).to receive(:get).with('/api/v2/connections/connectionId', { fields: nil, include_fields: true })
       expect { @instance.connection('connectionId') }.not_to raise_error
     end
     it 'is expected raise an error when calling with empty id' do
@@ -113,7 +113,7 @@ describe Auth0::Api::V2::Connections do
   context '.delete_connection_user' do
     it { expect(@instance).to respond_to(:delete_connection_user) }
     it 'is expected to call delete to /api/v2/connections/connectionId/users' do
-      expect(@instance).to receive(:delete).with('/api/v2/connections/connectionId/users', email: 'email@test.com')
+      expect(@instance).to receive(:delete).with('/api/v2/connections/connectionId/users', { email: 'email@test.com' })
       @instance.delete_connection_user('connectionId', 'email@test.com')
     end
 

data/spec/lib/auth0/api/v2/device_credentials_spec.rb

@@ -13,13 +13,13 @@ describe Auth0::Api::V2::DeviceCredentials do
     it { expect(@instance).to respond_to(:list_device_credentials) }
     it 'is expected to send get request to /api/v2/device-credentials' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/device-credentials',
+        '/api/v2/device-credentials', {
         fields: nil,
         include_fields: nil,
         user_id: nil,
         client_id: client_id,
         type: nil
-      )
+      })
       expect { @instance.device_credentials(client_id) }.not_to raise_error
     end
     it 'is expect to raise an error when type is not one of \'public_key\', \'refresh_token\', \'rotating_refresh_token\'' do
@@ -34,13 +34,13 @@ describe Auth0::Api::V2::DeviceCredentials do
     it { expect(@instance).to respond_to(:create_device_public_key) }
     it 'is expected to send post to /api/v2/device-credentials' do
       expect(@instance).to receive(:post).with(
-        '/api/v2/device-credentials',
+        '/api/v2/device-credentials', {
         device_name: 'device_name',
         value: 'value',
         device_id: 'device_id',
         client_id: 'client_id',
         type: 'public_key'
-      )
+      })
       expect { @instance.create_device_credential('device_name', 'value', 'device_id', 'client_id') }
         .not_to raise_error
     end

data/spec/lib/auth0/api/v2/emails_spec.rb

@@ -9,11 +9,11 @@ describe Auth0::Api::V2::Emails do
   context '.get_email' do
     it { expect(@instance).to respond_to(:get_provider) }
     it 'expect client to send get to /api/v2/emails/provider with fields' do
-      expect(@instance).to receive(:get).with('/api/v2/emails/provider', fields: 'some', include_fields: true)
+      expect(@instance).to receive(:get).with('/api/v2/emails/provider', { fields: 'some', include_fields: true })
       expect { @instance.get_provider(fields: 'some', include_fields: true) }.not_to raise_error
     end
     it 'expect client to send get to /api/v2/emails/provider with empty fields' do
-      expect(@instance).to receive(:get).with('/api/v2/emails/provider', fields: nil, include_fields: nil)
+      expect(@instance).to receive(:get).with('/api/v2/emails/provider', { fields: nil, include_fields: nil })
       expect { @instance.get_provider }.not_to raise_error
     end
   end

data/spec/lib/auth0/api/v2/grants_spec.rb

@@ -0,0 +1,91 @@
+require 'spec_helper'
+describe Auth0::Api::V2::Grants do
+  before :all do
+    dummy_instance = DummyClass.new
+    dummy_instance.extend(Auth0::Api::V2::Grants)
+    @instance = dummy_instance
+  end
+
+  context '.grants' do
+    it { expect(@instance).to respond_to(:grants) }
+    it { expect(@instance).to respond_to(:get_all_grants) }
+
+    it 'is expected to get /api/v2/grants/' do
+      expect(@instance).to receive(:get).with(
+        '/api/v2/grants', {
+        client_id: nil,
+        user_id: nil,
+        audience: nil,
+        page: nil,
+        per_page: nil,
+        include_totals: nil
+      })
+      expect { @instance.grants }.not_to raise_error
+    end
+
+    it 'is expected to send get /api/v2/grants/ with client_id and audience' do
+      audience = "https://samples.auth0.com/api/v2/"
+
+      expect(@instance).to receive(:get).with(
+        '/api/v2/grants', {
+        client_id: '1',
+        user_id: nil,
+        audience: audience,
+        page: nil,
+        per_page: nil,
+        include_totals: nil
+      })
+      expect { @instance.grants(client_id: '1', audience: audience) }.not_to raise_error
+    end
+
+    it 'is expected to send get /api/v2/grants/ with client_id and user_id' do
+      expect(@instance).to receive(:get).with(
+        '/api/v2/grants', {
+        client_id: '1',
+        user_id: '1',
+        audience: nil,
+        page: nil,
+        per_page: nil,
+        include_totals: nil
+      })
+      expect { @instance.grants(client_id: '1', user_id: '1') }.not_to raise_error
+    end
+
+    it 'is expected to send get /api/v2/grants/ with pagination' do
+      expect(@instance).to receive(:get).with(
+        '/api/v2/grants', {
+        client_id: nil,
+        user_id: nil,
+        audience: nil,
+        page: 1,
+        per_page: 2,
+        include_totals: nil
+      })
+      expect { @instance.grants(page: 1, per_page: 2) }.not_to raise_error
+    end
+
+    it 'is expected to send get /api/v2/grants/ with include_totals' do
+      expect(@instance).to receive(:get).with(
+        '/api/v2/grants', {
+        client_id: nil,
+        user_id: nil,
+        audience: nil,
+        page: 1,
+        per_page: 2,
+        include_totals: true
+      })
+      expect { @instance.grants(page: 1, per_page: 2, include_totals: true) }.not_to raise_error
+    end
+  end
+
+  context '.delete_grant' do
+    it { expect(@instance).to respond_to(:delete_grant) }
+    it 'is expected to send delete to /api/v2/grants/1?user_id=1' do
+      expect(@instance).to receive(:delete).with('/api/v2/grants/1', user_id: '1')
+      expect { @instance.delete_grant('1', '1') }.not_to raise_error
+    end
+    it { expect { @instance.delete_grant('', '') }.to raise_error 'Must specify a grant id as id' }
+    it { expect { @instance.delete_grant('', '1') }.to raise_error 'Must specify a grant id as id' }
+    it { expect { @instance.delete_grant('1', '') }.to raise_error 'Must specify a user id' }
+  end
+end

data/spec/lib/auth0/api/v2/jobs_spec.rb

@@ -25,24 +25,24 @@ describe Auth0::Api::V2::Jobs do
     it { expect(@instance).to respond_to(:import_users) }
     it 'expect client to send post to /api/v2/jobs/users-imports' do
       expect(@instance).to receive(:post_file).with(
-        '/api/v2/jobs/users-imports',
+        '/api/v2/jobs/users-imports', {
         users: 'file',
         connection_id: 'connnection_id',
         upsert: false,
         external_id: nil,
         send_completion_email: true
-      )
+      })
       expect { @instance.import_users('file', 'connnection_id') }.not_to raise_error
     end
     it 'expect client to send post to /api/v2/jobs/users-imports with options' do
       expect(@instance).to receive(:post_file).with(
-        '/api/v2/jobs/users-imports',
+        '/api/v2/jobs/users-imports', {
         users: 'file',
         connection_id: 'connnection_id',
         upsert: true,
         external_id: 'external_1',
         send_completion_email: false
-      )
+      })
       expect do
         @instance.import_users(
           'file',
@@ -61,12 +61,12 @@ describe Auth0::Api::V2::Jobs do
     it { expect { @instance.export_users }.not_to raise_error }
     it 'sends post to /api/v2/jobs/users-exports with correct params' do
       expect(@instance).to receive(:post).with(
-        '/api/v2/jobs/users-exports',
+        '/api/v2/jobs/users-exports', {
         fields: [{ name: 'author' }],
         connection_id: 'test-connection',
         format: 'csv',
         limit: 10
-      )
+      })
       @instance.export_users(
         fields: ['author'],
         connection_id: 'test-connection',
@@ -83,9 +83,9 @@ describe Auth0::Api::V2::Jobs do
 
     it 'should post to the jobs email verification endpoint' do
       expect(@instance).to receive(:post).with(
-        '/api/v2/jobs/verification-email',
+        '/api/v2/jobs/verification-email', {
         user_id: 'test_user_id'
-      )
+      })
       expect do
         @instance.send_verification_email('test_user_id')
       end.not_to raise_error
@@ -93,21 +93,22 @@ describe Auth0::Api::V2::Jobs do
 
     it 'should post to the jobs email verification endpoint with a client_id' do
       expect(@instance).to receive(:post).with(
-        '/api/v2/jobs/verification-email',
+        '/api/v2/jobs/verification-email', {
         user_id: 'test_user_id',
         client_id: 'test_client_id'
-      )
+      })
       expect do
         @instance.send_verification_email('test_user_id', 'test_client_id')
       end.not_to raise_error
     end
 
     it 'expect client to accept hash identity' do
-      expect(@instance).to receive(:post).with('/api/v2/jobs/verification-email', user_id: 'user_id',
-                                                                                  identity: {
-                                                                                    provider: "auth0",
-                                                                                    user_id: "user_id"
-                                                                                  })
+      expect(@instance).to receive(:post).with('/api/v2/jobs/verification-email', {
+        user_id: 'user_id',
+        identity: {
+        provider: "auth0",
+        user_id: "user_id"
+      }})
       expect {
         @instance.send_verification_email('user_id', identity: { provider: "auth0", user_id: "user_id"}) 
       }.not_to raise_error
@@ -120,10 +121,10 @@ describe Auth0::Api::V2::Jobs do
     end
 
     it 'expect client to accept organization_id' do
-      expect(@instance).to receive(:post).with('/api/v2/jobs/verification-email',
+      expect(@instance).to receive(:post).with('/api/v2/jobs/verification-email', {
         user_id: 'user_id',
         organization_id: 'org_id'
-      )
+      })
 
       expect {
         @instance.send_verification_email('user_id', organization_id: 'org_id')

data/spec/lib/auth0/api/v2/log_streams_spec.rb

@@ -33,7 +33,7 @@ describe Auth0::Api::V2::LogStreams do
     it { expect(@instance).to respond_to(:create_log_stream) }
     it 'is expected to call post /api/v2/log-streams' do
       expect(@instance).to receive(:post).with(
-        '/api/v2/log-streams',
+        '/api/v2/log-streams', {
         name: 'test',
         type: 'https',
         sink: {
@@ -42,7 +42,7 @@ describe Auth0::Api::V2::LogStreams do
           httpContentFormat: "JSONLINES",
           httpAuthorization: "string"
         }
-      )
+      })
 
       @instance.create_log_stream('test', 'https',
         httpEndpoint: "https://mycompany.com",
@@ -75,7 +75,7 @@ describe Auth0::Api::V2::LogStreams do
   context '.patch_log_stream' do
     it { expect(@instance).to respond_to(:patch_log_stream) }
     it 'is expected to send patch to /api/v2/log_streams/test' do
-      expect(@instance).to receive(:patch).with('/api/v2/log-streams/test', status: 'paused')
+      expect(@instance).to receive(:patch).with('/api/v2/log-streams/test', { status: 'paused' })
       expect { @instance.patch_log_stream('test', 'paused') }.not_to raise_error
     end
     it { expect { @instance.patch_log_stream('', nil) }.to raise_error 'Must specify a log stream id' }

data/spec/lib/auth0/api/v2/logs_spec.rb

@@ -11,7 +11,7 @@ describe Auth0::Api::V2::Logs do
     it { expect(@instance).to respond_to(:get_logs) }
     it 'is expected to call /api/v2/logs' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/logs',
+        '/api/v2/logs', {
         q: nil,
         page: nil,
         per_page: nil,
@@ -21,7 +21,7 @@ describe Auth0::Api::V2::Logs do
         include_totals: nil,
         from: nil,
         take: nil
-      )
+      })
       expect { @instance.logs }.not_to raise_error
     end
     it 'is expect to rise an error when take is higher than 100' do

data/spec/lib/auth0/api/v2/organizations_spec.rb

@@ -17,25 +17,25 @@ describe Auth0::Api::V2::Organizations do
 
     it 'is expected to get /api/v2/organizations' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/organizations',
+        '/api/v2/organizations', {
         per_page: nil,
         page: nil,
         from: nil,
         take: nil,
         include_totals: nil
-      )
+      })
       expect { @instance.organizations }.not_to raise_error
     end
 
     it 'is expected to get /api/v2/organizations with custom parameters' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/organizations',
+        '/api/v2/organizations', {
         per_page: 10,
         page: 1,
         from: 'org_id',
         take: 50,
         include_totals: true
-      )
+      })
       expect do
         @instance.organizations(
           per_page: 10,
@@ -72,9 +72,9 @@ describe Auth0::Api::V2::Organizations do
 
     it 'is expected to post to /api/v2/organizations' do
       expect(@instance).to receive(:post).with(
-        '/api/v2/organizations',
+        '/api/v2/organizations', {
           name: 'test_org'
-        )
+        })
       expect do
         @instance.create_organization(
           name: 'test_org'
@@ -130,9 +130,9 @@ describe Auth0::Api::V2::Organizations do
 
     it 'is expected to patch /api/v2/organizations/org_id' do
       expect(@instance).to receive(:patch).with(
-        '/api/v2/organizations/org_id',
+        '/api/v2/organizations/org_id', {
         name: 'name'
-      )
+      })
       @instance.patch_organization(
         'org_id',
         name: 'name'
@@ -446,13 +446,13 @@ describe Auth0::Api::V2::Organizations do
 
     it 'is expected to get members for an org' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/organizations/org_id/members',
+        '/api/v2/organizations/org_id/members', {
         per_page: nil,
         page: nil,
         from: nil,
         take: nil,
         include_totals: nil
-      )
+      })
       expect do
         @instance.get_organizations_members('org_id')
       end.not_to raise_error
@@ -460,13 +460,13 @@ describe Auth0::Api::V2::Organizations do
 
     it 'is expected to get /api/v2/organizations with custom parameters' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/organizations/org_id/members',
+        '/api/v2/organizations/org_id/members', {
         per_page: 10,
         page: 1,
         from: 'org_id',
         take: 50,
         include_totals: true
-      )
+      })
       expect do
         @instance.get_organizations_members(
           'org_id',
@@ -595,7 +595,7 @@ describe Auth0::Api::V2::Organizations do
     end
 
     it 'is expected to delete /api/v2/organizations/org_id/members/user_id/roles' do
-      expect(@instance).to receive(:delete).with(
+      expect(@instance).to receive(:delete_with_body).with(
         '/api/v2/organizations/org_id/members/user_id/roles', {
           roles: ['123', '456']
         }

data/spec/lib/auth0/api/v2/resource_servers_spec.rb

@@ -12,10 +12,10 @@ describe Auth0::Api::V2::ResourceServers do
     it { expect(@instance).to respond_to(:get_resource_servers) }
     it 'is expected to call get /api/v2/resource-servers' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/resource-servers',
+        '/api/v2/resource-servers', {
         page: nil,
         per_page: nil,
-      )
+      })
       expect { @instance.resource_servers }.not_to raise_error
     end
   end
@@ -35,14 +35,14 @@ describe Auth0::Api::V2::ResourceServers do
     it { expect(@instance).to respond_to(:create_resource_server) }
     it 'is expected to call post /api/v2/resource-servers' do
       expect(@instance).to receive(:post).with(
-        '/api/v2/resource-servers',
+        '/api/v2/resource-servers', {
         identifier: 'test',
         name: 'name',
         signing_alg: 'signing_alg',
         signing_secret: 'signing_secret',
         token_lifetime: 'token_lifetime',
         scopes: 'scopes'
-      )
+      })
 
       @instance.create_resource_server('test', name: 'name',
                                                signing_alg: 'signing_alg',
@@ -77,7 +77,7 @@ describe Auth0::Api::V2::ResourceServers do
   context '.patch_resource_server' do
     it { expect(@instance).to respond_to(:patch_resource_server) }
     it 'is expected to send patch to /api/v2/resource_servers/1' do
-      expect(@instance).to receive(:patch).with('/api/v2/resource-servers/1', fields: 'fields')
+      expect(@instance).to receive(:patch).with('/api/v2/resource-servers/1', { fields: 'fields' })
       expect { @instance.patch_resource_server('1', fields: 'fields') }.not_to raise_error
     end
     it { expect { @instance.patch_resource_server('', nil) }.to raise_error Auth0::InvalidParameter }