Correctly serialize review requests with only-fields
Review Request #10169 — Created Sept. 22, 2018 and submitted — Latest diff uploaded
If the
dvcs_featureis disabled and a request is made to the review
request WebAPI resource with theonly-fields/only_fieldsquery/post
parameter set to a subset of fields that does not include the
created_with_historyfield, the server would raise an exception and
return a HTTP 500. This was because the field was already excluded from
the set of serialized fields.We now are more defensive about removing this field so as not to raise a
KeyErrorif the field is not in the dictionary.
Ran unit tests.