Setup CI for testing (scalapy#3)

* Setup CI for testing

* Integration tests

* Use coursier setup-action

Author: kiendang

.github/scripts/integration_test.sh

@@ -0,0 +1,4 @@
+#!/bin/sh
+
+sbt +publishLocal
+sbt -Dplugin.ci=true -Dplugin.python.executable="$1" -Dplugin.scalapy.version="$2" scripted

.github/workflows/ci.yml

@@ -0,0 +1,35 @@
+name: CI
+on:
+  push:
+    branches:
+      - main
+    tags:
+      - v*
+  pull_request:
+
+jobs:
+  test:
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest]
+        jdk: [8]
+        python: [3.7, 3.8, 3.9]
+    steps:
+      - uses: actions/checkout@master
+      - uses: coursier/[email protected]
+      - uses: coursier/setup-action@v1
+        with:
+          jvm: ${{ matrix.jdk }}
+          apps: sbtn
+      - name: Set up Python ${{ matrix.python }}
+        uses: actions/setup-python@v2
+        with:
+          python: ${{ matrix.python }}
+      - name: Unit tests
+        run: sbt +test
+        shell: bash
+      - name: Integration tests again latest ScalaPy stable
+        run: .github/scripts/integration_test.sh python3 0.5.0
+        shell: bash

build.sbt

@@ -1,7 +1,4 @@
 import Dependencies._
-import com.typesafe.sbt.SbtGit.git
-import java.nio.file.{Files, Paths}
-import java.io.File
 
 inThisBuild(
   List(
@@ -23,7 +20,9 @@ lazy val scala212 = "2.12.14"
 lazy val scala213 = "2.13.6"
 lazy val scala3   = "3.0.0"
 
-ThisBuild / scalaVersion := scala213
+lazy val enableScripted = Option(sys.props("plugin.ci")).map(_.trim.nonEmpty).getOrElse(false)
+
+ThisBuild / scalaVersion := (if (enableScripted) scala212 else scala213)
 
 ThisBuild / scalafixDependencies += organizeImports
 
@@ -33,7 +32,27 @@ def warnUnusedImports(scalaVersion: String) =
     case _            => Nil
   }
 
+def scriptedPlugin = if (enableScripted) Seq(ScriptedPlugin) else Nil
+
+def scriptedSettings = if (enableScripted) {
+  Seq(
+    scriptedLaunchOpts := {
+      scriptedLaunchOpts.value ++ {
+        Seq(s"-Dplugin.scalapy.version=$scalapyVersion") ++
+          Option(sys.props("plugin.python.executable"))
+            .map("-Dplugin.python.executable=" + _)
+            .toSeq ++
+          Seq("-Xmx1024M", "-Dplugin.version=" + version.value)
+      }
+    },
+    scriptedBufferLog := false
+  )
+} else Nil
+
+lazy val scalapyVersion = Option(sys.props("plugin.scalapy.version")).map(_.trim).getOrElse("0.5.0")
+
 lazy val root = (project in file("."))
+  .enablePlugins(scriptedPlugin: _*)
   .settings(
     name := "Python Native Libs",
     crossScalaVersions := Seq(scala212, scala213, scala3),
@@ -48,13 +67,14 @@ lazy val root = (project in file("."))
     semanticdbVersion := scalafixSemanticdb.revision,
     scalacOptions ++= warnUnusedImports(scalaVersion.value)
   )
+  .settings(scriptedSettings)
 
 lazy val docs = project
   .in(file("python-docs"))
   .settings(
     mdocVariables := Map(
       "VERSION"         -> "0.1.3",
-      "SCALAPY_VERSION" -> "0.5.0",
+      "SCALAPY_VERSION" -> scalapyVersion,
       "PYTHON"          -> "/usr/bin/python3"
     )
   )

project/build.properties

@@ -1 +1 @@
-sbt.version=1.5.2
+sbt.version=1.5.5

src/main/scala/ai/kien/python/Python.scala

@@ -51,7 +51,7 @@ class Python private[python] (
     nativeLibraryPaths <- nativeLibraryPaths
     libPathFlags = nativeLibraryPaths.map("-L" + _)
     flags = rawLdflags
-      .split("\\s+(?=-)")
+      .split("\\s+")
       .filter(f => f.nonEmpty && !libPathFlags.contains(f))
   } yield libPathFlags ++ flags
 

src/sbt-test/integration/scalapy/build.sbt

@@ -0,0 +1,35 @@
+import ai.kien.python.Python
+
+lazy val scala212 = "2.12.14"
+lazy val scala213 = "2.13.6"
+lazy val scala3   = "3.0.0"
+
+ThisBuild / scalaVersion := scala213
+
+lazy val scalapyVersion = sys.props("plugin.scalapy.version")
+
+lazy val python = Python(
+  Option(sys.props("plugin.python.executable")).filter(_.trim.nonEmpty)
+)
+
+lazy val pythonLdFlags = python.ldflags.get
+
+lazy val javaOpts = python.scalapyProperties.get.map { case (k, v) =>
+  s"""-D$k=$v"""
+}.toSeq
+
+lazy val root = crossProject(JVMPlatform, NativePlatform)
+  .in(file("."))
+  .settings(
+    crossScalaVersions := Seq(scala212, scala213)
+  )
+  .jvmSettings(
+    fork := true,
+    javaOptions ++= javaOpts,
+    libraryDependencies += "me.shadaj" %% "scalapy-core" % scalapyVersion
+  )
+  .nativeSettings(
+    libraryDependencies += "me.shadaj" %%% "scalapy-core" % scalapyVersion,
+    nativeLinkStubs := true,
+    nativeLinkingOptions ++= pythonLdFlags
+  )

src/sbt-test/integration/scalapy/project/.gitignore

@@ -0,0 +1 @@
+project/

src/sbt-test/integration/scalapy/project/build.properties

@@ -0,0 +1 @@
+sbt.version=1.5.5

src/sbt-test/integration/scalapy/project/plugins.sbt

@@ -0,0 +1,11 @@
+addSbtPlugin("org.portable-scala" % "sbt-scala-native-crossproject" % "1.1.0")
+addSbtPlugin("org.scala-native"   % "sbt-scala-native"              % "0.4.0")
+
+sys.props.get("plugin.version") match {
+  case Some(v) => libraryDependencies += "ai.kien" %% "python-native-libs" % v
+  case _ =>
+    sys.error(
+      """|The system property 'plugin.version' is not defined.
+         |Specify this property using the scriptedLaunchOpts -D.""".stripMargin
+    )
+}

src/sbt-test/integration/scalapy/shared/src/main/scala/project/Main.scala

@@ -0,0 +1,11 @@
+package project
+
+import me.shadaj.scalapy.py
+import me.shadaj.scalapy.py.{PyQuote, SeqConverters}
+
+object Main {
+  def main(args: Array[String]): Unit = {
+    println(py.Dynamic.global.list(Seq(1, 2, 3).toPythonProxy))
+    println(py"'Hello from ScalaPy!'")
+  }
+}

src/sbt-test/integration/scalapy/test

@@ -0,0 +1,2 @@
+> +rootJVM/run
+> +rootNative/run