Skip to content

ci: clean up tests.#340

Merged
hantmac merged 2 commits into
databendlabs:mainfrom
youngsofun:fix
Aug 13, 2025
Merged

ci: clean up tests.#340
hantmac merged 2 commits into
databendlabs:mainfrom
youngsofun:fix

Conversation

@youngsofun
Copy link
Copy Markdown
Member

  1. use testng`s own asserts, reduce unnecessary test deps.
  2. rm deps Gson.
  3. unify test group name to uppercase.

@youngsofun
Copy link
Copy Markdown
Member Author

cc @hantmac

Run jwgmeligmeyling/checkstyle-github-action@master
  with:
    path: **/checkstyle-result.xml
    token: ***
    name: Checkstyle Report
    title: Checkstyle Results
  env:
    JAVA_HOME: /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/11.0.28-6/x64
    JAVA_HOME_11_X64: /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/11.0.28-6/x64
With the provided path, there will be 2 results uploaded
Creating annotations for /home/runner/work/databend-jdbc/databend-jdbc/databend-client/target/checkstyle-result.xml
Creating annotations for /home/runner/work/databend-jdbc/databend-jdbc/databend-jdbc/target/checkstyle-result.xml
Uploading 0 / 0 annotations to GitHub as Checkstyle Report with conclusion success
Error: HttpError: Resource not accessible by integration

@youngsofun youngsofun requested review from ZhiHanZ and hantmac August 13, 2025 14:17
@hantmac hantmac merged commit b056d7a into databendlabs:main Aug 13, 2025
2 of 3 checks passed
@hantmac hantmac mentioned this pull request Nov 12, 2025
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