You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-issues@jackrabbit.apache.org by "Michael Dürig (JIRA)" <ji...@apache.org> on 2015/12/18 13:50:46 UTC

[jira] [Commented] (OAK-3803) Clean up the fixtures code in core and jcr modules

    [ https://issues.apache.org/jira/browse/OAK-3803?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15063917#comment-15063917 ] 

Michael Dürig commented on OAK-3803:
------------------------------------

Big +1 from my side from the patch. I think this cleans up a lot of legacy in the test. 

[~mreutegg], WDYT?

> Clean up the fixtures code in core and jcr modules
> --------------------------------------------------
>
>                 Key: OAK-3803
>                 URL: https://issues.apache.org/jira/browse/OAK-3803
>             Project: Jackrabbit Oak
>          Issue Type: Task
>          Components: core, jcr
>            Reporter: Tomek Rękawek
>              Labels: tech-debt
>             Fix For: 1.4
>
>         Attachments: OAK-3803.patch
>
>
> oak-core and oak-jcr modules uses the fixture mechanism to provide NodeStore implementations to the unit/integration tests. There is a few problems with the fixture implementation:
> * the {{NodeStoreFixture}} class is duplicated between two modules and supports different set of options (eg. the oak-core version doesn't support the RDB node store at all, while the oak-jcr doesn't support MemoryNodeStore)
> * it isn't possible to set the MongoDB URL manually from the Maven command line (it can be done for the RDB, though), which makes running the tests on a Mongo replica hard,
> * the Mongo fixture doesn't remove the test database after the test is done.
> There should be just one NodeStoreFixture implementation (the oak-jcr can reuse the oak-core version), supporting all values of the {{Fixture}} enum. The Mongo fixture should be more customisable and also should clean-up the database.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)