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

remove traceur runtime requirement #24

Open
jeffbcross opened this issue Dec 17, 2014 · 0 comments
Open

remove traceur runtime requirement #24

jeffbcross opened this issue Dec 17, 2014 · 0 comments
Milestone

Comments

@jeffbcross
Copy link
Contributor

As discussed in #21, it's problematic for benchpress runtime libraries to add $traceurRuntime and System to the window, as code under test may be using a conflicting version of Traceur. In the case of benchpress, the dependency injection library requires the traceur runtime to be added.

Proposed fix: write a lightweight DI implementation with subset of the angular/di.js API, that will ship with the bp.js runtime, but continue to use the real di.js for testing since it has nice mock utilities.

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

No branches or pull requests

1 participant