Prevent migration changes to info.xml file from affecting selection of version when merging HearThisPacks

src/HearThis/Communication/WindowsLink.cs

@@ -60,6 +60,11 @@ public bool TryListFiles(string androidPath, out string list)
 			foreach (var file in Directory.EnumerateFiles(path, "*.*"))
 			{
 				var filename = Path.GetFileName(file);
+				// REVIEW: We need to consider whether/when changes to info.xml files
+				// might need to be regarded as significant for determining which
+				// version to use in merge, since the "Check For Problems" view makes
+				// it more likely for the info file to change without any clips being
+				// modified.
 				if (filename == "info.xml")
 					continue;
 				sb.Append(filename);

src/HearThis/Script/ChapterInfo.cs

@@ -415,7 +415,10 @@ private void Save(string filePath, bool preserveModifiedTime = false)
 				throw new Exception($"Unable to save {GetType().Name} file: " + filePath, error);
 
 			if (preserveModifiedTime)
+			{
 				finfo.LastWriteTimeUtc = modified;
+				finfo.Attributes |= FileAttributes.Archive;
+			}
 		}
 
 		public string ToXmlString()

src/HearThisTests/ClipRepositoryTests.cs

@@ -1642,13 +1642,14 @@ public void ShiftClipsAtOrAfterBlockIfAllClipsAreBeforeDate_AllFilesModifiedBefo
 			const int kTestChapter = 1;
 
 			var chapterFolder = ClipRepository.GetChapterFolder(testProject, kTestBook, kTestChapter);
-			ChapterRecordingInfoBase info;
+			TestChapterInfo info;
 			if (includeClip0)
 				info = new TestChapterInfo(1, 2, 3, 8); // Intentionally omitted 4, just to make sure the logic is okay with having one missing.
 			else
 				info = new TestChapterInfo(2, 3, 8); // Intentionally omitted 4, just to make sure the logic is okay with having one missing.
 			info.RecordingInfo[1].SkippedChanged += sender => { }; // code requires us to have a handler before we can set it.
 			info.RecordingInfo[1].Skipped = true;
+			info.ExpectedPreserveModifiedTime = true;
 
 			try
 			{
@@ -1663,13 +1664,15 @@ public void ShiftClipsAtOrAfterBlockIfAllClipsAreBeforeDate_AllFilesModifiedBefo
 				// SUT
 				Assert.IsTrue(ClipRepository.ShiftClipsAtOrAfterBlockIfAllClipsAreBeforeDate(
 					testProject, kTestBook, kTestChapter, 1, DateTime.UtcNow, () => info));
+
 				Assert.AreEqual(includeClip0 ? 5 : 4, Directory.GetFiles(chapterFolder).Length);
 				Assert.That(File.Exists(Path.Combine(chapterFolder, "8.wav")));
 				Assert.That(File.Exists(Path.Combine(chapterFolder, "4.wav")));
 				Assert.That(File.Exists(Path.Combine(chapterFolder, "3.skip")));
 				Assert.That(File.Exists(Path.Combine(chapterFolder, "2.wav")));
 				Assert.IsFalse(File.Exists(Path.Combine(chapterFolder, "1.wav")));
 				Assert.AreEqual(includeClip0, File.Exists(file0));
+				Assert.AreEqual(1, info.SaveCallCount);
 
 				int i = 0;
 				if (includeClip0)
@@ -2152,6 +2155,7 @@ private class TestChapterInfo : ChapterRecordingInfoBase
 			private readonly List<ScriptLine> _recordings;
 
 			public int SaveCallCount { get; private set; }
+			public bool ExpectedPreserveModifiedTime { get; set; }
 
 			public TestChapterInfo(params int[] scriptLineNumbers)
 			{
@@ -2168,6 +2172,7 @@ public override void OnScriptBlockRecorded(ScriptLine selectedScriptBlock)
 
 			public override void Save(bool preserveModifiedTime = false)
 			{
+				Assert.AreEqual(ExpectedPreserveModifiedTime, preserveModifiedTime);
 				SaveCallCount++;
 			}
 		}

src/HearThisTests/ScriptProviderBaseTests.cs

@@ -802,6 +802,7 @@ public override void OnScriptBlockRecorded(ScriptLine scriptBlock)
 
 			public override void Save(bool preserveModifiedTime = false)
 			{
+				Assert.IsTrue(preserveModifiedTime);
 				SaveCallCount++;
 			}
 		}