-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge multi-inheritance POC #10242 #10243
Conversation
} | ||
|
||
@Test | ||
public void testMoveArchiveAndResetFromTwoParents() |
Check warning
Code scanning / Pmd (reported by Codacy)
JUnit tests should include assert() or fail()
@@ -152,10 +154,62 @@ | |||
|
|||
} | |||
|
|||
@Test | |||
public void testSyncProjectFromTwoParents() |
Check warning
Code scanning / Pmd (reported by Codacy)
JUnit tests should include assert() or fail()
@@ -14,10 +14,10 @@ | |||
|
|||
|
|||
@Test | |||
public void createProjectWithParent() | |||
public void createProjectWithOneParent() |
Check warning
Code scanning / Pmd (reported by Codacy)
JUnit tests should include assert() or fail()
modules/lib/lib-project/src/main/java/com/enonic/xp/lib/project/mapper/ProjectMapper.java
Outdated
Show resolved
Hide resolved
@@ -759,4 +765,31 @@ private Context adminContext() | |||
.authInfo( AuthenticationInfo.copyOf( ContextAccessor.current().getAuthInfo() ).principals( RoleKeys.ADMIN ).build() ) | |||
.build(); | |||
} | |||
|
|||
private boolean createsCircleDependency( final ProjectName projectName, final List<ProjectName> parentNames ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
strange name for method. What about shoudCreateCircleDependency
?
* Multi-inheritance in layers #10242 * NPE on project create after multi-inheritance implementation #10338 * Wrong parent can be selected in multi-inheritance sync #10343 (#10344) * Wrong parent can be selected for sync during layer init #10346 (#10347) * layers cycle when project gets created #10350 (#10355) * Config to enable multilayer #10351 (#10359) --------- Co-authored-by: rymsha <rymsha@yandex.ru>
No description provided.