Run tests in CI (#290)

Co-authored-by: Varun Saini <61795485+vrn-sn@users.noreply.github.com>

Author: Nicell

.github/actions/setup-and-build/action.yml

@@ -1,7 +1,10 @@
-name: "Setup and Build Lute"
-description: "Install dependencies, configure and build Lute"
+name: "Setup and Build"
+description: "Install dependencies, configure and build target"
 
 inputs:
+  target:
+    description: "Which target to build"
+    required: true
   config:
     description: "Build configuration (debug or release)"
     required: false
@@ -57,23 +60,23 @@ runs:
         path: extern
         key: extern-${{ hashFiles('extern/*.tune') }}
 
-    - name: Fetch Lute
+    - name: Fetch ${{ inputs.target }}
       if: steps.cache-extern.outputs.cache-hit != 'true'
       shell: bash
-      run: lute ./tools/luthier.luau fetch lute
+      run: lute ./tools/luthier.luau fetch ${{ inputs.target }}
 
-    - name: Configure Lute
+    - name: Configure ${{ inputs.target }}
       shell: bash
-      run: lute ./tools/luthier.luau configure lute --config ${{ inputs.config }} ${{ inputs.options }}
+      run: lute ./tools/luthier.luau configure ${{ inputs.target }} --config ${{ inputs.config }} ${{ inputs.options }}
 
-    - name: Build Lute
+    - name: Build ${{ inputs.target }}
       shell: bash
-      run: lute ./tools/luthier.luau build lute --config ${{ inputs.config }} ${{ inputs.options }}
+      run: lute ./tools/luthier.luau build ${{ inputs.target }} --config ${{ inputs.config }} ${{ inputs.options }}
 
     - name: Get Artifact Path
       id: artifact_path
       shell: bash
       run: |
-        exe_path=$(lute ./tools/luthier.luau build lute --config ${{ inputs.config }} --which)
+        exe_path=$(lute ./tools/luthier.luau build ${{ inputs.target }} --config ${{ inputs.config }} --which)
         echo "Executable path: $exe_path"  # Debug print
         echo "exe_path=$exe_path" >> "$GITHUB_OUTPUT"

.github/workflows/ci.yml

@@ -28,10 +28,30 @@ jobs:
       - name: Setup and Build Lute
         uses: ./.github/actions/setup-and-build
         with:
+          target: Lute.CLI
           config: debug
           options: ${{ matrix.options }}
           token: ${{ secrets.GITHUB_TOKEN }}
 
+  test:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Setup and Build Tests
+        id: build_tests
+        uses: ./.github/actions/setup-and-build
+        with:
+          target: Lute.Test
+          config: debug
+          options: --c-compiler clang --cxx-compiler clang++
+          token: ${{ secrets.GITHUB_TOKEN }}
+
+      - name: Run Tests
+        run: ${{ steps.build_tests.outputs.exe_path }}
+
   check-format:
     runs-on: ubuntu-latest
 

.github/workflows/release.yml

@@ -80,6 +80,7 @@ jobs:
       id: build_lute
       uses: ./.github/actions/setup-and-build
       with:
+        target: Lute.CLI
         config: release
         options: ${{ matrix.options }}
         token: ${{ secrets.GITHUB_TOKEN }}

tests/src/require.test.cpp

@@ -6,6 +6,8 @@
 #include "Luau/Require.h"
 #include "lualib.h"
 
+#include <memory>
+
 class LuaStateFixture
 {
 public:

tools/luthier.luau

@@ -271,18 +271,14 @@ local function call(cmd: { string }, passedCwd: string?): number
 end
 
 local function build()
-	local targetName = args:get("target")
+	local exeName = getExeName(args:get("target"))
 	local projectPath = getProjectPath()
 
-	if isWindows then
-		targetName ..= ".exe"
-	end
-
 	if args:has("clean") then
 		call({ "ninja", "-C", projectPath, "clean" })
 	end
 
-	local cmd = { "ninja", "-C", projectPath, targetName }
+	local cmd = { "ninja", "-C", projectPath, exeName }
 
 	return call(cmd)
 end

tools/luthier.py

@@ -313,20 +313,17 @@ def call(*args):
     return sp.call(*args, env=env)
 
 def build(args):
-    targetName = args.target
+    exeName = getExeName(args.target)
     projectPath = getProjectPath(args)
 
-    if isWindows:
-        targetName = targetName + '.exe'
-
     if args.clean:
         call(['ninja', '-C', projectPath, 'clean'])
 
     cmd = [
         'ninja',
         '-C',
         projectPath,
-        targetName
+        exeName
     ]
 
     return call(cmd)