ksconf sort: fix output glitch & glob unit test

- Re-enable sort unittest for glob expansion for #59.  Support for this feature
  was added in v0.7.7 and the test should have been reintroduced at that time.
- Fix confusing output message.  Both "Error trying to process file" and
  "Replaced file" messages were incorrectly shown at the same time.

docs/source/changelog.rst

@@ -50,6 +50,12 @@ More changes:
    Use ``{{layers_list}}`` to capture what unique layer names made their way into a tarball, and
    use ``{{layers_hash}}`` when that list get too long to be manageable.
 
+Bug fixes:
+
+*  Fixed ``sort`` bug where the user was incorrectly told that a file with errors was unexpectedly also successfully replaced.
+   The contradictory output messages have been cleaned up.
+   For clarity, this only occurred for inline replacement mode, and was purely an output issue, not a file handling problem.
+
 
 Ksconf 0.9
 ----------

ksconf/commands/sort.py

@@ -100,6 +100,7 @@ def run(self, args):
                     self.stderr.write("Error trying to process file {0}.  "
                                       "Error:  {1}\n".format(conf, e))
                     failure = True
+                    continue
                 if smart_rc == SMART_NOCHANGE:
                     if not args.quiet:
                         self.stderr.write("Nothing to update.  "

tests/test_cli_sort.py

@@ -71,16 +71,13 @@ def test_sort_inplace_returncodes(self):
             self.assertEqual(ko.returncode, EXIT_CODE_SUCCESS)
             self.assertRegex(ko.stderr, "^Nothing to update")
 
-    r''' # Leaving this enabled makes too much noise...
-    @unittest.expectedFailure
     def test_sort_glob(self):
-        # Not yet implemented.  Currently relying on the shell to do this.
+        # Implemented in 0.7.7 for Windows.  Don't rely on shell for expansion
         glob_pattern = self.twd.get_path("*.conf")
         with ksconf_cli:
             ko = ksconf_cli("sort", "-i", glob_pattern)
             self.assertEqual(ko.returncode, EXIT_CODE_BAD_CONF_FILE)
-            self.assertRegex(ko.stderr, r"badfile\.conf")
-    '''
+            self.assertRegex(ko.stderr, r"(?i)error[^\r\n]+badfile\.conf")
 
     def test_bad_file(self):
         with ksconf_cli: