Cleaning data associated with the card upon deletion.

chrome/browser/resources/google_now/cards_unittest.gtestjs

Index: chrome/browser/resources/google_now/cards_unittest.gtestjs
diff --git a/chrome/browser/resources/google_now/cards_unittest.gtestjs b/chrome/browser/resources/google_now/cards_unittest.gtestjs
index aca37c65cce1e398dc8bf9f77c530866f0545b31..38e60c83b2da4082a4d53ae4a77f5d94e6ec6b97 100644
--- a/chrome/browser/resources/google_now/cards_unittest.gtestjs
+++ b/chrome/browser/resources/google_now/cards_unittest.gtestjs
@@ -36,10 +36,12 @@ function setUpCardManagerTest(fixture) {
       'chrome.alarms.clear',
       'chrome.alarms.create',
       'chrome.notifications.clear',
+      'chrome.storage.local.set',
       'instrumented.alarms.onAlarm.addListener',
       'instrumented.notifications.create',
       'instrumented.notifications.update',
-      'instrumented.storage.local.get'
+      'instrumented.storage.local.get',
+      'tasks.add'
   ]);
 
   chrome.runtime = {}; // No error.
@@ -398,13 +400,80 @@ TEST_F('GoogleNowCardsUnitTest', 'onAlarmShowHasDataUpdate', function() {
 TEST_F('GoogleNowCardsUnitTest', 'onAlarmHide', function() {
   // Tests onAlarm for the 'hide' alarm.
   var test = setUpCardManagerTest(this);
+  var tasksAddSavedArgs = new SaveMockArguments();
+  this.mockApis.expects(once()).
+      tasks_add(tasksAddSavedArgs.match(eq(CLEAR_CARD_TASK_NAME)), tasksAddSavedArgs.match(ANYTHING)).

[robliao, 2013/09/27 19:47:11]

Linebreak

[vadimt, 2013/09/27 20:10:56]

Done.

+      will(invokeCallback(tasksAddSavedArgs,1));
   this.mockApis.expects(once()).
       chrome_notifications_clear(testCardId, ANYTHING);
   this.mockApis.expects(once()).
       chrome_alarms_clear(expectedShowAlarmId);
   this.mockApis.expects(once()).
       chrome_alarms_clear(expectedHideAlarmId);
-
+  var storageGetSavedArgs = new SaveMockArguments();
+  this.mockApis.expects(once()).
+      instrumented_storage_local_get(
+          storageGetSavedArgs.match(
+              eqJSON(['notificationsData', 'notificationGroups'])),
+          storageGetSavedArgs.match(ANYTHING)).
+      will(invokeCallback(
+          storageGetSavedArgs,
+          1,
+          {
+            notificationsData: {
+              'TEST CARD ID': {testField: 'TEST VALUE'},
+              'TEST CARD ID 1': {testField: 'TEST VALUE 1'}
+            },
+            notificationGroups: {
+              groupA: {
+                cards: [
+                  {
+                    chromeNotificationId: 'TEST CARD ID',
+                    testField: 'TEST VALUE',
+                  },
+                  {
+                    chromeNotificationId: 'TEST CARD ID 1',
+                    testField: 'TEST VALUE 1',
+                  }
+                ]
+              },
+              groupB: {
+                cards: [
+                  {
+                    chromeNotificationId: 'TEST CARD ID 0',
+                    testField: 'TEST VALUE 0',
+                  },
+                  {
+                    chromeNotificationId: 'TEST CARD ID',
+                    testField: 'TEST VALUE',
+                  }
+                ]
+              }
+            }}));
+  this.mockApis.expects(once()).
+      chrome_storage_local_set(eqJSON({
+        notificationsData: {
+          'TEST CARD ID 1': {testField: 'TEST VALUE 1'}
+        },
+        notificationGroups: {
+          groupA: {
+            cards: [
+              {
+                chromeNotificationId: 'TEST CARD ID 1',
+                testField: 'TEST VALUE 1',
+              }
+            ]
+          },
+          groupB: {
+            cards: [
+              {
+                chromeNotificationId: 'TEST CARD ID 0',
+                testField: 'TEST VALUE 0',
+              }
+            ]
+          }
+        }
+      }));
 
   // Call tested method.
   test.alarmCallback({name: expectedHideAlarmId});