Include skipped diff and empty lines from active revision when diffing (

#9928)

src/dotnet/APIView/APIViewUnitTests/APIViewUnitTests.csproj

@@ -51,6 +51,12 @@
     <None Update="SampleTestFiles\azure-core-1.47.0-sources4.json">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>
+    <None Update="SampleTestFiles\azure-schemaregistry_python.json">
+      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
+    </None>
+    <None Update="SampleTestFiles\azure-schemaregistry_python_diff.json">
+      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
+    </None>
     <None Update="SampleTestFiles\azure.data.tables.12.9.0_doc_change.json">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>

src/dotnet/APIView/APIViewUnitTests/CodeFileTests.cs

@@ -185,5 +185,35 @@ public async Task TestCodeFileComparisonWithChangeInSkippedLines()
             var diff = CodeFileHelpers.FindDiff(codeFileA.ReviewLines, codeFileB.ReviewLines);
             Assert.False(FindAnyDiffLine(diff));
         }
+
+        [Fact]
+        public async Task VerifyPythonDiff()
+        {
+            var codeFileA = new CodeFile();
+            var codeFileB = new CodeFile();
+            var filePath = Path.Combine("SampleTestFiles", "azure-schemaregistry_python.json");
+            var fileInfo = new FileInfo(filePath);
+            var fileStream = fileInfo.Open(FileMode.Open, FileAccess.Read, FileShare.Read);
+            codeFileA = await CodeFile.DeserializeAsync(fileStream);
+            filePath = Path.Combine("SampleTestFiles", "azure-schemaregistry_python_diff.json");
+            fileInfo = new FileInfo(filePath);
+            fileStream = fileInfo.Open(FileMode.Open, FileAccess.Read, FileShare.Read);
+            codeFileB = await CodeFile.DeserializeAsync(fileStream);
+            bool isSame = CodeFileHelpers.AreCodeFilesSame(codeFileA, codeFileB);
+            Assert.False(isSame);
+
+            var diff = CodeFileHelpers.FindDiff(codeFileA.ReviewLines, codeFileB.ReviewLines);
+            Assert.True(FindAnyDiffLine(diff));
+
+            //Verify first line in diff view is the global text
+            Assert.True(diff.First().LineId == "GLOBAL");
+
+            //Verify that last line of namespace line's children is empty line
+            var namespaceLine = diff.First(l=>l.LineId == "azure.schemaregistry");
+            Assert.Equal("namespace azure.schemaregistry", namespaceLine.ToString());
+
+            Assert.True(namespaceLine.Children.Last().Tokens.Count == 0);
+
+        }
     }
 }