Disable warning CS0618 when using Obsolete with a message

Author: ltrzesniewski

src/Abc.Zebus.MessageDsl.Tests/MessageDsl/CSharpGeneratorTests.cs

@@ -352,7 +352,7 @@ public async Task should_handle_obsolete_attribute_1()
             }
         });
 
-        code.ShouldNotContain("#pragma warning disable 612");
+        code.ShouldNotContain("#pragma warning disable CS0612, CS0618");
     }
 
     [Test]
@@ -370,7 +370,7 @@ public async Task should_handle_obsolete_attribute_2()
             }
         });
 
-        code.ShouldContain("#pragma warning disable 612");
+        code.ShouldContain("#pragma warning disable CS0612, CS0618");
     }
 
     [Test]
@@ -382,7 +382,19 @@ public async Task should_handle_obsolete_attribute_3()
             Attributes = { new AttributeDefinition("Obsolete") }
         });
 
-        code.ShouldContain("#pragma warning disable 612");
+        code.ShouldContain("#pragma warning disable CS0612, CS0618");
+    }
+
+    [Test]
+    public async Task should_handle_obsolete_attribute_4()
+    {
+        var code = await Verify(new MessageDefinition
+        {
+            Name = "FooExecuted",
+            Attributes = { new AttributeDefinition("Obsolete", "\"Custom message\"") }
+        });
+
+        code.ShouldContain("#pragma warning disable CS0612, CS0618");
     }
 
     [Test]

src/Abc.Zebus.MessageDsl.Tests/MessageDsl/ParsedContractsTests.cs

@@ -138,6 +138,19 @@ public void should_handle_attributes()
         msg.Parameters[0].Attributes[0].TypeName.ShouldEqual(new TypeName("Lol"));
     }
 
+    [Test]
+    public void should_handle_attributes_2()
+    {
+        var contracts = ParseValid("""[Transient(), Obsolete] FooExecuted(int id);""");
+
+        var msg = contracts.Messages.ExpectedSingle();
+        msg.Attributes.Count.ShouldEqual(2);
+        msg.Attributes[0].TypeName.ShouldEqual(new TypeName("Transient"));
+        msg.Attributes[0].Parameters.ShouldNotBeNull().ShouldBeEmpty();
+        msg.Attributes[1].TypeName.ShouldEqual(new TypeName("Obsolete"));
+        msg.Attributes[1].Parameters.ShouldBeNull();
+    }
+
     [Test]
     public void should_set_message_as_transient()
     {

src/Abc.Zebus.MessageDsl.Tests/MessageDsl/Snapshots/CSharpGeneratorTests.should_handle_obsolete_attribute_2.verified.cs

@@ -8,7 +8,7 @@
 using Abc.Zebus;
 using ProtoBuf;
 
-#pragma warning disable 612
+#pragma warning disable CS0612, CS0618
 
 [ProtoContract]
 [System.Diagnostics.DebuggerNonUserCode]

src/Abc.Zebus.MessageDsl.Tests/MessageDsl/Snapshots/CSharpGeneratorTests.should_handle_obsolete_attribute_3.verified.cs

@@ -8,7 +8,7 @@
 using Abc.Zebus;
 using ProtoBuf;
 
-#pragma warning disable 612
+#pragma warning disable CS0612, CS0618
 
 [ProtoContract]
 [System.Diagnostics.DebuggerNonUserCode]

src/Abc.Zebus.MessageDsl.Tests/MessageDsl/Snapshots/CSharpGeneratorTests.should_handle_obsolete_attribute_4.verified.cs

@@ -0,0 +1,19 @@
+//------------------------------------------------------------------------------
+// <auto-generated>
+//     This code was generated by a tool.
+// </auto-generated>
+//------------------------------------------------------------------------------
+
+using System;
+using Abc.Zebus;
+using ProtoBuf;
+
+#pragma warning disable CS0612, CS0618
+
+[ProtoContract]
+[System.Diagnostics.DebuggerNonUserCode]
+[System.CodeDom.Compiler.GeneratedCode("Abc.Zebus.MessageDsl", "1.2.3.4")]
+[Obsolete("Custom message")]
+public sealed partial class FooExecuted : IEvent
+{
+}

src/Abc.Zebus.MessageDsl/Generator/CSharpGenerator.cs

@@ -108,7 +108,7 @@ private void WritePragmas()
 
         if (hasObsolete)
         {
-            Writer.WriteLine("#pragma warning disable 612");
+            Writer.WriteLine("#pragma warning disable CS0612, CS0618");
             Writer.WriteLine("");
         }
     }