ci(core): 在 push/PR 上运行 core 单元测试 (#529)

* ci(core): run core unit tests on push/PR and add JaCoCo report

Adds .github/workflows/core-test.yml to run `mvn test -pl core` and
upload the JaCoCo report. Adds the JaCoCo plugin to core/pom.xml in
report-only mode (no coverage gate).

* ci(core): drop JaCoCo, fix core unit test fork crash

- Remove jacoco-maven-plugin from core/pom.xml and the coverage upload
  step from the workflow; keep only `mvn test -pl core`.
- Add surefire config pointing benchmark-conf at the repo-root config
  dir so ConfigDescriptor finds function.xml/config.properties. Under
  surefire the fork's working dir is core/, where these files don't
  exist, so Config.initInnerFunction() did System.exit(0) and crashed
  the forked VM ("terminated without properly saying goodbye").
- Fix stale OPERATION_PROPORTION in OperationControllerTest (11 -> 13
  values) to match the current count of normal operations.

Author: SpriCoder

.github/workflows/core-test.yml

@@ -0,0 +1,25 @@
+name: core-unit-test
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+    branches:
+      - master
+jobs:
+  core-unit-test:
+    runs-on: ubuntu-latest
+    steps:
+      - name: checkout
+        uses: actions/checkout@v4
+        with:
+          path: iot-benchmark
+      - name: Set java 8
+        uses: actions/setup-java@v4
+        with:
+          distribution: 'zulu'
+          java-version: 8
+      - name: run core unit tests
+        run: |
+          cd ${{ github.workspace }}/iot-benchmark
+          mvn -B test -pl core

core/pom.xml

@@ -114,7 +114,27 @@
     </dependency>
   </dependencies>
   <build>
-
+    <plugins>
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-surefire-plugin</artifactId>
+        <version>3.5.4</version>
+        <configuration>
+          <!--
+            Tests trigger ConfigDescriptor, which resolves config.properties and
+            function.xml relative to "benchmark-conf" (default: configuration/conf).
+            Surefire forks with workingDirectory=core/, where these files don't
+            exist, so Config.initInnerFunction() hits a missing function.xml and
+            calls System.exit(0) (crashing the fork). Point benchmark-conf at the
+            repo-root config dir so the files resolve. The root config.properties
+            is fully commented out, so config stays at defaults.
+          -->
+          <systemPropertyVariables>
+            <benchmark-conf>${project.parent.basedir}/configuration/conf</benchmark-conf>
+          </systemPropertyVariables>
+        </configuration>
+      </plugin>
+    </plugins>
   </build>
 
 </project>

core/src/test/java/cn/edu/tsinghua/iot/benchmark/client/OperationControllerTest.java

@@ -42,15 +42,15 @@ public void after() {}
 
   @Test
   public void testGetNextOperationType() {
-    config.setOPERATION_PROPORTION("1:0:0:0:0:0:0:0:0:0:0");
+    config.setOPERATION_PROPORTION("1:0:0:0:0:0:0:0:0:0:0:0:0");
     OperationController operationController = new OperationController(0);
 
     int loop = 10000;
     for (int i = 0; i < loop; i++) {
       Assert.assertEquals(Operation.INGESTION, operationController.getNextOperationType());
     }
 
-    config.setOPERATION_PROPORTION("0:1:0:0:0:0:0:0:0:0:0");
+    config.setOPERATION_PROPORTION("0:1:0:0:0:0:0:0:0:0:0:0:0");
     operationController = new OperationController(0);
     for (int i = 0; i < loop; i++) {
       assertEquals(Operation.PRECISE_QUERY, operationController.getNextOperationType());