Skip to content

Basic blueprints tests #129

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 30 commits into from
Feb 1, 2017
Merged

Basic blueprints tests #129

merged 30 commits into from
Feb 1, 2017

Conversation

@kimroen
Copy link
Owner

@kimroen kimroen commented Dec 11, 2016 •
edited
Loading

Takes over from #106

Fixes #101

These tests are extremely closely based on the ones found in the ember-cli-legacy-blueprints tests, much like the templates themselves.

To do

  • Set up running on Travis
  • ~~~Make the route and resource tests run~~~
  • Figure out what to do with the view tests (they are failing)
  • ~~~Make a list of all the options that need testing per blueprint, and then write tests for those~~~
  • Describe how to run tests in README

Improvements?

kimroen added 24 commits December 10, 2016 17:21
@kimroen
Install blueprint test helpers
29e6f54
@kimroen
Set up the first test
7092846
@kimroen
Add inital test for adapter blueprint
4b63982
@kimroen
Add inital test for adapter-test blueprint
844df1b
@kimroen
Add inital test for addon-import blueprint
dba6358
@kimroen
Add inital test for component and component-test blueprints
66dccac
@kimroen
Add inital test for component-addon blueprint
266119d
@kimroen
Add inital test for controller and controller-test blueprint
0fce919
@kimroen
Add inital test for helper blueprint
679521a
@kimroen
Add inital test for helper-addon blueprint
8e11700
@kimroen
Add inital test for initializer blueprint
3e5eab6
@kimroen
Add inital test for initializer-addon blueprint
a55e401
@kimroen
Add inital test for mixin blueprint
7f61cb6
@kimroen
Add inital test for model blueprint
d5080f9
@kimroen
Add inital test for resource blueprint
b8a36fa 
Skipped for now, because the default ember project generated doesn't have a `router.coffee`.
@kimroen
Add inital test for route blueprint
3f53682 
Skipping for now, because there's no system for generating the `router.coffee` file.
@kimroen
Add inital test for route-addon blueprint
ea0f86a
@kimroen
Add inital test for serializer blueprint
745db6c
@kimroen
Add inital test for service blueprint
b3aded0
@kimroen
Add inital test for template blueprint
572db33 
This one is so basic that we might just remove it - we just want to make sure that templates are generated even when the coffee blueprints are being used, and that happens just fine through the other blueprint tests.
@kimroen
Add inital test for test-helper blueprint
4bc05a7
@kimroen
Add inital test for transform blueprint
adea23e
@kimroen
Add inital test for util blueprint
010eaac
@kimroen
Add inital test for view blueprint
105bc4a 
The view-test blueprint tests are not passing for these and I'm not sure why, so they are skipped for now.
@kimroen kimroen mentioned this pull request Dec 11, 2016
Closed
@kimroen
Fix view-test blueprint
d397293 
The upstream implementation doesn't use the newer separation of qunit and mocha test file structures, so the imported blueprint implementation didn't find the files where it expected to. I renamed the file to where the ember-legacy-blueprints view-test blueprint expected to find it.
@kimroen
Update travis config
a80fa12 
- Install phantomjs differently
- Cache different caches
@kimroen kimroen self-assigned this Jan 31, 2017
@kimroen
Move ember-try running to a script and add node tests
faa0fff
@kimroen
Trigger running node tests differently
fefdaba 
I'm setting $NODE_TESTS to a specific value and checking for that instead of digging in to how to check for existence so that I can move on.
@kimroen
Move before install and install steps to scripts
24f8b6b 
This is done to skip irrelevant installations based on test type.
@kimroen
Add bare minimum instructions for developing and running tests
e3f6603 
Heavily borrowed from the `ember-cli-legacy-blueprints` addon.
@kimroen
Copy link
Owner Author

kimroen commented Feb 1, 2017

Merging this now so I can keep iterating separately.

@kimroen kimroen changed the title [WIP] Blueprints tests Basic blueprints tests Feb 1, 2017
@kimroen kimroen merged commit 4231a98 into master Feb 1, 2017
@kimroen kimroen deleted the blueprints-tests branch February 1, 2017 21:29
@kimroen kimroen mentioned this pull request Feb 1, 2017
Merged
kimroen added a commit that referenced this pull request Feb 6, 2017
@kimroen
Update changelog for #123, #124, and #129
e7a13cd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@kimroen kimroen

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Set up tests

1 participant

@kimroen