Skip to content
Open
Changes from all commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
a53997b
enable simple fees
joshmarinacci Feb 5, 2026
89de6e9
spotless
joshmarinacci Feb 5, 2026
10079d3
Merge branch 'main' into simplefees-consensus-tests-fixes
joshmarinacci Feb 5, 2026
0be9900
improve FeeResult debug output
joshmarinacci Feb 5, 2026
b6c2498
spotless
joshmarinacci Feb 5, 2026
86aff88
fix issue
joshmarinacci Feb 5, 2026
82e8d6d
disable simple fees
joshmarinacci Feb 5, 2026
ecf6e90
fix file service tests
joshmarinacci Feb 5, 2026
40b7da8
spotless
joshmarinacci Feb 5, 2026
85b8bc1
spotless
joshmarinacci Feb 5, 2026
a3bdd00
fixes for TopicCreateSuite
joshmarinacci Feb 5, 2026
80cdeff
Merge branch 'main' into simplefees-consensus-tests-fixes
joshmarinacci Feb 5, 2026
7da617c
Update hedera-node/test-clients/src/main/java/com/hedera/services/bdd…
joshmarinacci Feb 6, 2026
5d3cef4
fix compile error
joshmarinacci Feb 6, 2026
6a18978
turn off simple fees again
joshmarinacci Feb 6, 2026
f1f401c
Merge branch 'main' into simplefees-consensus-tests-fixes
joshmarinacci Feb 6, 2026
d9b387b
move util function to FeesChargingUtils
joshmarinacci Feb 6, 2026
de21579
Merge remote-tracking branch 'origin/simplefees-consensus-tests-fixes…
joshmarinacci Feb 6, 2026
c95d1d9
move util function again and rename
joshmarinacci Feb 6, 2026
f044b3c
Merge branch 'main' into simplefees-consensus-tests-fixes
joshmarinacci Feb 6, 2026
f92e047
Merge branch 'main' into simplefees-consensus-tests-fixes
joshmarinacci Feb 6, 2026
4cb4742
remove duplicated constant
joshmarinacci Feb 6, 2026
8d5778e
add more fee estimation util methods
joshmarinacci Feb 6, 2026
9d4d7a4
fix tests when running under simple fees
joshmarinacci Feb 6, 2026
4c00014
Merge branch 'main' into simplefees-moar-fixes
joshmarinacci Feb 9, 2026
11db33d
Merge branch 'main' into simplefees-moar-fixes
joshmarinacci Feb 9, 2026
4e771e5
Merge branch 'main' into simplefees-moar-fixes
joshmarinacci Feb 9, 2026
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,10 @@ public class PrivilegedOpsTest {
final Stream<DynamicTest> freezeAdminPrivilegesAsExpected() {
return defaultHapiSpec("freezeAdminPrivilegesAsExpected")
.given(
cryptoCreate(FREEZE_ADMIN),
cryptoCreate(ADDRESS_BOOK_CONTROL),
cryptoCreate(EXCHANGE_RATE_CONTROL),
cryptoCreate(FEE_SCHEDULE_CONTROL),
Comment on lines +79 to +82
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't understand, how does this help?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Without these the test fails with INSUFFICIENT_PAYER_BALANCE instead of AUTHORIZATION_FAILED, when running under simple fees.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is strange that the missing accounts fail with INSUFFICIENT_PAYER_BALANCE instead of something more accurate like NO_SUCH_USER.

cryptoCreate(CIVILIAN),
cryptoTransfer(tinyBarsFromTo(GENESIS, EXCHANGE_RATE_CONTROL, ONE_MILLION_HBARS)))
.when(
Expand Down
Loading