FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add template instrumentation. by rwjblue · Pull Request #103 · ember-cli-code-coverage/ember-cli-code-coverage · GitHub

Add template instrumentation. - #103

Open
rwjblue wants to merge 7 commits into
masterfrom
template-instrumentation
Open

Add template instrumentation.#103
rwjblue wants to merge 7 commits into
masterfrom
template-instrumentation

Conversation

rwjblue commented Mar 19, 2017
edited by RobbieTheWagner
Loading

Copy link
Copy Markdown
Collaborator

This PR adds support for instrumentation of templates roughly the same as JS instrumentation (using a custom set of helpers and a template AST transform to instrument templates).

There are still some issues with this approach that we need to vet/review.

  • Need more testing with "real world" templates.
  • Add logic in treeFor to prevent app and addon tree from being emitted unless COVERAGE=true.
  • Figure out how to handle instrumentation of usages of htmlbars-inline-precompiler.
  • Fix issues with istanbul parsing coverage data.
  • Decide if we are willing to do a major version bump, or if we have to rewrite in ES5.

Paired on this with @rondale-sc.

rwjblue commented Mar 19, 2017

Copy link
Copy Markdown
Collaborator Author

Decide if we are willing to do a major version bump, or if we have to rewrite in ES5.

@kategengler - Thoughts on this one?

rwjblue commented Mar 19, 2017

Copy link
Copy Markdown
Collaborator Author

OK, this is working now. Here is an example after running the test suite of Ember Observer:

Copy link
Copy Markdown
Collaborator

That's amazing!

Copy link
Copy Markdown
Collaborator

Seconding amazing!

For:

Decide if we are willing to do a major version bump, or if we have to rewrite in ES5.

I'm fine with a major version bump.

Copy link
Copy Markdown

@rwjblue This does look amazing - any updates?

Copy link
Copy Markdown
Collaborator

@rwjblue how can I help get this across the finish line?

Copy link
Copy Markdown

Yes, same thoughts about amazing and how can we get this merged?

Also, would it show coverage for the js side of helpers/components?

Copy link
Copy Markdown
Collaborator

@knownasilya this is likely on hold, due to the new spike. We'll be updating a lot in the coming weeks.

Copy link
Copy Markdown

This looks really nice. Would like to see this make it in 👍

Copy link
Copy Markdown

@kategengler @rwjblue Are there any plans on this being completed soon?

rwjblue commented Sep 17, 2020

Copy link
Copy Markdown
Collaborator Author

The general idea here is still valid, just need to complete off the to do items.

harishv98 commented May 5, 2021
edited
Loading

Copy link
Copy Markdown

@rwjblue If this is on hold, is there any way i can contribute and get this merged ?
Really looking forward to use this.

Copy link
Copy Markdown

Given that template only components are a prevalent thing, this would seem even more important

mukilane mentioned this pull request Jun 12, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.


Back | FazBrowse Home | New Git URL