Don't error if new keys are added to student response

It's common when developing with APIs to add in new keys to the response
if this happens we don't want this code to error.

Author: zetter-rpf

lib/profile_api_client.rb

@@ -255,7 +255,9 @@ def build_student(attrs)
       symbolized_attrs[:email] ||= nil
       symbolized_attrs[:ssoProviders] ||= []
 
-      Student.new(**symbolized_attrs)
+      allowed_keys = ProfileApiClient::Student.members
+
+      Student.new(**symbolized_attrs.slice(*allowed_keys))
     end
 
     def unauthorized!(response)

spec/lib/profile_api_client_spec.rb

@@ -553,6 +553,15 @@ def update_school_student
       expect(school_student_response).to eq(expected)
     end
 
+    it 'returns student even if response contains unexpected keys' do
+      data = { id: student_id, schoolId: school.id, name: 'name', username: 'username', email: 'test@example.com', ssoProviders: [], createdAt: '', updatedAt: '', discardedAt: '' }
+      response = data.merge({ unexpectedKey: 'unexpectedValue' })
+      expected = ProfileApiClient::Student.new(**data)
+      stub_request(:get, student_url)
+        .to_return(status: 200, body: response.to_json, headers: { 'content-type' => 'application/json' })
+      expect(school_student_response).to eq(expected)
+    end
+
     private
 
     def school_student