#290: Add tests with many files (#291)

exasol · Dec 11, 2023 · 57105cb · 57105cb
1 parent 7e38d1d
commit 57105cb
Show file tree

Hide file tree

Showing 21 changed files with 482 additions and 448 deletions.
diff --git a/.github/workflows/ci-build-extension.yml b/.github/workflows/ci-build-extension.yml
@@ -7,7 +7,7 @@ on:
   pull_request:
 
 jobs:
-  build:
+  build-extension:
     runs-on: ubuntu-22.04
     concurrency:
       group: ${{ github.workflow }}-${{ github.ref }}

diff --git a/.github/workflows/ci-build-next-java.yml b/.github/workflows/ci-build-next-java.yml
@@ -18,7 +18,7 @@ jobs:
         with:
           fetch-depth: 0
       - name: Set up JDK 17
-        uses: actions/setup-java@v3
+        uses: actions/setup-java@v4
         with:
           distribution: "temurin"
           java-version: 17

diff --git a/.github/workflows/ci-build.yml b/.github/workflows/ci-build.yml
@@ -1,22 +1,20 @@
 name: CI Build
-
 on:
   push:
     branches:
       - main
   pull_request:
 
 jobs:
-  build:
-    name: Build with Exasol ${{ matrix.exasol_db_version }}
-    runs-on: ubuntu-20.04 # UDFs fail with "VM error: Internal error: VM crashed" on ubuntu-latest
+  matrix-build:
+    runs-on: ubuntu-20.04
     concurrency:
       group: ${{ github.workflow }}-${{ github.ref }}-${{ matrix.exasol_db_version }}
       cancel-in-progress: true
     strategy:
       fail-fast: false
       matrix:
-        exasol_db_version: ["7.1.24", "8.23.1"]
+        exasol_db_version: ["8.23.1", "7.1.24"]
     env:
       DEFAULT_EXASOL_DB_VERSION: "8.23.1"
     steps:
@@ -29,7 +27,7 @@ jobs:
         with:
           fetch-depth: 0
       - name: Set up JDK 11 & 17
-        uses: actions/setup-java@v3
+        uses: actions/setup-java@v4
         with:
           distribution: "temurin"
           java-version: |
@@ -46,7 +44,7 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v4
         with:
-          go-version: "1.20"
+          go-version: "1.21"
           cache: false
 
       - name: Cache Go modules
@@ -62,11 +60,20 @@ jobs:
       - name: Enable testcontainer reuse
         run: echo 'testcontainers.reuse.enable=true' > "$HOME/.testcontainers.properties"
       - name: Run scalafix linting
-        run: mvn --batch-mode clean compile test-compile scalastyle:check scalafix:scalafix
+        if: ${{ matrix.exasol_db_version == env.DEFAULT_EXASOL_DB_VERSION }}
+        run: mvn --batch-mode clean compile test-compile scalastyle:check scalafix:scalafix spotless:check
+      - name: Build extension
+        run: |
+          mvn package -DskipTests
+          cd extension
+          npm ci
+          npm run build
+          npm run test
+          npm run lint
       - name: Run tests and build with Maven
         run: >
           JAVA_HOME=$JAVA_HOME_11_X64
-          mvn --batch-mode clean verify
+          mvn --batch-mode verify
           -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn
           -DtrimStackTrace=false
           -Dcom.exasol.dockerdb.image=${{ matrix.exasol_db_version }}
@@ -75,7 +82,7 @@ jobs:
           # not forward it to the test. So we use this environment variable,
           # see BaseIntegrationTest.scala.getExasolDockerImageVersion()
           EXASOL_DB_VERSION: ${{ matrix.exasol_db_version }}
-      - name: Publish Test Report
+      - name: Publish Test Report for Exasol ${{ matrix.exasol_db_version }}
         uses: scacap/action-surefire-report@v1
         if: ${{ always() && github.event.pull_request.head.repo.full_name == github.repository && github.actor != 'dependabot[bot]' }}
         with:
@@ -93,3 +100,9 @@ jobs:
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
+
+  build:
+    needs: matrix-build
+    runs-on: ubuntu-latest
+    steps:
+      - run: echo "Build successful"
diff --git a/.github/workflows/linting.yml b/.github/workflows/linting.yml
diff --git a/.github/workflows/release_droid_prepare_original_checksum.yml b/.github/workflows/release_droid_prepare_original_checksum.yml
@@ -16,15 +16,15 @@ jobs:
         with:
           fetch-depth: 0
       - name: Set up JDK 11
-        uses: actions/setup-java@v3
+        uses: actions/setup-java@v4
         with:
           distribution: "temurin"
           java-version: 11
           cache: "maven"
       - name: Set up Go
         uses: actions/setup-go@v3
         with:
-          go-version: 1.19
+          go-version: "1.21"
       - name: Enable testcontainer reuse
         run: echo 'testcontainers.reuse.enable=true' > "$HOME/.testcontainers.properties"
       - name: Run tests and build with Maven

diff --git a/.github/workflows/release_droid_upload_github_release_assets.yml b/.github/workflows/release_droid_upload_github_release_assets.yml
@@ -16,7 +16,7 @@ jobs:
         with:
           fetch-depth: 0
       - name: Set up JDK 11
-        uses: actions/setup-java@v3
+        uses: actions/setup-java@v4
         with:
           distribution: "temurin"
           java-version: 11

diff --git a/.project-keeper.yml b/.project-keeper.yml
@@ -14,3 +14,9 @@ excludes:
   - "E-PK-CORE-18: Outdated content: '.github/workflows/ci-build-next-java.yml'"
   - "E-PK-CORE-18: Outdated content: '.github/workflows/release_droid_upload_github_release_assets.yml'"
   - "E-PK-CORE-18: Outdated content: '.github/workflows/release_droid_prepare_original_checksum.yml'"
+build:
+  runnerOs: ubuntu-20.04
+  freeDiskSpace: false
+  exasolDbVersions:
+    - "8.23.1"
+    - "7.1.24"
diff --git a/.vscode/settings.json b/.vscode/settings.json
@@ -1,8 +1,8 @@
 {
   "editor.formatOnSave": true,
   "editor.codeActionsOnSave": {
-    "source.organizeImports": true,
-    "source.generate.finalModifiers": true
+    "source.organizeImports": "explicit",
+    "source.generate.finalModifiers": "explicit"
   },
   "java.saveActions.organizeImports": true,
   "java.sources.organizeImports.starThreshold": 3,