Minor test coverage improvements

Author: mstefarov

fNbt.Test/CompoundTests.cs

@@ -107,6 +107,7 @@ public void GettersAndSetters() {
             Assert.Throws<InvalidOperationException>(() => nestedInt[0] = nestedInt);
 
             // Trying to use string indexers on non-NbtCompound tags
+            Assert.Throws<InvalidOperationException>(() => dummyTag = childList["test"]);
             Assert.Throws<InvalidOperationException>(() => childList["test"] = nestedInt);
             Assert.Throws<InvalidOperationException>(() => nestedInt["test"] = nestedInt);
 

fNbt.Test/NbtFileTests.cs

@@ -233,6 +233,7 @@ public void ReadRootTag() {
 
         void ReadRootTagInternal(String fileName, NbtCompression compression) {
             Assert.Throws<ArgumentOutOfRangeException>(() => NbtFile.ReadRootTagName(fileName, compression, true, -1));
+            Assert.Throws<ArgumentOutOfRangeException>(() => NbtFile.ReadRootTagName(fileName, (NbtCompression)255, true, 0));
 
             Assert.AreEqual("Level", NbtFile.ReadRootTagName(fileName));
             Assert.AreEqual("Level", NbtFile.ReadRootTagName(fileName, compression, true, 0));

fNbt.Test/NbtReaderTests.cs

@@ -290,6 +290,9 @@ public void ReadToDescendantTest() {
             // Ensure ReadToDescendant returns false when at end-of-stream
             while (reader.ReadToFollowing()) { }
             Assert.IsFalse(reader.ReadToDescendant("*"));
+
+            // Ensure that this works even on the root
+            Assert.IsFalse(new NbtReader(TestFiles.MakeReaderTest()).ReadToDescendant("*"));
         }
 
 

fNbt/NbtFile.cs

@@ -501,8 +501,7 @@ public long SaveToStream([NotNull] Stream stream, NbtCompression compression) {
                     RootTag.WriteTag(writer);
                     break;
 
-                default:
-                    throw new ArgumentOutOfRangeException("compression");
+                // Can't be AutoDetect or unknown: parameter is already validated
             }
 
             if (stream.CanSeek) {