Opened 4 years ago

Closed 4 years ago

#2739 closed defect (invalid)

[Send notifications to a subset of the user base] Test notification handler with caching enabled

Reported by: Ross Assignee:
Priority: P3 Milestone:
Module: Sitescripts Keywords:
Cc: fhd Blocked By:
Blocking: #2162 Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Ross Verified working: no
Review URL(s):

Description

Background

Various notification changes have been tested for #2162. These were tested with caching disabled and should be tested again with caching enabled (as mentioned in #2277) after the latest fixes are confirmed.

What to test

Test that notifications are cached for X minutes?

Change History (3)

comment:1 Changed 4 years ago by Ross

  • Summary changed from [Send notifications to a subset of the user base] Test with handler with caching enabled to [Send notifications to a subset of the user base] Test notification handler with caching enabled

@fhd How exactly should caching work here?

comment:2 Changed 4 years ago by Ross

  • Ready unset

comment:3 Changed 4 years ago by fhd

  • Resolution set to invalid
  • Status changed from new to closed

To answer your question: We're caching requests with the same lastVersion request value for one minute.

However, closing this issue. It's more of a task issue (which we don't have currently, we typically use other systems for that), something we should test. Not exactly a proven defect (yet :D).

Note: See TracTickets for help on using tickets.