Skip to content

Enable hooks acceptance test#12814

Draft
xin-hedera wants to merge 3 commits intomainfrom
enable-hooks-acceptance-test
Draft

Enable hooks acceptance test#12814
xin-hedera wants to merge 3 commits intomainfrom
enable-hooks-acceptance-test

Conversation

@xin-hedera
Copy link
Contributor

Description:

Related issue(s):

Fixes #

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Xin Li <xin@hashgraph.com>
@xin-hedera xin-hedera added this to the 0.148.0 milestone Jan 23, 2026
@lfdt-bot
Copy link

lfdt-bot commented Jan 23, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@codacy-production
Copy link

codacy-production bot commented Jan 23, 2026

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
-39.21% (target: -1.00%)
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (037d1b8) 27283 25615 93.89%
Head commit (c0f8588) 52771 (+25488) 28852 (+3237) 54.67% (-39.21%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#12814) 0 0 ∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Signed-off-by: Xin Li <xin@hashgraph.com>
@xin-hedera
Copy link
Contributor Author

blocked by SDK bug.

There also seems to be communication issue between CN v0.70.0-rc.2 and BN v0.26.0-rc1 so acceptance tests can't run at all with blockstream.

@steven-sheehy steven-sheehy removed this from the 0.148.0 milestone Feb 5, 2026
Signed-off-by: Xin Li <xin@hashgraph.com>
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.

3 participants