From c0bc52353ac840a73902b75e20bca0faf04aaed7 Mon Sep 17 00:00:00 2001 From: GGu Date: Thu, 5 May 2016 13:48:40 +0800 Subject: [PATCH] TUP-4528: Fix some junits in common se --- .../org.talend.core.runtime.test/fragment.xml | 8 +- .../AbstractItemMigrationTaskTest.java | 2 +- .../repository/ERepositoryObjectTypeTest.java | 18 +++ .../MigrationTaskExtensionEPReaderTest.java | 26 ++-- .../org.talend.utils.test/data/testfile | 0 .../org/talend/utils/io/FilesUtilsTest.java | 111 ++++++++++-------- 6 files changed, 103 insertions(+), 62 deletions(-) delete mode 100644 test/plugins/org.talend.utils.test/data/testfile diff --git a/test/plugins/org.talend.core.runtime.test/fragment.xml b/test/plugins/org.talend.core.runtime.test/fragment.xml index c78de08416..ba8cc45d1e 100644 --- a/test/plugins/org.talend.core.runtime.test/fragment.xml +++ b/test/plugins/org.talend.core.runtime.test/fragment.xml @@ -4,14 +4,14 @@ @@ -42,7 +42,7 @@ folder="metadata/exmple_demo" isResouce="true" label="job.Ext2" - parentNodeType="JOBS" + parentNodeType="JOB_DOC" products="DI" type="JOB_EXT2"> diff --git a/test/plugins/org.talend.core.runtime.test/src/org/talend/core/model/migration/AbstractItemMigrationTaskTest.java b/test/plugins/org.talend.core.runtime.test/src/org/talend/core/model/migration/AbstractItemMigrationTaskTest.java index 308827e4b5..25ab6774f5 100644 --- a/test/plugins/org.talend.core.runtime.test/src/org/talend/core/model/migration/AbstractItemMigrationTaskTest.java +++ b/test/plugins/org.talend.core.runtime.test/src/org/talend/core/model/migration/AbstractItemMigrationTaskTest.java @@ -112,7 +112,7 @@ public class AbstractItemMigrationTaskTest { @Test public void testGetAllTypes() { List allTypes = jobAndCodeItemMigrationTask.getAllTypes(); - assertEquals(5, allTypes.size());// 2 types and 3 extended types + assertEquals(4, allTypes.size());// 1 types and 3 extended types } @Test diff --git a/test/plugins/org.talend.core.runtime.test/src/org/talend/core/model/repository/ERepositoryObjectTypeTest.java b/test/plugins/org.talend.core.runtime.test/src/org/talend/core/model/repository/ERepositoryObjectTypeTest.java index 6553ca9c0d..77fc3ce1e4 100644 --- a/test/plugins/org.talend.core.runtime.test/src/org/talend/core/model/repository/ERepositoryObjectTypeTest.java +++ b/test/plugins/org.talend.core.runtime.test/src/org/talend/core/model/repository/ERepositoryObjectTypeTest.java @@ -265,6 +265,24 @@ public class ERepositoryObjectTypeTest { unusedTypes = new ArrayList(); unusedTypes.add(ERepositoryObjectType.SVG_BUSINESS_PROCESS); unusedTypes.add(ERepositoryObjectType.SNIPPETS); + + // for example tests + ERepositoryObjectType t = ERepositoryObjectType.getType("CODE_EXT"); + if (t != null) { + unusedTypes.add(t); + } + t = ERepositoryObjectType.getType("JOB_DOC_EXT"); + if (t != null) { + unusedTypes.add(t); + } + t = ERepositoryObjectType.getType("JOB_EXT"); + if (t != null) { + unusedTypes.add(t); + } + t = ERepositoryObjectType.getType("JOB_EXT2"); + if (t != null) { + unusedTypes.add(t); + } } @After diff --git a/test/plugins/org.talend.core.runtime.test/src/org/talend/migration/MigrationTaskExtensionEPReaderTest.java b/test/plugins/org.talend.core.runtime.test/src/org/talend/migration/MigrationTaskExtensionEPReaderTest.java index 8ca845fc22..04fddfe30a 100644 --- a/test/plugins/org.talend.core.runtime.test/src/org/talend/migration/MigrationTaskExtensionEPReaderTest.java +++ b/test/plugins/org.talend.core.runtime.test/src/org/talend/migration/MigrationTaskExtensionEPReaderTest.java @@ -12,9 +12,12 @@ // ============================================================================ package org.talend.migration; +import static org.hamcrest.CoreMatchers.equalTo; +import static org.hamcrest.CoreMatchers.hasItem; import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertFalse; import static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertThat; import static org.junit.Assert.assertTrue; import java.io.IOException; @@ -46,9 +49,10 @@ public class MigrationTaskExtensionEPReaderTest { assertFalse(migrationTaskExtensions.isEmpty()); List listOfExtensions = migrationTaskExtensions.get(ERepositoryObjectType.JOB_DOC); assertNotNull(listOfExtensions); - assertTrue("list of extensions should have one extension", listOfExtensions.size() == 2); //$NON-NLS-1$ - assertEquals(listOfExtensions.get(0), ERepositoryObjectType.getType("JOB_EXT")); //$NON-NLS-1$ - assertEquals(listOfExtensions.get(1), ERepositoryObjectType.getType("JOB_EXT2")); //$NON-NLS-1$ + assertThat(listOfExtensions.size(), equalTo(3)); //$NON-NLS-1$ + assertThat(listOfExtensions, hasItem(ERepositoryObjectType.getType("JOB_EXT"))); + assertThat(listOfExtensions, hasItem(ERepositoryObjectType.getType("JOB_EXT2"))); + assertThat(listOfExtensions, hasItem(ERepositoryObjectType.getType("JOB_DOC_EXT"))); } @Test @@ -74,9 +78,10 @@ public class MigrationTaskExtensionEPReaderTest { MigrationTaskExtensionEPReader migrationTaskExtensionEPReader = new MigrationTaskExtensionEPReader(); Set objectTypeExtensions = migrationTaskExtensionEPReader.getObjectTypeExtensions(Collections .singleton(ERepositoryObjectType.JOB_DOC)); - assertEquals(2, objectTypeExtensions.size()); - assertEquals("JOB_EXT2", ((ERepositoryObjectType) objectTypeExtensions.toArray()[0]).getType()); - assertEquals("JOB_EXT", ((ERepositoryObjectType) objectTypeExtensions.toArray()[1]).getType()); + assertEquals(3, objectTypeExtensions.size()); + assertThat(objectTypeExtensions, hasItem(ERepositoryObjectType.getType("JOB_EXT"))); + assertThat(objectTypeExtensions, hasItem(ERepositoryObjectType.getType("JOB_EXT2"))); + assertThat(objectTypeExtensions, hasItem(ERepositoryObjectType.getType("JOB_DOC_EXT"))); objectTypeExtensions = migrationTaskExtensionEPReader.getObjectTypeExtensions(Collections .singleton(ERepositoryObjectType.BUSINESS_PROCESS)); assertTrue(objectTypeExtensions.isEmpty()); @@ -87,10 +92,11 @@ public class MigrationTaskExtensionEPReaderTest { MigrationTaskExtensionEPReader migrationTaskExtensionEPReader = new MigrationTaskExtensionEPReader(); Set objectTypeExtensions = migrationTaskExtensionEPReader.getObjectTypeExtensions(Arrays.asList( ERepositoryObjectType.JOB_DOC, ERepositoryObjectType.CODE)); - assertEquals(3, objectTypeExtensions.size()); - assertEquals("JOB_EXT2", ((ERepositoryObjectType) objectTypeExtensions.toArray()[0]).getType()); - assertEquals("JOB_EXT", ((ERepositoryObjectType) objectTypeExtensions.toArray()[1]).getType()); - assertEquals("CODE_EXT", ((ERepositoryObjectType) objectTypeExtensions.toArray()[2]).getType()); + assertEquals(4, objectTypeExtensions.size()); + assertThat(objectTypeExtensions, hasItem(ERepositoryObjectType.getType("JOB_EXT"))); + assertThat(objectTypeExtensions, hasItem(ERepositoryObjectType.getType("JOB_EXT2"))); + assertThat(objectTypeExtensions, hasItem(ERepositoryObjectType.getType("JOB_DOC_EXT"))); + assertThat(objectTypeExtensions, hasItem(ERepositoryObjectType.getType("CODE_EXT"))); objectTypeExtensions = migrationTaskExtensionEPReader.getObjectTypeExtensions(Collections .singleton(ERepositoryObjectType.BUSINESS_PROCESS)); assertTrue(objectTypeExtensions.isEmpty()); diff --git a/test/plugins/org.talend.utils.test/data/testfile b/test/plugins/org.talend.utils.test/data/testfile deleted file mode 100644 index e69de29bb2..0000000000 diff --git a/test/plugins/org.talend.utils.test/src/org/talend/utils/io/FilesUtilsTest.java b/test/plugins/org.talend.utils.test/src/org/talend/utils/io/FilesUtilsTest.java index 0caa49f6b3..40521ad8b5 100644 --- a/test/plugins/org.talend.utils.test/src/org/talend/utils/io/FilesUtilsTest.java +++ b/test/plugins/org.talend.utils.test/src/org/talend/utils/io/FilesUtilsTest.java @@ -12,12 +12,14 @@ // ============================================================================ package org.talend.utils.io; -import static org.junit.Assert.*; +import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertTrue; import java.io.File; import java.io.IOException; import org.junit.After; +import org.junit.Before; import org.junit.Test; /** @@ -25,14 +27,37 @@ import org.junit.Test; */ public class FilesUtilsTest { + private File testBaseFolder, testTmpFolder, testDataFolder; + + @Before + public void beforeTest() throws IOException { + testBaseFolder = File.createTempFile(this.getClass().getSimpleName(), ""); + testBaseFolder.delete(); + testBaseFolder.mkdirs(); + testDataFolder = new File(testBaseFolder, "data"); + testTmpFolder = new File(testBaseFolder, "temp"); + } + + @After + public void clearFolder() throws IOException { + if (testBaseFolder.exists() && testBaseFolder.isDirectory()) { + File files[] = testBaseFolder.listFiles(); + for (File file2 : files) { + deleteFile(file2); + } + } + } + /** * Test method for {@link org.talend.utils.io.FilesUtils#isSVNFolder(java.io.File)}. */ @Test public void testIsSVNFolderFile() { - File file = new File("META-INF"); + File file = new File(testBaseFolder, "META-INF"); + file.mkdirs(); assertTrue(file.exists()); assertFalse(FilesUtils.isSVNFolder(file)); + deleteFile(file); } /** @@ -40,10 +65,10 @@ public class FilesUtilsTest { */ @Test public void testIsEmptyFolder() { - File file = new File("temp"); - file.mkdirs(); - assertTrue(file.exists()); - assertTrue(FilesUtils.isEmptyFolder("temp")); + + testTmpFolder.mkdirs(); + assertTrue(testTmpFolder.exists()); + assertTrue(FilesUtils.isEmptyFolder(testTmpFolder.getAbsolutePath())); } /** @@ -55,12 +80,18 @@ public class FilesUtilsTest { */ @Test public void testCopyFolder() throws IOException { - File source = new File("data"); - File target = new File("temp"); + File source = testDataFolder; + source.mkdirs(); + File testFile = new File(source, "testfile"); + testFile.createNewFile(); + + File target = testTmpFolder; + testTmpFolder.mkdirs(); assertTrue(source.exists()); assertTrue(target.exists()); FilesUtils.copyFolder(source, target, false, null, null, false); - assertTrue(new File("temp//testfile").exists()); + assertTrue(new File(testTmpFolder, "testfile").exists()); + deleteFile(source); } private void deleteFile(File file) { @@ -77,33 +108,18 @@ public class FilesUtilsTest { } } - /** - * - * DOC Administrator Comment method "clearFolder". - * - * @throws IOException - */ - @After - public void clearFolder() throws IOException { - File file = new File("temp"); - if (file.isDirectory()) { - File files[] = file.listFiles(); - for (File file2 : files) { - deleteFile(file2); - } - } - } - /** * Test method for {@link org.talend.utils.io.FilesUtils#copyDirectory(java.io.File, java.io.File)}. */ @Test public void testCopyDirectory() { - File source = new File("data"); - File target = new File("temp"); + File source = testDataFolder; + source.mkdirs(); + File target = testTmpFolder; + target.mkdirs(); assertTrue(source.isDirectory() && target.isDirectory()); FilesUtils.copyDirectory(source, target); - assertTrue(new File("temp//data").exists()); + assertTrue(new File(testTmpFolder, "data").exists()); } /** @@ -111,7 +127,8 @@ public class FilesUtilsTest { */ @Test public void testCreateFolderFile() { - File file = new File("temp//testfolder"); + testTmpFolder.mkdirs(); + File file = new File(testTmpFolder, "testfolder"); assertFalse(file.exists()); FilesUtils.createFolder(file); assertTrue(file.exists()); @@ -124,7 +141,7 @@ public class FilesUtilsTest { */ @Test public void testCreateFoldersIfNotExistsString() throws IOException { - File file = new File("temp//testfolder1/testfolder2"); + File file = new File(testTmpFolder, "testfolder1/testfolder2"); assertFalse(file.exists()); FilesUtils.createFoldersIfNotExists(file.getAbsolutePath()); assertTrue(file.exists()); @@ -137,10 +154,10 @@ public class FilesUtilsTest { */ @Test public void testCreateFoldersIfNotExistsStringBooleanIsFilePath() { - String baseDirectory = "temp//testfolder1/testfolder2"; - File directory = new File(baseDirectory); + String baseDirectory = "testfolder1/testfolder2"; + File directory = new File(testTmpFolder, baseDirectory); assertFalse(directory.exists()); - File filePath = new File(baseDirectory + "/myfile"); + File filePath = new File(testTmpFolder, baseDirectory + "/myfile"); boolean pathIsFilePath = true; FilesUtils.createFoldersIfNotExists(filePath.getAbsolutePath(), pathIsFilePath); assertTrue(directory.exists()); @@ -154,8 +171,8 @@ public class FilesUtilsTest { */ @Test public void testCreateFoldersIfNotExistsStringBooleanIsNotFilePath() { - String baseDirectory = "temp//testfolder1/testfolder2"; - File directory = new File(baseDirectory); + String baseDirectory = "testfolder1//testfolder2"; + File directory = new File(testTmpFolder, baseDirectory); assertFalse(directory.exists()); boolean pathIsFilePath = false; FilesUtils.createFoldersIfNotExists(directory.getAbsolutePath(), pathIsFilePath); @@ -169,7 +186,8 @@ public class FilesUtilsTest { */ @Test public void testDeleteFile() throws IOException { - File file = new File("temp//testFiles"); + testTmpFolder.mkdirs(); + File file = new File(testTmpFolder, "testFiles"); assertFalse(file.exists()); file.createNewFile(); assertTrue(file.exists()); @@ -179,8 +197,7 @@ public class FilesUtilsTest { @Test public void testDeleteFolder() throws IOException { - File tmpFolder = new File("temp"); - File testFolder = new File(tmpFolder, "test1111111111111"); + File testFolder = new File(testTmpFolder, "test1111111111111"); FilesUtils.deleteFolder(testFolder, true); assertFalse(testFolder.exists()); @@ -230,28 +247,28 @@ public class FilesUtilsTest { assertFalse(FilesUtils.allInSameFolder(null)); assertFalse(FilesUtils.allInSameFolder(new File("abc"))); - File tmpFolder = new File("temp"); - assertTrue(FilesUtils.allInSameFolder(tmpFolder)); + testTmpFolder.mkdirs(); + assertTrue(FilesUtils.allInSameFolder(testTmpFolder)); - File abcFile = new File(tmpFolder, "abc.txt"); - assertFalse(FilesUtils.allInSameFolder(tmpFolder, "abc.txt")); + File abcFile = new File(testTmpFolder, "abc.txt"); + assertFalse(FilesUtils.allInSameFolder(testTmpFolder, "abc.txt")); assertFalse(FilesUtils.allInSameFolder(abcFile, "abc.txt")); if (!abcFile.exists()) { abcFile.createNewFile(); } - assertTrue(FilesUtils.allInSameFolder(tmpFolder, "abc.txt")); + assertTrue(FilesUtils.allInSameFolder(testTmpFolder, "abc.txt")); assertTrue(FilesUtils.allInSameFolder(abcFile, "abc.txt")); - File xyzFile = new File(tmpFolder, "xyz.txt"); + File xyzFile = new File(testTmpFolder, "xyz.txt"); if (!xyzFile.exists()) { xyzFile.createNewFile(); } - assertTrue(FilesUtils.allInSameFolder(tmpFolder, "abc.txt", "xyz.txt")); + assertTrue(FilesUtils.allInSameFolder(testTmpFolder, "abc.txt", "xyz.txt")); assertTrue(FilesUtils.allInSameFolder(abcFile, "abc.txt", "xyz.txt")); assertTrue(FilesUtils.allInSameFolder(xyzFile, "abc.txt", "xyz.txt")); - assertFalse(FilesUtils.allInSameFolder(tmpFolder, "abc.txt", "xyz.txt", "XXXX123.txt")); + assertFalse(FilesUtils.allInSameFolder(testTmpFolder, "abc.txt", "xyz.txt", "XXXX123.txt")); assertFalse(FilesUtils.allInSameFolder(abcFile, "abc.txt", "xyz.txt", "XXXX123.txt")); assertFalse(FilesUtils.allInSameFolder(xyzFile, "abc.txt", "xyz.txt", "XXXX123.txt")); }