Remove Fx firstrun test. Bug 877202. #1753

Merged
merged 1 commit into from Mar 6, 2014

Conversation

Projects
None yet
2 participants
Member

jpetto commented Mar 4, 2014

  • Removes images, CSS & JS files, & templates.
  • Class-based view remains for later use.
Member

alexgibson commented Mar 5, 2014

Looks like you may also need to remove some of the tests here that are no longer needed?

Member

jpetto commented Mar 5, 2014

Yes indeed. Issued the PR just before flying out the door yesterday. What's that thing haste makes? Fixing up tests now.

Remove Fx firstrun test. Bug 877202.
- Removes images, CSS & JS files, & templates.

- Class-based view remains for later use.

- Updated class-based views & tests to use/expect a list
  of template names (conform to django intention)
Member

alexgibson commented Mar 6, 2014

LGTM, r+

alexgibson added a commit that referenced this pull request Mar 6, 2014

@alexgibson alexgibson merged commit 1c64a0e into mozilla:master Mar 6, 2014

1 check passed

default Jenkins build 'bedrock_github' #3404 has succeeded
Details

@jpetto jpetto deleted the jpetto:bug-877202-remove-fx-firstrun-tests branch Nov 13, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment