Merge pull request #1101 from lahma/install-fonts

Install required font on Linux CI run
nissl-lab · Jun 24, 2023 · fac36d0 · fac36d0
2 parents 8469d3d + 31f9508
commit fac36d0
Show file tree

Hide file tree

Showing 6 changed files with 27 additions and 4 deletions.
diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml
@@ -26,6 +26,7 @@ jobs:
   windows-latest:
     name: windows-latest
     runs-on: windows-latest
+    timeout-minutes: 20
     steps:
       - uses: actions/checkout@v3
       - name: 'Cache: .nuke/temp, ~/.nuget/packages'
@@ -45,6 +46,7 @@ jobs:
   ubuntu-latest:
     name: ubuntu-latest
     runs-on: ubuntu-latest
+    timeout-minutes: 20
     steps:
       - uses: actions/checkout@v3
       - name: 'Cache: .nuke/temp, ~/.nuget/packages'

diff --git a/.github/workflows/PR.yml b/.github/workflows/PR.yml
@@ -22,6 +22,7 @@ jobs:
   windows-latest:
     name: windows-latest
     runs-on: windows-latest
+    timeout-minutes: 20
     steps:
       - uses: actions/checkout@v3
       - name: 'Cache: .nuke/temp, ~/.nuget/packages'
@@ -41,6 +42,7 @@ jobs:
   ubuntu-latest:
     name: ubuntu-latest
     runs-on: ubuntu-latest
+    timeout-minutes: 20
     steps:
       - uses: actions/checkout@v3
       - name: 'Cache: .nuke/temp, ~/.nuget/packages'

diff --git a/.nuke/build.schema.json b/.nuke/build.schema.json
@@ -79,6 +79,7 @@
             "enum": [
               "Clean",
               "Compile",
+              "InstallFonts",
               "Pack",
               "Restore",
               "Test"
@@ -97,6 +98,7 @@
             "enum": [
               "Clean",
               "Compile",
+              "InstallFonts",
               "Pack",
               "Restore",
               "Test"

diff --git a/build/Build.GitHubAction.cs b/build/Build.GitHubAction.cs
@@ -4,13 +4,15 @@
     GitHubActionsImage.WindowsLatest,
     GitHubActionsImage.UbuntuLatest,
     OnPushBranches = new[] { "main", "master" },
-    InvokedTargets = new[] { nameof(Clean), nameof(Test), nameof(Pack) }
+    InvokedTargets = new[] { nameof(Clean), nameof(Test), nameof(Pack) },
+    TimeoutMinutes = 20
 )]
 [GitHubActions("PR",
     GitHubActionsImage.WindowsLatest,
     GitHubActionsImage.UbuntuLatest,
     On = new [] { GitHubActionsTrigger.PullRequest },
-    InvokedTargets = new[] { nameof(Clean), nameof(Test), nameof(Pack) }
+    InvokedTargets = new[] { nameof(Clean), nameof(Test), nameof(Pack) },
+    TimeoutMinutes = 20
 )]
 partial class Build
 {

diff --git a/build/Build.cs b/build/Build.cs
@@ -74,7 +74,7 @@ protected override void OnBuildInitialized()
         });
 
     Target Test => _ => _
-        .DependsOn(Compile)
+        .DependsOn(Compile, InstallFonts)
         .Executes(() =>
         {
             DotNetTest(_ =>_
@@ -87,6 +87,17 @@ protected override void OnBuildInitialized()
             );
         });
 
+    Target InstallFonts => _ => _
+        .OnlyWhenDynamic(() => RuntimeInformation.IsOSPlatform(OSPlatform.Linux) && Host is GitHubActions)
+        .Executes(() =>
+        {
+            ProcessTasks.StartProcess("sudo", "apt install -y fonts-noto-color-emoji");
+            ProcessTasks.StartProcess("mkdir", "-p /usr/local/share/fonts");
+            ProcessTasks.StartProcess("cp", "/usr/share/fonts/truetype/noto/NotoColorEmoji.ttf /usr/local/share/fonts/");
+            ProcessTasks.StartProcess("chmod", "644 /usr/local/share/fonts/NotoColorEmoji.ttf");
+            ProcessTasks.StartProcess("fc-cache", "-fv");
+        });
+
     Target Pack => _ => _
         .After(Test)
         .Produces(ArtifactsDirectory / "**")

diff --git a/testcases/main/HSSF/UserModel/TestHSSFSheet.cs b/testcases/main/HSSF/UserModel/TestHSSFSheet.cs
@@ -20,6 +20,7 @@ namespace TestCases.HSSF.UserModel
     using System.IO;
     using System;
     using System.Configuration;
+    using System.Runtime.InteropServices;
     using NPOI.HSSF.UserModel;
     using NPOI.HSSF.Model;
     using NPOI.HSSF.Record;
@@ -853,7 +854,10 @@ public void TestAutoSizeRow()
             sheet.AutoSizeRow(row.RowNum);
 
             Assert.AreNotEqual(100, row.Height);
-            Assert.AreEqual(506, row.Height);
+
+            // there's slight difference due to fonts
+            var expectedRowHeight = RuntimeInformation.IsOSPlatform(OSPlatform.Windows) ? 506 : 528;
+            Assert.AreEqual(expectedRowHeight, row.Height);
 
             workbook.Close();
         }