add linux CI

[jmarolf]

No description provided.

[codecov]

Codecov Report

Merging #4444 (a4bf6dd) into master (46c1e52) will decrease coverage by 0.01%.
The diff coverage is 76.00%.

@@            Coverage Diff             @@
##           master    #4444      +/-   ##
==========================================
- Coverage   95.85%   95.84%   -0.02%     
==========================================
  Files        1179     1182       +3     
  Lines      268733   268752      +19     
  Branches    16224    16226       +2     
==========================================
- Hits       257607   257575      -32     
- Misses       9062     9104      +42     
- Partials     2064     2073       +9     

[jmarolf]

@mavasani looking at the test results it appears e have some tests that rely on windows style paths

[omajid]

Fixing some hardcoded windows paths makes the tests run work much better for me:

diff --git a/src/Test.Utilities/CSharpCodeFixVerifier`2+Test.cs b/src/Test.Utilities/CSharpCodeFixVerifier`2+Test.cs
index 23b6ec5e4..f8d221d5a 100644
--- a/src/Test.Utilities/CSharpCodeFixVerifier`2+Test.cs
+++ b/src/Test.Utilities/CSharpCodeFixVerifier`2+Test.cs
@@ -55,7 +55,7 @@ public Test()
                             DocumentId.CreateNewId(projectId, debugName: ".editorconfig"),
                             ".editorconfig",
                             SourceText.From($"is_global = true" + Environment.NewLine + AnalyzerConfigDocument),
-                            filePath: @"z:\.editorconfig");
+                            filePath: @"/tmp/.editorconfig");
                     }
 
                     return solution;
diff --git a/src/Test.Utilities/VisualBasicCodeFixVerifier`2+Test.cs b/src/Test.Utilities/VisualBasicCodeFixVerifier`2+Test.cs
index 6c0074678..1c76fcd64 100644
--- a/src/Test.Utilities/VisualBasicCodeFixVerifier`2+Test.cs
+++ b/src/Test.Utilities/VisualBasicCodeFixVerifier`2+Test.cs
@@ -33,7 +33,7 @@ public Test()
                             DocumentId.CreateNewId(projectId, debugName: ".editorconfig"),
                             ".editorconfig",
                             SourceText.From($"is_global = true" + Environment.NewLine + AnalyzerConfigDocument),
-                            filePath: @"z:\.editorconfig");
+                            filePath: @"/tmp/.editorconfig");
                     }
 
                     return solution;

The paths should be conditionalized correctly based on OS.

Another set of failures is based on assuming the newlines in the file are always \r\n. On Linux, line-endings are \n. Some files are written to disk (and get \n line-endings) then the test fails because it expects \r\n. This seems to work:

diff --git a/src/PublicApiAnalyzers/UnitTests/DeclarePublicApiAnalyzerTests.cs b/src/PublicApiAnalyzers/UnitTests/DeclarePublicApiAnalyzerTests.cs
index 6b943c43e..191b4014e 100644
--- a/src/PublicApiAnalyzers/UnitTests/DeclarePublicApiAnalyzerTests.cs
+++ b/src/PublicApiAnalyzers/UnitTests/DeclarePublicApiAnalyzerTests.cs
@@ -2094,6 +2094,8 @@ public class {|RS0016:{|RS0016:C2|}|} { }
         [InlineData("\r\n\r\n", "\r\n")]
         public async Task TestPreserveTrailingNewline(string originalEndOfFile, string expectedEndOfFile)
         {
+            originalEndOfFile = originalEndOfFile.Replace("\r\n", Environment.NewLine, StringComparison.Ordinal);
+            expectedEndOfFile = expectedEndOfFile.Replace("\r\n", Environment.NewLine, StringComparison.Ordinal);
             var source = @"
 public class C
 {

[mavasani]

@jmarolf Are you blocked here?