ci: run integration tests always on main, on all os

Author: AriPerkkio

.github/workflows/integration-test-cli.yaml

@@ -8,15 +8,29 @@ jobs:
   cli-integration-test:
     name: CLI Integration Tests
     # Note: `prepare-release.yaml` sets this commit message
-    if: ${{ contains(github.event.pull_request.title, 'release CLI') || github.event_name == 'workflow_dispatch' }}
-    runs-on: ubuntu-latest
+    if: ${{ contains(github.event.pull_request.title, 'release CLI') || github.event_name == 'workflow_dispatch' || github.ref_name == 'main' }}
+
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest]
+        node-version: [18, 20, 22]
+        include:
+          - os: macos-latest
+            node_version: 20
+          - os: windows-latest
+            node_version: 20.13.1
+
     steps:
       - name: Checkout
         uses: actions/checkout@v4
         with:
           fetch-depth: 0
 
       - uses: ./.github/actions/setup-and-build
+        with:
+          node-version: ${{ matrix.node-version }}
 
       - name: Update template's versions
         working-directory: ./packages/cli

integration/cli/create-tutorial.test.ts

@@ -4,17 +4,16 @@ import { execa } from 'execa';
 import { temporaryDirectory } from 'tempy';
 import { describe, beforeEach, afterAll, expect, it } from 'vitest';
 
+const isWindows = process.platform === 'win32';
 const baseDir = path.resolve(__dirname, '../..');
-
 const cli = path.join(baseDir, 'packages/cli/dist/index.js');
+const tmp = temporaryDirectory();
 
 interface TestContext {
   projectName: string;
   dest: string;
 }
 
-const tmp = temporaryDirectory();
-
 beforeEach<TestContext>(async (context) => {
   context.projectName = Math.random().toString(36).substring(7);
   context.dest = path.join(tmp, context.projectName);
@@ -35,7 +34,7 @@ describe.each(['npm', 'pnpm', 'yarn'])('%s', (packageManager) => {
     expect(filesToJSON(projectFiles)).toMatchFileSnapshot(`${snapshotPrefix}-created.json`);
   });
 
-  it<TestContext>('should create and build a project', async ({ projectName, dest }) => {
+  it.skipIf(isWindows)<TestContext>('should create and build a project', async ({ projectName, dest }) => {
     await createProject(projectName, packageManager, { cwd: tmp, install: true });
 
     await execa(packageManager, ['run', 'build'], {
@@ -50,19 +49,22 @@ describe.each(['npm', 'pnpm', 'yarn'])('%s', (packageManager) => {
     expect(filesToJSON(distFiles)).toMatchFileSnapshot(`${snapshotPrefix}-built.json`);
   });
 
-  it<TestContext>('created project contains overwritten UnoCSS config', async ({ projectName, dest }) => {
-    await createProject(projectName, packageManager, { cwd: tmp });
+  it.skipIf(isWindows)<TestContext>(
+    'created project contains overwritten UnoCSS config',
+    async ({ projectName, dest }) => {
+      await createProject(projectName, packageManager, { cwd: tmp });
 
-    const unoConfig = await fs.readFile(`${dest}/uno.config.ts`, 'utf8');
+      const unoConfig = await fs.readFile(`${dest}/uno.config.ts`, 'utf8');
 
-    expect(unoConfig).toBe(`\
+      expect(unoConfig).toBe(`\
 import { defineConfig } from '@tutorialkit/theme';
 
 export default defineConfig({
   // add your UnoCSS config here: https://unocss.dev/guide/config-file
 });
 `);
-  });
+    },
+  );
 });
 
 async function createProject(name: string, packageManager: string, options: { cwd: string; install?: boolean }) {

integration/theme-resolving/inline-content.test.ts

@@ -26,6 +26,5 @@ test('getInlineContentForPackage finds files from @tutorialkit/astro', async ()
     root: `${tmp}/theme-test`,
   });
 
-  console.log(content);
   expect(content.length).toBeGreaterThan(0);
 });