You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@chemistry.apache.org by "Michael Dürig (JIRA)" <ji...@apache.org> on 2009/12/16 16:33:18 UTC
[jira] Created: (CMIS-80) JCR tests: setup fails due to missing
mix:unstructured
JCR tests: setup fails due to missing mix:unstructured
------------------------------------------------------
Key: CMIS-80
URL: https://issues.apache.org/jira/browse/CMIS-80
Project: Chemistry
Issue Type: Bug
Components: jcr
Reporter: Michael Dürig
Attachments: CMIS-80.patch
As noted in CMIS-73 the tests don't pass because in JcrNewFolder save() tries to do a node.addMixin("mix:unstructured") which doesn't work as this mixin is not defined.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CMIS-80) JCR tests: setup fails due to missing
mix:unstructured
Posted by "Serge Huber (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CMIS-80?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12791575#action_12791575 ]
Serge Huber commented on CMIS-80:
---------------------------------
Thank you Florent !
> JCR tests: setup fails due to missing mix:unstructured
> ------------------------------------------------------
>
> Key: CMIS-80
> URL: https://issues.apache.org/jira/browse/CMIS-80
> Project: Chemistry
> Issue Type: Bug
> Components: jcr
> Reporter: Michael Dürig
> Attachments: CMIS-80.patch, External_definition_file.patch
>
>
> As noted in CMIS-73 the tests don't pass because in JcrNewFolder save() tries to do a node.addMixin("mix:unstructured") which doesn't work as this mixin is not defined.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CMIS-80) JCR tests: setup fails due to missing
mix:unstructured
Posted by "Michael Dürig (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CMIS-80?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Michael Dürig updated CMIS-80:
------------------------------
Attachment: CMIS-80.patch
This patch registers a mix:unstructured with the repository.
> JCR tests: setup fails due to missing mix:unstructured
> ------------------------------------------------------
>
> Key: CMIS-80
> URL: https://issues.apache.org/jira/browse/CMIS-80
> Project: Chemistry
> Issue Type: Bug
> Components: jcr
> Reporter: Michael Dürig
> Attachments: CMIS-80.patch
>
>
> As noted in CMIS-73 the tests don't pass because in JcrNewFolder save() tries to do a node.addMixin("mix:unstructured") which doesn't work as this mixin is not defined.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CMIS-80) JCR tests: setup fails due to missing
mix:unstructured
Posted by "Michael Dürig (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CMIS-80?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12791410#action_12791410 ]
Michael Dürig commented on CMIS-80:
-----------------------------------
I also suggest to switch to Jackrabbit 2 (Apache Jackrabbit 2.0 beta4 ATM). If this is desired I can try to come up with a patch.
> JCR tests: setup fails due to missing mix:unstructured
> ------------------------------------------------------
>
> Key: CMIS-80
> URL: https://issues.apache.org/jira/browse/CMIS-80
> Project: Chemistry
> Issue Type: Bug
> Components: jcr
> Reporter: Michael Dürig
> Attachments: CMIS-80.patch
>
>
> As noted in CMIS-73 the tests don't pass because in JcrNewFolder save() tries to do a node.addMixin("mix:unstructured") which doesn't work as this mixin is not defined.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CMIS-80) JCR tests: setup fails due to missing
mix:unstructured
Posted by "Serge Huber (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CMIS-80?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Serge Huber updated CMIS-80:
----------------------------
Attachment: Node_type_registration_with_CND_file.patch
Here's my proposition for this patch, just in case :)
> JCR tests: setup fails due to missing mix:unstructured
> ------------------------------------------------------
>
> Key: CMIS-80
> URL: https://issues.apache.org/jira/browse/CMIS-80
> Project: Chemistry
> Issue Type: Bug
> Components: jcr
> Reporter: Michael Dürig
> Attachments: CMIS-80.patch, Node_type_registration_with_CND_file.patch
>
>
> As noted in CMIS-73 the tests don't pass because in JcrNewFolder save() tries to do a node.addMixin("mix:unstructured") which doesn't work as this mixin is not defined.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CMIS-80) JCR tests: setup fails due to missing
mix:unstructured
Posted by "Michael Dürig (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CMIS-80?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12791409#action_12791409 ]
Michael Dürig commented on CMIS-80:
-----------------------------------
Currently chemistry-jcr requires a repository with the mix:unstructured mixin. My patch registers such a node type with the repository enabling the test cases to actually run. Some of them even pass! While requiring mix:unstructured might not be desired in the long run, I think the patch can help to identify other problem with chemistry-jcr.
> JCR tests: setup fails due to missing mix:unstructured
> ------------------------------------------------------
>
> Key: CMIS-80
> URL: https://issues.apache.org/jira/browse/CMIS-80
> Project: Chemistry
> Issue Type: Bug
> Components: jcr
> Reporter: Michael Dürig
> Attachments: CMIS-80.patch
>
>
> As noted in CMIS-73 the tests don't pass because in JcrNewFolder save() tries to do a node.addMixin("mix:unstructured") which doesn't work as this mixin is not defined.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CMIS-80) JCR tests: setup fails due to missing
mix:unstructured
Posted by "Michael Dürig (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CMIS-80?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12791515#action_12791515 ]
Michael Dürig commented on CMIS-80:
-----------------------------------
> (but why is it deprecated ?)
Because this interface will be replaced by the standard JCR 2.0 API in Apache Jackrabbit 2.0.
> JCR tests: setup fails due to missing mix:unstructured
> ------------------------------------------------------
>
> Key: CMIS-80
> URL: https://issues.apache.org/jira/browse/CMIS-80
> Project: Chemistry
> Issue Type: Bug
> Components: jcr
> Reporter: Michael Dürig
> Attachments: CMIS-80.patch, External_definition_file.patch
>
>
> As noted in CMIS-73 the tests don't pass because in JcrNewFolder save() tries to do a node.addMixin("mix:unstructured") which doesn't work as this mixin is not defined.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CMIS-80) JCR tests: setup fails due to missing
mix:unstructured
Posted by "Serge Huber (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CMIS-80?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Serge Huber updated CMIS-80:
----------------------------
Attachment: (was: Node_type_registration_with_CND_file.patch)
> JCR tests: setup fails due to missing mix:unstructured
> ------------------------------------------------------
>
> Key: CMIS-80
> URL: https://issues.apache.org/jira/browse/CMIS-80
> Project: Chemistry
> Issue Type: Bug
> Components: jcr
> Reporter: Michael Dürig
> Attachments: CMIS-80.patch, External_definition_file.patch
>
>
> As noted in CMIS-73 the tests don't pass because in JcrNewFolder save() tries to do a node.addMixin("mix:unstructured") which doesn't work as this mixin is not defined.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Resolved: (CMIS-80) JCR tests: setup fails due to missing
mix:unstructured
Posted by "Florent Guillaume (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CMIS-80?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Florent Guillaume resolved CMIS-80.
-----------------------------------
Resolution: Fixed
I've committed a mix of both your patches.
r891381
The tests are still disabled until they all pass, but it's now just a matter of running them one by one and fixing what's missing :)
> JCR tests: setup fails due to missing mix:unstructured
> ------------------------------------------------------
>
> Key: CMIS-80
> URL: https://issues.apache.org/jira/browse/CMIS-80
> Project: Chemistry
> Issue Type: Bug
> Components: jcr
> Reporter: Michael Dürig
> Attachments: CMIS-80.patch, External_definition_file.patch
>
>
> As noted in CMIS-73 the tests don't pass because in JcrNewFolder save() tries to do a node.addMixin("mix:unstructured") which doesn't work as this mixin is not defined.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CMIS-80) JCR tests: setup fails due to missing
mix:unstructured
Posted by "Serge Huber (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CMIS-80?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Serge Huber updated CMIS-80:
----------------------------
Attachment: External_definition_file.patch
I've regenerated the patch file by including your suggestions :
- renamed NodeTypeRegistryImpl with JackrabbitNodeTypeRegistry (but why is it deprecated ?)
- added the child node definition to the mixin type
- removed the pom.xml file from the patch, as the tests are not executing correctly so that it no longer interferes with the building of the project.
> JCR tests: setup fails due to missing mix:unstructured
> ------------------------------------------------------
>
> Key: CMIS-80
> URL: https://issues.apache.org/jira/browse/CMIS-80
> Project: Chemistry
> Issue Type: Bug
> Components: jcr
> Reporter: Michael Dürig
> Attachments: CMIS-80.patch, External_definition_file.patch
>
>
> As noted in CMIS-73 the tests don't pass because in JcrNewFolder save() tries to do a node.addMixin("mix:unstructured") which doesn't work as this mixin is not defined.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CMIS-80) JCR tests: setup fails due to missing
mix:unstructured
Posted by "Serge Huber (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CMIS-80?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12791412#action_12791412 ]
Serge Huber commented on CMIS-80:
---------------------------------
Actually I also had worked on something like this, except that I added an external CND to be able to add definitions at a later point if needed.
I'll attach a patch if there is interest ?
> JCR tests: setup fails due to missing mix:unstructured
> ------------------------------------------------------
>
> Key: CMIS-80
> URL: https://issues.apache.org/jira/browse/CMIS-80
> Project: Chemistry
> Issue Type: Bug
> Components: jcr
> Reporter: Michael Dürig
> Attachments: CMIS-80.patch
>
>
> As noted in CMIS-73 the tests don't pass because in JcrNewFolder save() tries to do a node.addMixin("mix:unstructured") which doesn't work as this mixin is not defined.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CMIS-80) JCR tests: setup fails due to missing
mix:unstructured
Posted by "Michael Dürig (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CMIS-80?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12791429#action_12791429 ]
Michael Dürig commented on CMIS-80:
-----------------------------------
Serge, I'm not sure whether this is required but I think it would be a good idea to add a residual definition for a node also:
+ * (nt:base) = nt:unstructured multiple version
Also using JackrabbitNodeTypeManager instead of NodeTypeManagerImpl would be slightly cleaner. Just nitpicking...
> JCR tests: setup fails due to missing mix:unstructured
> ------------------------------------------------------
>
> Key: CMIS-80
> URL: https://issues.apache.org/jira/browse/CMIS-80
> Project: Chemistry
> Issue Type: Bug
> Components: jcr
> Reporter: Michael Dürig
> Attachments: CMIS-80.patch, Node_type_registration_with_CND_file.patch
>
>
> As noted in CMIS-73 the tests don't pass because in JcrNewFolder save() tries to do a node.addMixin("mix:unstructured") which doesn't work as this mixin is not defined.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.