I think a helper method for `revisionHistoryToJson` that doesn't require `atts` might help -- then you could avoid these changes in the tests just to add empty arrays. It's hard to understand what tests are actually changed to test attachments and which are changed just because of the changed method signature but are not expecting different outputs.