From 90c8932fc7316b5afaae350395624b6fd2e73a97 Mon Sep 17 00:00:00 2001 From: Jon Skeet Date: Tue, 12 May 2015 09:47:19 +0100 Subject: Convert back to using NUnit, which is now loaded via NuGet. This includes the NUnit test adapter which allows NUnit tests to be run under VS without any extra plugins. Unfortunate the compatibility tests using the abstract test fixture class show up as "external" tests, and aren't well presented - but they do run. --- .../ProtocolBuffers.Test/GeneratedMessageTest.cs | 180 ++++++++++----------- 1 file changed, 90 insertions(+), 90 deletions(-) (limited to 'csharp/src/ProtocolBuffers.Test/GeneratedMessageTest.cs') diff --git a/csharp/src/ProtocolBuffers.Test/GeneratedMessageTest.cs b/csharp/src/ProtocolBuffers.Test/GeneratedMessageTest.cs index a9052954..984c36e6 100644 --- a/csharp/src/ProtocolBuffers.Test/GeneratedMessageTest.cs +++ b/csharp/src/ProtocolBuffers.Test/GeneratedMessageTest.cs @@ -38,7 +38,7 @@ using System; using System.Collections.Generic; using Google.ProtocolBuffers.Collections; using Google.ProtocolBuffers.TestProtos; -using Xunit; +using NUnit.Framework; namespace Google.ProtocolBuffers { @@ -53,14 +53,14 @@ namespace Google.ProtocolBuffers extensionsReflectionTester = ReflectionTester.CreateTestAllExtensionsInstance(); } - [Fact] + [Test] public void RepeatedAddPrimitiveBeforeBuild() { TestAllTypes message = new TestAllTypes.Builder {RepeatedInt32List = {1, 2, 3}}.Build(); TestUtil.AssertEqual(new int[] {1, 2, 3}, message.RepeatedInt32List); } - [Fact] + [Test] public void AddPrimitiveFailsAfterBuild() { TestAllTypes.Builder builder = new TestAllTypes.Builder(); @@ -71,7 +71,7 @@ namespace Google.ProtocolBuffers Assert.Throws(() => list.Add(2)); } - [Fact] + [Test] public void RepeatedAddMessageBeforeBuild() { TestAllTypes message = new TestAllTypes.Builder @@ -79,11 +79,11 @@ namespace Google.ProtocolBuffers RepeatedNestedMessageList = {new TestAllTypes.Types.NestedMessage.Builder {Bb = 10}.Build()} }.Build(); - Assert.Equal(1, message.RepeatedNestedMessageCount); - Assert.Equal(10, message.RepeatedNestedMessageList[0].Bb); + Assert.AreEqual(1, message.RepeatedNestedMessageCount); + Assert.AreEqual(10, message.RepeatedNestedMessageList[0].Bb); } - [Fact] + [Test] public void AddMessageFailsAfterBuild() { TestAllTypes.Builder builder = new TestAllTypes.Builder(); @@ -93,14 +93,14 @@ namespace Google.ProtocolBuffers Assert.Throws(() => list.Add(new TestAllTypes.Types.NestedMessage.Builder { Bb = 10 }.Build())); } - [Fact] + [Test] public void DefaultInstance() { - Assert.Same(TestAllTypes.DefaultInstance, TestAllTypes.DefaultInstance.DefaultInstanceForType); - Assert.Same(TestAllTypes.DefaultInstance, TestAllTypes.CreateBuilder().DefaultInstanceForType); + Assert.AreSame(TestAllTypes.DefaultInstance, TestAllTypes.DefaultInstance.DefaultInstanceForType); + Assert.AreSame(TestAllTypes.DefaultInstance, TestAllTypes.CreateBuilder().DefaultInstanceForType); } - [Fact] + [Test] public void Accessors() { TestAllTypes.Builder builder = TestAllTypes.CreateBuilder(); @@ -109,7 +109,7 @@ namespace Google.ProtocolBuffers TestUtil.AssertAllFieldsSet(message); } - [Fact] + [Test] public void SettersRejectNull() { TestAllTypes.Builder builder = TestAllTypes.CreateBuilder(); @@ -127,7 +127,7 @@ namespace Google.ProtocolBuffers () => builder.AddRepeatedNestedMessage((TestAllTypes.Types.NestedMessage.Builder) null)); } - [Fact] + [Test] public void RepeatedSetters() { TestAllTypes.Builder builder = TestAllTypes.CreateBuilder(); @@ -137,7 +137,7 @@ namespace Google.ProtocolBuffers TestUtil.AssertRepeatedFieldsModified(message); } - [Fact] + [Test] public void RepeatedAppend() { TestAllTypes.Builder builder = TestAllTypes.CreateBuilder(); @@ -151,11 +151,11 @@ namespace Google.ProtocolBuffers TestAllTypes message = builder.Build(); TestUtil.AssertEqual(message.RepeatedInt32List, new int[] {1, 2, 3, 4}); TestUtil.AssertEqual(message.RepeatedForeignEnumList, new ForeignEnum[] {ForeignEnum.FOREIGN_BAZ}); - Assert.Equal(1, message.RepeatedForeignMessageCount); - Assert.Equal(12, message.GetRepeatedForeignMessage(0).C); + Assert.AreEqual(1, message.RepeatedForeignMessageCount); + Assert.AreEqual(12, message.GetRepeatedForeignMessage(0).C); } - [Fact] + [Test] public void RepeatedAppendRejectsNull() { TestAllTypes.Builder builder = TestAllTypes.CreateBuilder(); @@ -170,7 +170,7 @@ namespace Google.ProtocolBuffers () => builder.AddRangeRepeatedBytes(new[] {TestUtil.ToBytes("one"), null})); } - [Fact] + [Test] public void SettingForeignMessageUsingBuilder() { TestAllTypes message = TestAllTypes.CreateBuilder() @@ -181,10 +181,10 @@ namespace Google.ProtocolBuffers // Create expected version passing foreign message instance explicitly. .SetOptionalForeignMessage(ForeignMessage.CreateBuilder().SetC(123).Build()) .Build(); - Assert.Equal(expectedMessage, message); + Assert.AreEqual(expectedMessage, message); } - [Fact] + [Test] public void SettingRepeatedForeignMessageUsingBuilder() { TestAllTypes message = TestAllTypes.CreateBuilder() @@ -195,10 +195,10 @@ namespace Google.ProtocolBuffers // Create expected version passing foreign message instance explicitly. .AddRepeatedForeignMessage(ForeignMessage.CreateBuilder().SetC(456).Build()) .Build(); - Assert.Equal(expectedMessage, message); + Assert.AreEqual(expectedMessage, message); } - [Fact] + [Test] public void SettingRepeatedValuesUsingRangeInCollectionInitializer() { int[] values = {1, 2, 3}; @@ -206,29 +206,29 @@ namespace Google.ProtocolBuffers { RepeatedSint32List = {values} }.Build(); - Assert.True(Lists.Equals(values, message.RepeatedSint32List)); + Assert.IsTrue(Lists.Equals(values, message.RepeatedSint32List)); } - [Fact] + [Test] public void SettingRepeatedValuesUsingIndividualValuesInCollectionInitializer() { TestAllTypes message = new TestAllTypes.Builder { RepeatedSint32List = {6, 7} }.Build(); - Assert.True(Lists.Equals(new int[] {6, 7}, message.RepeatedSint32List)); + Assert.IsTrue(Lists.Equals(new int[] {6, 7}, message.RepeatedSint32List)); } - [Fact] + [Test] public void Defaults() { TestUtil.AssertClear(TestAllTypes.DefaultInstance); TestUtil.AssertClear(TestAllTypes.CreateBuilder().Build()); - Assert.Equal("\u1234", TestExtremeDefaultValues.DefaultInstance.Utf8String); + Assert.AreEqual("\u1234", TestExtremeDefaultValues.DefaultInstance.Utf8String); } - [Fact] + [Test] public void ReflectionGetters() { TestAllTypes.Builder builder = TestAllTypes.CreateBuilder(); @@ -237,7 +237,7 @@ namespace Google.ProtocolBuffers reflectionTester.AssertAllFieldsSetViaReflection(message); } - [Fact] + [Test] public void ReflectionSetters() { TestAllTypes.Builder builder = TestAllTypes.CreateBuilder(); @@ -246,7 +246,7 @@ namespace Google.ProtocolBuffers TestUtil.AssertAllFieldsSet(message); } - [Fact] + [Test] public void ReflectionClear() { TestAllTypes.Builder builder = TestAllTypes.CreateBuilder(); @@ -256,14 +256,14 @@ namespace Google.ProtocolBuffers TestUtil.AssertClear(message); } - [Fact] + [Test] public void ReflectionSettersRejectNull() { TestAllTypes.Builder builder = TestAllTypes.CreateBuilder(); reflectionTester.AssertReflectionSettersRejectNull(builder); } - [Fact] + [Test] public void ReflectionRepeatedSetters() { TestAllTypes.Builder builder = TestAllTypes.CreateBuilder(); @@ -273,14 +273,14 @@ namespace Google.ProtocolBuffers TestUtil.AssertRepeatedFieldsModified(message); } - [Fact] + [Test] public void TestReflectionRepeatedSettersRejectNull() { TestAllTypes.Builder builder = TestAllTypes.CreateBuilder(); reflectionTester.AssertReflectionRepeatedSettersRejectNull(builder); } - [Fact] + [Test] public void ReflectionDefaults() { TestUtil.TestInMultipleCultures(() => @@ -295,7 +295,7 @@ namespace Google.ProtocolBuffers // ================================================================= // Extensions. - [Fact] + [Test] public void ExtensionAccessors() { TestAllExtensions.Builder builder = TestAllExtensions.CreateBuilder(); @@ -304,7 +304,7 @@ namespace Google.ProtocolBuffers TestUtil.AssertAllExtensionsSet(message); } - [Fact] + [Test] public void ExtensionRepeatedSetters() { TestAllExtensions.Builder builder = TestAllExtensions.CreateBuilder(); @@ -314,14 +314,14 @@ namespace Google.ProtocolBuffers TestUtil.AssertRepeatedExtensionsModified(message); } - [Fact] + [Test] public void ExtensionDefaults() { TestUtil.AssertExtensionsClear(TestAllExtensions.DefaultInstance); TestUtil.AssertExtensionsClear(TestAllExtensions.CreateBuilder().Build()); } - [Fact] + [Test] public void ExtensionReflectionGetters() { TestAllExtensions.Builder builder = TestAllExtensions.CreateBuilder(); @@ -330,7 +330,7 @@ namespace Google.ProtocolBuffers extensionsReflectionTester.AssertAllFieldsSetViaReflection(message); } - [Fact] + [Test] public void ExtensionReflectionSetters() { TestAllExtensions.Builder builder = TestAllExtensions.CreateBuilder(); @@ -339,14 +339,14 @@ namespace Google.ProtocolBuffers TestUtil.AssertAllExtensionsSet(message); } - [Fact] + [Test] public void ExtensionReflectionSettersRejectNull() { TestAllExtensions.Builder builder = TestAllExtensions.CreateBuilder(); extensionsReflectionTester.AssertReflectionSettersRejectNull(builder); } - [Fact] + [Test] public void ExtensionReflectionRepeatedSetters() { TestAllExtensions.Builder builder = TestAllExtensions.CreateBuilder(); @@ -356,14 +356,14 @@ namespace Google.ProtocolBuffers TestUtil.AssertRepeatedExtensionsModified(message); } - [Fact] + [Test] public void ExtensionReflectionRepeatedSettersRejectNull() { TestAllExtensions.Builder builder = TestAllExtensions.CreateBuilder(); extensionsReflectionTester.AssertReflectionRepeatedSettersRejectNull(builder); } - [Fact] + [Test] public void ExtensionReflectionDefaults() { TestUtil.TestInMultipleCultures(() => @@ -375,33 +375,33 @@ namespace Google.ProtocolBuffers }); } - [Fact] + [Test] public void ClearExtension() { // ClearExtension() is not actually used in TestUtil, so try it manually. - Assert.False(TestAllExtensions.CreateBuilder() + Assert.IsFalse(TestAllExtensions.CreateBuilder() .SetExtension(Unittest.OptionalInt32Extension, 1) .ClearExtension(Unittest.OptionalInt32Extension) .HasExtension(Unittest.OptionalInt32Extension)); - Assert.Equal(0, TestAllExtensions.CreateBuilder() + Assert.AreEqual(0, TestAllExtensions.CreateBuilder() .AddExtension(Unittest.RepeatedInt32Extension, 1) .ClearExtension(Unittest.RepeatedInt32Extension) .GetExtensionCount(Unittest.RepeatedInt32Extension)); } - [Fact] + [Test] public void ExtensionMergeFrom() { TestAllExtensions original = TestAllExtensions.CreateBuilder() .SetExtension(Unittest.OptionalInt32Extension, 1).Build(); TestAllExtensions merged = TestAllExtensions.CreateBuilder().MergeFrom(original).Build(); - Assert.True((merged.HasExtension(Unittest.OptionalInt32Extension))); - Assert.Equal(1, (int) merged.GetExtension(Unittest.OptionalInt32Extension)); + Assert.IsTrue((merged.HasExtension(Unittest.OptionalInt32Extension))); + Assert.AreEqual(1, (int) merged.GetExtension(Unittest.OptionalInt32Extension)); } /* Removed multiple files option for the moment - [Fact] + [Test] public void MultipleFilesOption() { // We mostly just want to check that things compile. MessageWithNoOuter message = MessageWithNoOuter.CreateBuilder() @@ -410,63 +410,63 @@ namespace Google.ProtocolBuffers .SetNestedEnum(MessageWithNoOuter.Types.NestedEnum.BAZ) .SetForeignEnum(EnumWithNoOuter.BAR) .Build(); - Assert.Equal(message, MessageWithNoOuter.ParseFrom(message.ToByteString())); + Assert.AreEqual(message, MessageWithNoOuter.ParseFrom(message.ToByteString())); - Assert.Equal(MultiFileProto.DescriptorProtoFile, MessageWithNoOuter.DescriptorProtoFile.File); + Assert.AreEqual(MultiFileProto.DescriptorProtoFile, MessageWithNoOuter.DescriptorProtoFile.File); FieldDescriptor field = MessageWithNoOuter.DescriptorProtoFile.FindDescriptor("foreign_enum"); - Assert.Equal(MultiFileProto.DescriptorProtoFile.FindTypeByName("EnumWithNoOuter") + Assert.AreEqual(MultiFileProto.DescriptorProtoFile.FindTypeByName("EnumWithNoOuter") .FindValueByNumber((int)EnumWithNoOuter.BAR), message[field]); - Assert.Equal(MultiFileProto.DescriptorProtoFile, ServiceWithNoOuter.DescriptorProtoFile.File); + Assert.AreEqual(MultiFileProto.DescriptorProtoFile, ServiceWithNoOuter.DescriptorProtoFile.File); - Assert.False(TestAllExtensions.DefaultInstance.HasExtension(MultiFileProto.ExtensionWithOuter)); + Assert.IsFalse(TestAllExtensions.DefaultInstance.HasExtension(MultiFileProto.ExtensionWithOuter)); }*/ - [Fact] + [Test] public void OptionalFieldWithRequiredSubfieldsOptimizedForSize() { TestOptionalOptimizedForSize message = TestOptionalOptimizedForSize.DefaultInstance; - Assert.True(message.IsInitialized); + Assert.IsTrue(message.IsInitialized); message = TestOptionalOptimizedForSize.CreateBuilder().SetO( TestRequiredOptimizedForSize.CreateBuilder().BuildPartial() ).BuildPartial(); - Assert.False(message.IsInitialized); + Assert.IsFalse(message.IsInitialized); message = TestOptionalOptimizedForSize.CreateBuilder().SetO( TestRequiredOptimizedForSize.CreateBuilder().SetX(5).BuildPartial() ).BuildPartial(); - Assert.True(message.IsInitialized); + Assert.IsTrue(message.IsInitialized); } - [Fact] + [Test] public void OptimizedForSizeMergeUsesAllFieldsFromTarget() { TestOptimizedForSize withFieldSet = new TestOptimizedForSize.Builder {I = 10}.Build(); TestOptimizedForSize.Builder builder = new TestOptimizedForSize.Builder(); builder.MergeFrom(withFieldSet); TestOptimizedForSize built = builder.Build(); - Assert.Equal(10, built.I); + Assert.AreEqual(10, built.I); } - [Fact] + [Test] public void UninitializedExtensionInOptimizedForSizeMakesMessageUninitialized() { TestOptimizedForSize.Builder builder = new TestOptimizedForSize.Builder(); builder.SetExtension(TestOptimizedForSize.TestExtension2, new TestRequiredOptimizedForSize.Builder().BuildPartial()); - Assert.False(builder.IsInitialized); - Assert.False(builder.BuildPartial().IsInitialized); + Assert.IsFalse(builder.IsInitialized); + Assert.IsFalse(builder.BuildPartial().IsInitialized); builder = new TestOptimizedForSize.Builder(); builder.SetExtension(TestOptimizedForSize.TestExtension2, new TestRequiredOptimizedForSize.Builder {X = 10}.BuildPartial()); - Assert.True(builder.IsInitialized); - Assert.True(builder.BuildPartial().IsInitialized); + Assert.IsTrue(builder.IsInitialized); + Assert.IsTrue(builder.BuildPartial().IsInitialized); } - [Fact] + [Test] public void ToBuilder() { TestAllTypes.Builder builder = TestAllTypes.CreateBuilder(); @@ -475,40 +475,40 @@ namespace Google.ProtocolBuffers TestUtil.AssertAllFieldsSet(message.ToBuilder().Build()); } - [Fact] + [Test] public void FieldConstantValues() { - Assert.Equal(TestAllTypes.Types.NestedMessage.BbFieldNumber, 1); - Assert.Equal(TestAllTypes.OptionalInt32FieldNumber, 1); - Assert.Equal(TestAllTypes.OptionalGroupFieldNumber, 16); - Assert.Equal(TestAllTypes.OptionalNestedMessageFieldNumber, 18); - Assert.Equal(TestAllTypes.OptionalNestedEnumFieldNumber, 21); - Assert.Equal(TestAllTypes.RepeatedInt32FieldNumber, 31); - Assert.Equal(TestAllTypes.RepeatedGroupFieldNumber, 46); - Assert.Equal(TestAllTypes.RepeatedNestedMessageFieldNumber, 48); - Assert.Equal(TestAllTypes.RepeatedNestedEnumFieldNumber, 51); + Assert.AreEqual(TestAllTypes.Types.NestedMessage.BbFieldNumber, 1); + Assert.AreEqual(TestAllTypes.OptionalInt32FieldNumber, 1); + Assert.AreEqual(TestAllTypes.OptionalGroupFieldNumber, 16); + Assert.AreEqual(TestAllTypes.OptionalNestedMessageFieldNumber, 18); + Assert.AreEqual(TestAllTypes.OptionalNestedEnumFieldNumber, 21); + Assert.AreEqual(TestAllTypes.RepeatedInt32FieldNumber, 31); + Assert.AreEqual(TestAllTypes.RepeatedGroupFieldNumber, 46); + Assert.AreEqual(TestAllTypes.RepeatedNestedMessageFieldNumber, 48); + Assert.AreEqual(TestAllTypes.RepeatedNestedEnumFieldNumber, 51); } - [Fact] + [Test] public void ExtensionConstantValues() { - Assert.Equal(TestRequired.SingleFieldNumber, 1000); - Assert.Equal(TestRequired.MultiFieldNumber, 1001); - Assert.Equal(Unittest.OptionalInt32ExtensionFieldNumber, 1); - Assert.Equal(Unittest.OptionalGroupExtensionFieldNumber, 16); - Assert.Equal(Unittest.OptionalNestedMessageExtensionFieldNumber, 18); - Assert.Equal(Unittest.OptionalNestedEnumExtensionFieldNumber, 21); - Assert.Equal(Unittest.RepeatedInt32ExtensionFieldNumber, 31); - Assert.Equal(Unittest.RepeatedGroupExtensionFieldNumber, 46); - Assert.Equal(Unittest.RepeatedNestedMessageExtensionFieldNumber, 48); - Assert.Equal(Unittest.RepeatedNestedEnumExtensionFieldNumber, 51); + Assert.AreEqual(TestRequired.SingleFieldNumber, 1000); + Assert.AreEqual(TestRequired.MultiFieldNumber, 1001); + Assert.AreEqual(Unittest.OptionalInt32ExtensionFieldNumber, 1); + Assert.AreEqual(Unittest.OptionalGroupExtensionFieldNumber, 16); + Assert.AreEqual(Unittest.OptionalNestedMessageExtensionFieldNumber, 18); + Assert.AreEqual(Unittest.OptionalNestedEnumExtensionFieldNumber, 21); + Assert.AreEqual(Unittest.RepeatedInt32ExtensionFieldNumber, 31); + Assert.AreEqual(Unittest.RepeatedGroupExtensionFieldNumber, 46); + Assert.AreEqual(Unittest.RepeatedNestedMessageExtensionFieldNumber, 48); + Assert.AreEqual(Unittest.RepeatedNestedEnumExtensionFieldNumber, 51); } - [Fact] + [Test] public void EmptyPackedValue() { TestPackedTypes empty = new TestPackedTypes.Builder().Build(); - Assert.Equal(0, empty.SerializedSize); + Assert.AreEqual(0, empty.SerializedSize); } } } \ No newline at end of file -- cgit v1.2.3