auth0 5.6.0 → 5.8.0

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

@@ -24,8 +24,9 @@ describe Auth0::Api::V2::Branding do
     it { expect(@instance).to respond_to(:patch_branding) }
     it 'is expected to call post /api/v2/branding' do
       expect(@instance).to receive(:patch).with(
-        '/api/v2/branding',
-        template: 'Test'
+        '/api/v2/branding', {
+          template: 'Test'
+        }
       )
       expect { @instance.patch_branding({ template: 'Test' }) }.not_to raise_error
     end
@@ -52,7 +53,7 @@ describe Auth0::Api::V2::Branding do
     it { expect(@instance).to respond_to(:put_branding_templates_for_universal_login) }
     it 'is expected to call put /api/v2/branding/templates/universal-login' do
       expect(@instance).to receive(:put).with(
-        '/api/v2/branding/templates/universal-login', template: 'Template'
+        '/api/v2/branding/templates/universal-login', { template: 'Template' }
       )
       expect do
         @instance.put_branding_templates_for_universal_login(template: 'Template')

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

@@ -12,12 +12,12 @@ describe Auth0::Api::V2::ClientGrants do
 
     it 'is expected to get /api/v2/client-grants/' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/client-grants',
+        '/api/v2/client-grants', {
         client_id: nil,
         audience: nil,
         page: nil,
         per_page: nil
-      )
+      })
       expect { @instance.client_grants }.not_to raise_error
     end
 
@@ -25,23 +25,23 @@ describe Auth0::Api::V2::ClientGrants do
       audience = "https://samples.auth0.com/api/v2/"
 
       expect(@instance).to receive(:get).with(
-        '/api/v2/client-grants',
+        '/api/v2/client-grants', {
         client_id: '1',
         audience: audience,
         page: nil,
         per_page: nil
-      )
-      expect { @instance.client_grants(client_id: '1', audience: audience) }.not_to raise_error
+      })
+      expect { @instance.client_grants(client_id: '1', audience: audience ) }.not_to raise_error
     end
 
     it 'is expected to send get /api/v2/client-grants/ with pagination' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/client-grants',
+        '/api/v2/client-grants', {
         client_id: nil,
         audience: nil,
         page: 1,
         per_page: 2
-      )
+      })
       expect { @instance.client_grants(page: 1, per_page: 2) }.not_to raise_error
     end
   end
@@ -49,7 +49,7 @@ describe Auth0::Api::V2::ClientGrants do
   context '.create_client_grant' do
     it { expect(@instance).to respond_to(:create_client_grant) }
     it 'is expected to send post to /api/v2/client-grants' do
-      expect(@instance).to receive(:post).with('/api/v2/client-grants', opt: 'test body')
+      expect(@instance).to receive(:post).with('/api/v2/client-grants', { opt: 'test body' })
       expect { @instance.create_client_grant(opt: 'test body') }.not_to raise_error
     end
   end

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

@@ -1,29 +1,58 @@
 require 'spec_helper'
 describe Auth0::Api::V2::DeviceCredentials do
-  attr_reader :client_id
-
   before :all do
     dummy_instance = DummyClass.new
     dummy_instance.extend(Auth0::Api::V2::DeviceCredentials)
     @instance = dummy_instance
-    @client_id = Faker::Lorem.word
   end
   context '.device_credentials' do
     it { expect(@instance).to respond_to(:device_credentials) }
     it { expect(@instance).to respond_to(:list_device_credentials) }
-    it 'is expected to send get request to /api/v2/device-credentials' do
+    it 'is expected to send get request with no parameters to /api/v2/device-credentials' do
+      expect(@instance).to receive(:get).with(
+        '/api/v2/device-credentials', {
+        fields: nil,
+        include_fields: nil,
+        user_id: nil,
+        client_id: nil,
+        type: nil,
+        page: nil,
+        per_page: nil,
+        include_totals: nil
+      })
+      expect { @instance.device_credentials }.not_to raise_error
+    end
+    it 'is expected to send get request with client_id to /api/v2/device-credentials' do
+      client_id = Faker::Lorem.word
       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
-      )
+        type: nil,
+        page: nil,
+        per_page: nil,
+        include_totals: nil
+      })
       expect { @instance.device_credentials(client_id) }.not_to raise_error
     end
+    it 'is expected to send get request with options to /api/v2/device-credentials' do
+      expect(@instance).to receive(:get).with(
+        '/api/v2/device-credentials', {
+        fields: 'name',
+        include_fields: true,
+        user_id: '1',
+        client_id: 'client_id',
+        type: 'rotating_refresh_token',
+        page: 1,
+        per_page: 10,
+        include_totals: true
+      })
+      expect { @instance.device_credentials('client_id', fields: 'name', include_fields: true, user_id: '1', type: 'rotating_refresh_token', page: 1, per_page: 10, include_totals: true) }.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
-      expect { @instance.device_credentials(client_id, type: 'invalid_type') }.to raise_error(
+      expect { @instance.device_credentials('client_id', type: 'invalid_type') }.to raise_error(
         'Type must be one of \'public_key\', \'refresh_token\', \'rotating_refresh_token\''
       )
     end
@@ -34,13 +63,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

@@ -12,14 +12,14 @@ describe Auth0::Api::V2::Grants do
 
     it 'is expected to get /api/v2/grants/' do
       expect(@instance).to receive(:get).with(
-        '/api/v2/grants',
+        '/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
 
@@ -27,53 +27,53 @@ describe Auth0::Api::V2::Grants do
       audience = "https://samples.auth0.com/api/v2/"
 
       expect(@instance).to receive(:get).with(
-        '/api/v2/grants',
+        '/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',
+        '/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',
+        '/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',
+        '/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
@@ -81,7 +81,7 @@ describe Auth0::Api::V2::Grants do
   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).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' }

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