Skip to content
This repository was archived by the owner on Dec 31, 2022. It is now read-only.

Update component and component-test blueprints #30

Merged
merged 2 commits into from
Feb 18, 2019

Conversation

simonihmig
Copy link
Contributor

Adds support for ember-mocha 0.14

Tests are copied from ember.js, after making them based on fixtures through emberjs/ember.js#17198

@simonihmig
Copy link
Contributor Author

Failing tests on Windows are weird. The contents seem to match, and as this is only failing on Windows I would assume this has something to do with line endings. But I already checked that line endings are set to UNIX-style in my IDE, and core.autocrlf is set to input in git as expected. And I even rewrote some fixture parts manually to make sure there are no strange invisible characters from copy/pasting...

Any ideas?

Adds support for ember-mocha 0.14
Tests are copied from ember.js, after making them based on fixtures through emberjs/ember.js#17198
@simonihmig simonihmig force-pushed the component-blueprint branch 2 times, most recently from 8acd88a to e0d1e90 Compare November 30, 2018 22:37
@simonihmig
Copy link
Contributor Author

Ok, green now ✅and ready to be merged!

The last commit fixes the line endings on Windows: a03dcca

Will create a backport PR for ember...

@chriskrycho
Copy link
Member

@simonihmig as with the other PRs you opened, I'm sorry this sat so long! I will try to get this sorted this week!

Copy link
Member

@jamescdavis jamescdavis left a comment

Choose a reason for hiding this comment

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

This looks great! I've restarted Travis so we can get a fresh build since this has sat for a bit. Will merge once it passes.


let content = fs.readFileSync(path.join(__dirname, '../fixtures', filePath), { encoding: 'utf-8' });
if (options.replace) {
content = content.replace(/<%= (\w+) =%>/g, (_match, key) => options.replace[key] || '');
Copy link
Member

Choose a reason for hiding this comment

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

Nice! You've made it so fixtures can be factories!

@jamescdavis jamescdavis merged commit 00cb0f3 into typed-ember:master Feb 18, 2019
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.

3 participants