tree d05438e47fed8e25c20ae1e26f88cad75a9fecef
parent 6c73a2e246d1d0d4cf2690837c7056fea757f128
author jhorvitz <jhorvitz@google.com> 1634598959 -0700
committer Copybara-Service <copybara-worker@google.com> 1634599018 -0700

Rewrite `SerializationContextTest` to avoid using mocks.

The current approach of mocking `CodedOutputStream` is dubious because we are verifying calls to `final` methods. In this case, Mockito does not actually verify the specified method call, but instead verifies calls to the methods called within the body of the specified method.

It's not too hard to use real instances for `CodedOutputStream` and `ObjectCodecRegistry` in this test, which is generally better than mocks anyway.

PiperOrigin-RevId: 404089580