[MGNLPN-70] Rewrite tests that use RepositoryTestCase to use mocks instead Created: 09/Apr/14 Updated: 15/Apr/14 Resolved: 09/Apr/14 |
|
| Status: | Closed |
| Project: | Magnolia Personalization |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.0 |
| Type: | Task | Priority: | Neutral |
| Reporter: | Jaroslav Simak | Assignee: | Jaroslav Simak |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Task DoR: |
Empty
|
||||||||
| Date of First Response: | |||||||||
| Epic Link: | Personalisation | ||||||||
| Sprint: | 5.3 Sprint 6 | ||||||||
| Description |
|
Build time if p13n modules takes quite a lot of time because RepositoryTestCases are heavily used. We need to rewrite tests to use mocks to lower down build time. |
| Comments |
| Comment by Jaroslav Simak [ 09/Apr/14 ] |
| Comment by Roman Kovařík [ 09/Apr/14 ] |
|
Please remove:
+ // removeProperties(destination); commented out - causes test fails with mocks but not with real nodes, why?
from CopyUtil when integrating. It also would be good if we could rename CopyUtil to prevent collision with info.magnolia.cms.core.version.CopyUtil. |