CCM NG: Fixed test failures in FolderManagerTest
git-svn-id: https://svn.libreccm.org/ccm/ccm_ng@6007 8810af33-2d31-482b-a856-94f89814c4df
parent
87464c7a87
commit
f96cd9f8d9
|
|
@ -138,6 +138,7 @@ public class FolderManagerTest {
|
||||||
com.arsdigita.ui.admin.applications.DefaultApplicationInstanceForm.class)
|
com.arsdigita.ui.admin.applications.DefaultApplicationInstanceForm.class)
|
||||||
.addClass(
|
.addClass(
|
||||||
com.arsdigita.ui.admin.applications.DefaultApplicationSettingsPane.class)
|
com.arsdigita.ui.admin.applications.DefaultApplicationSettingsPane.class)
|
||||||
|
.addClass(org.libreccm.imexport.Exportable.class)
|
||||||
.addPackage(org.librecms.dispatcher.ItemResolver.class.getPackage())
|
.addPackage(org.librecms.dispatcher.ItemResolver.class.getPackage())
|
||||||
.addPackage(com.arsdigita.util.Lockable.class.getPackage())
|
.addPackage(com.arsdigita.util.Lockable.class.getPackage())
|
||||||
.addPackage(com.arsdigita.web.BaseServlet.class.getPackage())
|
.addPackage(com.arsdigita.web.BaseServlet.class.getPackage())
|
||||||
|
|
@ -179,14 +180,14 @@ public class FolderManagerTest {
|
||||||
"content_section_id"})
|
"content_section_id"})
|
||||||
@InSequence(1000)
|
@InSequence(1000)
|
||||||
public void createDocumentsFolder() {
|
public void createDocumentsFolder() {
|
||||||
final Optional<Folder> parent = folderRepo.findById(-2005L);
|
final Optional<Folder> parent = folderRepo.findById(2005L);
|
||||||
assertThat(parent.isPresent(), is(true));
|
assertThat(parent.isPresent(), is(true));
|
||||||
|
|
||||||
final Folder test = folderManager.createFolder("test", parent.get());
|
final Folder test = folderManager.createFolder("test", parent.get());
|
||||||
|
|
||||||
assertThat(test, is(not(nullValue())));
|
assertThat(test, is(not(nullValue())));
|
||||||
assertThat(test.getName(), is(equalTo("test")));
|
assertThat(test.getName(), is(equalTo("test")));
|
||||||
assertThat(test.getSection().getObjectId(), is(-1100L));
|
assertThat(test.getSection().getObjectId(), is(1100L));
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
|
|
@ -203,14 +204,14 @@ public class FolderManagerTest {
|
||||||
"content_section_id"})
|
"content_section_id"})
|
||||||
@InSequence(1100)
|
@InSequence(1100)
|
||||||
public void createAssetsFolder() {
|
public void createAssetsFolder() {
|
||||||
final Optional<Folder> parent = folderRepo.findById(-2013L);
|
final Optional<Folder> parent = folderRepo.findById(2013L);
|
||||||
assertThat(parent.isPresent(), is(true));
|
assertThat(parent.isPresent(), is(true));
|
||||||
|
|
||||||
final Folder test = folderManager.createFolder("test", parent.get());
|
final Folder test = folderManager.createFolder("test", parent.get());
|
||||||
|
|
||||||
assertThat(test, is(not(nullValue())));
|
assertThat(test, is(not(nullValue())));
|
||||||
assertThat(test.getName(), is(equalTo("test")));
|
assertThat(test.getName(), is(equalTo("test")));
|
||||||
assertThat(test.getSection().getObjectId(), is(-1100L));
|
assertThat(test.getSection().getObjectId(), is(1100L));
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test(expected = IllegalArgumentException.class)
|
@Test(expected = IllegalArgumentException.class)
|
||||||
|
|
@ -234,7 +235,7 @@ public class FolderManagerTest {
|
||||||
@ShouldThrowException(IllegalArgumentException.class)
|
@ShouldThrowException(IllegalArgumentException.class)
|
||||||
@InSequence(1500)
|
@InSequence(1500)
|
||||||
public void createFolderNullName() {
|
public void createFolderNullName() {
|
||||||
final Optional<Folder> parent = folderRepo.findById(-2005L);
|
final Optional<Folder> parent = folderRepo.findById(2005L);
|
||||||
assertThat(parent.isPresent(), is(true));
|
assertThat(parent.isPresent(), is(true));
|
||||||
|
|
||||||
final Folder test = folderManager.createFolder(null, parent.get());
|
final Folder test = folderManager.createFolder(null, parent.get());
|
||||||
|
|
@ -250,7 +251,7 @@ public class FolderManagerTest {
|
||||||
@ShouldThrowException(IllegalArgumentException.class)
|
@ShouldThrowException(IllegalArgumentException.class)
|
||||||
@InSequence(1500)
|
@InSequence(1500)
|
||||||
public void createFolderEmptyName() {
|
public void createFolderEmptyName() {
|
||||||
final Optional<Folder> parent = folderRepo.findById(-2005L);
|
final Optional<Folder> parent = folderRepo.findById(2005L);
|
||||||
assertThat(parent.isPresent(), is(true));
|
assertThat(parent.isPresent(), is(true));
|
||||||
|
|
||||||
final Folder test = folderManager.createFolder(" ", parent.get());
|
final Folder test = folderManager.createFolder(" ", parent.get());
|
||||||
|
|
@ -267,7 +268,7 @@ public class FolderManagerTest {
|
||||||
@InSequence(2000)
|
@InSequence(2000)
|
||||||
public void deleteFolder() {
|
public void deleteFolder() {
|
||||||
//docs-1-1-1
|
//docs-1-1-1
|
||||||
final Folder folder = folderRepo.findById(-2007L).get();
|
final Folder folder = folderRepo.findById(2007L).get();
|
||||||
folderManager.deleteFolder(folder);
|
folderManager.deleteFolder(folder);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
@ -280,7 +281,7 @@ public class FolderManagerTest {
|
||||||
@ShouldThrowException(IllegalArgumentException.class)
|
@ShouldThrowException(IllegalArgumentException.class)
|
||||||
@InSequence(2100)
|
@InSequence(2100)
|
||||||
public void deleteNonEmptyFolder() {
|
public void deleteNonEmptyFolder() {
|
||||||
final Folder folder = folderRepo.findById(-2008L).get();
|
final Folder folder = folderRepo.findById(2008L).get();
|
||||||
folderManager.deleteFolder(folder);
|
folderManager.deleteFolder(folder);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
@ -293,7 +294,7 @@ public class FolderManagerTest {
|
||||||
@ShouldThrowException(IllegalArgumentException.class)
|
@ShouldThrowException(IllegalArgumentException.class)
|
||||||
@InSequence(2100)
|
@InSequence(2100)
|
||||||
public void deleteNonEmptySubFolder() {
|
public void deleteNonEmptySubFolder() {
|
||||||
final Folder folder = folderRepo.findById(-2006L).get();
|
final Folder folder = folderRepo.findById(2006L).get();
|
||||||
folderManager.deleteFolder(folder);
|
folderManager.deleteFolder(folder);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
@ -306,7 +307,7 @@ public class FolderManagerTest {
|
||||||
@ShouldThrowException(IllegalArgumentException.class)
|
@ShouldThrowException(IllegalArgumentException.class)
|
||||||
@InSequence(2210)
|
@InSequence(2210)
|
||||||
public void deleteRootFolder() {
|
public void deleteRootFolder() {
|
||||||
final Folder folder = folderRepo.findById(-2003L).get();
|
final Folder folder = folderRepo.findById(2003L).get();
|
||||||
folderManager.deleteFolder(folder);
|
folderManager.deleteFolder(folder);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
@ -332,8 +333,8 @@ public class FolderManagerTest {
|
||||||
@InSequence(3000)
|
@InSequence(3000)
|
||||||
public void moveFolder() {
|
public void moveFolder() {
|
||||||
//docs-1-1-2 to docs-2
|
//docs-1-1-2 to docs-2
|
||||||
final Folder folder = folderRepo.findById(-2008L).get();
|
final Folder folder = folderRepo.findById(2008L).get();
|
||||||
final Folder target = folderRepo.findById(-2010L).get();
|
final Folder target = folderRepo.findById(2010L).get();
|
||||||
|
|
||||||
folderManager.moveFolder(folder, target);
|
folderManager.moveFolder(folder, target);
|
||||||
}
|
}
|
||||||
|
|
@ -349,8 +350,8 @@ public class FolderManagerTest {
|
||||||
public void moveFolderTargetFolderSameName() {
|
public void moveFolderTargetFolderSameName() {
|
||||||
//docs-1/downloads to /docs-2/
|
//docs-1/downloads to /docs-2/
|
||||||
|
|
||||||
final Folder folder = folderRepo.findById(-2009L).get();
|
final Folder folder = folderRepo.findById(2009L).get();
|
||||||
final Folder target = folderRepo.findById(-2010L).get();
|
final Folder target = folderRepo.findById(2010L).get();
|
||||||
|
|
||||||
folderManager.moveFolder(folder, target);
|
folderManager.moveFolder(folder, target);
|
||||||
}
|
}
|
||||||
|
|
@ -364,8 +365,8 @@ public class FolderManagerTest {
|
||||||
@ShouldThrowException(IllegalArgumentException.class)
|
@ShouldThrowException(IllegalArgumentException.class)
|
||||||
@InSequence(3100)
|
@InSequence(3100)
|
||||||
public void moveDocumentsFolderToAssetsFolder() {
|
public void moveDocumentsFolderToAssetsFolder() {
|
||||||
final Folder folder = folderRepo.findById(-2009L).get();
|
final Folder folder = folderRepo.findById(2009L).get();
|
||||||
final Folder target = folderRepo.findById(-2014L).get();
|
final Folder target = folderRepo.findById(2014L).get();
|
||||||
|
|
||||||
folderManager.moveFolder(folder, target);
|
folderManager.moveFolder(folder, target);
|
||||||
}
|
}
|
||||||
|
|
@ -379,8 +380,8 @@ public class FolderManagerTest {
|
||||||
@ShouldThrowException(IllegalArgumentException.class)
|
@ShouldThrowException(IllegalArgumentException.class)
|
||||||
@InSequence(3110)
|
@InSequence(3110)
|
||||||
public void moveAssetsFolderToDocumentsFolder() {
|
public void moveAssetsFolderToDocumentsFolder() {
|
||||||
final Folder folder = folderRepo.findById(-2014L).get();
|
final Folder folder = folderRepo.findById(2014L).get();
|
||||||
final Folder target = folderRepo.findById(-2010L).get();
|
final Folder target = folderRepo.findById(2010L).get();
|
||||||
|
|
||||||
folderManager.moveFolder(folder, target);
|
folderManager.moveFolder(folder, target);
|
||||||
}
|
}
|
||||||
|
|
@ -394,7 +395,7 @@ public class FolderManagerTest {
|
||||||
@ShouldThrowException(IllegalArgumentException.class)
|
@ShouldThrowException(IllegalArgumentException.class)
|
||||||
@InSequence(3110)
|
@InSequence(3110)
|
||||||
public void moveFolderToItself() {
|
public void moveFolderToItself() {
|
||||||
final Folder folder = folderRepo.findById(-2008L).get();
|
final Folder folder = folderRepo.findById(2008L).get();
|
||||||
|
|
||||||
folderManager.moveFolder(folder, folder);
|
folderManager.moveFolder(folder, folder);
|
||||||
}
|
}
|
||||||
|
|
@ -408,7 +409,7 @@ public class FolderManagerTest {
|
||||||
@ShouldThrowException(NullPointerException.class)
|
@ShouldThrowException(NullPointerException.class)
|
||||||
@InSequence(3200)
|
@InSequence(3200)
|
||||||
public void moveFolderNull() {
|
public void moveFolderNull() {
|
||||||
final Folder target = folderRepo.findById(-2010L).get();
|
final Folder target = folderRepo.findById(2010L).get();
|
||||||
folderManager.moveFolder(null, target);
|
folderManager.moveFolder(null, target);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
@ -421,7 +422,7 @@ public class FolderManagerTest {
|
||||||
@ShouldThrowException(NullPointerException.class)
|
@ShouldThrowException(NullPointerException.class)
|
||||||
@InSequence(3210)
|
@InSequence(3210)
|
||||||
public void moveFolderTargetNull() {
|
public void moveFolderTargetNull() {
|
||||||
final Folder folder = folderRepo.findById(-2008L).get();
|
final Folder folder = folderRepo.findById(2008L).get();
|
||||||
|
|
||||||
folderManager.moveFolder(folder, null);
|
folderManager.moveFolder(folder, null);
|
||||||
}
|
}
|
||||||
|
|
@ -435,8 +436,8 @@ public class FolderManagerTest {
|
||||||
@ShouldThrowException(IllegalArgumentException.class)
|
@ShouldThrowException(IllegalArgumentException.class)
|
||||||
@InSequence(3300)
|
@InSequence(3300)
|
||||||
public void moveFolderWithLiveItems() {
|
public void moveFolderWithLiveItems() {
|
||||||
final Folder folder = folderRepo.findById(-2011L).get();
|
final Folder folder = folderRepo.findById(2011L).get();
|
||||||
final Folder target = folderRepo.findById(-2010L).get();
|
final Folder target = folderRepo.findById(2010L).get();
|
||||||
|
|
||||||
folderManager.moveFolder(folder, target);
|
folderManager.moveFolder(folder, target);
|
||||||
}
|
}
|
||||||
|
|
@ -450,8 +451,8 @@ public class FolderManagerTest {
|
||||||
@ShouldThrowException(IllegalArgumentException.class)
|
@ShouldThrowException(IllegalArgumentException.class)
|
||||||
@InSequence(3310)
|
@InSequence(3310)
|
||||||
public void moveFolderWithLiveItemsInSubFolder() {
|
public void moveFolderWithLiveItemsInSubFolder() {
|
||||||
final Folder folder = folderRepo.findById(-2010L).get();
|
final Folder folder = folderRepo.findById(2010L).get();
|
||||||
final Folder target = folderRepo.findById(-2005L).get();
|
final Folder target = folderRepo.findById(2005L).get();
|
||||||
|
|
||||||
folderManager.moveFolder(folder, target);
|
folderManager.moveFolder(folder, target);
|
||||||
}
|
}
|
||||||
|
|
@ -465,8 +466,8 @@ public class FolderManagerTest {
|
||||||
@ShouldThrowException(IllegalArgumentException.class)
|
@ShouldThrowException(IllegalArgumentException.class)
|
||||||
@InSequence(3320)
|
@InSequence(3320)
|
||||||
public void moveFolderToOtherSection() {
|
public void moveFolderToOtherSection() {
|
||||||
final Folder folder = folderRepo.findById(-2008L).get();
|
final Folder folder = folderRepo.findById(2008L).get();
|
||||||
final Folder target = folderRepo.findById(-2003L).get();
|
final Folder target = folderRepo.findById(2003L).get();
|
||||||
|
|
||||||
folderManager.moveFolder(folder, target);
|
folderManager.moveFolder(folder, target);
|
||||||
}
|
}
|
||||||
|
|
@ -476,21 +477,21 @@ public class FolderManagerTest {
|
||||||
+ "FolderManagerTest/data.xml")
|
+ "FolderManagerTest/data.xml")
|
||||||
@InSequence(3400)
|
@InSequence(3400)
|
||||||
public void getFolderPath() {
|
public void getFolderPath() {
|
||||||
final Folder infoRoot = folderRepo.findById(-2001L).get();
|
final Folder infoRoot = folderRepo.findById(2001L).get();
|
||||||
final Folder infoAssets = folderRepo.findById(-2002L).get();
|
final Folder infoAssets = folderRepo.findById(2002L).get();
|
||||||
final Folder projectsRoot = folderRepo.findById(-2003L).get();
|
final Folder projectsRoot = folderRepo.findById(2003L).get();
|
||||||
final Folder projectsAssets = folderRepo.findById(-2004L).get();
|
final Folder projectsAssets = folderRepo.findById(2004L).get();
|
||||||
final Folder docs1 = folderRepo.findById(-2005L).get();
|
final Folder docs1 = folderRepo.findById(2005L).get();
|
||||||
final Folder docs11 = folderRepo.findById(-2006L).get();
|
final Folder docs11 = folderRepo.findById(2006L).get();
|
||||||
final Folder docs111 = folderRepo.findById(-2007L).get();
|
final Folder docs111 = folderRepo.findById(2007L).get();
|
||||||
final Folder docs112 = folderRepo.findById(-2008L).get();
|
final Folder docs112 = folderRepo.findById(2008L).get();
|
||||||
final Folder downloads1 = folderRepo.findById(-2009L).get();
|
final Folder downloads1 = folderRepo.findById(2009L).get();
|
||||||
final Folder docs2 = folderRepo.findById(-2010L).get();
|
final Folder docs2 = folderRepo.findById(2010L).get();
|
||||||
final Folder docs21 = folderRepo.findById(-2011L).get();
|
final Folder docs21 = folderRepo.findById(2011L).get();
|
||||||
final Folder downloads2 = folderRepo.findById(-2012L).get();
|
final Folder downloads2 = folderRepo.findById(2012L).get();
|
||||||
final Folder assets1 = folderRepo.findById(-2013L).get();
|
final Folder assets1 = folderRepo.findById(2013L).get();
|
||||||
final Folder assets11 = folderRepo.findById(-2014L).get();
|
final Folder assets11 = folderRepo.findById(2014L).get();
|
||||||
final Folder assets12 = folderRepo.findById(-2015L).get();
|
final Folder assets12 = folderRepo.findById(2015L).get();
|
||||||
|
|
||||||
assertThat(folderManager.getFolderPath(infoRoot),
|
assertThat(folderManager.getFolderPath(infoRoot),
|
||||||
is(equalTo("/info_root/")));
|
is(equalTo("/info_root/")));
|
||||||
|
|
@ -529,21 +530,21 @@ public class FolderManagerTest {
|
||||||
+ "FolderManagerTest/data.xml")
|
+ "FolderManagerTest/data.xml")
|
||||||
@InSequence(3410)
|
@InSequence(3410)
|
||||||
public void getFolderPathWithContentSection() {
|
public void getFolderPathWithContentSection() {
|
||||||
final Folder infoRoot = folderRepo.findById(-2001L).get();
|
final Folder infoRoot = folderRepo.findById(2001L).get();
|
||||||
final Folder infoAssets = folderRepo.findById(-2002L).get();
|
final Folder infoAssets = folderRepo.findById(2002L).get();
|
||||||
final Folder projectsRoot = folderRepo.findById(-2003L).get();
|
final Folder projectsRoot = folderRepo.findById(2003L).get();
|
||||||
final Folder projectsAssets = folderRepo.findById(-2004L).get();
|
final Folder projectsAssets = folderRepo.findById(2004L).get();
|
||||||
final Folder docs1 = folderRepo.findById(-2005L).get();
|
final Folder docs1 = folderRepo.findById(2005L).get();
|
||||||
final Folder docs11 = folderRepo.findById(-2006L).get();
|
final Folder docs11 = folderRepo.findById(2006L).get();
|
||||||
final Folder docs111 = folderRepo.findById(-2007L).get();
|
final Folder docs111 = folderRepo.findById(2007L).get();
|
||||||
final Folder docs112 = folderRepo.findById(-2008L).get();
|
final Folder docs112 = folderRepo.findById(2008L).get();
|
||||||
final Folder downloads1 = folderRepo.findById(-2009L).get();
|
final Folder downloads1 = folderRepo.findById(2009L).get();
|
||||||
final Folder docs2 = folderRepo.findById(-2010L).get();
|
final Folder docs2 = folderRepo.findById(2010L).get();
|
||||||
final Folder docs21 = folderRepo.findById(-2011L).get();
|
final Folder docs21 = folderRepo.findById(2011L).get();
|
||||||
final Folder downloads2 = folderRepo.findById(-2012L).get();
|
final Folder downloads2 = folderRepo.findById(2012L).get();
|
||||||
final Folder assets1 = folderRepo.findById(-2013L).get();
|
final Folder assets1 = folderRepo.findById(2013L).get();
|
||||||
final Folder assets11 = folderRepo.findById(-2014L).get();
|
final Folder assets11 = folderRepo.findById(2014L).get();
|
||||||
final Folder assets12 = folderRepo.findById(-2015L).get();
|
final Folder assets12 = folderRepo.findById(2015L).get();
|
||||||
|
|
||||||
assertThat(folderManager.getFolderPath(infoRoot, true),
|
assertThat(folderManager.getFolderPath(infoRoot, true),
|
||||||
is(equalTo("info:/info_root/")));
|
is(equalTo("info:/info_root/")));
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue