merging protos created from the same source should be a no-op #114
Labels
priority: p2
Moderately-important priority. Fix may not be included in next release.
type: bug
Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
In the course of working on #112 , I discovered that merging a synthetic proto with itself, which one would expect to be a no-op, actually changes the protos.
Two ways to see this;
src/test/java/com/google/cloud/discotoproto3converter/DiscoToProto3ConverterAppTest.java
, re-enable theassertEquals
inconvertWithIdempotentMerge
and examine the test failure. Some fields get extra comments.The text was updated successfully, but these errors were encountered: