Run component tests, unit tests and static analysis in distro build (#29)

---------

Co-authored-by: Sergey Kleyman <Sergey.Kleyman@elastic.co>

Author: intuibase

.github/workflows/build.yml

@@ -23,6 +23,9 @@ concurrency:
   cancel-in-progress: ${{ github.ref != 'refs/heads/main' }}
 
 jobs:
+  test-php-static-and-unit:
+    uses: ./.github/workflows/test-php-static-and-unit.yml
+
   build-compiler:
     uses: ./.github/workflows/native-dev-tools-build.yml
     secrets: inherit
@@ -58,13 +61,21 @@ jobs:
       - build-packages
     uses: ./.github/workflows/test-otel-unit.yml
 
+  test-packages-component-tests:
+    needs:
+      - build-packages
+    uses: ./.github/workflows/test-packages-component-tests.yml
+
+
   # The very last job to report whether the Workflow passed.
   # This will act as the Branch Protection gatekeeper
   ci:
     needs:
       - tests-phpt
+      - test-php-static-and-unit
       - build-packages
       - test-otel-unit
+      - test-packages-component-tests
     runs-on: ubuntu-latest
     steps:
       - name: report

.github/workflows/generate-component-tests-matrix.yml

@@ -0,0 +1,29 @@
+---
+
+# Generates matrix for component tests
+name: generate-component-tests-matrix
+
+on:
+  workflow_call:
+    outputs:
+      matrix:
+        description: "Matrix for component tests"
+        value: ${{ jobs.generate-component-tests-matrix.outputs.matrix }}
+
+permissions:
+  contents: read
+
+jobs:
+  generate-component-tests-matrix:
+    name: generate-component-tests-matrix
+    timeout-minutes: 5
+    runs-on: ubuntu-latest
+    outputs:
+      matrix: ${{ steps.generate.outputs.matrix }}
+    steps:
+      - uses: actions/checkout@v6
+      - id: generate
+        run: |
+          MATRIX_JSON=$(./tools/test/component/generate_matrix.sh | jq --raw-input --slurp -c 'split("\n") | map(select(length > 0)) | map({ "row": . } )')
+          echo "matrix={\"include\":${MATRIX_JSON}}"
+          echo "matrix={\"include\":${MATRIX_JSON}}" >> $GITHUB_OUTPUT

.github/workflows/test-packages-component-tests.yml

@@ -0,0 +1,49 @@
+---
+
+name: test-packages-component-tests
+
+on:
+  workflow_call: ~
+
+permissions:
+  contents: read
+
+env:
+  BUILD_PACKAGES_SUB_DIR: build/packages
+  COMPONENT_TESTS_LOGS_SUB_DIR: build/component_tests_logs
+
+jobs:
+  generate-component-tests-matrix:
+    uses: ./.github/workflows/generate-component-tests-matrix.yml
+
+  run-component-tests-in-docker-for-one-matrix-row:
+    name: row
+    runs-on: 'ubuntu-latest'
+    needs: generate-component-tests-matrix
+    timeout-minutes: 300
+    strategy:
+      fail-fast: false
+      matrix: ${{ fromJson(needs.generate-component-tests-matrix.outputs.matrix) }}
+    env:
+      MATRIX_ROW: ${{ matrix.row }}
+    steps:
+      - uses: actions/checkout@v6
+      - name: Download built packages
+        uses: actions/download-artifact@v7
+        with:
+          pattern: packages-*
+          path: ${{ env.BUILD_PACKAGES_SUB_DIR }}
+
+      - name: Run component tests
+        continue-on-error: false
+        run: |
+          mv "${PWD}/${{ env.BUILD_PACKAGES_SUB_DIR }}/"*/* "${PWD}/${{ env.BUILD_PACKAGES_SUB_DIR }}/"
+          rm -rf "${PWD}/${{ env.BUILD_PACKAGES_SUB_DIR }}/packages-"*
+          ./tools/test/component/test_packages_one_matrix_row_in_docker.sh --matrix_row "${MATRIX_ROW}" --packages_dir "${PWD}/${{ env.BUILD_PACKAGES_SUB_DIR }}" --logs_dir "${PWD}/${{ env.COMPONENT_TESTS_LOGS_SUB_DIR }}"
+      - uses: actions/upload-artifact@v6
+        if: always()
+        continue-on-error: false
+        with:
+          name: test-packages-component-tests-${{ matrix.row }}
+          path: |
+            ${{ env.COMPONENT_TESTS_LOGS_SUB_DIR }}/*

.github/workflows/test-php-static-and-unit.yml

@@ -0,0 +1,39 @@
+---
+
+name: test-php-static-and-unit
+
+on:
+  workflow_call: ~
+  workflow_dispatch: ~
+
+permissions:
+  contents: read
+
+env:
+  UNIT_TESTS_LOGS_SUB_DIR: build/unit_tests_logs
+
+jobs:
+  generate-php-versions:
+    uses: ./.github/workflows/generate-php-versions.yml
+
+  static-check-and-unit-tests:
+    name: static check and unit tests
+    runs-on: ubuntu-latest
+    needs: generate-php-versions
+    timeout-minutes: 30
+    strategy:
+      fail-fast: false
+      matrix: ${{ fromJson(needs.generate-php-versions.outputs.php-versions) }}
+    env:
+      PHP_VERSION: ${{ matrix.php-version }}
+    steps:
+      - uses: actions/checkout@v6
+      - name: Invoke test_php_static_and_unit.sh
+        run: ./tools/test/test_php_static_and_unit.sh --php_versions "${PHP_VERSION}" --logs_dir "${PWD}/${{ env.UNIT_TESTS_LOGS_SUB_DIR }}"
+      - uses: actions/upload-artifact@v6
+        if: always()
+        continue-on-error: false
+        with:
+          name: test_php_static_and_unit-log-${{ matrix.php-version }}
+          path: |
+            ${{ env.UNIT_TESTS_LOGS_SUB_DIR }}/*

.gitignore

@@ -0,0 +1,86 @@
+# Generated files
+prod/php/OpenTelemetry/Distro/PhpPartVersion.php
+prod/php/OpenTelemetry/Distro/Log/LogFeature.php
+
+# Mac
+.DS_Store
+
+# JetBrains PhpStorm
+.idea/
+
+# Visual Studio
+.vs/
+
+# Visual Studio Code
+.vscode/
+
+# Composer
+composer.lock
+composer.phar
+vendor/
+prod/php/vendor_*/
+
+# PHP_CodeSniffer
+.php_cs.cache
+
+# PHPUnit
+.phpunit.result.cache
+
+#phpt results folder
+prod/native/extension/phpt/results/
+prod/native/phpbridge_extension/phpt/results/
+
+#native build folder
+prod/native/_build/
+
+# Built documentation
+html_docs
+
+# Packaging artifacts and generated unit test reports
+build/*
+
+# Backup files when using sed
+.bck
+
+# For local development
+z_local*
+
+# Per user PHPStan configuration
+phpstan.neon
+
+# Prerequisites
+*.d
+
+# Compiled Object files
+*.slo
+*.lo
+*.o
+*.obj
+
+# Precompiled Headers
+*.gch
+*.pch
+
+# Compiled Dynamic libraries
+*.so
+*.dylib
+*.dll
+
+# Fortran module files
+*.mod
+*.smod
+
+# Compiled Static libraries
+*.lai
+*.la
+*.a
+*.lib
+
+# Executables
+*.exe
+*.out
+*.app
+
+# Docs
+.artifacts
+docs/.artifacts

composer.json

@@ -53,8 +53,8 @@
     "autoload-dev": {
         "psr-4": {
             "OpenTelemetry\\Distro\\": "./prod/php/OpenTelemetry/Distro/",
-            "ElasticOTelTests\\": "./tests/ElasticOTelTests/",
-            "ElasticOTelTools\\Build\\": "./tools/build/"
+            "OTelDistroTests\\": "./tests/OTelDistroTests/",
+            "OpenTelemetry\\DistroTools\\Build\\": "./tools/build/"
         }
     },
     "config": {
@@ -74,7 +74,7 @@
             "php tools/build/direct_composer_install_update_not_allowed.php update"
         ],
         "generate_lock_use_current_json": [
-            "@putenv OTEL_PHP_PHP_TOOLS_ALLOW_DIRECT_COMPOSER_COMMAND=true",
+            "@putenv OTEL_PHP_TOOLS_ALLOW_DIRECT_COMPOSER_COMMAND=true",
             "composer --ignore-platform-req=ext-mysqli --ignore-platform-req=ext-pgsql --ignore-platform-req=ext-opentelemetry --no-interaction --no-plugins --no-scripts --no-install update"
         ],
         "parallel-lint": [

generated_composer_lock_files/dev.json

@@ -53,8 +53,8 @@
     "autoload-dev": {
         "psr-4": {
             "OpenTelemetry\\Distro\\": "./prod/php/OpenTelemetry/Distro/",
-            "ElasticOTelTests\\": "./tests/ElasticOTelTests/",
-            "ElasticOTelTools\\Build\\": "./tools/build/"
+            "OTelDistroTests\\": "./tests/OTelDistroTests/",
+            "OpenTelemetry\\DistroTools\\Build\\": "./tools/build/"
         }
     },
     "config": {
@@ -74,7 +74,7 @@
             "php tools/build/direct_composer_install_update_not_allowed.php update"
         ],
         "generate_lock_use_current_json": [
-            "@putenv OTEL_PHP_PHP_TOOLS_ALLOW_DIRECT_COMPOSER_COMMAND=true",
+            "@putenv OTEL_PHP_TOOLS_ALLOW_DIRECT_COMPOSER_COMMAND=true",
             "composer --ignore-platform-req=ext-mysqli --ignore-platform-req=ext-pgsql --ignore-platform-req=ext-opentelemetry --no-interaction --no-plugins --no-scripts --no-install update"
         ],
         "parallel-lint": [