[MAGNOLIA-3413] Fix all the cache filter tests and cover not yet tested scenarios Created: 25/Nov/10  Updated: 02/Feb/15  Resolved: 22/May/12

Status: Closed
Project: Magnolia
Component/s: cache
Affects Version/s: None
Fix Version/s: 4.5.3

Type: Bug Priority: Critical
Reporter: Jan Haderka Assignee: Ondrej Chytil
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: JPEG File failing tests.jpg    
Issue Links:
causality
caused by MAGNOLIA-3411 in case of a 302 (not modified) respo... Closed
is causing MAGNOLIA-4058 Re-activate all disabled (@Ignore) Un... Closed
relation
is related to MGNLADVCACHE-18 Create set of integration tests to te... Closed
supersession
is superseded by MGNLCACHE-42 Fix HttpServletResponse mocks in GZip... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:

 Description   

Since fixing MAGNOLIA-3411 the
CacheFilterTest.testJustSends304WithNoBodyIfRequestHeadersAskForItEvenOnCommitedResponse is commented. The test either needs to be fixed or it is obsolete and should be removed completely.



 Comments   
Comment by Daniel Lipp [ 21/Mar/12 ]

As a next step we should reactivate the currently ignored tests - 44 right now.

I'd recommend to first migrate the ignored tests to Mockito - should make it much simpler to reactivate them.

Generated at Mon Feb 12 03:46:14 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.