[MAGNOLIA-5155] Remove redundancy of magnolia-test-public-webapp Created: 03/Jul/13  Updated: 25/Jul/13  Resolved: 04/Jul/13

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

Type: Task Priority: Neutral
Reporter: Magnolia International Assignee: Magnolia International
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Cloners
is cloned by MGNLEE-294 Remove redundancy of magnolia-ee-test... Closed
dependency
depends upon MAGNOLIA-3094 Also use contextPath when resolving m... Closed
supersession
is superseded by MAGNOLIA-5157 Further simplify test-webapp Closed
Template:
Acceptance criteria:
Empty
Task DoR:
Empty

 Description   

Since MAGNOLIA-3094, we actually do not need magnolia-test-public-webapp, since the magnolia.properties can be resolved/found by using the contextPath.

This could simplify the test-webapp quite a bit: we only need one, and it doesn't need to re-specify the whole magnolia.properties file anymore.



 Comments   
Comment by Magnolia International [ 03/Jul/13 ]
  • Have a local fix, but for some reason info.magnolia.module.servletsanity.servlets.UnicodeNormalizatonFilterTestServlet now fails - indeed, magnolia.utf8.enabled is not true, so the UnicodeNormalizationFilter isn't enabled, so the UnicodeNormalizationRequestWrapper isn't used. I can't see how this property was set to true before, since it isn't present in any of the magnolia.properties files.
  • Setting this property explicitly indeed fixes the test, but I'd like to investigate history before committing.
Generated at Mon Feb 12 04:02:30 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.