diff --git a/csharp/src/Google.Protobuf.Test/GeneratedMessageTest.Proto2.cs b/csharp/src/Google.Protobuf.Test/GeneratedMessageTest.Proto2.cs index 704f2e5af3..1e22674c7c 100644 --- a/csharp/src/Google.Protobuf.Test/GeneratedMessageTest.Proto2.cs +++ b/csharp/src/Google.Protobuf.Test/GeneratedMessageTest.Proto2.cs @@ -261,11 +261,14 @@ namespace Google.Protobuf Assert.True(message.IsInitialized()); } + // Code was accidentally left in message parser that threw exceptions when missing required fields after parsing. + // We've decided to not throw exceptions on missing fields, instead leaving it up to the consumer how they + // want to check and handle missing fields. [Test] public void RequiredFieldsNoThrow() { - TestRequired.Parser.ParseFrom(new byte[0]); - (TestRequired.Parser as MessageParser).ParseFrom(new byte[0]); + Assert.DoesNotThrow(() => TestRequired.Parser.ParseFrom(new byte[0])); + Assert.DoesNotThrow(() => (TestRequired.Parser as MessageParser).ParseFrom(new byte[0])); } [Test]