Setup integration tests

Author: ericdallo

.github/workflows/test.yml

@@ -43,8 +43,106 @@ jobs:
       - name: Install Clojure
         uses: DeLaGuardo/setup-clojure@master
         with:
-          cli: '1.12.0.1495'
-          bb: '0.9.161'
+          cli: '1.12.0.1530'
+          bb: '1.12.200'
 
       - name: Run tests
         run: bb test
+  jvm-integration-test:
+    runs-on: ${{matrix.os}}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-22.04]
+        jdk: [17]
+        include:
+          - os: windows-latest
+            jdk: 17
+          - os: macos-latest
+            jdk: 17
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Set up JDK ${{ matrix.jdk }}
+        uses: actions/setup-java@v3
+        with:
+          distribution: 'adopt'
+          java-version: ${{ matrix.jdk }}
+
+      - name: Install clojure build tools
+        uses: DeLaGuardo/setup-clojure@master
+        with:
+          cli: '1.12.0.1530'
+          bb: '1.12.200'
+
+      - name: Generate embedded binary
+        run: bb prod-cli
+
+      - name: Run integration tests
+        run: bb integration-test
+
+  graalvm-build:
+    runs-on: ubuntu-22.04
+    strategy:
+      fail-fast: false
+    steps:
+      - uses: actions/checkout@v4
+      - name: Install Clojure
+        uses: DeLaGuardo/setup-clojure@master
+        with:
+          cli: '1.12.0.1530'
+          bb: '1.12.200'
+
+      - uses: graalvm/setup-graalvm@v1
+        with:
+          java-version: '21'
+          distribution: 'graalvm'
+          native-image-musl: 'true'
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+
+      - name: Build static Linux native image
+        env:
+          ECA_JAR: "eca.jar"
+          ECA_XMX: "-J-Xmx6g"
+          ECA_STATIC: true
+          ECA_MUSL: true
+        run: |
+          bb native-cli
+
+      - name: Compress binary
+        uses: svenstaro/upx-action@v2
+        with:
+          file: eca
+
+      - name: Upload
+        uses: actions/upload-artifact@v4
+        with:
+          name: eca-native
+          path: eca
+
+  graalvm-integration-test:
+    needs: graalvm-build
+    runs-on: ubuntu-22.04
+    strategy:
+      fail-fast: false
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Install clojure build tools
+        uses: DeLaGuardo/setup-clojure@master
+        with:
+          cli: '1.12.0.1530'
+          bb: '1.12.200'
+
+      - uses: actions/download-artifact@v4
+        name: eca-native
+
+      - name: Setup binary
+        run: |
+          cp -rf eca-native/eca eca
+          chmod +x eca
+
+      - name: Run integration tests
+        run: bb integration-test

bb.edn

@@ -7,10 +7,11 @@
  :tasks {debug-cli make/debug-cli
          debug-graal make/debug-graal
          prod-jar make/prod-jar
+         prod-cli make/prod-cli
          native-cli make/native-cli
 
          test make/unit-test
 
          tag make/tag
          get-last-changelog-entry make/get-last-changelog-entry
-         integration-test entrypoint/run-all}}
+         integration-test make/integration-test}}

build.clj

@@ -65,6 +65,10 @@
 (defn prod-jar [opts]
   (aot-jar (merge opts {:extra-aliases [:native]})))
 
+(defn prod-cli [opts]
+  (aot-jar opts)
+  (bin {}))
+
 (defn native-cli [opts]
   (println "Building native image...")
   (if-let [graal-home (System/getenv "GRAALVM_HOME")]

integration-test/entrypoint.clj

@@ -1,6 +1,7 @@
 (ns entrypoint
   (:require
-   [clojure.test :as t]))
+   [clojure.test :as t]
+   [integration.eca :as eca]))
 
 (def namespaces
   '[integration.initialize-test])
@@ -27,11 +28,8 @@
      ~@body))
 
 #_{:clojure-lsp/ignore [:clojure-lsp/unused-public-var]}
-(defn run-all [& args]
-  (when-not (first args)
-    (println "First arg must be path to eca binary")
-    (System/exit 0))
-
+(defn run-all [binary]
+  (alter-var-root #'eca/*eca-binary-path* (constantly binary))
   (apply require namespaces)
 
   (let [timeout-minutes (if (re-find #"(?i)win|mac" (System/getProperty "os.name"))

integration-test/integration/eca.clj

@@ -6,6 +6,7 @@
    [clojure.test :refer [use-fixtures]]
    [integration.client :as client]))
 
+(def ^:dynamic *eca-binary-path* nil)
 (def ^:dynamic *eca-process* nil)
 (def ^:dynamic *mock-client* nil)
 
@@ -17,7 +18,7 @@
               {:dir "integration-test/sample-test/"})))
 
 (defn start-process! []
-  (let [server (start-server (first *command-line-args*))
+  (let [server (start-server *eca-binary-path*)
         client (client/client (:in server) (:out server))]
     (client/start client nil)
     (async/go-loop []

scripts/make.clj

@@ -4,7 +4,8 @@
    [babashka.fs :as fs]
    [babashka.process :as p]
    [babashka.tasks :refer [shell]]
-   [clojure.string :as string]))
+   [clojure.string :as string]
+   [entrypoint]))
 
 (def windows? (#'fs/windows?))
 
@@ -59,6 +60,10 @@
   (build "prod-jar")
   (mv-here "target/eca.jar"))
 
+(defn prod-cli []
+  (build "prod-cli")
+  (mv-here (fs/path (eca-bin-filename :script))))
+
 (defn native-cli
   "Build the native `eca[.exe]` cli executable with `graalvm`."
   []
@@ -86,3 +91,18 @@
   (println :running-unit-tests...)
   (clj! ["-M:test"])
   (println))
+
+(defn integration-test
+  "Run the integration tests in 'test/integration-test/' using `./eca[.bat|.exe]`.
+
+  There should only be one eca executable found, throws error
+  otherwise."
+  []
+  (let [eca-bins (->> [:native :script] (map eca-bin-filename) distinct)
+        eca-bins-found (->> eca-bins (filter fs/exists?) (into #{}))]
+
+    (case (count eca-bins-found)
+      0 (throw (ex-info "No eca executables found." {:searched-for eca-bins}))
+      1 (entrypoint/run-all (str (first eca-bins-found)))
+      (throw (ex-info "More than one eca executables found. Can only work with one."
+                      {:bin-found eca-bins-found})))))