Issue 3048 - Mark inactive notifications instead of removing them

sitescripts/notifications/test/parser.py

 # coding: utf-8
 
 # This file is part of the Adblock Plus web scripts,
 # Copyright (C) 2006-2015 Eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
@@ skipping 45 matching lines @@
 severity = information
 title.en-US = The title
 message.en-US = The message
 """)
     notifications = parser.load_notifications()
     self.assertEqual(len(notifications), 1)
     self.assertEqual(notifications[0]["id"], "1")
     self.assertEqual(notifications[0]["severity"], "information")
     self.assertEqual(notifications[0]["title"]["en-US"], "The title")
     self.assertEqual(notifications[0]["message"]["en-US"], "The message")
+    self.assertNotIn("inactive", notifications[0])
 
   def test_inactive(self):
     self.notification_to_load = ("1", """
 inactive = True
 """)
     notifications = parser.load_notifications()
     self.assertEqual(len(notifications), 1)
     self.assertTrue(notifications[0]["inactive"])
 
   def test_in_range(self):
@@ skipping 38 matching lines @@
     current_time = datetime.datetime.now()
     hour_delta = datetime.timedelta(hours=1)
     start_time = current_time - hour_delta
     end_time = current_time + hour_delta
     self.notification_to_load = ("1", """
 start = %s
 end = %s
 """ % (_format_time(start_time), _format_time(end_time)))
     notifications = parser.load_notifications()
     self.assertEqual(len(notifications), 1)
+    self.assertNotIn("inactive", notifications[0])
     self.assertNotIn("start", notifications[0])
     self.assertNotIn("end", notifications[0])
 
 if __name__ == "__main__":
   unittest.main()