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

Helios 14 #107

Closed
wants to merge 150 commits into from
Closed

Helios 14 #107

wants to merge 150 commits into from

Conversation

aleeusgr
Copy link
Owner

@aleeusgr aleeusgr commented May 27, 2023

No description provided.

…and a template for a transaction test

moving on to testing transactions, I need to think about beforeAll, beforeEach and where does NetworkEmulator belong
now need to add validatorAddress to the scope
figuring out test organisation I need to have a simple example that will show me what is valid
added lockAda as an importable statement
needed to get access to newer features around time

BREAKING CHANGE: 106
@aleeusgr
Copy link
Owner Author

 FAIL  tests/helios-vesting-lock.test.ts > a vesting contract lockAda transaction > tests lockAda tx import            
Error: TypeError on line 7: expected 2 args for main                                                                   
 ❯ Function.new node_modules/@hyperionbt/helios/helios.js:1421:10                                                      
 ❯ Function.typeError node_modules/@hyperionbt/helios/helios.js:1463:30                                                
 ❯ Site.typeError node_modules/@hyperionbt/helios/helios.js:1338:20                                                    
 ❯ FuncStatement.typeError node_modules/@hyperionbt/helios/helios.js:1735:21                                           
 ❯ MintingProgram.evalTypesInternal node_modules/@hyperionbt/helios/helios.js:36832:15                                 
 ❯ MintingProgram.evalTypes node_modules/@hyperionbt/helios/helios.js:37134:15                                         
 ❯ Function.new node_modules/@hyperionbt/helios/helios.js:36016:28                                                     
 ❯ Module.lockAda src/lockAda.ts:46:56                                                                                 
     46|  tx.addInputs(inputUtxos);                                                                                    
     47|                                                                                                               
     48|  const mintScript =`minting nft     

new helios version requires stating arg to main explicitly

BREAKING CHANGE: 106, 105, 104
old-tests/lock-ada.test.ts Outdated Show resolved Hide resolved
src/lockAda.ts Outdated
const txId: TxId = TxId::new(TX_ID)
const outputId: TxOutputId = TxOutputId::new(txId, ` + inputUtxos[0].utxoIdx + `)

func main(_, _, ctx: ScriptContext) -> Bool {
Copy link
Owner Author

Choose a reason for hiding this comment

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

is the error here?

src/matching-keyhash.js Outdated Show resolved Hide resolved
src/nft.hl Outdated Show resolved Hide resolved
const txId: TxId = TxId::new(TX_ID)
const outputId: TxOutputId = TxOutputId::new(txId, ` + inputUtxos[0].utxoIdx + `)

func main(_, _, ctx: ScriptContext) -> Bool {
Copy link
Owner Author

Choose a reason for hiding this comment

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

same change as in src/lockAda.ts

fixing errors after updating to helios-0.14.0

105
@aleeusgr
Copy link
Owner Author

805ef82

@aleeusgr aleeusgr closed this May 27, 2023
@aleeusgr aleeusgr deleted the helios-14 branch April 5, 2024 07:02
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.

None yet

1 participant