Skip to content

Conversation

@Dkwcs
Copy link
Contributor

@Dkwcs Dkwcs commented Dec 23, 2025

Description of change

New aa tests that use random object have been implemented:
randomness.move

Links to any relevant issues

Fixed #9583 .

How the change has been tested

Run iota-adapter-transactional-tests tests
With cargo simtest

@Dkwcs Dkwcs self-assigned this Dec 23, 2025
@Dkwcs Dkwcs requested a review from a team as a code owner December 23, 2025 11:02
@Dkwcs Dkwcs added the vm-language Issues related to the VM & Language Team label Dec 23, 2025
@vercel
Copy link

vercel bot commented Dec 23, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

6 Skipped Deployments
Project Deployment Review Updated (UTC)
apps-backend Ignored Ignored Preview Jan 13, 2026 5:20pm
apps-ui-kit Ignored Ignored Preview Jan 13, 2026 5:20pm
iota-evm-bridge Ignored Ignored Preview Jan 13, 2026 5:20pm
iota-multisig-toolkit Ignored Ignored Preview Jan 13, 2026 5:20pm
rebased-explorer Ignored Ignored Preview Jan 13, 2026 5:20pm
wallet-dashboard Ignored Ignored Preview Jan 13, 2026 5:20pm

@iota-ci iota-ci added the sc-platform Issues related to the Smart Contract Platform group. label Dec 23, 2025
@Dkwcs Dkwcs linked an issue Dec 23, 2025 that may be closed by this pull request
@Dkwcs Dkwcs requested review from miker83z and valeriyr December 23, 2025 13:54
#[authenticator]
public fun authenticate(
_account: &AbstractAccount,
_random: &mut Random,
Copy link
Contributor

Choose a reason for hiding this comment

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

This test can be removed. It doesn't check randomness, but the verifier rule that is related to mutable references.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed f0cbfb9

Copy link
Contributor

@miker83z miker83z left a comment

Choose a reason for hiding this comment

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

could you implement a test similar to the attack ptb as in the docs?

@Dkwcs
Copy link
Contributor Author

Dkwcs commented Jan 9, 2026

could you implement a test similar to the attack ptb as in the docs?

ead4034

@miker83z miker83z force-pushed the vm-lang/aa-auth/9583-Test-randomness-for-authenticators branch from 5f6b14d to 1d39ec5 Compare January 13, 2026 17:19
@miker83z miker83z merged commit b89b213 into vm-lang/aa-auth/8805-beta-feature-branch Jan 13, 2026
39 of 40 checks passed
@miker83z miker83z deleted the vm-lang/aa-auth/9583-Test-randomness-for-authenticators branch January 13, 2026 17:49
miker83z added a commit that referenced this pull request Jan 19, 2026
…ators (#9693)

# Description of change

New aa tests that use random object have been implemented:
`randomness.move`

## Links to any relevant issues

Fixed #9583 .

## How the change has been tested

Run iota-adapter-transactional-tests tests
With `cargo simtest`

---------

Co-authored-by: miker83z <mirko.zichichi@iota.org>
miker83z added a commit that referenced this pull request Jan 26, 2026
…ators (#9693)

# Description of change

New aa tests that use random object have been implemented:
`randomness.move`

## Links to any relevant issues

Fixed #9583 .

## How the change has been tested

Run iota-adapter-transactional-tests tests
With `cargo simtest`

---------

Co-authored-by: miker83z <mirko.zichichi@iota.org>
miker83z added a commit that referenced this pull request Jan 26, 2026
…ators (#9693)

# Description of change

New aa tests that use random object have been implemented:
`randomness.move`

## Links to any relevant issues

Fixed #9583 .

## How the change has been tested

Run iota-adapter-transactional-tests tests
With `cargo simtest`

---------

Co-authored-by: miker83z <mirko.zichichi@iota.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sc-platform Issues related to the Smart Contract Platform group. vm-language Issues related to the VM & Language Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[AA] Test randomness for authenticators

4 participants