Skip to content

Conversation

muffinresearch
Copy link
Contributor

@muffinresearch muffinresearch commented Jun 3, 2016

Fixes up 'invalid' used as eventAction and adds more tests to cover that part of the code to make sure events are fired correctly.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling b6b1395 on muffinresearch:fix-invalid-uninstall into 8bb6e30 on mozilla:master.

const guid = '@uninstall';
const installURL = 'https://mysite.com/download.xpi';


Copy link
Contributor

Choose a reason for hiding this comment

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

This looks superfluous.

@mstriemer
Copy link
Contributor

r+wc

@muffinresearch muffinresearch force-pushed the fix-invalid-uninstall branch from b6b1395 to 7851373 Compare June 3, 2016 20:54
@muffinresearch muffinresearch force-pushed the fix-invalid-uninstall branch from 7851373 to 73b73ba Compare June 3, 2016 20:55
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 73b73ba on muffinresearch:fix-invalid-uninstall into 8bb6e30 on mozilla:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 73b73ba on muffinresearch:fix-invalid-uninstall into 8bb6e30 on mozilla:master.

@muffinresearch muffinresearch merged commit 11d6f69 into mozilla:master Jun 3, 2016
@muffinresearch muffinresearch deleted the fix-invalid-uninstall branch June 3, 2016 20:58
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