Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Currency contract for e2e tests #3040

Merged
merged 4 commits into from
Nov 30, 2021
Merged

Currency contract for e2e tests #3040

merged 4 commits into from
Nov 30, 2021

Conversation

piotr-iohk
Copy link
Contributor

Comments

Contract test from #3036 adjusted for testnet e2e tests. Appears to work fine:

  E2E Balance -> Sign -> Submit
    cannot balance on empty wallet
    ping-pong
    game
    mint-burn
    withdrawal
    currency <----------------

Executed full test suite against this branch:

Issue Number

ADP-1222, ADP-1234

@piotr-iohk piotr-iohk self-assigned this Nov 30, 2021
@piotr-iohk piotr-iohk changed the title Currency contract test for e2e tests Currency contract for e2e tests Nov 30, 2021
Copy link
Contributor

@HeinrichApfelmus HeinrichApfelmus left a comment

Choose a reason for hiding this comment

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

Looks good to me! 😊

# $ toBits "00" <> Bits.bits (fromIntegral idx :: Integer) <> toBits "001100"
# ```
def plutus_encode_idx(int)
raise "Not supported index. (0-127) are supported." if int > 127
Copy link
Contributor

Choose a reason for hiding this comment

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

"7 bits are enough for everyone" — Bill Gates
😂

@piotr-iohk
Copy link
Contributor Author

bors r+

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Nov 30, 2021

Build succeeded:

@iohk-bors iohk-bors bot merged commit 40eb034 into master Nov 30, 2021
@iohk-bors iohk-bors bot deleted the piotr/adp-1222/currency branch November 30, 2021 15:52
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