fix account show presenter tests

app/presenters/account_show_presenter.rb

@@ -144,10 +144,11 @@ def connected_apps
     user.connected_apps.includes([:service_provider_record, :email_address])
   end
 
-  def show_change_option
+  def hide_change_option
+    binding.pry
     if requested_attributes
       requested_attributes.include?('all_emails') ||
-        !requested_attributes.include?('email')
+        requested_attributes.include?('email')
     end
   end
 

app/views/accounts/_connected_app.html.erb

@@ -17,7 +17,7 @@
           timestamp_html: render(TimeComponent.new(time: identity.created_at)),
         ) %>
     <br />
-    <% if !@presenter.show_change_option %>
+    <% if !@presenter.hide_change_option %>
       <strong>
         <%= identity.email_address&.email || t('account.connected_apps.email_not_selected') %>
       </strong>

spec/presenters/account_show_presenter_spec.rb

@@ -679,7 +679,7 @@
     end
   end
 
-  describe '#hide_change_option' do
+  describe '#show_change_option' do
     let(:view_context) { ActionController::Base.new.view_context }
     let(:service_provider) { create(:service_provider) }
     let(:view_context) { ActionController::Base.new.view_context }
@@ -693,7 +693,7 @@
       )
     end
 
-    it 'returns false if option is not available' do
+    it 'returns false when option should be shown' do
       user = User.new
       account_show = AccountShowPresenter.new(
         decrypted_pii: {},
@@ -702,10 +702,25 @@
         sp_name: nil,
         user: user,
         locked_for_session: false,
-        requested_attributes: false,
+        requested_attributes: ['ssn'],
+      )
+
+      expect(account_show.hide_change_option).to eq(false)
+    end
+
+    it 'returns true when option should be hidden' do
+      user = User.new
+      account_show = AccountShowPresenter.new(
+        decrypted_pii: {},
+        sp_session_request_url: nil,
+        authn_context: nil,
+        sp_name: nil,
+        user: user,
+        locked_for_session: false,
+        requested_attributes: ['all_emails'],
       )
 
-      expect(account_show.requested_attributes).to eq(false)
+      expect(account_show.hide_change_option).to eq(true)
     end
   end
 end