Minor refactoring; coverage improved

Source/TargetBinaryStorage/InMemoryBinaryStorage/BinaryStorage.cs

@@ -9,7 +9,7 @@
     {
         private Dictionary<string, IBlobInfo> Blobs { get; } = new Dictionary<string, IBlobInfo>();
 
-        private Dictionary<BlockId, BlockInfo> Blocks { get; } = new Dictionary<BlockId, BlockInfo>();
+        private Dictionary<BlockId, byte[]> Blocks { get; } = new Dictionary<BlockId, byte[]>();
 
         public Task<IBlobInfo> GetBlobIfExists(string blobId)
         {
@@ -33,13 +33,13 @@
         {
             return Task.Run(() =>
             {
-                this.Blocks[new BlockId(blobId, blockKey)] = new BlockInfo(blobId, partNumber, blockKey, block);
+                this.Blocks[new BlockId(blobId, blockKey)] = block;
             });
         }
 
         public Task<byte[]> RetrieveBlock(string blobId, string blockKey)
         {
-            return Task.Run(() => this.Blocks[new BlockId(blobId, blockKey)].BlockData);
+            return Task.Run(() => this.Blocks[new BlockId(blobId, blockKey)]);
         }
     }
 }

Source/TargetBinaryStorage/InMemoryBinaryStorage/BlockInfo.cs

@@ -1,21 +0,0 @@
-namespace EternalArrowBackup.TargetBinaryStorage.InMemoryBinaryStorage
-{
-    internal class BlockInfo
-    {
-        public BlockInfo(string blobId, int partNumber, string blockKey, byte[] blockData)
-        {
-            this.BlobId = blobId;
-            this.PartNumber = partNumber;
-            this.BlockKey = blockKey;
-            this.BlockData = blockData;
-        }
-
-        public string BlobId { get; }
-
-        public int PartNumber { get; }
-
-        public string BlockKey { get; }
-
-        public byte[] BlockData { get; }
-    }
-}

Tests/ContentTransformer/ClearText/DataWithSignatureTests.cs

@@ -0,0 +1,22 @@
+namespace EternalArrowBackup.ContentTransformer.ClearText.Tests
+{
+    using System;
+    using Xunit;
+
+    public static class DataWithSignatureTests
+    {
+        [Fact]
+        [Trait("Category", "Simple")]
+        public static void TestDataWithSignatureConstructor()
+        {
+            Assert.ThrowsAny<ArgumentException>(() => new DataWithSignature(null, new byte[0]));
+            Assert.ThrowsAny<ArgumentException>(() => new DataWithSignature(new byte[0], null));
+            Assert.ThrowsAny<ArgumentException>(() => new DataWithSignature(new byte[0], new byte[65536]));
+            Assert.ThrowsAny<ArgumentException>(() => new DataWithSignature(new byte[0], new byte[100000]));
+            new DataWithSignature(new byte[0], new byte[0]);
+            new DataWithSignature(new byte[100], new byte[100]);
+            new DataWithSignature(new byte[1000], new byte[65535]);
+            new DataWithSignature(new byte[100000], new byte[65535]);
+        }
+    }
+}

Tests/ContentTransformer/ClearText/EncryptorAndHasherTests.cs

@@ -1,5 +1,6 @@
 namespace EternalArrowBackup.ContentTransformer.ClearText.Tests
 {
+    using System;
     using System.Text;
     using System.Threading.Tasks;
     using EternalArrowBackup.Hasher.Sha256;
@@ -26,11 +27,13 @@
             encrypted[0]++;
             decryptionResult = await encryptor.GetOriginalData(encrypted);
             Assert.False(decryptionResult.IsSuccessful);
+            Assert.Throws<NotImplementedException>(() => decryptionResult.Data);
 
             encrypted[0]--;
             encrypted[encrypted.Length - 1]--;
             decryptionResult = await encryptor.GetOriginalData(encrypted);
             Assert.False(decryptionResult.IsSuccessful);
+            Assert.Throws<NotImplementedException>(() => decryptionResult.Data);
         }
     }
 }