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

[ENHANCEMENT] Adding Report option to 'ember test' #4106

Merged
merged 1 commit into from May 19, 2015
Merged

[ENHANCEMENT] Adding Report option to 'ember test' #4106

merged 1 commit into from May 19, 2015

Conversation

step2yeung
Copy link
Contributor

This passes a reporter option to testem, which will invoke test reporting option in testem using the specified test reporter.
This option will only be invoked in testem CI, it will be ignored in testem development mode.

@@ -85,7 +86,7 @@ module.exports = Command.extend({
var testOptions = this.assign({}, commandOptions, {
outputPath: outputPath,
project: this.project,
configFile: this._generateCustomConfigFile(commandOptions)
configFile: this._generateCustomConfigFile(commandOptions),
Copy link
Member

Choose a reason for hiding this comment

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

Remove the trailing comma.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed.

@step2yeung step2yeung changed the title Adding Report option to 'ember test' [ENHANCEMENT] Adding Report option to 'ember test' May 19, 2015
@stefanpenner
Copy link
Contributor

#4114 should fix the appveyor failure

stefanpenner added a commit that referenced this pull request May 19, 2015
[ENHANCEMENT] Adding Report option to 'ember test'
@stefanpenner stefanpenner merged commit f9ff5ca into ember-cli:master May 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants