Test and fix PatchProgress.ProgressFraction

Patches are now only counted after needs are checked, so this shouldn't
consider needs unsatisfied nodes

ModuleManager/Progress/PatchProgress.cs

@@ -16,7 +16,7 @@ public float ProgressFraction
             get
             {
                 if (Counter.totalPatches > 0)
-                    return (Counter.appliedPatches + Counter.needsUnsatisfied) / (float)Counter.totalPatches;
+                    return Counter.appliedPatches / (float)Counter.totalPatches;
                 return 0;
             }
         }

ModuleManagerTests/Progress/PatchProgressTest.cs

@@ -276,5 +276,37 @@ public void TestException__Url()
             Assert.Equal(2, progress.Counter.errorFiles["abc/def.cfg"]);
             logger.Received().Exception("An exception was tossed", e2);
         }
+
+        [Fact]
+        public void TestProgressFraction()
+        {
+            Assert.Equal(0, progress.ProgressFraction);
+
+            progress.Counter.needsUnsatisfied.Increment();
+            progress.Counter.needsUnsatisfied.Increment();
+
+            progress.Counter.totalPatches.Increment();
+            progress.Counter.totalPatches.Increment();
+            progress.Counter.totalPatches.Increment();
+            progress.Counter.totalPatches.Increment();
+
+            Assert.Equal(0, progress.ProgressFraction);
+
+            progress.Counter.appliedPatches.Increment();
+
+            Assert.Equal(0.25, progress.ProgressFraction);
+
+            progress.Counter.appliedPatches.Increment();
+
+            Assert.Equal(0.5, progress.ProgressFraction);
+
+            progress.Counter.appliedPatches.Increment();
+
+            Assert.Equal(0.75, progress.ProgressFraction);
+
+            progress.Counter.appliedPatches.Increment();
+
+            Assert.Equal(1, progress.ProgressFraction);
+        }
     }
 }