Skip to content
This repository has been archived by the owner on Mar 3, 2021. It is now read-only.

Various fixes in remix-tests #1151

Merged
merged 3 commits into from
Apr 3, 2019
Merged

Various fixes in remix-tests #1151

merged 3 commits into from
Apr 3, 2019

Conversation

yann300
Copy link
Collaborator

@yann300 yann300 commented Apr 2, 2019

that mostly concern the runTestSources.ts

@yann300 yann300 requested a review from 0mkara April 2, 2019 19:28
@@ -2,4 +2,4 @@ export { runTestFiles } from './runTestFiles'
export { runTestSources } from './runTestSources'
export { runTest } from './testRunner'
export * from './types'
export { assertLibCode } from '../sol/tests.sol.js'
export const assertLibCode = require('../sol/tests.sol.js')
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not sure about the benefit of exporting assertLibCode as const.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

the change is more about using require over from. it was curiously not working in the ide

Copy link
Collaborator

Choose a reason for hiding this comment

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

Is remix-ide code using ES6 ?

@yann300 yann300 merged commit d18880e into master Apr 3, 2019
@0mkara 0mkara deleted the remixideIntegration branch April 3, 2019 11:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants