[Feature] Added failure only notifications

src/AppDelegate.m

@@ -67,6 +67,12 @@ - (IBAction)showPreferences:(id)sender {
 }
 
 - (BOOL)shouldShowNotificationFor:(TravisEvent *)eventData {
+  if ([[Preferences sharedPreferences] failureOnlyNotificationEnabled]) {
+    if (eventData.status != TravisEventStatusFailed) {
+      return FALSE;
+    }
+  }
+
   RepositoryFilter *filter;
   if ([[Preferences sharedPreferences] firehoseEnabled]) {
     filter = [RepositoryFilter filterThatAcceptsAllRepositories];
@@ -94,9 +100,14 @@ - (void)eventFetcher:(TravisEventFetcher *)eventFetcher gotEvent:(TravisEvent *)
 #pragma mark - NSUserNotificationCenterDelegate
 
 - (void)userNotificationCenter:(NSUserNotificationCenter *)center didActivateNotification:(NSUserNotification *)notification {
-  if ([notification activationType] == NSUserNotificationActivationTypeContentsClicked) {
+  if ([notification activationType] == NSUserNotificationActivationTypeContentsClicked || [notification activationType] == NSUserNotificationActivationTypeActionButtonClicked) {
     [[NSWorkspace sharedWorkspace] openURL:[NSURL URLWithString:[notification userInfo][@"URL"]]];
   }
 }
 
+- (BOOL)userNotificationCenter:(NSUserNotificationCenter *)center shouldPresentNotification:(NSUserNotification *)notification {
+  // Show notifications when the Preferences panel is visible.
+  return TRUE;
+}
+
 @end

src/Preferences.h

@@ -21,4 +21,7 @@
 - (BOOL)firehoseEnabled;
 - (void)setFirehoseEnabled:(BOOL)firehoseEnabled;
 
+- (BOOL)failureOnlyNotificationEnabled;
+- (void)setFailureOnlyNotificationEnabled:(BOOL)failuresOnlyNotificationsEnabled;
+
 @end

src/Preferences.m

@@ -10,6 +10,7 @@
 
 static NSString * const kRepositoriesSetting = @"repositories";
 static NSString * const kFirehoseSetting = @"firehose";
+static NSString * const kFailuresOnlyNotificationsEnabled = @"failuresOnly";
 
 @implementation Preferences
 
@@ -66,4 +67,15 @@ - (void)setFirehoseEnabled:(BOOL)firehoseEnabled {
   [userDefaults synchronize];
 }
 
+- (BOOL)failureOnlyNotificationEnabled {
+  NSUserDefaults *userDefaults = [NSUserDefaults standardUserDefaults];
+  return [userDefaults boolForKey:kFailuresOnlyNotificationsEnabled];
+}
+
+- (void)setFailureOnlyNotificationEnabled:(BOOL)failureOnlyNotificationEnabled {
+  NSUserDefaults *userDefaults = [NSUserDefaults standardUserDefaults];
+  [userDefaults setBool:failureOnlyNotificationEnabled forKey:kFailuresOnlyNotificationsEnabled];
+  [userDefaults synchronize];
+}
+
 @end

src/PreferencesController.h

@@ -18,11 +18,15 @@
 @property (assign) IBOutlet NSPanel *preferencesPanel;
 @property (weak) IBOutlet NSButtonCell *firehoseEnabledButtonCell;
 @property (weak) IBOutlet NSButtonCell *firehoseDisabledButtonCell;
+@property (weak) IBOutlet NSButtonCell *failureOnlyNotificationEnabledButtonCell;
+@property (weak) IBOutlet NSButtonCell *failureOnlyNotificationDisabledButtonCell;
 
 - (IBAction)addRepository:(id)sender;
 - (IBAction)removeRepository:(id)sender;
 - (IBAction)enableFirehose:(id)sender;
 - (IBAction)disableFirehose:(id)sender;
+- (IBAction)enableFailureOnlyNotification:(id)sender;
+- (IBAction)disableFailureOnlyNotification:(id)sender;
 - (IBAction)close:(id)sender;
 
 @end

src/PreferencesController.m

@@ -56,6 +56,14 @@ - (IBAction)disableFirehose:(id)sender {
   [[self preferences] setFirehoseEnabled:NO];
 }
 
+- (IBAction)enableFailureOnlyNotification:(id)sender {
+  [[self preferences] setFailureOnlyNotificationEnabled:YES];
+}
+
+- (IBAction)disableFailureOnlyNotification:(id)sender {
+  [[self preferences] setFailureOnlyNotificationEnabled:NO];
+}
+
 #pragma mark - NSWindowDelegate
 
 - (void)windowDidBecomeKey:(NSNotification *)notification {
@@ -66,6 +74,13 @@ - (void)windowDidBecomeKey:(NSNotification *)notification {
     [[self firehoseEnabledButtonCell] setObjectValue:@(NO)];
     [[self firehoseDisabledButtonCell] setObjectValue:@(YES)];
   }
+  if ([[self preferences] failureOnlyNotificationEnabled]) {
+    [[self failureOnlyNotificationEnabledButtonCell] setObjectValue:@(YES)];
+    [[self failureOnlyNotificationDisabledButtonCell] setObjectValue:@(NO)];
+  } else {
+    [[self failureOnlyNotificationEnabledButtonCell] setObjectValue:@(NO)];
+    [[self failureOnlyNotificationDisabledButtonCell] setObjectValue:@(YES)];
+  }
 }
 
 #pragma mark - NSTableViewDataSource

test/PreferencesControllerTests.m

@@ -41,4 +41,14 @@ - (void)testDisableFirehoseSetting {
   [verify(_preferences) setFirehoseEnabled:NO];
 }
 
+- (void)testEnableFailureOnlyNotificationSetting {
+  [_preferencesController enableFailureOnlyNotification:nil];
+  [verify(_preferences) setFailureOnlyNotificationEnabled:YES];
+}
+
+- (void)testDisableFailureOnlyNotificationSetting {
+  [_preferencesController disableFailureOnlyNotification:nil];
+  [verify(_preferences) setFailureOnlyNotificationEnabled:NO];
+}
+
 @end