GODRIVER-2587 Implement modifyCollection for the unified test runner

[joyjwang]

GODRIVER-2587

Summary

Background & Motivation

[mongodb-drivers-pr-bot]

API Change Report

./v2/mongo/options

incompatible changes

BSONOptions.ObjectIDAsHexString: removed

./v2/x/mongo/driver

incompatible changes

##CursorOptions.MarshalValueEncoderFn: changed from func(io.Writer) ./v2/bson.Encoder to func(io.Writer) (./v2/bson.Encoder, error)

[prestonvasquez]

@joyjwang Can you sync the JSON and YAML for tests the use collMod in the Go Driver: https://github.com/search?q=repo%3Amongodb%2Fspecifications+%5C%22collMod%5C%22+language%3AJSON&type=code

[prestonvasquez]

@joyjwang Can you sync the JSON and YAML for tests the use collMod in the Go Driver: github.com/search?q=repo%3Amongodb%2Fspecifications+%5C%22collMod%5C%22+language%3AJSON&type=code

@joyjwang I apologize, I used the wrong search string. I meant to used "modifyCollection". Looking through the tests it looks like everything is covered, we can revert c1efef5 to keep this PR focused.

We can also put the PR in "ready for review".

[joyjwang]

@prestonvasquez the sync does add the test testdata/collection-management/modifyCollection-errorResponse.json which uses "modifyCollection", so would it be best to keep that and only revert the others?

[prestonvasquez]

@prestonvasquez the sync does add the test testdata/collection-management/modifyCollection-errorResponse.json which uses "modifyCollection", so would it be best to keep that and only revert the others?

Yes, that sounds right.

[prestonvasquez]

LGTM!

[joyjwang]

drivers-pr-bot please backport to v1