[PLUTO-696] TCK: Contesting V2 tests that expect PortletURL.setParameter(String,String...) to remove the value when null is passed in a V2.0 portlet

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.0.0
• Fix Version/s: 3.0.1
• Component/s: tck
• Assignee: Neil Griffin (Inactive)
• Reporter: Neil Griffin (Inactive)
• Created: 06/Mar/18 21:05
• Updated: 14/Dec/22 14:37
• Resolved: 24/Mar/18 18:36

Description

Apache Pluto fails the legacy Portlet 2.0 TCK SetParameterWithValuesIllegalArgumentExceptionTestPortlet because PortletURL.setParameter(String name, String... values) does not throw IllegalArgumentException when null is passed for the value.

The following V 2.0 tests in the new Portlet 3.0 TCK are testing the Portlet 3.0 behavior, which is to remove the parameter from the URL when null is passed:

• V2URLTests_BaseURL_ApiRenderActurl_setParameterB7
• V2URLTests_BaseURL_ApiRenderRenurl_setParameterB7
• V2URLTests_BaseURL_ApiRenderResurl_setParameterB7

In order to fix this problem, these tests should test for IllegalArgumentException, and testing for removal should be done in V3URLTests.

Issue Links

• is related to: PLUTO-697 Spec: Fortify Section A.3.2 titled "Clarifications that may make V1.0 Portlets Non-compliant" with additional details