Extract converter.yml and add tests.

Author: pawchen

.github/workflows/converter.yml

@@ -0,0 +1,50 @@
+name: Build the converter and optionally run it
+
+on:
+  workflow_call:
+    inputs:
+      test:
+        type: boolean
+      run:
+        type: boolean
+      is_index:
+        type: boolean
+
+jobs:
+  
+  build:
+    env:
+      build_config: Release
+      book_folder: ${{ github.event.repository.name }}${{ case(inputs.is_index, '/index', '') }}
+    runs-on: ubuntu-latest
+    steps:
+      - name: Set $build_config_lower
+        run: echo "build_config_lower=${build_config@L}" >> "$GITHUB_ENV"
+      - name: Checkout Book
+        if: ${{ run && !inputs.is_index }}
+        uses: actions/checkout@v6
+        with:
+          path: ${{ env.book_folder }}
+      - name: Checkout Index with Converter
+        uses: actions/checkout@v6
+        with:
+          repository: originlab/originlab.github.io
+          path: originlab.github.io
+      - uses: actions/setup-dotnet@v5
+        with:
+          dotnet-version: '10.x'
+      #- run: ls -l -R
+      - name: Build converter
+        run: dotnet build -c $build_config originlab.github.io/converter/generator/generator.csproj
+      - name: Test converter
+        if: ${{ inputs.test }}
+        run: dotnet run -c $build_config --project originlab.github.io/converter/tests/tests.csproj
+      - name: Run Converter
+        if: ${{ inputs.run }}
+        run: dotnet "originlab.github.io/converter/artifacts/bin/generator/$build_config_lower/generator.dll" "$book_folder" ${{ case(inputs.is_index, 'merge', '') }}
+      - name: Upload pages artifact
+        if: ${{ inputs.run }}
+        uses: actions/upload-pages-artifact@v5
+        with:
+          path: public_html
+

.github/workflows/publish_book.yml

@@ -13,30 +13,11 @@ concurrency:
 jobs:
 
   build:
-    env:
-      book_folder: ${{ github.event.repository.name }}${{ case(inputs.is_index, '/index', '') }}
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout Book
-        if: ${{ !inputs.is_index }}
-        uses: actions/checkout@v6
-        with:
-          path: ${{ env.book_folder }}
-      - name: Checkout Index with Converter
-        uses: actions/checkout@v6
-        with:
-          repository: originlab/originlab.github.io
-          path: originlab.github.io
-      - uses: actions/setup-dotnet@v5
-        with:
-          dotnet-version: '10.x'
-      #- run: ls -l -R
-      - name: Run Converter
-        run: dotnet run -c Release --project originlab.github.io/converter/generator/generator.csproj -- ${{ env.book_folder }} ${{ case(inputs.is_index, 'merge', '') }}
-      - name: Upload pages artifact
-        uses: actions/upload-pages-artifact@v5
-        with:
-          path: public_html
+    uses: ./.github/workflows/converter.yml
+    with:
+      test: ${{ inputs.is_index }}
+      run: true
+      is_index: ${{ inputs.is_index }}
 
   deploy:
     needs: build

converter/converter.slnx

@@ -4,10 +4,12 @@
     <File Path=".editorconfig" />
   </Folder>
   <Folder Name="/Solution Items/Workflows/">
+    <File Path="../.github/workflows/converter.yml" />
     <File Path="../.github/workflows/deploy.yml" />
     <File Path="../.github/workflows/publish_book.yml" />
   </Folder>
   <Project Path="generator/generator.csproj" />
   <Project Path="templates/templates.csproj" />
+  <Project Path="tests/tests.csproj" Id="0ced4798-e7da-48fa-8462-9c8c97036502" />
   <Project Path="utils/utils.csproj" Id="a84489a3-d5e2-4908-b1a4-6947fab586d4" />
 </Solution>

converter/tests/UtilsTests.cs

@@ -0,0 +1,35 @@
+namespace OriginLab.DocumentGeneration.Tests;
+
+public class UtilsTests
+{
+    [Theory]
+    [InlineData("", new string[0])]
+    [InlineData("", new string?[] { null })]
+    [InlineData("", new[] { "" })]
+    [InlineData("", new[] { "", null })]
+    [InlineData("/a", new[] { "a", null })]
+    [InlineData("/a", new[] { "a", "" })]
+    [InlineData("/a/b", new[] { "a", "b" })]
+    [InlineData("/b", new[] { null, "b" })]
+    [InlineData("/b", new[] { "", "b" })]
+    public void TryPrefixEachSkipsEmptyItems(string expected, string?[] items)
+    {
+        Assert.Equal(expected, '/'.TryPrefixEach(items));
+    }
+
+    [Theory]
+    [InlineData("", new string[0])]
+    [InlineData("", new string?[] { null })]
+    [InlineData("", new[] { "" })]
+    [InlineData("", new[] { "", null })]
+    [InlineData("/a/", new[] { "a", null })]
+    [InlineData("/a/", new[] { "a", "" })]
+    [InlineData("/a/b/", new[] { "a", "b" })]
+    [InlineData("/b/", new[] { null, "b" })]
+    [InlineData("/b/", new[] { "", "b" })]
+    public void TrySurroundEachSkipsEmptyItems(string expected, string?[] items)
+    {
+        Assert.Equal(expected, '/'.TrySurroundEach(items));
+    }
+
+}

converter/tests/tests.csproj

@@ -0,0 +1,23 @@
+<Project Sdk="Microsoft.NET.Sdk">
+
+  <PropertyGroup>
+    <OutputType>Exe</OutputType>
+    <IsPackable>false</IsPackable>
+    <IsTestProject>true</IsTestProject>
+    <RootNamespace>OriginLab.DocumentGeneration.Tests</RootNamespace>
+  </PropertyGroup>
+
+  <ItemGroup>
+    <PackageReference Include="xunit.v3" Version="1.0.1" />
+    <PackageReference Include="Microsoft.Testing.Extensions.CodeCoverage" Version="17.14.1" />
+  </ItemGroup>
+
+  <ItemGroup>
+    <ProjectReference Include="..\generator\generator.csproj" />
+  </ItemGroup>
+
+  <ItemGroup>
+    <Using Include="Xunit" />
+  </ItemGroup>
+
+</Project>