Skip to content

Write a routing acceptance test#85

Merged
joostdevries merged 6 commits intoember-cli:masterfrom
Gaurav0:routing_acceptance_test
Jul 28, 2015
Merged

Write a routing acceptance test#85
joostdevries merged 6 commits intoember-cli:masterfrom
Gaurav0:routing_acceptance_test

Conversation

@Gaurav0
Copy link
Copy Markdown
Contributor

@Gaurav0 Gaurav0 commented Jul 26, 2015

This adds an acceptance test using ember-cli-mirage and a whole bunch of interesting hacks.

@Gaurav0 Gaurav0 changed the title [WIP] [FAILING] Try to write a routing acceptance test Write a routing acceptance test Jul 27, 2015
@Gaurav0
Copy link
Copy Markdown
Contributor Author

Gaurav0 commented Jul 27, 2015

I got it working!!! Please review.

Comment thread app/adapters/application.js Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we store this in config/environment.js? That allows us to have a different value in the test environment from development/production, and should allow you to remove the host computed.

@Gaurav0
Copy link
Copy Markdown
Contributor Author

Gaurav0 commented Jul 28, 2015

@rwjblue Updated.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'd move this to helpers

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The main reason I haven't done so is that this uses closure over login, gist_id, and commit.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Understood.

joostdevries pushed a commit that referenced this pull request Jul 28, 2015
@joostdevries joostdevries merged commit 184193c into ember-cli:master Jul 28, 2015
@Gaurav0 Gaurav0 deleted the routing_acceptance_test branch March 21, 2016 13:47
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.

3 participants