From 21a0f0c554a046b60b5ddad70fefc8ab8735b1e2 Mon Sep 17 00:00:00 2001
From: Robert Peralta <rapito@users.noreply.github.com>
Date: Tue, 12 Sep 2023 06:21:09 -0400
Subject: [PATCH] Make trigger_id and action_name optional for actions index
 endpoint (#478)

Co-authored-by: Steve Hobbs <steve.hobbs@hey.com>
Co-authored-by: Steve Hobbs <steve.hobbs@okta.com>
---
 Gemfile.lock                          |  3 +++
 lib/auth0/api/v2/actions.rb           | 13 ++++++------
 spec/lib/auth0/api/v2/actions_spec.rb | 30 ++++++++++++++++++---------
 3 files changed, 29 insertions(+), 17 deletions(-)

diff --git a/Gemfile.lock b/Gemfile.lock
index e10855c8..7a69e287 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -225,7 +225,10 @@ GEM
     zeitwerk (2.6.11)
 
 PLATFORMS
+  aarch64-linux
   arm64-darwin-21
+  arm64-darwin-22
+  x86_64-darwin-21
   x86_64-linux
 
 DEPENDENCIES
diff --git a/lib/auth0/api/v2/actions.rb b/lib/auth0/api/v2/actions.rb
index cb4c3951..6d1af603 100644
--- a/lib/auth0/api/v2/actions.rb
+++ b/lib/auth0/api/v2/actions.rb
@@ -16,18 +16,16 @@ module Actions
         # @param page [integer] The page number. Zero based.
         # @param installed [boolean] When true, return only installed actions. When false, return only custom actions. Returns all actions by default.
         # @return [json] Actions and pagination info
-        def actions(trigger_id, action_name, deployed: nil, per_page: nil, page: nil, installed: nil)
-          raise Auth0::MissingTriggerId, 'Must supply a valid trigger_id' if trigger_id.to_s.empty?
-          raise Auth0::MissingActionName, 'Must supply a valid action_name' if action_name.to_s.empty?
-
+        def actions(trigger_id = nil, action_name = nil, deployed: nil, per_page: nil, page: nil, installed: nil)
           request_params = {
-            trigger_id: trigger_id,
-            action_name: action_name,
+            triggerId: trigger_id,
+            actionName: action_name,
             deployed: deployed,
             per_page: per_page,
             page: page,
             installed: installed
           }
+          
           path = "#{actions_path}/actions"
           get(path, request_params)
         end
@@ -38,7 +36,8 @@ def actions(trigger_id, action_name, deployed: nil, per_page: nil, page: nil, in
         # @param body [hash] See https://auth0.com/docs/api/management/v2/#!/actions/post_action for available options
         # @return [json] Returns the created action.
         def create_action(body = {})
-          post(actions_path, body)
+          path = "#{actions_path}/actions"
+          post(path, body)
         end
 
         # Retrieve the set of triggers currently available within actions. A trigger is an extensibility point to which actions can be bound.
diff --git a/spec/lib/auth0/api/v2/actions_spec.rb b/spec/lib/auth0/api/v2/actions_spec.rb
index 28e1d264..d1f0e5af 100644
--- a/spec/lib/auth0/api/v2/actions_spec.rb
+++ b/spec/lib/auth0/api/v2/actions_spec.rb
@@ -15,11 +15,28 @@
       expect(@instance).to respond_to(:get_actions)
     end
 
+    it 'is expected to support all optional arguments' do
+      expect(@instance).to receive(:get).with(
+        '/api/v2/actions/actions', {
+          triggerId: nil,
+          actionName: nil,
+          deployed: nil,
+          per_page: nil,
+          page: nil,
+          installed: nil
+        }
+      )
+
+      expect do
+        @instance.actions()
+      end.not_to raise_error
+    end
+
     it 'is expected to get /api/v2/actions with custom parameters' do
       expect(@instance).to receive(:get).with(
         '/api/v2/actions/actions', {
-        trigger_id: 'post-login',
-        action_name: 'loginHandler',
+        triggerId: 'post-login',
+        actionName: 'loginHandler',
         deployed: true,
         per_page: 10,
         page: 1,
@@ -37,13 +54,6 @@
       end.not_to raise_error
     end
 
-    it 'is expected to raise an exception when the trigger id is empty' do
-      expect { @instance.actions(nil, nil) }.to raise_exception(Auth0::MissingTriggerId)
-    end
-
-    it 'is expected to raise an exception when the action name is empty' do
-      expect { @instance.actions(1, nil) }.to raise_exception(Auth0::MissingActionName)
-    end
   end
 
   context '.action' do
@@ -71,7 +81,7 @@
 
     it 'is expected to post to /api/v2/actions' do
       expect(@instance).to receive(:post).with(
-        '/api/v2/actions', {
+        '/api/v2/actions/actions', {
           name: 'test_org'
         })
       expect do