Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: add integration / render test #10

Merged
merged 2 commits into from
Aug 17, 2020
Merged

Conversation

mydea
Copy link
Contributor

@mydea mydea commented Aug 17, 2020

I started working on this in parallel and added an integration test (that was formerly failing but is now succeeding). Not sure if this is interesting, feel free to merge it or close it if you prefer, I already made the work ;)

Related to #7

Copy link
Contributor

@buschtoens buschtoens left a comment

Choose a reason for hiding this comment

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

Great, thank you!

The more tests the better. And it's good to have one that actually renders to the template! 😇

tests/integration/cached-decorator-test.ts Outdated Show resolved Hide resolved
tests/integration/cached-decorator-test.ts Outdated Show resolved Hide resolved
@buschtoens buschtoens changed the title chore(test): Add integration test test: add integration / render test Aug 17, 2020
@buschtoens buschtoens merged commit 66486f8 into ember-polyfills:main Aug 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants