diff --git a/app/controllers/api/ownership_transfers_controller.rb b/app/controllers/api/ownership_transfers_controller.rb index 5c4775585..004d59841 100644 --- a/app/controllers/api/ownership_transfers_controller.rb +++ b/app/controllers/api/ownership_transfers_controller.rb @@ -12,7 +12,7 @@ def show if @ownership_transfer.blank? || cannot?(:read, @ownership_transfer) head :not_found elsif current_user_is_requester? - render json: { status: @ownership_transfer.status, you_are: 'owner', nominee_name: nominee_name }, status: :ok + render json: { status: @ownership_transfer.status, you_are: 'owner', nominee_name:, nominee_email: }, status: :ok else render json: { status: @ownership_transfer.status, you_are: 'nominee' }, status: :ok end @@ -49,5 +49,9 @@ def current_user_is_requester? def nominee_name User.from_userinfo(ids: @ownership_transfer.nominated_user_id).first&.name end + + def nominee_email + @ownership_transfer.email_address + end end end diff --git a/spec/features/ownership_transfer/viewing_ownership_transfer_status_spec.rb b/spec/features/ownership_transfer/viewing_ownership_transfer_status_spec.rb index 1cde48c9c..cf03b3f22 100644 --- a/spec/features/ownership_transfer/viewing_ownership_transfer_status_spec.rb +++ b/spec/features/ownership_transfer/viewing_ownership_transfer_status_spec.rb @@ -63,6 +63,13 @@ expect(json['nominee_name']).to eq(nominee.name) end + it 'includes the nominated teacher\'s email' do + get("/api/schools/#{school.id}/ownership_transfer", headers:) + + json = JSON.parse(response.body) + expect(json['nominee_email']).to eq(ownership_transfer.email_address) + end + it 'includes the transfer status' do get("/api/schools/#{school.id}/ownership_transfer", headers:) @@ -114,7 +121,7 @@ expect(response).to have_http_status(:ok) json = JSON.parse(response.body) - expect(json).to include('status' => 'completed', 'you_are' => 'owner', 'nominee_name' => nominee.name) + expect(json).to include('status' => 'completed', 'you_are' => 'owner', 'nominee_name' => nominee.name, 'nominee_email' => ownership_transfer.email_address) end end