Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Bug 915639 email notification tests #12199

Merged
merged 1 commit into from Sep 27, 2013

Conversation

jrburke
Copy link
Contributor

@jrburke jrburke commented Sep 13, 2013

Bulk integration tests for recent email stories. Adds all the helpers to the apps/email/test/marionette/lib folder, creates some new helpers.

Modified two HTML files to add a class name to make it easier to find them for tests.

@@ -0,0 +1,30 @@
/*global module, console, process */
Copy link
Contributor

Choose a reason for hiding this comment

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

I like this! It'd be nice if we could abstract it from email. Perhaps we could put this function in https://github.com/mozilla-b2g/marionette-helper as MarionetteHelper#debug. Then if I am the Calendar app, I could do

var client = marionette.client();
client.helper.debug('calendar', Calendar.prototype);
var app = new Calendar(client);
...

Thoughts?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

jrburke added a commit that referenced this pull request Sep 27, 2013
Bug 915639 email notification tests r=gaye
@jrburke jrburke merged commit 49e118a into mozilla-b2g:master Sep 27, 2013
@jrburke jrburke deleted the bug915639-email-notify-tests branch September 27, 2013 01:44
jrburke added a commit that referenced this pull request Oct 1, 2013
Bug 915639 email notification tests r=gaye(cherry picked from commit 49e118a)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants