[dna-issues] [JBoss JIRA] Updated: (DNA-346) JR TCK Tests for SetPropertyStringTest and SetPropertyValueTest Fail

[Brian Carothers (JIRA)]

     [ https://jira.jboss.org/jira/browse/DNA-346?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Brian Carothers updated DNA-346:
--------------------------------

    Attachment: DNA-346.patch


Fixed a couple of minor defects that were keeping the tests from passing as per Javadoc in section 7.1.5 of the 1.0.1 spec.

- Added code to "compact" value arrays, that is, Node.setProperty(String, Value[]) and Node.setProperty(String, Value[], int) should ignore any null values in the value array.
- Added code (AbstractJcrNode.checkType) to throw a ValueFormatException if the values in the value arrays in Node.setProperty(String, Value[]) and Node.setProperty(String, Value[], int) are not all of the same type.  The spec permits coercing the values into the type expected by the property, but it does not permit different types of values in the array.
- Added code (AbstractJcrNode.checkCardinalityOfExistingProperty) to throw a ValueFormatException if one attempts to set a multi-valued value onto an existing single-valued array (or vice versa) directly through the node.

> JR TCK Tests for SetPropertyStringTest and SetPropertyValueTest Fail
> --------------------------------------------------------------------
>
>                 Key: DNA-346
>                 URL: https://jira.jboss.org/jira/browse/DNA-346
>             Project: DNA
>          Issue Type: Bug
>          Components: JCR
>    Affects Versions: 0.4
>            Reporter: Brian Carothers
>         Attachments: DNA-346.patch
>
>
> The L2 TCK tests SetPropertyStringTest and SetPropertyValueTest fail.  Opening this defect as a placeholder for the patch.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira