-
Notifications
You must be signed in to change notification settings - Fork 334
Test Standard.Microsoft in (mocked) dual JVM mode
#14037
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
531a763
Run Standard.Microsoft in dual JVM mode
JaroslavTulach 6b1622d
Merge remote-tracking branch 'origin/develop' into wip/jtulach/Micros…
JaroslavTulach c40ad9c
Merging with latest HidableValue changes
JaroslavTulach 4bbffbd
Allow to pass a HideableValue factory from the other JVM
JaroslavTulach 20f573f
Allow SQLServer_Dialect to pass its own version of JDBCUtils into fil…
JaroslavTulach d4a1ac7
Merge with origin/develop
JaroslavTulach 20529f8
Running test/Microsoft_Tests in a special - dual JVM mode
JaroslavTulach 39e1f0b
Derive ProblemAggregator and ColumnFetcher classes from ColumnFetcher…
JaroslavTulach f9c9e0a
Convert non-Java_Column by proxying its ColumnStorage interface
JaroslavTulach d3d90c3
Improve debugging by associating other JVM objects with EpbLanguage
JaroslavTulach 51366e2
Sort ColumnStorage interfaces rather than classes
JaroslavTulach 933c556
Properly choose fetcher type even for proxied interfaces
JaroslavTulach 3422d84
Generate CI actions to run test/Microsoft_Tests in dual JVM mode
vitvakatu a48b2c8
Let Table be the factory for appropriate Row instance
JaroslavTulach c7be997
Associate JDBC_Connection with the right java.sql.SQLException to catch
JaroslavTulach bfb1e61
Manually shortening too long generated names to their original values
JaroslavTulach 407939d
To compatibly add a defaulted argument, it must be the last
JaroslavTulach a8eb695
Recording API changes necessary to run Standard.Microsoft in dual JVM…
JaroslavTulach f91584d
Need polyglot imports to let EnsoLibraryFeature register those clases…
JaroslavTulach 8af9c47
Easy way to opt-out of compiling Standard.Microsoft into NI via ENSO_…
JaroslavTulach 8a93181
Shorten CI job names by overriding id_key_base
JaroslavTulach a32206e
VSCode debug configuration for enso-build-ci-gen
JaroslavTulach 9ba82d4
Other JVM objects are now loaded by EpbLanguage
JaroslavTulach 7606734
Avoid Audit tests when Standard.Microsoft tests are running in dual J…
JaroslavTulach 5bc3de1
Let's name the argument problem_aggregator
JaroslavTulach 544c8fb
Removing unused polyglot java import
JaroslavTulach 623b6bb
Returning private comment back
JaroslavTulach c9ce1cb
Merge with latest develop
JaroslavTulach 22a8d10
Provide JDBCUtils as positional argument
JaroslavTulach 6f9c500
Call the argument jdbc_utils
JaroslavTulach f908502
Merge branch 'wip/jtulach/MicrosoftServerInHotSpot' of enso:enso-org/…
JaroslavTulach f460c68
Formatting
JaroslavTulach fcd4313
Providing sql_exception as a positional argument
JaroslavTulach 84426cd
Merge branch 'wip/jtulach/MicrosoftServerInHotSpot' of enso:enso-org/…
JaroslavTulach 7c289aa
Using full sql_exception atom constructor name and removing self argu…
JaroslavTulach a6830b0
Adjusting Standard.Database API
JaroslavTulach File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -407,7 +407,54 @@ jobs: | |
| run: ./run git-clean | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| enso-build-ci-gen-job-standard-library-tests-graal-vm-ce-standard-library-in-native-linux-amd64: | ||
| license-check-linux-amd64: | ||
| name: Verify License Packages (linux, amd64) | ||
| runs-on: | ||
| - self-hosted | ||
| - Linux | ||
| steps: | ||
| - if: runner.os == 'Windows' | ||
| name: Setup required bazel environment | ||
| run: "\n\"BAZEL_SH=C:\\Program Files\\Git\\bin\\bash.exe\" >> $env:GITHUB_ENV\n\"BAZEL_VC=C:\\BuildTools\\VC\" >> $env:GITHUB_ENV\n " | ||
| shell: pwsh | ||
| - name: Setup bazel environment | ||
| uses: bazel-contrib/[email protected] | ||
| with: | ||
| bazelrc: build --remote_cache=grpcs://${{ vars.ENSO_BAZEL_CACHE_URI }} --remote_cache_header="authorization=Basic ${{ secrets.ENSO_BAZEL_CACHE_TOKEN }}" | ||
| output-base: ${{ runner.os == 'Windows' && 'c:/_bazel' || '' }} | ||
| - name: Expose Artifact API and context information. | ||
| uses: actions/github-script@v7 | ||
| with: | ||
| script: "\n core.exportVariable(\"ACTIONS_RUNTIME_TOKEN\", process.env[\"ACTIONS_RUNTIME_TOKEN\"])\n core.exportVariable(\"ACTIONS_RUNTIME_URL\", process.env[\"ACTIONS_RUNTIME_URL\"])\n core.exportVariable(\"GITHUB_RETENTION_DAYS\", process.env[\"GITHUB_RETENTION_DAYS\"])\n console.log(context)\n " | ||
| - name: Checking out the repository | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| clean: false | ||
| - if: runner.os == 'macOS' | ||
| name: Setup nodejs version | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version-file: .node-version | ||
| - if: runner.os != 'Linux' | ||
| run: npm install -g [email protected] && corepack --version | ||
| - name: Build Script Setup | ||
| run: ./run --help || (git clean -ffdx && ./run --help) | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| - if: "${{ (contains(github.event.pull_request.labels.*.name, 'CI: Clean build required') || (github.ref == 'refs/heads/develop') || inputs.clean_build_required) }}" | ||
| name: Clean before | ||
| run: ./run git-clean | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| - run: ./run backend sbt '--' verifyLicensePackages | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| - if: "${{ (always()) && (contains(github.event.pull_request.labels.*.name, 'CI: Clean build required') || (github.ref == 'refs/heads/develop') || inputs.clean_build_required) }}" | ||
| name: Clean after | ||
| run: ./run git-clean | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| stdlib-graal-vm-ce-standard-library-in-native-linux-amd64: | ||
| name: Standard Library Native Tests (GraalVM CE) (linux, amd64) | ||
| needs: | ||
| - enso-build-ci-gen-job-build-engine-distribution-graal-vm-ce-linux-amd64 | ||
|
|
@@ -492,7 +539,7 @@ jobs: | |
| GRAAL_EDITION: GraalVM CE | ||
| permissions: | ||
| checks: write | ||
| enso-build-ci-gen-job-standard-library-tests-graal-vm-ce-standard-library-in-native-windows-amd64: | ||
| stdlib-graal-vm-ce-standard-library-in-native-windows-amd64: | ||
| name: Standard Library Native Tests (GraalVM CE) (windows, amd64) | ||
| needs: | ||
| - enso-build-ci-gen-job-build-engine-distribution-graal-vm-ce-windows-amd64 | ||
|
|
@@ -577,7 +624,7 @@ jobs: | |
| GRAAL_EDITION: GraalVM CE | ||
| permissions: | ||
| checks: write | ||
| enso-build-ci-gen-job-standard-library-tests-graal-vm-ce-standard-library-linux-amd64: | ||
| stdlib-graal-vm-ce-standard-library-linux-amd64: | ||
| name: Standard Library JVM Tests (GraalVM CE) (linux, amd64) | ||
| needs: | ||
| - enso-build-ci-gen-job-build-engine-distribution-graal-vm-ce-linux-amd64 | ||
|
|
@@ -662,7 +709,7 @@ jobs: | |
| GRAAL_EDITION: GraalVM CE | ||
| permissions: | ||
| checks: write | ||
| enso-build-ci-gen-job-standard-library-tests-graal-vm-ce-standard-library-windows-amd64: | ||
| stdlib-graal-vm-ce-standard-library-windows-amd64: | ||
| name: Standard Library JVM Tests (GraalVM CE) (windows, amd64) | ||
| needs: | ||
| - enso-build-ci-gen-job-build-engine-distribution-graal-vm-ce-windows-amd64 | ||
|
|
@@ -747,7 +794,7 @@ jobs: | |
| GRAAL_EDITION: GraalVM CE | ||
| permissions: | ||
| checks: write | ||
| enso-build-ci-gen-job-standard-library-tests-graal-vm-ce-std-microsoft-linux-amd64: | ||
| stdlib-graal-vm-ce-std-microsoft-std-mock-dual-microsoft-linux-amd64: | ||
| name: Standard Library Microsoft Tests (GraalVM CE) (linux, amd64) | ||
| needs: | ||
| - enso-build-ci-gen-job-build-engine-distribution-graal-vm-ce-linux-amd64 | ||
|
|
@@ -800,7 +847,7 @@ jobs: | |
| run: |- | ||
| tar -xvf built-distribution.tar -C . | ||
| rm built-distribution.tar | ||
| - run: ./run backend test std-microsoft | ||
| - run: ./run backend test std-microsoft std-mock-dual-microsoft | ||
| env: | ||
| ENSO_LIB_S3_AWS_ACCESS_KEY_ID: ${{ secrets.ENSO_LIB_S3_AWS_ACCESS_KEY_ID }} | ||
| ENSO_LIB_S3_AWS_REGION: ${{ secrets.ENSO_LIB_S3_AWS_REGION }} | ||
|
|
@@ -832,7 +879,7 @@ jobs: | |
| GRAAL_EDITION: GraalVM CE | ||
| permissions: | ||
| checks: write | ||
| enso-build-ci-gen-job-standard-library-tests-graal-vm-ce-std-microsoft-windows-amd64: | ||
| stdlib-graal-vm-ce-std-microsoft-std-mock-dual-microsoft-windows-amd64: | ||
| name: Standard Library Microsoft Tests (GraalVM CE) (windows, amd64) | ||
| needs: | ||
| - enso-build-ci-gen-job-build-engine-distribution-graal-vm-ce-windows-amd64 | ||
|
|
@@ -885,7 +932,7 @@ jobs: | |
| run: |- | ||
| tar -xvf built-distribution.tar -C . | ||
| rm built-distribution.tar | ||
| - run: ./run backend test std-microsoft | ||
| - run: ./run backend test std-microsoft std-mock-dual-microsoft | ||
| env: | ||
| ENSO_LIB_S3_AWS_ACCESS_KEY_ID: ${{ secrets.ENSO_LIB_S3_AWS_ACCESS_KEY_ID }} | ||
| ENSO_LIB_S3_AWS_REGION: ${{ secrets.ENSO_LIB_S3_AWS_REGION }} | ||
|
|
@@ -917,52 +964,5 @@ jobs: | |
| GRAAL_EDITION: GraalVM CE | ||
| permissions: | ||
| checks: write | ||
| enso-build-ci-gen-job-verify-license-packages-linux-amd64: | ||
| name: Verify License Packages (linux, amd64) | ||
| runs-on: | ||
| - self-hosted | ||
| - Linux | ||
| steps: | ||
| - if: runner.os == 'Windows' | ||
| name: Setup required bazel environment | ||
| run: "\n\"BAZEL_SH=C:\\Program Files\\Git\\bin\\bash.exe\" >> $env:GITHUB_ENV\n\"BAZEL_VC=C:\\BuildTools\\VC\" >> $env:GITHUB_ENV\n " | ||
| shell: pwsh | ||
| - name: Setup bazel environment | ||
| uses: bazel-contrib/[email protected] | ||
| with: | ||
| bazelrc: build --remote_cache=grpcs://${{ vars.ENSO_BAZEL_CACHE_URI }} --remote_cache_header="authorization=Basic ${{ secrets.ENSO_BAZEL_CACHE_TOKEN }}" | ||
| output-base: ${{ runner.os == 'Windows' && 'c:/_bazel' || '' }} | ||
| - name: Expose Artifact API and context information. | ||
| uses: actions/github-script@v7 | ||
| with: | ||
| script: "\n core.exportVariable(\"ACTIONS_RUNTIME_TOKEN\", process.env[\"ACTIONS_RUNTIME_TOKEN\"])\n core.exportVariable(\"ACTIONS_RUNTIME_URL\", process.env[\"ACTIONS_RUNTIME_URL\"])\n core.exportVariable(\"GITHUB_RETENTION_DAYS\", process.env[\"GITHUB_RETENTION_DAYS\"])\n console.log(context)\n " | ||
| - name: Checking out the repository | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| clean: false | ||
| - if: runner.os == 'macOS' | ||
| name: Setup nodejs version | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version-file: .node-version | ||
| - if: runner.os != 'Linux' | ||
| run: npm install -g [email protected] && corepack --version | ||
| - name: Build Script Setup | ||
| run: ./run --help || (git clean -ffdx && ./run --help) | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| - if: "${{ (contains(github.event.pull_request.labels.*.name, 'CI: Clean build required') || (github.ref == 'refs/heads/develop') || inputs.clean_build_required) }}" | ||
| name: Clean before | ||
| run: ./run git-clean | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| - run: ./run backend sbt '--' verifyLicensePackages | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| - if: "${{ (always()) && (contains(github.event.pull_request.labels.*.name, 'CI: Clean build required') || (github.ref == 'refs/heads/develop') || inputs.clean_build_required) }}" | ||
| name: Clean after | ||
| run: ./run git-clean | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| env: | ||
| ENSO_BUILD_SKIP_VERSION_CHECK: "true" | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This section is just moved because the CI key is sorted and the sorting changed by changing the name.