Fix ParseComplete and remove impossible Buffer Null check

Seddryck · Sep 2, 2023 · 7cf92f1 · 7cf92f1
1 parent 8b1d672
commit 7cf92f1
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 7 deletions.
diff --git a/Pgnoli.Testing/Messages/Backend/Query/ParseCompleteTest.cs b/Pgnoli.Testing/Messages/Backend/Query/ParseCompleteTest.cs
@@ -12,7 +12,7 @@ public class ParseCompleteTest
         [Test]
         public void Write_Default_Success()
         {
-            var msg = new ParseComplete();
+            var msg = ParseComplete.Message.Build();
             msg.Write();
             var bytes = msg.GetBytes();
 

diff --git a/Pgnoli/Messages/Backend/Query/ParseComplete.cs b/Pgnoli/Messages/Backend/Query/ParseComplete.cs
@@ -6,14 +6,14 @@
 
 namespace Pgnoli.Messages.Backend.Query
 {
-    public class ParseComplete : CodeMessage
+    public sealed class ParseComplete : CodeMessage
     {
         public const char Code = '1';
 
-        protected internal ParseComplete(byte[] bytes)
+        internal ParseComplete(byte[] bytes)
             : base(Code, bytes) { }
 
-        protected internal ParseComplete()
+        internal ParseComplete()
             : base(Code) { }
 
         protected override int GetPayloadLength()

diff --git a/Pgnoli/Messages/Message.cs b/Pgnoli/Messages/Message.cs
@@ -36,9 +36,6 @@ internal virtual byte[] Write()
 
         public virtual int Read()
         {
-            if (Buffer is null)
-                throw new BufferNotAllocatedException();
-
             if (Buffer.Length == 0)
                 throw new BufferEmptyException();