-
Notifications
You must be signed in to change notification settings - Fork 563
Remix Tests Typescript implementation #1128
Conversation
remix-tests/src/compiler.ts
Outdated
} | ||
|
||
function writeTestAccountsContract (accounts: string[]) { | ||
var testAccountContract = require('../sol/tests_accounts.sol.js') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would recommand using ES6 lazy imports instead of node require
:
async function writeTestAccountsContract (accounts: string[]) {
const testAccountContract = await import('../sol/tests_accounts.sol.js')
...
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will have to wait till a Promisified
version of this library starts being developed.
remix-lib failing test is not related |
ethereum/populus is no longer maintained. Use ours.
@iurimatias is this ok to merge regarding the workshops next week? |
did the rebase worked ok? cause several commits actually belong to master. |
Yes it seems it worked. I can see |
i just don't get why this commit cc4dcb5 |
I rebased it after your fix last night. |
Remix Tests Typescript implementation