added comment and remove commented code

Author: Tien Nguyen

src/AdoNetCore.AseClient/Internal/FormatItem.cs

@@ -91,6 +91,12 @@ public static FormatItem CreateForParameter(AseParameter parameter, DbEnvironmen
                         break;
                     case DbType.String:
                         format.BlobType = BlobType.BLOB_UNICHAR;
+                        // This is far less than ideal but at the time of addressing this issue whereby if the
+                        // BlobType is a BLOB_UNICHAR then the UserType would need to be 36 when it
+                        // is a stored proc otherwise it would need to be zero (0).
+                        //
+                        // In the future, we'd need to overhaul how TDS_BLOB is structured especially
+                        // around BLOB_UNICHAR and the UserType that it should return in a more consistent way
                         if (commandType != CommandType.StoredProcedure)
                             format.UserType = 0;
 

src/AdoNetCore.AseClient/Internal/ValueWriter.cs

@@ -334,8 +334,6 @@ private static void WriteTDS_BLOB(object value, Stream stream, FormatItem format
                     stream.WriteNullableUShortPrefixedByteArray(format.ClassId);
                     stream.WriteBlobSpecificTerminator();
                     break;
-                    
-                    //throw new AseException($"TDS_BLOB support for {value.GetType().Name} not yet implemented");
             }
         }
 

test/AdoNetCore.AseClient.Tests/Integration/TextEchoProcedureTests.cs

@@ -477,7 +477,6 @@ public void Insert_Text_ShouldBeValid()
                         }
 
                         var count = connection.Execute("Select Fragment from test_text_table");
-                        //Assert.AreEqual(256, count);
                         transaction.Rollback();
                     }
                 }