Skip to content

build: refresh tester runtime jar - #203

Merged
slyunnn merged 1 commit into
matrixorigin:mainfrom
slyunnn:test/capture-runtime-jar
Aug 27, 2026
Merged

build: refresh tester runtime jar#203
slyunnn merged 1 commit into
matrixorigin:mainfrom
slyunnn:test/capture-runtime-jar

Conversation

@slyunnn

@slyunnn slyunnn commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

Refreshes the tracked lib/mo-tester-1.0-SNAPSHOT.jar from main after #202.\n\nMatrixOne BVT invokes run.sh, which loads the tracked lib jar when no target jar is present. This update makes the merged @capture runtime support available to that CI path.\n\nValidation: mvn -q test (74 tests, 0 failures, 0 errors, 0 skipped).

@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@slyunnn
slyunnn requested a review from Ariznawlll August 26, 2026 10:03

@Ariznawlll Ariznawlll left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed exact head 764d503bc6158945de284eeef1280273b24db68a. No blocking findings.

复核结果:

  • PR 中 tracked JAR 的 entry 集合与基于当前源码重新构建的 JAR 完全一致;逐项比较后,所有 .class 内容均字节级一致(0 个 class delta),差异仅为 manifest/pom.properties 中的构建者、JDK patch 版本和时间等构建元数据。
  • 使用隔离的 JDK 8 + Maven 环境执行 mvn clean package,74 tests 全部通过(0 failures / 0 errors / 0 skipped)。
  • 已从 tracked JAR 确认 @capture 所需的 capture name、runtime value replacement、scalar capture 和 failure handling 均已进入实际运行时产物;JAR 完整性检查通过。
  • 当前 head 的 case-format-checkBVT Test on Ubuntu/x64 均为 SUCCESS,其中 BVT 路径实际使用 tracked lib/mo-tester-1.0-SNAPSHOT.jar

Approved.

@slyunnn
slyunnn merged commit 334d3e1 into matrixorigin:main Aug 27, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants