Skip to content

test(permission): cover default active operation bitmap#936

Open
gummy789j wants to merge 1 commit into
release_v4.9.7from
followup/update-permission-op-labels
Open

test(permission): cover default active operation bitmap#936
gummy789j wants to merge 1 commit into
release_v4.9.7from
followup/update-permission-op-labels

Conversation

@gummy789j

Copy link
Copy Markdown
Collaborator

Summary

Follow-up to #935.

  • Extract the default active-permission operations bitmap into a named constant.
  • Use clearer labels for operations that are present in the bitmap but not listed in the wallet-cli friendly label map.
  • Add regression coverage to ensure the default active bitmap does not include ContractType 51 again.

Notes

The wording now distinguishes enum-backed unlisted operations from truly unknown operation codes:

  • Unlisted op 51 (ShieldedTransferContract)
  • Unknown op 255

Verification

  • git diff --check
  • Attempted ./gradlew test --tests org.tron.core.manager.UpdateAccountPermissionInteractiveTest, but this local machine does not have a Java Runtime installed.

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.

1 participant