Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

does not work when using with new traceur-runtime.js #21

Closed
tbosch opened this issue Dec 11, 2014 · 3 comments · Fixed by #23
Closed

does not work when using with new traceur-runtime.js #21

tbosch opened this issue Dec 11, 2014 · 3 comments · Fixed by #23

Comments

@tbosch
Copy link

tbosch commented Dec 11, 2014

Benchpress includes di.js which includes traceur-runtime.js in version 0.0.25. However, Angular2 is using traceur 0.0.74, and the two traceur-runtime.js are not compatible either way (code generated for the on does not run with the other).

I think we should either get rid of traceur-runtime within bp.js, or implement the frame approach, i.e. there won't be a bp.js in the application under test, only in the parent frame.

@jeffbcross
Copy link
Contributor

Thanks, will look into this.

@jeffbcross
Copy link
Contributor

Getting rid of traceur-runtime would be nice since it's more bytes, and since it's adding junk to the global object. Perhaps it's worth building a simplified DI for benchpress.

The auto-running context (ie benchmark runs in its own window or iframe) doesn't automatically solve the problem, because there still needs to be a runner present with the benchmark, to actually execute the code, time the tests, collect results, dispatch result events, etc.

I opened #24 with a proposal to properly address this issue, without giving up on DI altogether. In the meantime, I'll merge a fix that renames the $traceurRuntime and System globals during the build process so that they don't collide.

@jeffbcross
Copy link
Contributor

Published to npm @0.2.2

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants