Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

YUIDocs and unit test cases #152

Closed
wants to merge 4 commits into from
Closed

YUIDocs and unit test cases #152

wants to merge 4 commits into from

Conversation

codecounselor
Copy link
Contributor

I was playing around with YUI Docs which seem to work assuming you use the tags correctly (http:https://yui.github.io/yuidoc/syntax/index.html#primary-tags).

Here is a screenshot:
screenshot 2015-07-27 00 25 55
If you want to generate them, read this page: https://github.com/cibernox/ember-cli-yuidoc

Just wanted to share this, not sure it needs merged just yet. But if you like it feel free.

Still trying to figure out the best stack for tests. I don't know that I really like QUnit all that much.

@jkleinsc
Copy link
Member

@codecounselor Have you read the Ember.js testing guide:
http:https://guides.emberjs.com/v1.13.0/testing/

QUnit is the default testing framework used by Ember, though I would be open to using something else as long as you could still test via ember test.

@codecounselor
Copy link
Contributor Author

Yeah, how do you think I figured it out :) What I have to determine is if the helpers for the alternative frameworks (probably jasmine or mocha) are on par. https://github.com/subeeshb/ember-jasmine

I'm leaning towards Jasmine since I like the nested support for suites and setup/teardown and the general slant towards BDD. Mocha/Chai/Sinon seems to also be a popular choice.

QUnit is older and entirely flat. I'm also not sure what it provides in the way of mocks/fakes

@tangollama
Copy link
Member

@jkleinsc, let's confirm on this tomorrow. Any reason we shouldn't merge this PR?

@taras
Copy link
Contributor

taras commented Oct 7, 2015

I would argue to stay with QUnit to keep the project accessible to most Ember developers.

@jkleinsc
Copy link
Member

jkleinsc commented Oct 7, 2015

Closing this PR in favor of the work that @taras did.

@jkleinsc jkleinsc closed this Oct 7, 2015
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.

None yet

4 participants