Live breakpoint test (#3)

* test: working on context receiver test

* Fix gradle tasks using the npm command to work on Windows

* Fix SourceMapper for Windows

* Fix condition handling in LiveInstrumentRemote

* Reorganize tests

* breakpoint added listener

* Implement applyBreakpoint in ContextReceiver.ts

* Add authentication

* Implement breakpoint hits

* Check for side effects when evaluating conditions

* fix workflow

* not necessary

* fix workflow

* fix workflow

* fix workflow

* fix workflow

Co-authored-by: UltraDev <petzmagnus@gmail.com>

Author: BFergerson

.github/workflows/build.yml

@@ -16,7 +16,7 @@ on:
 jobs:
   build:
     runs-on: ubuntu-latest
-    timeout-minutes: 10
+    timeout-minutes: 5
     concurrency:
       group: ${{ github.workflow }}-${{ github.ref }}
       cancel-in-progress: true
@@ -42,18 +42,18 @@ jobs:
         id: properties
         shell: bash
         run: |
-          PROPERTIES="$(./gradlew properties --console=plain -q)"
-          VERSION="$(echo "$PROPERTIES" | grep "^version:" | cut -f2- -d ' ')"
+          VERSION=$(grep "projectVersion" gradle.properties | cut -d'=' -f2)
           echo "::set-output name=version::$VERSION"
           echo "SPP_RELEASE_VERSION=${VERSION/-SNAPSHOT/}" >> $GITHUB_ENV
+          SPP_RELEASE_VERSION=${VERSION/-SNAPSHOT/}
+          echo "::set-output name=release_version::$SPP_RELEASE_VERSION"
 
-      - run: ./gradlew assembleUp -Dbuild.profile=release -PprobeVersion=$SPP_RELEASE_VERSION
+      - name: Start platform
+        run: ./gradlew assembleUp -Dbuild.profile=release -PprobeVersion=$SPP_RELEASE_VERSION
 
       - name: Docker IPs
         run: docker inspect --format='{{.Name}} - {{range .NetworkSettings.Networks}}{{.IPAddress}}{{end}}' $(docker ps -aq)
 
-      - name: Set E2E_APP_HOST
-        run: E2E_APP_HOST=$(docker inspect -f '{{range.NetworkSettings.Networks}}{{.IPAddress}}{{end}}' $(docker ps -aqf "name=e2e-test")) && echo "E2E_APP_HOST=$E2E_APP_HOST" >> $GITHUB_ENV
       - name: Set SPP_PLATFORM_HOST
         run: SPP_PLATFORM_HOST=$(docker inspect -f '{{range.NetworkSettings.Networks}}{{.IPAddress}}{{end}}' $(docker ps -aqf "name=spp-platform")) && echo "SPP_PLATFORM_HOST=$SPP_PLATFORM_HOST" >> $GITHUB_ENV
       - name: Set SPP_JWT_TOKEN
@@ -63,15 +63,10 @@ jobs:
         run: |
           until $(curl --header "Authorization: Bearer ${{ env.SPP_JWT_TOKEN }}" --insecure --output /dev/null --silent --fail http://localhost:12800/health); do printf '.'; sleep 5; done
 
-      - name: Wait for probe
-        run: sleep 15s #todo: smarter waiting
-
-      - name: Verify probe connected
-        run: |
-          resp=$(curl --header "Authorization: Bearer ${{ env.SPP_JWT_TOKEN }}" --insecure http://localhost:12800/clients) && [[ $(echo $resp | jq -r ."probes"[0]."meta"."language") == "nodejs" ]] && echo $resp || (echo $resp && exit 1)
-
       - name: Run tests
-        run: npm run test
+        run: ./gradlew test
+        env:
+          CI: true
 
       - run: ./gradlew composeLogs
         if: ${{ always() }}

build.gradle.kts

@@ -1,8 +1,14 @@
+import org.apache.tools.ant.taskdefs.condition.Os
+
 plugins {
     id("com.avast.gradle.docker-compose") version "0.16.8"
 }
 
 tasks {
+    register<Exec>("test") {
+        executable = getNpm()
+        args("run", "build-test")
+    }
     register("cleanPackDir") {
         file("pack/").mkdirs()
         file("pack/").listFiles()?.forEach { it.delete() }
@@ -14,13 +20,13 @@ tasks {
 
     register<Exec>("makeDist") {
         dependsOn("cleanPackDir")
-        executable = "npm"
+        executable = getNpm()
         args("run", "build")
     }
 
     register<Exec>("buildDist") {
         dependsOn("makeDist")
-        executable = "npm"
+        executable = getNpm()
         args("pack", "--pack-destination=./pack")
     }
 
@@ -37,7 +43,15 @@ tasks {
 }
 
 dockerCompose {
-    dockerComposeWorkingDirectory.set(File("./e2e"))
     removeVolumes.set(true)
     waitForTcpPorts.set(false)
 }
+
+// Make npm work on Windows
+fun getNpm(): String {
+    return if (Os.isFamily(Os.FAMILY_WINDOWS)) {
+        "npm.cmd"
+    } else {
+        "npm"
+    }
+}

e2e/docker-compose.yml renamed to docker-compose.yml

@@ -1,27 +1,16 @@
 version: '3.3'
 services:
-  e2e-test:
-    container_name: e2e-test
-    build:
-      context: .
-    depends_on:
-      - spp-platform
-    links:
-      - spp-platform
-    ports:
-      - "5000:5000"
   spp-platform:
     image: sourceplusplus/spp-platform:latest
     container_name: spp-platform
     hostname: spp-platform
     ports:
       - "11800:11800"
-      - "11801:11801" #todo: shouldn't be necessary
       - "12800:12800"
+      - "5106:5106"
     environment:
-      - SPP_LOGGING_LEVEL=trace
+      - JAVA_OPTS=-agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=*:5106
+      - SPP_LOGGING_LEVEL=debug
       - SW_RECEIVER_GRPC_SSL_ENABLED=false #todo: fix this and config.secure = false
       - SPP_HTTP_SSL_ENABLED=false #todo; Change back to ssl once the library works with self-signed certs
-    volumes:
-      - ./config/spp-platform.crt:/skywalking/config/spp-platform.crt
-      - ./config/spp-platform.key:/skywalking/config/spp-platform.key
+      - SPP_GRPC_SSL_ENABLED=false

e2e/Dockerfile

@@ -1,12 +1,4 @@
 kotlin.code.style=official
 
 probeGroup=plus.sourceplus.probe
-
-vertxVersion=4.3.0
-
-jacksonVersion=2.13.1
-gsonVersion = 2.8.6
-jupiterVersion = 5.8.0
-logbackVersion = 1.2.6
-
-sppVersion=0.5.5-SNAPSHOT
+projectVersion=0.6.3-SNAPSHOT

e2e/E2ETest.js

@@ -14,6 +14,7 @@
   "scripts": {
     "build": "tsc",
     "test": "mocha test/*.js",
+    "build-test": "npm run build && npm run test",
     "prepublishOnly": "npm run build"
   },
   "peerDependencies": {