Set default method of button_component to POST

Author: jhutchison-nava

dpc-portal/app/components/core/button/button_component.rb

@@ -6,7 +6,7 @@ module Button
     class ButtonComponent < ViewComponent::Base
       attr_accessor :label, :destination, :method, :additional_classes
 
-      def initialize(label:, destination:, method: :get, additional_classes: nil, disabled: false)
+      def initialize(label:, destination:, method: :post, additional_classes: nil, disabled: false)
         super
         @label = label
         @destination = destination

dpc-portal/app/components/page/invitations/accept_invitation_component.html.erb

@@ -11,14 +11,12 @@
         </ul>
       </p>
     <%= render Core::Button::ButtonComponent.new(label: 'Verify information',
-        destination: confirm_organization_invitation_path(@organization, @invitation),
-        method: :post) %>
+        destination: confirm_organization_invitation_path(@organization, @invitation)) %>
     <% end %>
     <% if @invitation.credential_delegate? %>
     <p>Your identity has been verified. You can now accept your invite</p>
     <%= render Core::Button::ButtonComponent.new(label: 'Accept invite',
-        destination: register_organization_invitation_path(@organization, @invitation),
-        method: :post) %>
+        destination: register_organization_invitation_path(@organization, @invitation)) %>
     <% end %>
   </div>
 </div>

dpc-portal/app/components/page/invitations/register_component.html.erb

@@ -3,7 +3,6 @@
   <div>
     <p>Your role as Authorized Official and your organization's Medicare enrollment have been verified.</p>
     <%= render Core::Button::ButtonComponent.new(label: "Submit registration",
-        destination: register_organization_invitation_path(@organization, @invitation),
-        method: :post) %>
+        destination: register_organization_invitation_path(@organization, @invitation)) %>
   </div>
 </div>

dpc-portal/app/components/page/utility/error_component.html.erb

@@ -10,8 +10,7 @@
       when :ao_expired %>
         <%= render Core::Button::ButtonComponent.new(label: "Request new link",
             disabled: @invitation&.renewed?,
-            destination: renew_organization_invitation_path(@invitation.provider_organization, @invitation),
-            method: :post) %>
+            destination: renew_organization_invitation_path(@invitation.provider_organization, @invitation)) %>
       <% when :ao_accepted %>
         <%= link_to new_user_session_path, class: 'usa-button', data: { turbo: false } do %>
           Sign in with <span class="login-button__logo">Login.gov</span>

dpc-portal/spec/components/core/button/button_component_spec.rb

@@ -13,7 +13,7 @@
       let(:component) { described_class.new(label: 'label', destination: 'destination') }
       let(:expected_html) do
         <<~HTML
-          <form class="button_to" method="get" action="destination"><button class="usa-button" type="submit">label</button></form>
+          <form class="button_to" method="post" action="destination"><button class="usa-button" type="submit">label</button></form>
         HTML
       end
 
@@ -30,7 +30,7 @@
       end
       let(:expected_html) do
         <<~HTML
-          <form class="button_to" method="get" action="destination"><button class="usa-button--outline usa-button" type="submit">label</button></form>
+          <form class="button_to" method="post" action="destination"><button class="usa-button--outline usa-button" type="submit">label</button></form>
         HTML
       end
       it { is_expected.to match_html_fragment(expected_html) }
@@ -42,7 +42,7 @@
       end
       let(:expected_html) do
         <<~HTML
-          <form class="button_to" method="get" action="destination"><button class="usa-button" disabled="disabled" type="submit">label</button></form>
+          <form class="button_to" method="post" action="destination"><button class="usa-button" disabled="disabled" type="submit">label</button></form>
         HTML
       end
       it { is_expected.to match_html_fragment(expected_html) }

dpc-portal/spec/components/page/organization/credentials_componenent_spec.rb

@@ -20,23 +20,23 @@
       let(:org) { ComponentSupport::MockOrg.new(0) }
       it 'Should have Generate token button' do
         button = <<~BUTTON
-          <form class="button_to" method="get" action="/portal/organizations/#{org.path_id}/client_tokens/new">
+          <form class="button_to" method="post" action="/portal/organizations/#{org.path_id}/client_tokens/new">
             <button class="usa-button" type="submit">Generate token</button>
           </form>
         BUTTON
         is_expected.to include(normalize_space(button))
       end
       it 'Should have Create key button' do
         button = <<~BUTTON
-          <form class="button_to" method="get" action="/portal/organizations/#{org.path_id}/public_keys/new">
+          <form class="button_to" method="post" action="/portal/organizations/#{org.path_id}/public_keys/new">
             <button class="usa-button" type="submit">Add key</button>
           </form>
         BUTTON
         is_expected.to include(normalize_space(button))
       end
       it 'Should have Add IP button' do
         button = <<~BUTTON
-          <form class="button_to" method="get" action="/portal/organizations/#{org.path_id}/ip_addresses/new">
+          <form class="button_to" method="post" action="/portal/organizations/#{org.path_id}/ip_addresses/new">
             <button class="usa-button" type="submit">Add IP</button>
           </form>
         BUTTON