Consistently return from ProfileApiClient methods

So that the last line of the method body is the value being returned.
Note that in the case of `.create_safeguarding_flag` and
`.delete_safeguarding_flag` we return nil to indicate a successful
response so the conditional `raise` appears on the last line of the
method body.

Author: chrisroos

lib/profile_api_client.rb

@@ -135,17 +135,13 @@ def create_safeguarding_flag(token:, flag:)
         request.body = { flag: }
       end
 
-      return if response.status == 201 || response.status == 303
-
-      raise UnexpectedResponse, response
+      raise UnexpectedResponse, response unless [201, 303].include?(response.status)
     end
 
     def delete_safeguarding_flag(token:, flag:)
       response = connection(token).delete("/api/v1/safeguarding-flags/#{flag}")
 
-      return if response.status == 204
-
-      raise UnexpectedResponse, response
+      raise UnexpectedResponse, response unless response.status == 204
     end
 
     private