close
Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: OpenIntegrationEngine/engine
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
...
head repository: Mirthium-Project/engine
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: main
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 9 commits
  • 11 files changed
  • 2 contributors

Commits on Feb 7, 2026

  1. build: fail build if tests fail

    This will allow all tests for a given sub-project to finish, but will
    abort the rest of the build.
    
    Previously the build would finish successfully even if there were test
    failures.
    
    Signed-off-by: Tony Germano <tony@germano.name>
    tonygermano committed Feb 7, 2026
    Configuration menu
    Copy the full SHA
    1611f04 View commit details
    Browse the repository at this point in the history

Commits on Feb 8, 2026

  1. build: Standardize test output and decouple JaCoCo coverage

    Refactors the Ant build scripts across client, command, donkey, and
    server projects to improve CI integration and local developer
    experience.
    
    Changes:
    * Decoupled Coverage: Separated unit test execution (`test-run`) from
      code coverage instrumentation (`test-coverage`). Developers can now
      run tests locally without the performance penalty of the JaCoCo agent
      unless explicitly requested.
    * Standardized Output: Consolidated all test artifacts into a unified,
      Gradle-compliant directory structure:
      - Raw results: `build/test-results/test`
      - HTML/XML Reports: `build/reports/tests` and `build/reports/jacoco`
    
    Usage:
    - Run standard unit tests: `ant test-run`
    - Run tests with coverage: `ant test-coverage`
      (or `ant test-run -Dcoverage=true` from root)
    
    Signed-off-by: Tony Germano <tony@germano.name>
    tonygermano committed Feb 8, 2026
    Configuration menu
    Copy the full SHA
    4dc3514 View commit details
    Browse the repository at this point in the history
  2. build: Add comprehensive clean targets and legacy cleanup

    Improves build hygiene by ensuring all generated artifacts are removed
    during clean operations.
    
    Changes:
    * Global Clean: Added a top-level `clean` target in `mirth-build.xml`
      that cascades to all sub-projects.
    * Legacy Cleanup: Introduced `clean-legacy-dirs` target to remove
      obsolete directories (`junit-reports`, `code-coverage-reports`)
      ensuring developers do not view stale reports.
    * Deep Clean: Updated sub-project clean targets to include
      `${test_classes}` and `${build.dir}`, preventing stale class files
      from polluting subsequent builds.
    
    Signed-off-by: Tony Germano <tony@germano.name>
    tonygermano committed Feb 8, 2026
    Configuration menu
    Copy the full SHA
    da15020 View commit details
    Browse the repository at this point in the history

Commits on Feb 9, 2026

  1. ci: Implement secure test result reporting via GitHub Actions

    Establishes a split-workflow CI pipeline to safely run tests and report
    results, complying with security best practices for public repositories.
    
    Changes:
    * Build Workflow:
      - Updated the Ant build command to enable coverage
        (`-Dcoverage=true`).
      - Added a "Stage Test Results" step to aggregate XML reports from
        all sub-projects (client, server, donkey, etc.) while preserving
        directory structure.
      - Added an "Event File" job to capture PR metadata for secure
        downstream processing.
    * Reporting Workflow:
      - Created `upload_test_results.yaml` triggered by `workflow_run`.
      - Implemented the privileged report publishing step separately from
        the unprivileged build step.
      - configured `EnricoMi/publish-unit-test-result-action` to ingest the
        aggregated XML reports.
    
    Security:
    * Uses the `workflow_run` pattern to allow safe PR commenting from forks
      without exposing write tokens to the build environment.
    
    Signed-off-by: Tony Germano <tony@germano.name>
    tonygermano committed Feb 9, 2026
    Configuration menu
    Copy the full SHA
    f139d35 View commit details
    Browse the repository at this point in the history
  2. Merge pull request #1 from tonygermano/refactor-test-and-coverage

    Refactor test and coverage
    tonygermano authored Feb 9, 2026
    Configuration menu
    Copy the full SHA
    957f866 View commit details
    Browse the repository at this point in the history
  3. debug artifact files (#3)

    Signed-off-by: Tony Germano <tony@germano.name>
    tonygermano authored Feb 9, 2026
    Configuration menu
    Copy the full SHA
    b330f16 View commit details
    Browse the repository at this point in the history
  4. Debug artifacts (#4)

    * debug artifact files
    
    Signed-off-by: Tony Germano <tony@germano.name>
    
    * Revert "debug artifact files"
    
    This reverts commit 29dea9a.
    
    ---------
    
    Signed-off-by: Tony Germano <tony@germano.name>
    tonygermano authored Feb 9, 2026
    Configuration menu
    Copy the full SHA
    e9d0f2f View commit details
    Browse the repository at this point in the history
  5. Revert "debug artifact files"

    This reverts commit 29dea9a.
    tonygermano committed Feb 9, 2026
    Configuration menu
    Copy the full SHA
    fec1cbd View commit details
    Browse the repository at this point in the history
  6. add back event

    Signed-off-by: Tony Germano <tony@germano.name>
    tonygermano committed Feb 9, 2026
    Configuration menu
    Copy the full SHA
    476c74a View commit details
    Browse the repository at this point in the history
Loading