[SDK-3869] Support client credentials in management client (#437)

* feat: support for create credentials endpoint

* feat: add support for get client credentials

* feat: support getting single  client credential in api v2

* feat: add support for deleting a client credential

* chore: fill out doc comments for credentials

Author: stevehobbsdev

lib/auth0/api/v2/clients.rb

@@ -73,12 +73,54 @@ def patch_client(client_id, options)
           patch(path, options)
         end
 
+        # Creates credentials for a client
+        # @param client_id [string] The Id of the client to update
+        # @param options [hash] The payload to send to the endpoint
+        def create_client_credentials(client_id, options)
+          raise Auth0::MissingClientId, 'Must specify a client id' if client_id.to_s.empty?
+          raise Auth0::MissingParameter, 'Must specify a valid body' if options.to_s.empty?
+          post(client_credentials_path(client_id), options)
+        end
+
+        # Gets the credentials for a client
+        # @param client_id [string] The Id of the client
+        # @return [hash] The client credentials 
+        def client_credentials(client_id)
+          raise Auth0::MissingClientId, 'Must specify a client id' if client_id.to_s.empty?
+          get(client_credentials_path(client_id))
+        end
+        alias get_client_credentials client_credentials
+
+        # Gets a client credential by ID
+        # @param client_id [string] The Id of the client
+        # @param credential_id [string] The Id of the credential to retrieve
+        # @return [hash] The credential
+        def client_credential(client_id, credential_id)
+          raise Auth0::MissingClientId, 'Must specify a client id' if client_id.to_s.empty?
+          raise Auth0::MissingParameter, 'Must specify a credential id' if credential_id.to_s.empty?
+          get("#{client_credentials_path(client_id)}/#{credential_id}")
+        end
+        alias get_client_credential client_credential
+
+        # Deletes a credential from the specified client
+        # @param client_id [string] The Id of the client
+        # @param credential_id [string] The Id of the credential to delete
+        def delete_client_credential(client_id, credential_id)
+          raise Auth0::MissingClientId, 'Must specify a client id' if client_id.to_s.empty?
+          raise Auth0::MissingParameter, 'Must specify a credential id' if credential_id.to_s.empty?
+          delete("#{client_credentials_path(client_id)}/#{credential_id}")
+        end
+
         private
 
         # Clients API path
         def clients_path
           @clients_path ||= '/api/v2/clients'
         end
+
+        def client_credentials_path(client_id)
+          "#{clients_path}/#{client_id}/credentials"
+        end
       end
     end
   end

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

@@ -101,4 +101,55 @@
     it { expect { @instance.patch_client('', nil) }.to raise_error 'Must specify a client id' }
     it { expect { @instance.patch_client('some', nil) }.to raise_error 'Must specify a valid body' }
   end
+
+  context '.create_client_credentials' do
+    it { expect(@instance).to respond_to(:create_client_credentials) }
+
+    it 'is expected to send post to /api/v2/clients/1/credentials' do
+      payload = { credential_type: 'public_key', name: 'my credentials', pem: '' }
+
+      expect(@instance).to receive(:post).with('/api/v2/clients/1/credentials', payload)
+      expect { @instance.create_client_credentials('1', payload) }.not_to raise_error
+    end
+
+    it { expect { @instance.create_client_credentials('', nil) }.to raise_error 'Must specify a client id' }
+    it { expect { @instance.create_client_credentials('1', nil) }.to raise_error 'Must specify a valid body' }
+  end
+
+  context '.client_credentials' do
+    it { expect(@instance).to respond_to(:client_credentials) }
+    it { expect(@instance).to respond_to(:get_client_credentials) }
+
+    it 'is expected to send get to /api/v2/clients/1/credentials' do
+      expect(@instance).to receive(:get).with('/api/v2/clients/1/credentials')
+      expect { @instance.client_credentials('1') }.not_to raise_error 
+    end
+
+    it { expect { @instance.client_credentials('') }.to raise_error 'Must specify a client id' }
+  end
+
+  context '.client_credential' do
+    it { expect(@instance).to respond_to(:client_credential) }
+    it { expect(@instance).to respond_to(:get_client_credential) }
+
+    it 'is expected to send get to /api/v2/clients/1/credentials/2' do
+      expect(@instance).to receive(:get).with('/api/v2/clients/1/credentials/2')
+      expect { @instance.client_credential('1', '2') }.not_to raise_error
+    end
+
+    it { expect { @instance.client_credential('', '') }.to raise_error 'Must specify a client id' }
+    it { expect { @instance.client_credential('1', '') }.to raise_error 'Must specify a credential id' }
+  end
+
+  context '.delete_client_credential', focus: true do
+    it { expect(@instance).to respond_to(:delete_client_credential) }
+    
+    it 'is expected to delete /api/v2/clients/1/credentials/2' do
+      expect(@instance).to receive(:delete).with('/api/v2/clients/1/credentials/2')
+      expect { @instance.delete_client_credential('1', '2') }.not_to raise_error
+    end
+
+    it { expect { @instance.delete_client_credential('', '') }.to raise_error 'Must specify a client id' }
+    it { expect { @instance.delete_client_credential('1', '') }.to raise_error 'Must specify a credential id' }
+  end
 end