diff --git a/server/sonar-server/src/test/java/org/sonar/server/batch/IssuesActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/batch/IssuesActionTest.java index 66f2191f6c34..d22516d361e2 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/batch/IssuesActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/batch/IssuesActionTest.java @@ -99,7 +99,7 @@ public void before() { public void return_minimal_fields() throws Exception { ComponentDto project = ComponentTesting.newProjectDto(PROJECT_UUID).setKey(PROJECT_KEY); ComponentDto module = ComponentTesting.newModuleDto(MODULE_UUID, project).setKey(MODULE_KEY); - ComponentDto file = ComponentTesting.newFileDto(module, FILE_UUID).setKey(FILE_KEY).setPath(null); + ComponentDto file = ComponentTesting.newFileDto(module, null, FILE_UUID).setKey(FILE_KEY).setPath(null); db.getDbClient().componentDao().insert(db.getSession(), project, module, file); db.getSession().commit(); @@ -137,7 +137,7 @@ public void return_minimal_fields() throws Exception { public void issues_from_project() throws Exception { ComponentDto project = ComponentTesting.newProjectDto(PROJECT_UUID).setKey(PROJECT_KEY); ComponentDto module = ComponentTesting.newModuleDto(MODULE_UUID, project).setKey(MODULE_KEY); - ComponentDto file = ComponentTesting.newFileDto(module, FILE_UUID).setKey(FILE_KEY).setPath("src/org/struts/Action.java"); + ComponentDto file = ComponentTesting.newFileDto(module, null, FILE_UUID).setKey(FILE_KEY).setPath("src/org/struts/Action.java"); db.getDbClient().componentDao().insert(db.getSession(), project, module, file); db.getSession().commit(); @@ -175,7 +175,7 @@ public void issues_from_project() throws Exception { public void issues_from_module() throws Exception { ComponentDto project = ComponentTesting.newProjectDto(PROJECT_UUID).setKey(PROJECT_KEY); ComponentDto module = ComponentTesting.newModuleDto(MODULE_UUID, project).setKey(MODULE_KEY); - ComponentDto file = ComponentTesting.newFileDto(module, FILE_UUID).setKey(FILE_KEY).setPath("src/org/struts/Action.java"); + ComponentDto file = ComponentTesting.newFileDto(module, null, FILE_UUID).setKey(FILE_KEY).setPath("src/org/struts/Action.java"); db.getDbClient().componentDao().insert(db.getSession(), project, module, file); db.getSession().commit(); @@ -213,7 +213,7 @@ public void issues_from_module() throws Exception { public void issues_from_file() throws Exception { ComponentDto project = ComponentTesting.newProjectDto(PROJECT_UUID).setKey(PROJECT_KEY); ComponentDto module = ComponentTesting.newModuleDto(MODULE_UUID, project).setKey(MODULE_KEY); - ComponentDto file = ComponentTesting.newFileDto(module, FILE_UUID).setKey(FILE_KEY).setPath("src/org/struts/Action.java"); + ComponentDto file = ComponentTesting.newFileDto(module, null, FILE_UUID).setKey(FILE_KEY).setPath("src/org/struts/Action.java"); db.getDbClient().componentDao().insert(db.getSession(), project, module, file); db.getSession().commit(); @@ -289,7 +289,7 @@ public void project_issues_attached_file_on_removed_module() throws Exception { ComponentDto project = ComponentTesting.newProjectDto(PROJECT_UUID).setKey(PROJECT_KEY); // File and module are removed ComponentDto module = ComponentTesting.newModuleDto(MODULE_UUID, project).setKey(MODULE_KEY).setEnabled(false); - ComponentDto file = ComponentTesting.newFileDto(module, FILE_UUID).setKey(FILE_KEY).setPath("src/org/struts/Action.java").setEnabled(false); + ComponentDto file = ComponentTesting.newFileDto(module, null, FILE_UUID).setKey(FILE_KEY).setPath("src/org/struts/Action.java").setEnabled(false); db.getDbClient().componentDao().insert(db.getSession(), project, module, file); db.getSession().commit(); diff --git a/server/sonar-server/src/test/java/org/sonar/server/batch/ProjectDataLoaderMediumTest.java b/server/sonar-server/src/test/java/org/sonar/server/batch/ProjectDataLoaderMediumTest.java index f34241827628..59c2f1c992d8 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/batch/ProjectDataLoaderMediumTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/batch/ProjectDataLoaderMediumTest.java @@ -495,7 +495,7 @@ public void return_file_data_from_single_project() { tester.get(DbClient.class).componentDao().insert(dbSession, project); addDefaultProfile(); - ComponentDto file = ComponentTesting.newFileDto(project, "file"); + ComponentDto file = ComponentTesting.newFileDto(project, null, "file"); tester.get(DbClient.class).componentDao().insert(dbSession, file); tester.get(FileSourceDao.class).insert(newFileSourceDto(file).setSrcHash("123456")); @@ -515,7 +515,7 @@ public void return_file_data_from_multi_modules() { addDefaultProfile(); // File on project - ComponentDto projectFile = ComponentTesting.newFileDto(project, "projectFile"); + ComponentDto projectFile = ComponentTesting.newFileDto(project, null, "projectFile"); tester.get(DbClient.class).componentDao().insert(dbSession, projectFile); tester.get(FileSourceDao.class).insert(newFileSourceDto(projectFile).setSrcHash("123456")); @@ -523,7 +523,7 @@ public void return_file_data_from_multi_modules() { tester.get(DbClient.class).componentDao().insert(dbSession, module); // File on module - ComponentDto moduleFile = ComponentTesting.newFileDto(module, "moduleFile"); + ComponentDto moduleFile = ComponentTesting.newFileDto(module, null, "moduleFile"); tester.get(DbClient.class).componentDao().insert(dbSession, moduleFile); tester.get(FileSourceDao.class).insert(newFileSourceDto(moduleFile).setSrcHash("789456")); @@ -541,7 +541,7 @@ public void return_file_data_from_module() { addDefaultProfile(); // File on project - ComponentDto projectFile = ComponentTesting.newFileDto(project, "projectFile"); + ComponentDto projectFile = ComponentTesting.newFileDto(project, null, "projectFile"); tester.get(DbClient.class).componentDao().insert(dbSession, projectFile); tester.get(FileSourceDao.class).insert(newFileSourceDto(projectFile).setSrcHash("123456").setRevision("987654321")); @@ -550,7 +550,7 @@ public void return_file_data_from_module() { tester.get(DbClient.class).componentDao().insert(dbSession, module); // File on module - ComponentDto moduleFile = ComponentTesting.newFileDto(module, "moduleFile"); + ComponentDto moduleFile = ComponentTesting.newFileDto(module, null, "moduleFile"); tester.get(DbClient.class).componentDao().insert(dbSession, moduleFile); tester.get(FileSourceDao.class).insert(newFileSourceDto(moduleFile).setSrcHash("789456").setRevision("123456789")); diff --git a/server/sonar-server/src/test/java/org/sonar/server/component/ComponentServiceTest.java b/server/sonar-server/src/test/java/org/sonar/server/component/ComponentServiceTest.java index 5c3f5ee58f07..6e56f4981ce1 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/component/ComponentServiceTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/component/ComponentServiceTest.java @@ -114,7 +114,7 @@ public void check_module_keys_before_renaming() { ComponentDto module = ComponentTesting.newModuleDto(project).setKey("sample:root:module"); dbClient.componentDao().insert(dbSession, module); - ComponentDto file = newFileDto(module).setKey("sample:root:module:src/File.xoo"); + ComponentDto file = newFileDto(module, null).setKey("sample:root:module:src/File.xoo"); dbClient.componentDao().insert(dbSession, file); dbSession.commit(); @@ -301,7 +301,7 @@ public void should_return_project_uuids() { ComponentDto module = ComponentTesting.newModuleDto(project).setKey(moduleKey); dbClient.componentDao().insert(dbSession, module); String fileKey = "sample:root:module:Foo.xoo"; - ComponentDto file = newFileDto(module).setKey(fileKey); + ComponentDto file = newFileDto(module, null).setKey(fileKey); dbClient.componentDao().insert(dbSession, file); dbSession.commit(); diff --git a/server/sonar-server/src/test/java/org/sonar/server/component/ComponentServiceUpdateKeyTest.java b/server/sonar-server/src/test/java/org/sonar/server/component/ComponentServiceUpdateKeyTest.java index 8c48413bffb9..c4b35a88ec07 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/component/ComponentServiceUpdateKeyTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/component/ComponentServiceUpdateKeyTest.java @@ -70,7 +70,7 @@ public void setUp() { @Test public void update_project_key() { ComponentDto project = insertSampleRootProject(); - ComponentDto file = ComponentTesting.newFileDto(project).setKey("sample:root:src/File.xoo"); + ComponentDto file = ComponentTesting.newFileDto(project, null).setKey("sample:root:src/File.xoo"); dbClient.componentDao().insert(dbSession, file); dbSession.commit(); @@ -94,7 +94,7 @@ public void update_module_key() { ComponentDto module = ComponentTesting.newModuleDto(project).setKey("sample:root:module"); dbClient.componentDao().insert(dbSession, module); - ComponentDto file = ComponentTesting.newFileDto(module).setKey("sample:root:module:src/File.xoo"); + ComponentDto file = ComponentTesting.newFileDto(module, null).setKey("sample:root:module:src/File.xoo"); dbClient.componentDao().insert(dbSession, file); dbSession.commit(); @@ -171,7 +171,7 @@ public void fail_if_new_key_is_not_formatted_correctly() { public void fail_if_update_is_not_on_module_or_project() { setGlobalAdminPermission(); ComponentDto project = insertSampleRootProject(); - ComponentDto file = componentDb.insertComponent(newFileDto(project)); + ComponentDto file = componentDb.insertComponent(newFileDto(project, null)); expectedException.expect(IllegalArgumentException.class); expectedException.expectMessage("Component updated must be a module or a key"); diff --git a/server/sonar-server/src/test/java/org/sonar/server/component/ws/AppActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/component/ws/AppActionTest.java index 8c7f3cbba6e7..0d7ab34f99e7 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/component/ws/AppActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/component/ws/AppActionTest.java @@ -154,7 +154,7 @@ private void insertComponentsAndAnalysis() { ComponentDto module = ComponentTesting.newModuleDto(MODULE_UUID, project) .setLongName("SonarQube :: Plugin API") .setKey(MODULE_KEY); - ComponentDto file = ComponentTesting.newFileDto(module, FILE_UUID) + ComponentDto file = ComponentTesting.newFileDto(module, null, FILE_UUID) .setKey(FILE_KEY) .setName("Plugin.java") .setLongName("src/main/java/org/sonar/api/Plugin.java") diff --git a/server/sonar-server/src/test/java/org/sonar/server/component/ws/BulkUpdateKeyActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/component/ws/BulkUpdateKeyActionTest.java index f01f11d6f512..722e8cc6fb25 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/component/ws/BulkUpdateKeyActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/component/ws/BulkUpdateKeyActionTest.java @@ -91,7 +91,7 @@ public void json_example() { ComponentDto anotherProject = componentDb.insertComponent(newProjectDto().setKey("another_project")); componentDb.insertComponent(newModuleDto(anotherProject).setKey("my_new_project:module_1")); ComponentDto module2 = componentDb.insertComponent(newModuleDto(project).setKey("my_project:module_2")); - componentDb.insertComponent(newFileDto(module2)); + componentDb.insertComponent(newFileDto(module2, null)); String result = ws.newRequest() .setParam(PARAM_KEY, "my_project") @@ -117,7 +117,7 @@ public void dry_run_by_key() { public void bulk_update_project_key() { ComponentDto project = insertMyProject(); ComponentDto module = componentDb.insertComponent(newModuleDto(project).setKey("my_project:root:module")); - ComponentDto file = componentDb.insertComponent(newFileDto(module).setKey("my_project:root:module:src/File.xoo")); + ComponentDto file = componentDb.insertComponent(newFileDto(module, null).setKey("my_project:root:module:src/File.xoo")); BulkUpdateKeyWsResponse result = callByUuid(project.uuid(), FROM, TO); @@ -167,7 +167,7 @@ public void fail_to_bulk_update_with_invalid_new_key() { @Test public void fail_to_bulk_update_if_not_project_or_module() { ComponentDto project = insertMyProject(); - ComponentDto file = componentDb.insertComponent(newFileDto(project)); + ComponentDto file = componentDb.insertComponent(newFileDto(project, null)); expectedException.expect(IllegalArgumentException.class); expectedException.expectMessage("Component updated must be a module or a key"); diff --git a/server/sonar-server/src/test/java/org/sonar/server/component/ws/SearchActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/component/ws/SearchActionTest.java index 7c44116a5ab3..67e14ebef3ce 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/component/ws/SearchActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/component/ws/SearchActionTest.java @@ -92,13 +92,13 @@ public void search_json_example() { newModuleDto("module-uuid", project) .setName("Module Name") .setKey("module-key")); + ComponentDto directory = newDirectory(module, "path/to/directoy") + .setUuid("directory-uuid") + .setKey("directory-key") + .setName("Directory Name"); + componentDb.insertComponent(directory); componentDb.insertComponent( - newDirectory(module, "path/to/directoy") - .setUuid("directory-uuid") - .setKey("directory-key") - .setName("Directory Name")); - componentDb.insertComponent( - newFileDto(module, "file-uuid") + newFileDto(module, directory, "file-uuid") .setKey("file-key") .setLanguage("java") .setName("File Name")); diff --git a/server/sonar-server/src/test/java/org/sonar/server/component/ws/ShowActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/component/ws/ShowActionTest.java index ea19c676e14b..963954ea7e26 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/component/ws/ShowActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/component/ws/ShowActionTest.java @@ -153,7 +153,7 @@ private void insertJsonExampleComponentsAndSnapshots() { .setQualifier(Qualifiers.DIRECTORY); componentDb.insertComponent(directory); componentDb.insertComponent( - newFileDto(directory, "AVIF-FffA3Ax6PH2efPD") + newFileDto(directory, directory, "AVIF-FffA3Ax6PH2efPD") .setKey("com.sonarsource:java-markdown:src/main/java/com/sonarsource/markdown/impl/Rule.java") .setName("Rule.java") .setPath("src/main/java/com/sonarsource/markdown/impl/Rule.java") diff --git a/server/sonar-server/src/test/java/org/sonar/server/component/ws/TreeActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/component/ws/TreeActionTest.java index 7d9be1c97446..4c26595a6be6 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/component/ws/TreeActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/component/ws/TreeActionTest.java @@ -28,6 +28,7 @@ import java.io.IOException; import java.util.Date; import javax.annotation.CheckForNull; +import javax.annotation.Nullable; import org.apache.commons.io.IOUtils; import org.junit.Before; import org.junit.Rule; @@ -117,7 +118,7 @@ public void return_children() throws IOException { } ComponentDto directory = newDirectory(module, "directory-path-1"); componentDb.insertComponent(directory); - componentDb.insertComponent(newFileDto(directory, 10)); + componentDb.insertComponent(newFileDto(module, directory, 10)); db.commit(); componentDb.indexAllComponents(); @@ -147,7 +148,7 @@ public void return_descendants() throws IOException { } ComponentDto directory = newDirectory(module, "directory-path-1"); componentDb.insertComponent(directory); - componentDb.insertComponent(newFileDto(directory, 1)); + componentDb.insertComponent(newFileDto(module, directory, 1)); db.commit(); componentDb.indexAllComponents(); @@ -193,7 +194,7 @@ public void return_leaves() throws IOException { componentDb.insertComponent(newFileDto(module, 2)); ComponentDto directory = newDirectory(project, "directory-path-1"); componentDb.insertComponent(directory); - componentDb.insertComponent(newFileDto(directory, 3)); + componentDb.insertComponent(newFileDto(module, directory, 3)); db.commit(); componentDb.indexAllComponents(); @@ -385,11 +386,15 @@ private TreeWsResponse call(TestRequest request) { } } - private static ComponentDto newFileDto(ComponentDto parentComponent, int i) { - return ComponentTesting.newFileDto(parentComponent, "file-uuid-" + i) - .setName("file-name-" + i) - .setKey("file-key-" + i) - .setPath("file-path-" + i); + private static ComponentDto newFileDto(ComponentDto moduleOrProject, @Nullable ComponentDto directory, int i) { + return ComponentTesting.newFileDto(moduleOrProject, directory, "file-uuid-" + i) + .setName("file-name-" + i) + .setKey("file-key-" + i) + .setPath("file-path-" + i); + } + + private static ComponentDto newFileDto(ComponentDto moduleOrProject, int i) { + return newFileDto(moduleOrProject, null, i); } private ComponentDto initJsonExampleComponents() throws IOException { @@ -404,7 +409,7 @@ private ComponentDto initJsonExampleComponents() throws IOException { for (JsonElement componentAsJsonElement : components) { JsonObject componentAsJsonObject = componentAsJsonElement.getAsJsonObject(); String uuid = getJsonField(componentAsJsonObject, "id"); - componentDb.insertComponent(ComponentTesting.newChildComponent(uuid, project) + componentDb.insertComponent(ComponentTesting.newChildComponent(uuid, project, project) .setKey(getJsonField(componentAsJsonObject, "key")) .setName(getJsonField(componentAsJsonObject, "name")) .setLanguage(getJsonField(componentAsJsonObject, "language")) diff --git a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/issue/IntegrateIssuesVisitorTest.java b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/issue/IntegrateIssuesVisitorTest.java index 9e0522264ddf..80ab1300887d 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/issue/IntegrateIssuesVisitorTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/issue/IntegrateIssuesVisitorTest.java @@ -277,7 +277,7 @@ public void remove_uuid_of_original_file_from_componentsWithUnprocessedIssues_if private void addBaseIssue(RuleKey ruleKey) { ComponentDto project = ComponentTesting.newProjectDto(PROJECT_UUID).setKey(PROJECT_KEY); - ComponentDto file = ComponentTesting.newFileDto(project, FILE_UUID).setKey(FILE_KEY); + ComponentDto file = ComponentTesting.newFileDto(project, null, FILE_UUID).setKey(FILE_KEY); dbTester.getDbClient().componentDao().insert(dbTester.getSession(), project, file); RuleDto ruleDto = RuleTesting.newDto(ruleKey); diff --git a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/BuildComponentTreeStepTest.java b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/BuildComponentTreeStepTest.java index 43d7e2b3ecf5..4f67ac3b60c4 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/BuildComponentTreeStepTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/BuildComponentTreeStepTest.java @@ -174,8 +174,9 @@ public void compute_keys_and_uuids() { public void return_existing_uuids() { ComponentDto project = insertComponent(newProjectDto("ABCD").setKey(REPORT_PROJECT_KEY)); ComponentDto module = insertComponent(newModuleDto("BCDE", project).setKey(REPORT_MODULE_KEY)); - insertComponent(newDirectory(module, "CDEF", REPORT_DIR_KEY_1).setKey(REPORT_MODULE_KEY + ":" + REPORT_DIR_KEY_1)); - insertComponent(newFileDto(module, "DEFG").setKey(REPORT_MODULE_KEY + ":" + REPORT_FILE_KEY_1)); + ComponentDto directory = newDirectory(module, "CDEF", REPORT_DIR_KEY_1); + insertComponent(directory.setKey(REPORT_MODULE_KEY + ":" + REPORT_DIR_KEY_1)); + insertComponent(newFileDto(module, directory, "DEFG").setKey(REPORT_MODULE_KEY + ":" + REPORT_FILE_KEY_1)); reportReader.putComponent(componentWithKey(ROOT_REF, PROJECT, REPORT_PROJECT_KEY, MODULE_REF)); reportReader.putComponent(componentWithKey(MODULE_REF, MODULE, REPORT_MODULE_KEY, DIR_REF_1)); @@ -253,7 +254,7 @@ public void return_existing_uuids_when_components_were_removed() { ComponentDto project = insertComponent(newProjectDto("ABCD").setKey(REPORT_PROJECT_KEY)); ComponentDto removedModule = insertComponent(newModuleDto("BCDE", project).setKey(REPORT_MODULE_KEY).setEnabled(false)); ComponentDto removedDirectory = insertComponent(newDirectory(removedModule, "CDEF", REPORT_DIR_KEY_1).setKey(REPORT_MODULE_KEY + ":" + REPORT_DIR_KEY_1).setEnabled(false)); - insertComponent(newFileDto(removedDirectory, "DEFG").setKey(REPORT_MODULE_KEY + ":" + REPORT_FILE_KEY_1).setEnabled(false)); + insertComponent(newFileDto(removedModule, removedModule, "DEFG").setKey(REPORT_MODULE_KEY + ":" + REPORT_FILE_KEY_1).setEnabled(false)); reportReader.putComponent(componentWithKey(ROOT_REF, PROJECT, REPORT_PROJECT_KEY, MODULE_REF)); reportReader.putComponent(componentWithKey(MODULE_REF, MODULE, REPORT_MODULE_KEY, DIR_REF_1)); diff --git a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/LoadCrossProjectDuplicationsRepositoryStepTest.java b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/LoadCrossProjectDuplicationsRepositoryStepTest.java index 8e1857ac490c..93b7612efa4d 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/LoadCrossProjectDuplicationsRepositoryStepTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/LoadCrossProjectDuplicationsRepositoryStepTest.java @@ -340,7 +340,7 @@ private SnapshotDto createProjectSnapshot(ComponentDto project) { } private ComponentDto createFile(String fileKey, ComponentDto project) { - ComponentDto file = ComponentTesting.newFileDto(project) + ComponentDto file = ComponentTesting.newFileDto(project, null) .setKey(fileKey) .setLanguage(XOO_LANGUAGE); dbClient.componentDao().insert(dbSession, file); diff --git a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/PersistIssuesStepTest.java b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/PersistIssuesStepTest.java index 73a08c152310..711baf4f7e8d 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/PersistIssuesStepTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/PersistIssuesStepTest.java @@ -100,7 +100,7 @@ public void insert_new_issue() { dbClient.ruleDao().insert(session, rule); ComponentDto project = ComponentTesting.newProjectDto(); dbClient.componentDao().insert(session, project); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); dbClient.componentDao().insert(session, file); session.commit(); diff --git a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/ReportPersistAnalysisStepTest.java b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/ReportPersistAnalysisStepTest.java index 2ebd03163c3e..6a7a5ff102c5 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/ReportPersistAnalysisStepTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/ReportPersistAnalysisStepTest.java @@ -104,7 +104,7 @@ public void persist_analysis() { dbClient.componentDao().insert(dbTester.getSession(), moduleDto); ComponentDto directoryDto = ComponentTesting.newDirectory(moduleDto, "CDEF", "MODULE_KEY:src/main/java/dir").setKey("MODULE_KEY:src/main/java/dir"); dbClient.componentDao().insert(dbTester.getSession(), directoryDto); - ComponentDto fileDto = ComponentTesting.newFileDto(moduleDto, "DEFG").setKey("MODULE_KEY:src/main/java/dir/Foo.java"); + ComponentDto fileDto = ComponentTesting.newFileDto(moduleDto, directoryDto, "DEFG").setKey("MODULE_KEY:src/main/java/dir/Foo.java"); dbClient.componentDao().insert(dbTester.getSession(), fileDto); dbTester.getSession().commit(); @@ -173,7 +173,7 @@ public void only_persist_snapshots_with_periods_on_project_and_module() { ComponentDto directoryDto = ComponentTesting.newDirectory(moduleDto, "CDEF", "MODULE_KEY:src/main/java/dir").setKey("MODULE_KEY:src/main/java/dir"); dbClient.componentDao().insert(dbTester.getSession(), directoryDto); - ComponentDto fileDto = ComponentTesting.newFileDto(moduleDto, "DEFG").setKey("MODULE_KEY:src/main/java/dir/Foo.java"); + ComponentDto fileDto = ComponentTesting.newFileDto(moduleDto, directoryDto, "DEFG").setKey("MODULE_KEY:src/main/java/dir/Foo.java"); dbClient.componentDao().insert(dbTester.getSession(), fileDto); dbTester.getSession().commit(); diff --git a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/ReportPersistComponentsStepTest.java b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/ReportPersistComponentsStepTest.java index 32074d055ce8..ac5af070aa4a 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/ReportPersistComponentsStepTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/ReportPersistComponentsStepTest.java @@ -31,11 +31,11 @@ import org.sonar.db.DbTester; import org.sonar.db.component.ComponentDto; import org.sonar.db.component.ComponentTesting; -import org.sonar.server.computation.task.projectanalysis.component.TreeRootHolderRule; import org.sonar.server.computation.task.projectanalysis.component.Component; import org.sonar.server.computation.task.projectanalysis.component.FileAttributes; import org.sonar.server.computation.task.projectanalysis.component.MutableDbIdsRepositoryRule; import org.sonar.server.computation.task.projectanalysis.component.MutableDisabledComponentsHolder; +import org.sonar.server.computation.task.projectanalysis.component.TreeRootHolderRule; import org.sonar.server.computation.task.step.ComputationStep; import static org.assertj.core.api.Assertions.assertThat; @@ -45,7 +45,6 @@ import static org.mockito.Mockito.when; import static org.sonar.db.component.ComponentDto.UUID_PATH_OF_ROOT; import static org.sonar.db.component.ComponentDto.UUID_PATH_SEPARATOR; -import static org.sonar.db.component.ComponentDto.formatUuidPathFromParent; import static org.sonar.server.computation.task.projectanalysis.component.Component.Type.DIRECTORY; import static org.sonar.server.computation.task.projectanalysis.component.Component.Type.FILE; import static org.sonar.server.computation.task.projectanalysis.component.Component.Type.PROJECT; @@ -383,7 +382,7 @@ public void nothing_to_persist() { ComponentDto module = ComponentTesting.newModuleDto("BCDE", project).setKey(MODULE_KEY).setName("Module"); dbClient.componentDao().insert(dbTester.getSession(), module); ComponentDto directory = ComponentTesting.newDirectory(module, "src/main/java/dir").setUuid("CDEF").setKey("MODULE_KEY:src/main/java/dir"); - ComponentDto file = ComponentTesting.newFileDto(module, "DEFG").setPath("src/main/java/dir/Foo.java").setName("Foo.java").setKey("MODULE_KEY:src/main/java/dir/Foo.java"); + ComponentDto file = ComponentTesting.newFileDto(module, directory, "DEFG").setPath("src/main/java/dir/Foo.java").setName("Foo.java").setKey("MODULE_KEY:src/main/java/dir/Foo.java"); dbClient.componentDao().insert(dbTester.getSession(), directory, file); dbTester.getSession().commit(); @@ -528,7 +527,7 @@ public void update_module_uuid_when_moving_a_module() { .setName("Module B"); dbClient.componentDao().insert(dbTester.getSession(), moduleA, moduleB); ComponentDto directory = ComponentTesting.newDirectory(moduleB, "src/main/java/dir").setUuid("CDEF").setKey("MODULE_B:src/main/java/dir"); - ComponentDto file = ComponentTesting.newFileDto(moduleB, "DEFG").setPath("src/main/java/dir/Foo.java").setName("Foo.java").setKey("MODULE_B:src/main/java/dir/Foo.java"); + ComponentDto file = ComponentTesting.newFileDto(moduleB, directory, "DEFG").setPath("src/main/java/dir/Foo.java").setName("Foo.java").setKey("MODULE_B:src/main/java/dir/Foo.java"); dbClient.componentDao().insert(dbTester.getSession(), directory, file); dbTester.getSession().commit(); @@ -623,10 +622,7 @@ public void persist_components_that_were_previously_removed() { .setUuid("CDEF") .setKey("MODULE_KEY:src/main/java/dir") .setEnabled(false); - ComponentDto removedFile = ComponentTesting.newFileDto(removedModule, "DEFG") - // method ComponentTesting.newFileDto is buggy because it only takes the module/project and assumes it is the parent of the file, - // which is wrong here (and most of the time), it is the directory - .setUuidPath(formatUuidPathFromParent(removedDirectory)) + ComponentDto removedFile = ComponentTesting.newFileDto(removedModule, removedDirectory, "DEFG") .setPath("src/main/java/dir/Foo.java") .setName("Foo.java") .setKey("MODULE_KEY:src/main/java/dir/Foo.java") @@ -724,7 +720,7 @@ public void update_module_uuid_when_reactivating_removed_component() { dbClient.componentDao().insert(dbTester.getSession(), module, removedModule); ComponentDto directory = ComponentTesting.newDirectory(module, "src/main/java/dir").setUuid("CDEF").setKey("MODULE_KEY:src/main/java/dir"); // The file was attached to another module - ComponentDto removedFile = ComponentTesting.newFileDto(removedModule, "DEFG").setPath("src/main/java/dir/Foo.java").setName("Foo.java") + ComponentDto removedFile = ComponentTesting.newFileDto(removedModule, directory, "DEFG").setPath("src/main/java/dir/Foo.java").setName("Foo.java") .setKey("MODULE_KEY:src/main/java/dir/Foo.java").setEnabled(false); dbClient.componentDao().insert(dbTester.getSession(), directory, removedFile); dbTester.getSession().commit(); diff --git a/server/sonar-server/src/test/java/org/sonar/server/duplication/ws/DuplicationsJsonWriterTest.java b/server/sonar-server/src/test/java/org/sonar/server/duplication/ws/DuplicationsJsonWriterTest.java index 5bfda9d6d01d..d49e12a01e81 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/duplication/ws/DuplicationsJsonWriterTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/duplication/ws/DuplicationsJsonWriterTest.java @@ -70,9 +70,9 @@ public void setUp() { @Test public void write_duplications() { String key1 = "org.codehaus.sonar:sonar-ws-client:src/main/java/org/sonar/wsclient/services/PropertyDeleteQuery.java"; - ComponentDto file1 = ComponentTesting.newFileDto(project).setId(10L).setKey(key1).setLongName("PropertyDeleteQuery").setRootUuid("uuid_5"); + ComponentDto file1 = ComponentTesting.newFileDto(project, null).setId(10L).setKey(key1).setLongName("PropertyDeleteQuery").setRootUuid("uuid_5"); String key2 = "org.codehaus.sonar:sonar-ws-client:src/main/java/org/sonar/wsclient/services/PropertyUpdateQuery.java"; - ComponentDto file2 = ComponentTesting.newFileDto(project).setId(11L).setQualifier("FIL").setKey(key2).setLongName("PropertyUpdateQuery").setRootUuid("uuid_5"); + ComponentDto file2 = ComponentTesting.newFileDto(project, null).setId(11L).setQualifier("FIL").setKey(key2).setLongName("PropertyUpdateQuery").setRootUuid("uuid_5"); when(componentDao.selectByKey(session, key1)).thenReturn(Optional.of(file1)); when(componentDao.selectByKey(session, key2)).thenReturn(Optional.of(file2)); @@ -129,9 +129,9 @@ public void write_duplications() { @Test public void write_duplications_without_sub_project() { String key1 = "org.codehaus.sonar:sonar-ws-client:src/main/java/org/sonar/wsclient/services/PropertyDeleteQuery.java"; - ComponentDto file1 = ComponentTesting.newFileDto(project).setId(10L).setKey(key1).setLongName("PropertyDeleteQuery"); + ComponentDto file1 = ComponentTesting.newFileDto(project, null).setId(10L).setKey(key1).setLongName("PropertyDeleteQuery"); String key2 = "org.codehaus.sonar:sonar-ws-client:src/main/java/org/sonar/wsclient/services/PropertyUpdateQuery.java"; - ComponentDto file2 = ComponentTesting.newFileDto(project).setId(11L).setKey(key2).setLongName("PropertyUpdateQuery"); + ComponentDto file2 = ComponentTesting.newFileDto(project, null).setId(11L).setKey(key2).setLongName("PropertyUpdateQuery"); when(componentDao.selectByKey(session, key1)).thenReturn(Optional.of(file1)); when(componentDao.selectByKey(session, key2)).thenReturn(Optional.of(file2)); @@ -178,7 +178,7 @@ public void write_duplications_without_sub_project() { @Test public void write_duplications_with_a_removed_component() { String key1 = "org.codehaus.sonar:sonar-ws-client:src/main/java/org/sonar/wsclient/services/PropertyDeleteQuery.java"; - ComponentDto file1 = ComponentTesting.newFileDto(project).setId(10L).setKey(key1).setLongName("PropertyDeleteQuery"); + ComponentDto file1 = ComponentTesting.newFileDto(project, null).setId(10L).setKey(key1).setLongName("PropertyDeleteQuery"); when(componentDao.selectByKey(session, key1)).thenReturn(Optional.of(file1)); when(componentDao.selectByUuid(session, project.uuid())).thenReturn(Optional.of(project)); diff --git a/server/sonar-server/src/test/java/org/sonar/server/duplication/ws/DuplicationsParserTest.java b/server/sonar-server/src/test/java/org/sonar/server/duplication/ws/DuplicationsParserTest.java index c1addc7c9bc6..f33035ca6e6d 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/duplication/ws/DuplicationsParserTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/duplication/ws/DuplicationsParserTest.java @@ -71,15 +71,15 @@ public void setUp() { // Current file String key1 = "org.codehaus.sonar:sonar-plugin-api:src/main/java/org/sonar/api/utils/command/CommandExecutor.java"; - currentFile = newFileDto(project1).setKey(key1).setLongName("CommandExecutor"); + currentFile = newFileDto(project1, null).setKey(key1).setLongName("CommandExecutor"); // File on same project String key2 = "org.codehaus.sonar:sonar-plugin-api:src/main/java/com/sonar/orchestrator/util/CommandExecutor.java"; - fileOnSameProject = newFileDto(project1).setKey(key2).setLongName("CommandExecutor"); + fileOnSameProject = newFileDto(project1, null).setKey(key2).setLongName("CommandExecutor"); // File on different project String key3 = "com.sonarsource.orchestrator:sonar-orchestrator:src/main/java/com/sonar/orchestrator/util/CommandExecutor.java"; - fileOnDifferentProject = newFileDto(project2).setKey(key3).setLongName("CommandExecutor"); + fileOnDifferentProject = newFileDto(project2, null).setKey(key3).setLongName("CommandExecutor"); componentDao.insert(dbSession, currentFile, fileOnSameProject, fileOnDifferentProject); dbSession.commit(); @@ -194,9 +194,9 @@ public void duplication_on_removed_file() throws Exception { @Test public void compare_duplications() { - ComponentDto currentFile = newFileDto(project1).setId(11L); - ComponentDto fileOnSameProject = newFileDto(project1).setId(12L); - ComponentDto fileOnDifferentProject = newFileDto(project2).setId(13L); + ComponentDto currentFile = newFileDto(project1, null).setId(11L); + ComponentDto fileOnSameProject = newFileDto(project1, null).setId(12L); + ComponentDto fileOnDifferentProject = newFileDto(project2, null).setId(13L); DuplicationsParser.DuplicationComparator comparator = new DuplicationsParser.DuplicationComparator(currentFile.uuid(), currentFile.projectUuid()); diff --git a/server/sonar-server/src/test/java/org/sonar/server/issue/ActionServiceTest.java b/server/sonar-server/src/test/java/org/sonar/server/issue/ActionServiceTest.java index 9d8ce2a9480d..cd042c45c3de 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/issue/ActionServiceTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/issue/ActionServiceTest.java @@ -65,7 +65,7 @@ public void before() { when(dbClient.openSession(false)).thenReturn(session); project = newProjectDto(PROJECT_UUID).setKey(PROJECT_KEY); - issue = IssueTesting.newDto(newXooX1().setId(10), newFileDto(project), project).setKee(ISSUE_KEY); + issue = IssueTesting.newDto(newXooX1().setId(10), newFileDto(project, null), project).setKee(ISSUE_KEY); underTest = new ActionService(dbClient, userSession, issueService); } @@ -90,13 +90,13 @@ public void return_no_actions_when_not_logged() { @Test public void doest_not_return_assign_to_me_action_when_issue_already_assigned_to_user() { userSession.login("julien"); - IssueDto issue = IssueTesting.newDto(newXooX1().setId(10), newFileDto(project), project).setKee(ISSUE_KEY).setAssignee("julien"); + IssueDto issue = IssueTesting.newDto(newXooX1().setId(10), newFileDto(project, null), project).setKee(ISSUE_KEY).setAssignee("julien"); assertThat(underTest.listAvailableActions(issue.toDefaultIssue())).doesNotContain("assign_to_me"); } @Test public void return_only_comment_action_when_issue_has_a_resolution() { - IssueDto issue = IssueTesting.newDto(newXooX1().setId(10), newFileDto(project), project).setKee(ISSUE_KEY).setResolution(RESOLUTION_FIXED); + IssueDto issue = IssueTesting.newDto(newXooX1().setId(10), newFileDto(project, null), project).setKee(ISSUE_KEY).setResolution(RESOLUTION_FIXED); assertThat(underTest.listAvailableActions(issue.toDefaultIssue())).containsOnly("comment"); } diff --git a/server/sonar-server/src/test/java/org/sonar/server/issue/IssueBulkChangeServiceMediumTest.java b/server/sonar-server/src/test/java/org/sonar/server/issue/IssueBulkChangeServiceMediumTest.java index 2ddc44798c42..2f8bd826394b 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/issue/IssueBulkChangeServiceMediumTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/issue/IssueBulkChangeServiceMediumTest.java @@ -90,7 +90,7 @@ public void setUp() { SnapshotDto projectSnapshot = SnapshotTesting.newAnalysis(project); tester.get(SnapshotDao.class).insert(session, projectSnapshot); - file = ComponentTesting.newFileDto(project).setKey("MyComponent"); + file = ComponentTesting.newFileDto(project, null).setKey("MyComponent"); tester.get(ComponentDao.class).insert(session, file); // project can be seen by anyone diff --git a/server/sonar-server/src/test/java/org/sonar/server/issue/IssueCommentServiceMediumTest.java b/server/sonar-server/src/test/java/org/sonar/server/issue/IssueCommentServiceMediumTest.java index f98aed6c7a62..20647c01b3df 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/issue/IssueCommentServiceMediumTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/issue/IssueCommentServiceMediumTest.java @@ -84,7 +84,7 @@ public void setUp() { SnapshotDto projectSnapshot = SnapshotTesting.newAnalysis(project); tester.get(SnapshotDao.class).insert(session, projectSnapshot); - file = ComponentTesting.newFileDto(project); + file = ComponentTesting.newFileDto(project, null); tester.get(ComponentDao.class).insert(session, file); // project can be seen by anyone diff --git a/server/sonar-server/src/test/java/org/sonar/server/issue/IssueCommentServiceTest.java b/server/sonar-server/src/test/java/org/sonar/server/issue/IssueCommentServiceTest.java index c02a79c08eae..60707a7b75a9 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/issue/IssueCommentServiceTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/issue/IssueCommentServiceTest.java @@ -103,7 +103,7 @@ public void should_find_comment() { @Test public void should_add_comment() { - IssueDto issueDto = IssueTesting.newDto(RuleTesting.newXooX1().setId(500), ComponentTesting.newFileDto(ComponentTesting.newProjectDto()), ComponentTesting.newProjectDto()); + IssueDto issueDto = IssueTesting.newDto(RuleTesting.newXooX1().setId(500), ComponentTesting.newFileDto(ComponentTesting.newProjectDto(), null), ComponentTesting.newProjectDto()); when(issueService.getByKeyForUpdate(session, "ABCD")).thenReturn(issueDto); when(issueCommentService.findComments(session, "ABCD")).thenReturn(newArrayList(new DefaultIssueComment())); @@ -146,7 +146,7 @@ public void should_prevent_adding_null_comment() { @Test public void fail_if_comment_not_inserted_in_db() { - IssueDto issueDto = IssueTesting.newDto(RuleTesting.newXooX1().setId(500), ComponentTesting.newFileDto(ComponentTesting.newProjectDto()), ComponentTesting.newProjectDto()); + IssueDto issueDto = IssueTesting.newDto(RuleTesting.newXooX1().setId(500), ComponentTesting.newFileDto(ComponentTesting.newProjectDto(), null), ComponentTesting.newProjectDto()); when(issueService.getByKeyForUpdate(session, "ABCD")).thenReturn(issueDto); // Comment has not be inserted in db when(issueCommentService.findComments(session, "ABCD")).thenReturn(Collections.emptyList()); diff --git a/server/sonar-server/src/test/java/org/sonar/server/issue/IssueServiceMediumTest.java b/server/sonar-server/src/test/java/org/sonar/server/issue/IssueServiceMediumTest.java index 5a6d8846cda8..37153200261b 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/issue/IssueServiceMediumTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/issue/IssueServiceMediumTest.java @@ -344,7 +344,7 @@ private ComponentDto newProject() { } private ComponentDto newFile(ComponentDto project) { - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); tester.get(ComponentDao.class).insert(session, file); session.commit(); return file; diff --git a/server/sonar-server/src/test/java/org/sonar/server/issue/index/IssueIndexDebtTest.java b/server/sonar-server/src/test/java/org/sonar/server/issue/index/IssueIndexDebtTest.java index 72e023b37ec4..a10ad4212900 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/issue/index/IssueIndexDebtTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/issue/index/IssueIndexDebtTest.java @@ -84,9 +84,9 @@ public void facets_on_projects() { ComponentDto project2 = ComponentTesting.newProjectDto("EFGH"); indexIssues( - IssueTesting.newDoc("ISSUE1", ComponentTesting.newFileDto(project)).setEffort(10L), - IssueTesting.newDoc("ISSUE2", ComponentTesting.newFileDto(project)).setEffort(10L), - IssueTesting.newDoc("ISSUE3", ComponentTesting.newFileDto(project2)).setEffort(10L)); + IssueTesting.newDoc("ISSUE1", ComponentTesting.newFileDto(project, null)).setEffort(10L), + IssueTesting.newDoc("ISSUE2", ComponentTesting.newFileDto(project, null)).setEffort(10L), + IssueTesting.newDoc("ISSUE3", ComponentTesting.newFileDto(project2, null)).setEffort(10L)); SearchResult result = index.search(newQueryBuilder().build(), new SearchOptions().addFacets(newArrayList("projectUuids"))); @@ -98,9 +98,9 @@ public void facets_on_projects() { @Test public void facets_on_components() { ComponentDto project = ComponentTesting.newProjectDto("A"); - ComponentDto file1 = ComponentTesting.newFileDto(project, "ABCD"); - ComponentDto file2 = ComponentTesting.newFileDto(project, "BCDE"); - ComponentDto file3 = ComponentTesting.newFileDto(project, "CDEF"); + ComponentDto file1 = ComponentTesting.newFileDto(project, null, "ABCD"); + ComponentDto file2 = ComponentTesting.newFileDto(project, null, "BCDE"); + ComponentDto file3 = ComponentTesting.newFileDto(project, null, "CDEF"); indexIssues( IssueTesting.newDoc("ISSUE1", project), @@ -119,8 +119,8 @@ public void facets_on_components() { @Test public void facets_on_directories() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file1 = ComponentTesting.newFileDto(project).setPath("src/main/xoo/F1.xoo"); - ComponentDto file2 = ComponentTesting.newFileDto(project).setPath("F2.xoo"); + ComponentDto file1 = ComponentTesting.newFileDto(project, null).setPath("src/main/xoo/F1.xoo"); + ComponentDto file2 = ComponentTesting.newFileDto(project, null).setPath("F2.xoo"); indexIssues( IssueTesting.newDoc("ISSUE1", file1).setDirectoryPath("/src/main/xoo"), @@ -135,7 +135,7 @@ public void facets_on_directories() { @Test public void facets_on_severities() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setSeverity(Severity.INFO), @@ -151,7 +151,7 @@ public void facets_on_severities() { @Test public void facets_on_statuses() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setStatus(Issue.STATUS_CLOSED), @@ -167,7 +167,7 @@ public void facets_on_statuses() { @Test public void facets_on_resolutions() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setResolution(Issue.RESOLUTION_FALSE_POSITIVE), @@ -183,7 +183,7 @@ public void facets_on_resolutions() { @Test public void facets_on_languages() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); RuleKey ruleKey = RuleKey.of("repo", "X1"); indexIssues(IssueTesting.newDoc("ISSUE1", file).setRuleKey(ruleKey.toString()).setLanguage("xoo").setEffort(10L)); @@ -197,7 +197,7 @@ public void facets_on_languages() { @Test public void facets_on_assignees() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setAssignee("steph"), @@ -214,7 +214,7 @@ public void facets_on_assignees() { @Test public void facets_on_authors() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setAuthorLogin("steph"), @@ -249,9 +249,9 @@ public void deprecated_debt_facets() { ComponentDto project2 = ComponentTesting.newProjectDto("EFGH"); indexIssues( - IssueTesting.newDoc("ISSUE1", ComponentTesting.newFileDto(project)).setEffort(10L), - IssueTesting.newDoc("ISSUE2", ComponentTesting.newFileDto(project)).setEffort(10L), - IssueTesting.newDoc("ISSUE3", ComponentTesting.newFileDto(project2)).setEffort(10L)); + IssueTesting.newDoc("ISSUE1", ComponentTesting.newFileDto(project, null)).setEffort(10L), + IssueTesting.newDoc("ISSUE2", ComponentTesting.newFileDto(project, null)).setEffort(10L), + IssueTesting.newDoc("ISSUE3", ComponentTesting.newFileDto(project2, null)).setEffort(10L)); SearchResult result = index.search(IssueQuery.builder(userSessionRule).facetMode(DEPRECATED_FACET_MODE_DEBT).build(), new SearchOptions().addFacets(newArrayList("projectUuids"))); @@ -262,7 +262,7 @@ public void deprecated_debt_facets() { protected SearchOptions fixtureForCreatedAtFacet() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); IssueDoc issue0 = IssueTesting.newDoc("ISSUE0", file).setFuncCreationDate(DateUtils.parseDateTime("2011-04-25T01:05:13+0100")); IssueDoc issue1 = IssueTesting.newDoc("ISSUE1", file).setFuncCreationDate(DateUtils.parseDateTime("2014-09-01T12:34:56+0100")); diff --git a/server/sonar-server/src/test/java/org/sonar/server/issue/index/IssueIndexTest.java b/server/sonar-server/src/test/java/org/sonar/server/issue/index/IssueIndexTest.java index 987fb1dcde6b..3bca5f4b8007 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/issue/index/IssueIndexTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/issue/index/IssueIndexTest.java @@ -95,7 +95,7 @@ public void setUp() { @Test public void get_by_key() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); IssueDoc issue = IssueTesting.newDoc("ISSUE1", file) .setEffort(100L); indexIssues(issue); @@ -110,7 +110,7 @@ public void get_by_key() { @Test public void get_by_key_with_attributes() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); IssueDoc issue = IssueTesting.newDoc("ISSUE1", file).setAttributes((KeyValueFormat.format(ImmutableMap.of("jira-issue-key", "SONAR-1234")))); indexIssues(issue); @@ -121,7 +121,7 @@ public void get_by_key_with_attributes() { @Test(expected = IllegalStateException.class) public void comments_field_is_not_available() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); IssueDoc issue = IssueTesting.newDoc("ISSUE1", file); indexIssues(issue); @@ -132,7 +132,7 @@ public void comments_field_is_not_available() { @Test(expected = IllegalStateException.class) public void is_new_field_is_not_available() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); IssueDoc issue = IssueTesting.newDoc("ISSUE1", file); indexIssues(issue); @@ -150,8 +150,8 @@ public void filter_by_keys() { ComponentDto project = ComponentTesting.newProjectDto(); indexIssues( - IssueTesting.newDoc("1", ComponentTesting.newFileDto(project)), - IssueTesting.newDoc("2", ComponentTesting.newFileDto(project))); + IssueTesting.newDoc("1", ComponentTesting.newFileDto(project, null)), + IssueTesting.newDoc("2", ComponentTesting.newFileDto(project, null))); assertThat(underTest.search(IssueQuery.builder(userSessionRule).issueKeys(newArrayList("1", "2")).build(), new SearchOptions()).getDocs()).hasSize(2); assertThat(underTest.search(IssueQuery.builder(userSessionRule).issueKeys(newArrayList("1")).build(), new SearchOptions()).getDocs()).hasSize(1); @@ -166,11 +166,11 @@ public void filter_by_projects() { indexIssues( IssueTesting.newDoc("ISSUE1", project), - IssueTesting.newDoc("ISSUE2", ComponentTesting.newFileDto(project)), + IssueTesting.newDoc("ISSUE2", ComponentTesting.newFileDto(project, null)), IssueTesting.newDoc("ISSUE3", module), - IssueTesting.newDoc("ISSUE4", ComponentTesting.newFileDto(module)), + IssueTesting.newDoc("ISSUE4", ComponentTesting.newFileDto(module, null)), IssueTesting.newDoc("ISSUE5", subModule), - IssueTesting.newDoc("ISSUE6", ComponentTesting.newFileDto(subModule))); + IssueTesting.newDoc("ISSUE6", ComponentTesting.newFileDto(subModule, null))); assertThat(underTest.search(IssueQuery.builder(userSessionRule).projectUuids(newArrayList(project.uuid())).build(), new SearchOptions()).getDocs()).hasSize(6); assertThat(underTest.search(IssueQuery.builder(userSessionRule).projectUuids(newArrayList("unknown")).build(), new SearchOptions()).getDocs()).isEmpty(); @@ -182,9 +182,9 @@ public void facets_on_projects() { ComponentDto project2 = ComponentTesting.newProjectDto("EFGH"); indexIssues( - IssueTesting.newDoc("ISSUE1", ComponentTesting.newFileDto(project)), - IssueTesting.newDoc("ISSUE2", ComponentTesting.newFileDto(project)), - IssueTesting.newDoc("ISSUE3", ComponentTesting.newFileDto(project2))); + IssueTesting.newDoc("ISSUE1", ComponentTesting.newFileDto(project, null)), + IssueTesting.newDoc("ISSUE2", ComponentTesting.newFileDto(project, null)), + IssueTesting.newDoc("ISSUE3", ComponentTesting.newFileDto(project2, null))); SearchResult result = underTest.search(IssueQuery.builder(userSessionRule).build(), new SearchOptions().addFacets(newArrayList("projectUuids"))); assertThat(result.getFacets().getNames()).containsOnly("projectUuids"); @@ -196,7 +196,7 @@ public void filter_by_modules() { ComponentDto project = ComponentTesting.newProjectDto(); ComponentDto module = ComponentTesting.newModuleDto(project); ComponentDto subModule = ComponentTesting.newModuleDto(module); - ComponentDto file = ComponentTesting.newFileDto(subModule); + ComponentDto file = ComponentTesting.newFileDto(subModule, null); indexIssues( IssueTesting.newDoc("ISSUE3", module), @@ -229,9 +229,9 @@ public void filter_by_components_on_contextualized_search() { ComponentDto project = ComponentTesting.newProjectDto(); ComponentDto module = ComponentTesting.newModuleDto(project); ComponentDto subModule = ComponentTesting.newModuleDto(module); - ComponentDto file1 = ComponentTesting.newFileDto(project); - ComponentDto file2 = ComponentTesting.newFileDto(module); - ComponentDto file3 = ComponentTesting.newFileDto(subModule); + ComponentDto file1 = ComponentTesting.newFileDto(project, null); + ComponentDto file2 = ComponentTesting.newFileDto(module, null); + ComponentDto file3 = ComponentTesting.newFileDto(subModule, null); String view = "ABCD"; indexView(view, newArrayList(project.uuid())); @@ -262,11 +262,11 @@ public void filter_by_components_on_contextualized_search() { @Test public void filter_by_components_on_non_contextualized_search() { ComponentDto project = ComponentTesting.newProjectDto("project"); - ComponentDto file1 = ComponentTesting.newFileDto(project, "file1"); + ComponentDto file1 = ComponentTesting.newFileDto(project, null, "file1"); ComponentDto module = ComponentTesting.newModuleDto(project).setUuid("module"); - ComponentDto file2 = ComponentTesting.newFileDto(module, "file2"); + ComponentDto file2 = ComponentTesting.newFileDto(module, null, "file2"); ComponentDto subModule = ComponentTesting.newModuleDto(module).setUuid("subModule"); - ComponentDto file3 = ComponentTesting.newFileDto(subModule, "file3"); + ComponentDto file3 = ComponentTesting.newFileDto(subModule, null, "file3"); String view = "ABCD"; indexView(view, newArrayList(project.uuid())); @@ -293,9 +293,9 @@ public void filter_by_components_on_non_contextualized_search() { @Test public void facets_on_components() { ComponentDto project = ComponentTesting.newProjectDto("A"); - ComponentDto file1 = ComponentTesting.newFileDto(project, "ABCD"); - ComponentDto file2 = ComponentTesting.newFileDto(project, "BCDE"); - ComponentDto file3 = ComponentTesting.newFileDto(project, "CDEF"); + ComponentDto file1 = ComponentTesting.newFileDto(project, null, "ABCD"); + ComponentDto file2 = ComponentTesting.newFileDto(project, null, "BCDE"); + ComponentDto file3 = ComponentTesting.newFileDto(project, null, "CDEF"); indexIssues( IssueTesting.newDoc("ISSUE1", project), @@ -313,8 +313,8 @@ public void facets_on_components() { @Test public void filter_by_directories() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file1 = ComponentTesting.newFileDto(project).setPath("src/main/xoo/F1.xoo"); - ComponentDto file2 = ComponentTesting.newFileDto(project).setPath("F2.xoo"); + ComponentDto file1 = ComponentTesting.newFileDto(project, null).setPath("src/main/xoo/F1.xoo"); + ComponentDto file2 = ComponentTesting.newFileDto(project, null).setPath("F2.xoo"); indexIssues( IssueTesting.newDoc("ISSUE1", file1).setDirectoryPath("/src/main/xoo"), @@ -328,8 +328,8 @@ public void filter_by_directories() { @Test public void facets_on_directories() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file1 = ComponentTesting.newFileDto(project).setPath("src/main/xoo/F1.xoo"); - ComponentDto file2 = ComponentTesting.newFileDto(project).setPath("F2.xoo"); + ComponentDto file1 = ComponentTesting.newFileDto(project, null).setPath("src/main/xoo/F1.xoo"); + ComponentDto file2 = ComponentTesting.newFileDto(project, null).setPath("F2.xoo"); indexIssues( IssueTesting.newDoc("ISSUE1", file1).setDirectoryPath("/src/main/xoo"), @@ -343,7 +343,7 @@ public void facets_on_directories() { @Test public void filter_by_views() { ComponentDto project1 = ComponentTesting.newProjectDto(); - ComponentDto file1 = ComponentTesting.newFileDto(project1); + ComponentDto file1 = ComponentTesting.newFileDto(project1, null); ComponentDto project2 = ComponentTesting.newProjectDto(); indexIssues( // Project1 has 2 issues (one on a file and one on the project itself) @@ -369,7 +369,7 @@ public void filter_by_views() { @Test public void filter_by_severities() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setSeverity(Severity.INFO), @@ -383,7 +383,7 @@ public void filter_by_severities() { @Test public void facets_on_severities() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setSeverity(Severity.INFO), @@ -398,7 +398,7 @@ public void facets_on_severities() { @Test public void filter_by_statuses() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setStatus(Issue.STATUS_CLOSED), @@ -413,7 +413,7 @@ public void filter_by_statuses() { @Test public void facets_on_statuses() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setStatus(Issue.STATUS_CLOSED), @@ -428,7 +428,7 @@ public void facets_on_statuses() { @Test public void filter_by_resolutions() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setResolution(Issue.RESOLUTION_FALSE_POSITIVE), @@ -445,7 +445,7 @@ public void filter_by_resolutions() { @Test public void facets_on_resolutions() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setResolution(Issue.RESOLUTION_FALSE_POSITIVE), @@ -460,7 +460,7 @@ public void facets_on_resolutions() { @Test public void filter_by_resolved() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setStatus(Issue.STATUS_CLOSED).setResolution(Issue.RESOLUTION_FIXED), @@ -475,7 +475,7 @@ public void filter_by_resolved() { @Test public void filter_by_rules() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); RuleKey ruleKey = RuleKey.of("repo", "X1"); indexIssues(IssueTesting.newDoc("ISSUE1", file).setRuleKey(ruleKey.toString())); @@ -487,7 +487,7 @@ public void filter_by_rules() { @Test public void filter_by_languages() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); RuleKey ruleKey = RuleKey.of("repo", "X1"); indexIssues(IssueTesting.newDoc("ISSUE1", file).setRuleKey(ruleKey.toString()).setLanguage("xoo")); @@ -500,7 +500,7 @@ public void filter_by_languages() { @Test public void facets_on_languages() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); RuleKey ruleKey = RuleKey.of("repo", "X1"); indexIssues(IssueTesting.newDoc("ISSUE1", file).setRuleKey(ruleKey.toString()).setLanguage("xoo")); @@ -513,7 +513,7 @@ public void facets_on_languages() { @Test public void filter_by_assignees() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setAssignee("steph"), @@ -528,7 +528,7 @@ public void filter_by_assignees() { @Test public void facets_on_assignees() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setAssignee("steph"), @@ -544,7 +544,7 @@ public void facets_on_assignees() { @Test public void facets_on_assignees_supports_dashes() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setAssignee("j-b"), @@ -561,7 +561,7 @@ public void facets_on_assignees_supports_dashes() { @Test public void filter_by_assigned() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setAssignee("steph"), @@ -576,7 +576,7 @@ public void filter_by_assigned() { @Test public void filter_by_authors() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setAuthorLogin("steph"), @@ -591,7 +591,7 @@ public void filter_by_authors() { @Test public void facets_on_authors() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setAuthorLogin("steph"), @@ -607,7 +607,7 @@ public void facets_on_authors() { @Test public void filter_by_created_after() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setFuncCreationDate(parseDate("2014-09-20")), @@ -623,7 +623,7 @@ public void filter_by_created_after() { @Test public void filter_by_created_before() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setFuncCreationDate(parseDate("2014-09-20")), @@ -639,7 +639,7 @@ public void filter_by_created_before() { @Test public void filter_by_created_after_and_before() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setFuncCreationDate(parseDate("2014-09-20")), @@ -685,7 +685,7 @@ public void filter_by_created_after_and_before() { @Test public void filter_by_create_after_and_before_take_into_account_timezone() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setFuncCreationDate(parseDateTime("2014-09-20T00:00:00+0100")), @@ -732,7 +732,7 @@ public void filter_by_created_after_must_not_be_in_future() { @Test public void filter_by_created_at() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues(IssueTesting.newDoc("ISSUE1", file).setFuncCreationDate(parseDate("2014-09-20"))); @@ -872,7 +872,7 @@ public void facet_on_created_at_without_issues() { private SearchOptions fixtureForCreatedAtFacet() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); IssueDoc issue0 = IssueTesting.newDoc("ISSUE0", file).setFuncCreationDate(parseDateTime("2011-04-25T00:05:13+0000")); IssueDoc issue1 = IssueTesting.newDoc("ISSUE1", file).setFuncCreationDate(parseDateTime("2014-09-01T12:34:56+0100")); @@ -890,7 +890,7 @@ private SearchOptions fixtureForCreatedAtFacet() { @Test public void paging() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); for (int i = 0; i < 12; i++) { indexIssues(IssueTesting.newDoc("ISSUE" + i, file)); } @@ -913,7 +913,7 @@ public void paging() { @Test public void search_with_max_limit() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); List issues = newArrayList(); for (int i = 0; i < 500; i++) { String key = "ISSUE" + i; @@ -929,7 +929,7 @@ public void search_with_max_limit() { @Test public void sort_by_status() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setStatus(Issue.STATUS_OPEN), @@ -952,7 +952,7 @@ public void sort_by_status() { @Test public void sort_by_severity() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setSeverity(Severity.BLOCKER), @@ -981,7 +981,7 @@ public void sort_by_severity() { @Test public void sort_by_assignee() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setAssignee("steph"), @@ -1003,7 +1003,7 @@ public void sort_by_assignee() { @Test public void sort_by_creation_date() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setFuncCreationDate(parseDateTime("2014-09-23T00:00:00+0100")), @@ -1025,7 +1025,7 @@ public void sort_by_creation_date() { @Test public void sort_by_update_date() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setFuncUpdateDate(parseDateTime("2014-09-23T00:00:00+0100")), @@ -1047,7 +1047,7 @@ public void sort_by_update_date() { @Test public void sort_by_close_date() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); indexIssues( IssueTesting.newDoc("ISSUE1", file).setFuncCloseDate(parseDateTime("2014-09-23T00:00:00+0100")), @@ -1072,8 +1072,8 @@ public void sort_by_close_date() { @Test public void sort_by_file_and_line() { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file1 = ComponentTesting.newFileDto(project, "F1").setPath("src/main/xoo/org/sonar/samples/File.xoo"); - ComponentDto file2 = ComponentTesting.newFileDto(project, "F2").setPath("src/main/xoo/org/sonar/samples/File2.xoo"); + ComponentDto file1 = ComponentTesting.newFileDto(project, null, "F1").setPath("src/main/xoo/org/sonar/samples/File.xoo"); + ComponentDto file2 = ComponentTesting.newFileDto(project, null, "F2").setPath("src/main/xoo/org/sonar/samples/File2.xoo"); indexIssues( // file F1 @@ -1116,9 +1116,9 @@ public void authorized_issues_on_groups() { ComponentDto project2 = ComponentTesting.newProjectDto().setKey("project2"); ComponentDto project3 = ComponentTesting.newProjectDto().setKey("project3"); - ComponentDto file1 = ComponentTesting.newFileDto(project1).setKey("file1"); - ComponentDto file2 = ComponentTesting.newFileDto(project2).setKey("file2"); - ComponentDto file3 = ComponentTesting.newFileDto(project3).setKey("file3"); + ComponentDto file1 = ComponentTesting.newFileDto(project1, null).setKey("file1"); + ComponentDto file2 = ComponentTesting.newFileDto(project2, null).setKey("file2"); + ComponentDto file3 = ComponentTesting.newFileDto(project3, null).setKey("file3"); // project1 can be seen by sonar-users indexIssue(IssueTesting.newDoc("ISSUE1", file1), "sonar-users", null); @@ -1149,9 +1149,9 @@ public void authorized_issues_on_user() { ComponentDto project2 = ComponentTesting.newProjectDto().setKey("project2"); ComponentDto project3 = ComponentTesting.newProjectDto().setKey("project3"); - ComponentDto file1 = ComponentTesting.newFileDto(project1).setKey("file1"); - ComponentDto file2 = ComponentTesting.newFileDto(project2).setKey("file2"); - ComponentDto file3 = ComponentTesting.newFileDto(project3).setKey("file3"); + ComponentDto file1 = ComponentTesting.newFileDto(project1, null).setKey("file1"); + ComponentDto file2 = ComponentTesting.newFileDto(project2, null).setKey("file2"); + ComponentDto file3 = ComponentTesting.newFileDto(project3, null).setKey("file3"); // project1 can be seen by john, project2 by max, project3 cannot be seen by anyone indexIssue(IssueTesting.newDoc("ISSUE1", file1), null, "john"); @@ -1177,8 +1177,8 @@ public void authorized_issues_on_user_and_group() { ComponentDto project1 = ComponentTesting.newProjectDto().setKey("project1"); ComponentDto project2 = ComponentTesting.newProjectDto().setKey("project2"); - ComponentDto file1 = ComponentTesting.newFileDto(project1).setKey("file1"); - ComponentDto file2 = ComponentTesting.newFileDto(project2).setKey("file2"); + ComponentDto file1 = ComponentTesting.newFileDto(project1, null).setKey("file1"); + ComponentDto file2 = ComponentTesting.newFileDto(project2, null).setKey("file2"); // project1 can be seen by john and by sonar-users indexIssue(IssueTesting.newDoc("ISSUE1", file1), "sonar-users", "john"); @@ -1191,7 +1191,7 @@ public void authorized_issues_on_user_and_group() { @Test public void search_issues_for_batch_return_needed_fields() { ComponentDto project = ComponentTesting.newProjectDto("PROJECT"); - ComponentDto file = ComponentTesting.newFileDto(project).setPath("src/File.xoo"); + ComponentDto file = ComponentTesting.newFileDto(project, null).setPath("src/File.xoo"); IssueDoc issue = IssueTesting.newDoc("ISSUE", file) .setRuleKey("squid:S001") @@ -1229,7 +1229,7 @@ public void search_issues_for_batch() { ComponentDto project = ComponentTesting.newProjectDto(); ComponentDto module = ComponentTesting.newModuleDto(project); ComponentDto subModule = ComponentTesting.newModuleDto(module); - ComponentDto file = ComponentTesting.newFileDto(subModule); + ComponentDto file = ComponentTesting.newFileDto(subModule, null); indexIssues( IssueTesting.newDoc("ISSUE3", module), @@ -1260,8 +1260,8 @@ public void search_issues_for_batch_return_only_authorized_issues() { ComponentDto project1 = ComponentTesting.newProjectDto().setKey("project1"); ComponentDto project2 = ComponentTesting.newProjectDto().setKey("project2"); - ComponentDto file1 = ComponentTesting.newFileDto(project1).setKey("file1"); - ComponentDto file2 = ComponentTesting.newFileDto(project2).setKey("file2"); + ComponentDto file1 = ComponentTesting.newFileDto(project1, null).setKey("file1"); + ComponentDto file2 = ComponentTesting.newFileDto(project2, null).setKey("file2"); // project1 can be seen by sonar-users indexIssue(IssueTesting.newDoc("ISSUE1", file1), "sonar-users", null); diff --git a/server/sonar-server/src/test/java/org/sonar/server/issue/ws/SearchActionComponentsMediumTest.java b/server/sonar-server/src/test/java/org/sonar/server/issue/ws/SearchActionComponentsMediumTest.java index 5d92cce89c51..23fde1c21a66 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/issue/ws/SearchActionComponentsMediumTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/issue/ws/SearchActionComponentsMediumTest.java @@ -97,7 +97,7 @@ public void issues_on_different_projects() throws Exception { RuleDto rule = newRule(); ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(newFileDto(project, "F1").setKey("FK1")); + ComponentDto file = insertComponent(newFileDto(project, null, "F1").setKey("FK1")); IssueDto issue = IssueTesting.newDto(rule, file, project) .setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2") .setStatus("OPEN").setResolution("OPEN") @@ -108,7 +108,7 @@ public void issues_on_different_projects() throws Exception { ComponentDto project2 = insertComponent(ComponentTesting.newProjectDto("P2").setKey("PK2")); setDefaultProjectPermission(project2); - ComponentDto file2 = insertComponent(newFileDto(project2, "F2").setKey("FK2")); + ComponentDto file2 = insertComponent(newFileDto(project2, null, "F2").setKey("FK2")); IssueDto issue2 = IssueTesting.newDto(rule, file2, project2) .setKee("92fd47d4-b650-4037-80bc-7b112bd4eac2") .setStatus("OPEN").setResolution("OPEN") @@ -128,7 +128,7 @@ public void do_not_return_module_key_on_single_module_projects() throws IOExcept ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); ComponentDto module = insertComponent(newModuleDto("M1", project).setKey("MK1")); - ComponentDto file = insertComponent(newFileDto(module, "F1").setKey("FK1")); + ComponentDto file = insertComponent(newFileDto(module, null, "F1").setKey("FK1")); RuleDto newRule = newRule(); IssueDto issueInModule = IssueTesting.newDto(newRule, file, project).setKee("ISSUE_IN_MODULE"); IssueDto issueInRootModule = IssueTesting.newDto(newRule, project, project).setKee("ISSUE_IN_ROOT_MODULE"); @@ -157,7 +157,7 @@ public void do_not_return_module_key_on_single_module_projects() throws IOExcept public void search_by_project_uuid() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(newFileDto(project, "F1").setKey("FK1")); + ComponentDto file = insertComponent(newFileDto(project, null, "F1").setKey("FK1")); IssueDto issue = IssueTesting.newDto(newRule(), file, project).setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2"); db.issueDao().insert(session, issue); session.commit(); @@ -188,7 +188,7 @@ public void search_by_project_uuid() throws Exception { public void search_since_leak_period_on_project() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(newFileDto(project, "F1").setKey("FK1")); + ComponentDto file = insertComponent(newFileDto(project, null, "F1").setKey("FK1")); db.snapshotDao().insert(session, newAnalysis(project) .setPeriodDate(1, parseDateTime("2015-09-03T00:00:00+0100").getTime())); @@ -217,7 +217,7 @@ public void search_since_leak_period_on_file_in_module_project() throws Exceptio ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); ComponentDto module = insertComponent(newModuleDto(project)); - ComponentDto file = insertComponent(newFileDto(module, "F1").setKey("FK1")); + ComponentDto file = insertComponent(newFileDto(module, null, "F1").setKey("FK1")); db.snapshotDao().insert(session, newAnalysis(project).setPeriodDate(1, parseDateTime("2015-09-03T00:00:00+0100").getTime())); RuleDto rule = newRule(); @@ -249,9 +249,9 @@ public void project_facet_is_sticky() throws Exception { setDefaultProjectPermission(project1); setDefaultProjectPermission(project2); setDefaultProjectPermission(project3); - ComponentDto file1 = insertComponent(newFileDto(project1, "F1").setKey("FK1")); - ComponentDto file2 = insertComponent(newFileDto(project2, "F2").setKey("FK2")); - ComponentDto file3 = insertComponent(newFileDto(project3, "F3").setKey("FK3")); + ComponentDto file1 = insertComponent(newFileDto(project1, null, "F1").setKey("FK1")); + ComponentDto file2 = insertComponent(newFileDto(project2, null, "F2").setKey("FK2")); + ComponentDto file3 = insertComponent(newFileDto(project3, null, "F3").setKey("FK3")); RuleDto rule = newRule(); IssueDto issue1 = IssueTesting.newDto(rule, file1, project1).setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2"); IssueDto issue2 = IssueTesting.newDto(rule, file2, project2).setKee("2bd4eac2-b650-4037-80bc-7b1182fd47d4"); @@ -271,7 +271,7 @@ public void project_facet_is_sticky() throws Exception { public void search_by_file_uuid() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(newFileDto(project, "F1").setKey("FK1")); + ComponentDto file = insertComponent(newFileDto(project, null, "F1").setKey("FK1")); IssueDto issue = IssueTesting.newDto(newRule(), file, project).setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2"); db.issueDao().insert(session, issue); session.commit(); @@ -302,8 +302,8 @@ public void search_by_file_uuid() throws Exception { public void search_by_file_key() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(newFileDto(project, "F1").setKey("FK1")); - ComponentDto unitTest = insertComponent(newFileDto(project, "F2").setQualifier(Qualifiers.UNIT_TEST_FILE).setKey("FK2")); + ComponentDto file = insertComponent(newFileDto(project, null, "F1").setKey("FK1")); + ComponentDto unitTest = insertComponent(newFileDto(project, null, "F2").setQualifier(Qualifiers.UNIT_TEST_FILE).setKey("FK2")); RuleDto rule = newRule(); IssueDto issueOnFile = IssueTesting.newDto(rule, file, project).setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2"); IssueDto issueOnTest = IssueTesting.newDto(rule, unitTest, project).setKee("2bd4eac2-b650-4037-80bc-7b1182fd47d4"); @@ -326,9 +326,9 @@ public void search_by_file_key() throws Exception { public void display_file_facet() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); - ComponentDto file1 = insertComponent(newFileDto(project, "F1").setKey("FK1")); - ComponentDto file2 = insertComponent(newFileDto(project, "F2").setKey("FK2")); - ComponentDto file3 = insertComponent(newFileDto(project, "F3").setKey("FK3")); + ComponentDto file1 = insertComponent(newFileDto(project, null, "F1").setKey("FK1")); + ComponentDto file2 = insertComponent(newFileDto(project, null, "F2").setKey("FK2")); + ComponentDto file3 = insertComponent(newFileDto(project, null, "F3").setKey("FK3")); RuleDto newRule = newRule(); IssueDto issue1 = IssueTesting.newDto(newRule, file1, project).setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2"); IssueDto issue2 = IssueTesting.newDto(newRule, file2, project).setKee("2bd4eac2-b650-4037-80bc-7b1182fd47d4"); @@ -349,7 +349,7 @@ public void search_by_directory_path() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); ComponentDto directory = insertComponent(ComponentTesting.newDirectory(project, "D1", "src/main/java/dir")); - ComponentDto file = insertComponent(newFileDto(project, "F1").setKey("FK1").setPath(directory.path() + "/MyComponent.java")); + ComponentDto file = insertComponent(newFileDto(project, null, "F1").setKey("FK1").setPath(directory.path() + "/MyComponent.java")); IssueDto issue = IssueTesting.newDto(newRule(), file, project).setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2"); db.issueDao().insert(session, issue); session.commit(); @@ -384,8 +384,8 @@ public void search_by_directory_path_in_different_modules() throws Exception { ComponentDto module2 = insertComponent(newModuleDto("M2", project).setKey("MK2")); ComponentDto directory1 = insertComponent(ComponentTesting.newDirectory(module1, "D1", "src/main/java/dir")); ComponentDto directory2 = insertComponent(ComponentTesting.newDirectory(module2, "D2", "src/main/java/dir")); - ComponentDto file1 = insertComponent(newFileDto(module1, "F1").setKey("FK1").setPath(directory1.path() + "/MyComponent.java")); - insertComponent(newFileDto(module2, "F2").setKey("FK2").setPath(directory2.path() + "/MyComponent.java")); + ComponentDto file1 = insertComponent(newFileDto(module1, directory1, "F1").setKey("FK1").setPath(directory1.path() + "/MyComponent.java")); + insertComponent(newFileDto(module2, directory2, "F2").setKey("FK2").setPath(directory2.path() + "/MyComponent.java")); RuleDto rule = newRule(); IssueDto issue1 = IssueTesting.newDto(rule, file1, project).setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2"); db.issueDao().insert(session, issue1); @@ -434,8 +434,8 @@ public void display_module_facet() throws Exception { ComponentDto subModule1 = insertComponent(newModuleDto("SUBM1", module).setKey("SUBMK1")); ComponentDto subModule2 = insertComponent(newModuleDto("SUBM2", module).setKey("SUBMK2")); ComponentDto subModule3 = insertComponent(newModuleDto("SUBM3", module).setKey("SUBMK3")); - ComponentDto file1 = insertComponent(newFileDto(subModule1, "F1").setKey("FK1")); - ComponentDto file2 = insertComponent(newFileDto(subModule2, "F2").setKey("FK2")); + ComponentDto file1 = insertComponent(newFileDto(subModule1, null, "F1").setKey("FK1")); + ComponentDto file2 = insertComponent(newFileDto(subModule2, null, "F2").setKey("FK2")); RuleDto newRule = newRule(); IssueDto issue1 = IssueTesting.newDto(newRule, file1, project).setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2"); IssueDto issue2 = IssueTesting.newDto(newRule, file2, project).setKee("2bd4eac2-b650-4037-80bc-7b1182fd47d4"); @@ -456,7 +456,7 @@ public void display_directory_facet() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); ComponentDto directory = insertComponent(ComponentTesting.newDirectory(project, "D1", "src/main/java/dir")); - ComponentDto file = insertComponent(newFileDto(project, "F1").setKey("FK1").setPath(directory.path() + "/MyComponent.java")); + ComponentDto file = insertComponent(newFileDto(project, directory, "F1").setKey("FK1").setPath(directory.path() + "/MyComponent.java")); IssueDto issue = IssueTesting.newDto(newRule(), file, project).setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2"); db.issueDao().insert(session, issue); session.commit(); @@ -474,7 +474,7 @@ public void display_directory_facet() throws Exception { public void search_by_view_uuid() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(newFileDto(project, "F1").setKey("FK1")); + ComponentDto file = insertComponent(newFileDto(project, null, "F1").setKey("FK1")); insertIssue(IssueTesting.newDto(newRule(), file, project).setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2")); ComponentDto view = insertComponent(ComponentTesting.newProjectDto("V1").setQualifier(Qualifiers.VIEW).setKey("MyView")); @@ -493,7 +493,7 @@ public void search_by_view_uuid() throws Exception { public void search_by_view_uuid_return_only_authorized_view() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(newFileDto(project, "F1").setKey("FK1")); + ComponentDto file = insertComponent(newFileDto(project, null, "F1").setKey("FK1")); insertIssue(IssueTesting.newDto(newRule(), file, project).setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2")); ComponentDto view = insertComponent(ComponentTesting.newProjectDto("V1").setQualifier(Qualifiers.VIEW).setKey("MyView")); @@ -513,7 +513,7 @@ public void search_by_view_uuid_return_only_authorized_view() throws Exception { public void search_by_sub_view_uuid() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(newFileDto(project, "F1").setKey("FK1")); + ComponentDto file = insertComponent(newFileDto(project, null, "F1").setKey("FK1")); insertIssue(IssueTesting.newDto(newRule(), file, project).setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2")); ComponentDto view = insertComponent(ComponentTesting.newProjectDto("V1").setQualifier(Qualifiers.VIEW).setKey("MyView")); @@ -534,7 +534,7 @@ public void search_by_sub_view_uuid() throws Exception { public void search_by_sub_view_uuid_return_only_authorized_view() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(newFileDto(project, "F1").setKey("FK1")); + ComponentDto file = insertComponent(newFileDto(project, null, "F1").setKey("FK1")); insertIssue(IssueTesting.newDto(newRule(), file, project).setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2")); ComponentDto view = insertComponent(ComponentTesting.newProjectDto("V1").setQualifier(Qualifiers.VIEW).setKey("MyView")); @@ -556,7 +556,7 @@ public void search_by_sub_view_uuid_return_only_authorized_view() throws Excepti public void search_by_author() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(newFileDto(project, "F1").setKey("FK1")); + ComponentDto file = insertComponent(newFileDto(project, null, "F1").setKey("FK1")); RuleDto newRule = newRule(); IssueDto issue1 = IssueTesting.newDto(newRule, file, project).setAuthorLogin("leia").setKee("2bd4eac2-b650-4037-80bc-7b112bd4eac2"); IssueDto issue2 = IssueTesting.newDto(newRule, file, project).setAuthorLogin("luke@skywalker.name").setKee("82fd47d4-b650-4037-80bc-7b1182fd47d4"); @@ -582,7 +582,7 @@ public void search_by_author() throws Exception { public void search_by_developer() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(newFileDto(project, "F1").setKey("FK1")); + ComponentDto file = insertComponent(newFileDto(project, null, "F1").setKey("FK1")); ComponentDto developer = insertComponent(ComponentTesting.newDeveloper("Anakin Skywalker")); db.authorDao().insertAuthor(session, "vader", developer.getId()); db.authorDao().insertAuthor(session, "anakin@skywalker.name", developer.getId()); @@ -604,11 +604,11 @@ public void search_by_developer() throws Exception { public void search_by_developer_technical_project() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("P1").setKey("PK1")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(newFileDto(project, "F1").setKey("FK1")); + ComponentDto file = insertComponent(newFileDto(project, null, "F1").setKey("FK1")); ComponentDto otherProject = insertComponent(ComponentTesting.newProjectDto("P2").setKey("PK2")); setDefaultProjectPermission(otherProject); - ComponentDto otherFile = insertComponent(newFileDto(otherProject, "F2").setKey("FK2")); + ComponentDto otherFile = insertComponent(newFileDto(otherProject, null, "F2").setKey("FK2")); ComponentDto developer = insertComponent(ComponentTesting.newDeveloper("Anakin Skywalker")); ComponentDto technicalProject = insertComponent(ComponentTesting.newDevProjectCopy("COPY_P1", project, developer)); diff --git a/server/sonar-server/src/test/java/org/sonar/server/issue/ws/SearchActionMediumTest.java b/server/sonar-server/src/test/java/org/sonar/server/issue/ws/SearchActionMediumTest.java index aa04415b7d44..ee0a2966afb6 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/issue/ws/SearchActionMediumTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/issue/ws/SearchActionMediumTest.java @@ -118,7 +118,7 @@ public void response_contains_all_fields_except_additional_fields() throws Excep ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); IssueDto issue = IssueTesting.newDto(newRule(), file, project) .setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2") .setEffort(10L) @@ -146,7 +146,7 @@ public void issue_with_comments() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); IssueDto issue = IssueTesting.newDto(newRule(), file, project) .setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2"); db.issueDao().insert(session, issue); @@ -182,7 +182,7 @@ public void issue_with_comment_hidden() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); IssueDto issue = IssueTesting.newDto(newRule(), file, project) .setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2"); db.issueDao().insert(session, issue); @@ -216,7 +216,7 @@ public void load_additional_fields() throws Exception { db.userDao().insert(session, new UserDto().setLogin("fabrice").setName("Fabrice").setEmail("fabrice@email.com")); ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY").setLanguage("java")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY").setLanguage("js")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY").setLanguage("js")); IssueDto issue = IssueTesting.newDto(newRule(), file, project) .setKee("82fd47d4-b650-4037-80bc-7b112bd4eac2") @@ -237,7 +237,7 @@ public void issue_on_removed_file() throws Exception { RuleDto rule = newRule(); ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto removedFile = insertComponent(ComponentTesting.newFileDto(project).setUuid("REMOVED_FILE_ID") + ComponentDto removedFile = insertComponent(ComponentTesting.newFileDto(project, null).setUuid("REMOVED_FILE_ID") .setKey("REMOVED_FILE_KEY") .setEnabled(false)); @@ -261,7 +261,7 @@ public void issue_on_removed_file() throws Exception { public void issue_contains_component_id_for_eclipse() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); IssueDto issue = IssueTesting.newDto(newRule(), file, project); db.issueDao().insert(session, issue); session.commit(); @@ -276,7 +276,7 @@ public void apply_paging_with_one_component() throws Exception { RuleDto rule = newRule(); ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); for (int i = 0; i < QueryContext.MAX_LIMIT + 1; i++) { IssueDto issue = IssueTesting.newDto(rule, file, project); tester.get(IssueDao.class).insert(session, issue); @@ -293,7 +293,7 @@ public void components_contains_sub_projects() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("ProjectHavingModule")); setDefaultProjectPermission(project); ComponentDto module = insertComponent(ComponentTesting.newModuleDto(project).setKey("ModuleHavingFile")); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(module, "BCDE").setKey("FileLinkedToModule")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(module, null, "BCDE").setKey("FileLinkedToModule")); IssueDto issue = IssueTesting.newDto(newRule(), file, project); db.issueDao().insert(session, issue); session.commit(); @@ -307,7 +307,7 @@ public void components_contains_sub_projects() throws Exception { public void display_facets() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); IssueDto issue = IssueTesting.newDto(newRule(), file, project) .setIssueCreationDate(DateUtils.parseDate("2014-09-04")) .setIssueUpdateDate(DateUtils.parseDate("2017-12-04")) @@ -331,7 +331,7 @@ public void display_facets() throws Exception { public void display_facets_in_effort_mode() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); IssueDto issue = IssueTesting.newDto(newRule(), file, project) .setIssueCreationDate(DateUtils.parseDate("2014-09-04")) .setIssueUpdateDate(DateUtils.parseDate("2017-12-04")) @@ -356,7 +356,7 @@ public void display_facets_in_effort_mode() throws Exception { public void display_zero_valued_facets_for_selected_items() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); IssueDto issue = IssueTesting.newDto(newRule(), file, project) .setIssueCreationDate(DateUtils.parseDate("2014-09-04")) .setIssueUpdateDate(DateUtils.parseDate("2017-12-04")) @@ -384,7 +384,7 @@ public void filter_by_assigned_to_me() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); RuleDto rule = newRule(); IssueDto issue1 = IssueTesting.newDto(rule, file, project) .setIssueCreationDate(DateUtils.parseDate("2014-09-04")) @@ -428,7 +428,7 @@ public void filter_by_assigned_to_me_unauthenticated() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); RuleDto rule = newRule(); IssueDto issue1 = IssueTesting.newDto(rule, file, project) .setStatus("OPEN") @@ -458,7 +458,7 @@ public void assigned_to_me_facet_is_sticky_relative_to_assignees() throws Except ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); RuleDto rule = newRule(); IssueDto issue1 = IssueTesting.newDto(rule, file, project) .setIssueCreationDate(DateUtils.parseDate("2014-09-04")) @@ -501,7 +501,7 @@ public void sort_by_updated_at() throws Exception { RuleDto rule = newRule(); ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); db.issueDao().insert(session, IssueTesting.newDto(rule, file, project) .setKee("82fd47d4-b650-4037-80bc-7b112bd4eac1") .setIssueUpdateDate(DateUtils.parseDateTime("2014-11-02T00:00:00+0100"))); @@ -526,7 +526,7 @@ public void paging() throws Exception { RuleDto rule = newRule(); ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); for (int i = 0; i < 12; i++) { IssueDto issue = IssueTesting.newDto(rule, file, project); tester.get(IssueDao.class).insert(session, issue); @@ -547,7 +547,7 @@ public void paging_with_page_size_to_minus_one() throws Exception { RuleDto rule = newRule(); ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); for (int i = 0; i < 12; i++) { IssueDto issue = IssueTesting.newDto(rule, file, project); tester.get(IssueDao.class).insert(session, issue); @@ -568,7 +568,7 @@ public void deprecated_paging() throws Exception { RuleDto rule = newRule(); ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); for (int i = 0; i < 12; i++) { IssueDto issue = IssueTesting.newDto(rule, file, project); tester.get(IssueDao.class).insert(session, issue); @@ -596,7 +596,7 @@ public void default_page_size_is_100() throws Exception { public void display_deprecated_debt_fields() throws Exception { ComponentDto project = insertComponent(ComponentTesting.newProjectDto("PROJECT_ID").setKey("PROJECT_KEY")); setDefaultProjectPermission(project); - ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, "FILE_ID").setKey("FILE_KEY")); + ComponentDto file = insertComponent(ComponentTesting.newFileDto(project, null, "FILE_ID").setKey("FILE_KEY")); IssueDto issue = IssueTesting.newDto(newRule(), file, project) .setIssueCreationDate(DateUtils.parseDate("2014-09-04")) .setIssueUpdateDate(DateUtils.parseDate("2017-12-04")) diff --git a/server/sonar-server/src/test/java/org/sonar/server/measure/ws/ComponentActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/measure/ws/ComponentActionTest.java index 0e2cc1dfa815..b4b7e9866231 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/measure/ws/ComponentActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/measure/ws/ComponentActionTest.java @@ -149,7 +149,7 @@ public void developer_measure_by_developer_uuid() { componentDb.insertDeveloperAndSnapshot(developer); ComponentDto project = newProjectDto("project-uuid"); SnapshotDto projectSnapshot = componentDb.insertProjectAndSnapshot(project); - ComponentDto file = newFileDto(project, "file-uuid"); + ComponentDto file = newFileDto(project, null, "file-uuid"); componentDb.insertComponent(file); MetricDto ncloc = insertNclocMetric(); dbClient.measureDao().insert(dbSession, @@ -172,7 +172,7 @@ public void developer_measure_by_developer_key() { componentDb.insertDeveloperAndSnapshot(developer); ComponentDto project = newProjectDto(PROJECT_UUID); SnapshotDto projectSnapshot = componentDb.insertProjectAndSnapshot(project); - ComponentDto file = newFileDto(project, "file-uuid"); + ComponentDto file = newFileDto(project, null, "file-uuid"); componentDb.insertComponent(file); MetricDto ncloc = insertNclocMetric(); dbClient.measureDao().insert(dbSession, @@ -321,7 +321,7 @@ private void insertJsonExampleData() { .setPeriodDate(3, parseDateTime("2016-01-11T10:38:45+0100").getTime()) .setPeriodMode(3, "days") .setPeriodParam(3, "30"); - ComponentDto file = newFileDto(project) + ComponentDto file = newFileDto(project, null) .setUuid("AVIwDXE-bJbJqrw6wFv5") .setKey("MY_PROJECT:ElementImpl.java") .setName("ElementImpl.java") diff --git a/server/sonar-server/src/test/java/org/sonar/server/measure/ws/ComponentTreeActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/measure/ws/ComponentTreeActionTest.java index 158449e9487e..ebf301dfc8a1 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/measure/ws/ComponentTreeActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/measure/ws/ComponentTreeActionTest.java @@ -151,7 +151,7 @@ public void load_measures_and_periods() { userSession.anonymous().addProjectUuidPermissions(UserRole.ADMIN, "project-uuid"); ComponentDto directoryDto = newDirectory(projectDto, "directory-uuid", "path/to/directory").setName("directory-1"); componentDb.insertComponent(directoryDto); - ComponentDto file = newFileDto(directoryDto, "file-uuid").setName("file-1"); + ComponentDto file = newFileDto(directoryDto, null, "file-uuid").setName("file-1"); componentDb.insertComponent(file); MetricDto ncloc = insertNclocMetric(); MetricDto coverage = insertCoverageMetric(); @@ -181,7 +181,7 @@ public void load_measures_with_best_value() { userSession.anonymous().addProjectUuidPermissions(UserRole.ADMIN, "project-uuid"); ComponentDto directoryDto = newDirectory(projectDto, "directory-uuid", "path/to/directory").setName("directory-1"); componentDb.insertComponent(directoryDto); - ComponentDto file = newFileDto(directoryDto, "file-uuid").setName("file-1"); + ComponentDto file = newFileDto(directoryDto, null, "file-uuid").setName("file-1"); componentDb.insertComponent(file); MetricDto coverage = insertCoverageMetric(); dbClient.metricDao().insert(dbSession, newMetricDto() @@ -221,15 +221,15 @@ public void load_measures_with_best_value() { public void load_measures_multi_sort_with_metric_key_and_paginated() { ComponentDto projectDto = newProjectDto("project-uuid"); SnapshotDto projectSnapshot = componentDb.insertProjectAndSnapshot(projectDto); - ComponentDto file9 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-9").setName("file-1")); - ComponentDto file8 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-8").setName("file-1")); - ComponentDto file7 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-7").setName("file-1")); - ComponentDto file6 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-6").setName("file-1")); - ComponentDto file5 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-5").setName("file-1")); - ComponentDto file4 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-4").setName("file-1")); - ComponentDto file3 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-3").setName("file-1")); - ComponentDto file2 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-2").setName("file-1")); - ComponentDto file1 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-1").setName("file-1")); + ComponentDto file9 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-9").setName("file-1")); + ComponentDto file8 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-8").setName("file-1")); + ComponentDto file7 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-7").setName("file-1")); + ComponentDto file6 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-6").setName("file-1")); + ComponentDto file5 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-5").setName("file-1")); + ComponentDto file4 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-4").setName("file-1")); + ComponentDto file3 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-3").setName("file-1")); + ComponentDto file2 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-2").setName("file-1")); + ComponentDto file1 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-1").setName("file-1")); MetricDto coverage = insertCoverageMetric(); dbClient.measureDao().insert(dbSession, newMeasureDto(coverage, file1, projectSnapshot).setValue(1.0d), @@ -260,10 +260,10 @@ public void load_measures_multi_sort_with_metric_key_and_paginated() { public void sort_by_metric_value() { ComponentDto projectDto = newProjectDto("project-uuid"); SnapshotDto projectSnapshot = componentDb.insertProjectAndSnapshot(projectDto); - ComponentDto file4 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-4")); - ComponentDto file3 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-3")); - ComponentDto file1 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-1")); - ComponentDto file2 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-2")); + ComponentDto file4 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-4")); + ComponentDto file3 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-3")); + ComponentDto file1 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-1")); + ComponentDto file2 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-2")); MetricDto ncloc = newMetricDtoWithoutOptimization().setKey("ncloc").setValueType(ValueType.INT.name()).setDirection(1); dbClient.metricDao().insert(dbSession, ncloc); dbClient.measureDao().insert(dbSession, @@ -286,10 +286,10 @@ public void sort_by_metric_value() { public void remove_components_without_measure_on_the_metric_sort() { ComponentDto project = newProjectDto("project-uuid"); SnapshotDto projectSnapshot = componentDb.insertProjectAndSnapshot(project); - ComponentDto file1 = newFileDto(project, "file-uuid-1"); - ComponentDto file2 = newFileDto(project, "file-uuid-2"); - ComponentDto file3 = newFileDto(project, "file-uuid-3"); - ComponentDto file4 = newFileDto(project, "file-uuid-4"); + ComponentDto file1 = newFileDto(project, null, "file-uuid-1"); + ComponentDto file2 = newFileDto(project, null, "file-uuid-2"); + ComponentDto file3 = newFileDto(project, null, "file-uuid-3"); + ComponentDto file4 = newFileDto(project, null, "file-uuid-4"); componentDb.insertComponent(file1); componentDb.insertComponent(file2); componentDb.insertComponent(file3); @@ -321,9 +321,9 @@ public void remove_components_without_measure_on_the_metric_sort() { public void sort_by_metric_period() { ComponentDto projectDto = newProjectDto("project-uuid"); SnapshotDto projectSnapshot = componentDb.insertProjectAndSnapshot(projectDto); - ComponentDto file3 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-3")); - ComponentDto file1 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-1")); - ComponentDto file2 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-2")); + ComponentDto file3 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-3")); + ComponentDto file1 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-1")); + ComponentDto file2 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-2")); MetricDto ncloc = newMetricDtoWithoutOptimization().setKey("ncloc").setValueType(ValueType.INT.name()).setDirection(1); dbClient.metricDao().insert(dbSession, ncloc); dbClient.measureDao().insert(dbSession, @@ -346,10 +346,10 @@ public void sort_by_metric_period() { public void remove_components_without_measure_on_the_metric_period_sort() { ComponentDto projectDto = newProjectDto("project-uuid"); SnapshotDto projectSnapshot = componentDb.insertProjectAndSnapshot(projectDto); - ComponentDto file4 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-4")); - ComponentDto file3 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-3")); - ComponentDto file2 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-2")); - ComponentDto file1 = componentDb.insertComponent(newFileDto(projectDto, "file-uuid-1")); + ComponentDto file4 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-4")); + ComponentDto file3 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-3")); + ComponentDto file2 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-2")); + ComponentDto file1 = componentDb.insertComponent(newFileDto(projectDto, null, "file-uuid-1")); MetricDto ncloc = newMetricDtoWithoutOptimization().setKey("new_ncloc").setValueType(ValueType.INT.name()).setDirection(1); dbClient.metricDao().insert(dbSession, ncloc); dbClient.measureDao().insert(dbSession, @@ -402,8 +402,8 @@ public void load_developer_measures_by_developer_uuid() { ComponentDto project = newProjectDto("project-uuid").setKey("project-key"); componentDb.insertDeveloperAndSnapshot(developer); SnapshotDto projectSnapshot = componentDb.insertProjectAndSnapshot(project); - ComponentDto file1 = componentDb.insertComponent(newFileDto(project, "file1-uuid")); - ComponentDto file2 = componentDb.insertComponent(newFileDto(project, "file2-uuid")); + ComponentDto file1 = componentDb.insertComponent(newFileDto(project, null, "file1-uuid")); + ComponentDto file2 = componentDb.insertComponent(newFileDto(project, null, "file2-uuid")); componentDb.insertComponent(newDevProjectCopy("project-uuid-copy", project, developer)); MetricDto ncloc = insertNclocMetric(); dbClient.measureDao().insert(dbSession, @@ -435,7 +435,7 @@ public void load_developer_measures_by_developer_key() { ComponentDto project = newProjectDto("project-uuid").setKey("project-key"); componentDb.insertDeveloperAndSnapshot(developer); SnapshotDto projectSnapshot = componentDb.insertProjectAndSnapshot(project); - ComponentDto file1 = componentDb.insertComponent(newFileDto(project, "file1-uuid")); + ComponentDto file1 = componentDb.insertComponent(newFileDto(project, null, "file1-uuid")); componentDb.insertComponent(newDevProjectCopy("project-uuid-copy", project, developer)); MetricDto ncloc = insertNclocMetric(); dbClient.measureDao().insert(dbSession, @@ -462,7 +462,7 @@ public void load_measures_when_no_leave_qualifier() { String projectUuid = "project-uuid"; ComponentDto project = newProjectDto(projectUuid); componentDb.insertProjectAndSnapshot(project); - componentDb.insertComponent(newFileDto(project)); + componentDb.insertComponent(newFileDto(project, null)); insertNclocMetric(); ComponentTreeWsResponse result = call(ws.newRequest() @@ -482,7 +482,7 @@ public void fail_when_developer_is_unknown() { ComponentDto project = newProjectDto("project-uuid").setKey("project-key"); componentDb.insertDeveloperAndSnapshot(developer); SnapshotDto projectSnapshot = componentDb.insertProjectAndSnapshot(project); - ComponentDto file1 = componentDb.insertComponent(newFileDto(project, "file1-uuid")); + ComponentDto file1 = componentDb.insertComponent(newFileDto(project, null, "file1-uuid")); componentDb.insertComponent(newDevProjectCopy("project-uuid-copy", project, developer)); MetricDto ncloc = insertNclocMetric(); dbClient.measureDao().insert(dbSession, @@ -652,14 +652,14 @@ private void insertJsonExampleData() { .setPeriodMode(3, "days") .setPeriodParam(3, "30")); - ComponentDto file1 = componentDb.insertComponent(newFileDto(project) + ComponentDto file1 = componentDb.insertComponent(newFileDto(project, null) .setUuid("AVIwDXE-bJbJqrw6wFv5") .setKey("com.sonarsource:java-markdown:src/main/java/com/sonarsource/markdown/impl/ElementImpl.java") .setName("ElementImpl.java") .setLanguage("java") .setQualifier(Qualifiers.FILE) .setPath("src/main/java/com/sonarsource/markdown/impl/ElementImpl.java")); - ComponentDto file2 = componentDb.insertComponent(newFileDto(project) + ComponentDto file2 = componentDb.insertComponent(newFileDto(project, null) .setUuid("AVIwDXE_bJbJqrw6wFwJ") .setKey("com.sonarsource:java-markdown:src/test/java/com/sonarsource/markdown/impl/ElementImplTest.java") .setName("ElementImplTest.java") diff --git a/server/sonar-server/src/test/java/org/sonar/server/permission/ws/AddGroupActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/permission/ws/AddGroupActionTest.java index 6c01be3d0678..75469c254e9c 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/permission/ws/AddGroupActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/permission/ws/AddGroupActionTest.java @@ -206,7 +206,7 @@ public void fail_if_component_uuid_is_not_a_project() throws Exception { insertGroup("sonar-administrators"); ComponentDto project = newProjectDto("project-uuid").setKey("project-key"); insertComponent(project); - insertComponent(ComponentTesting.newFileDto(project, "file-uuid")); + insertComponent(ComponentTesting.newFileDto(project, null, "file-uuid")); commit(); newRequest() diff --git a/server/sonar-server/src/test/java/org/sonar/server/permission/ws/AddUserActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/permission/ws/AddUserActionTest.java index 26e0c56e1e4e..623a56097cc3 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/permission/ws/AddUserActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/permission/ws/AddUserActionTest.java @@ -169,7 +169,7 @@ public void fail_when_project_permission_without_project() throws Exception { @Test public void fail_when_component_is_not_a_project() throws Exception { expectedException.expect(BadRequestException.class); - insertComponent(newFileDto(newProjectDto("project-uuid"), "file-uuid")); + insertComponent(newFileDto(newProjectDto("project-uuid"), null, "file-uuid")); commit(); ws.newPostRequest(CONTROLLER, ACTION) diff --git a/server/sonar-server/src/test/java/org/sonar/server/permission/ws/RemoveGroupActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/permission/ws/RemoveGroupActionTest.java index 3df7726edc84..222a657629bd 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/permission/ws/RemoveGroupActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/permission/ws/RemoveGroupActionTest.java @@ -187,7 +187,7 @@ public void fail_when_project_project_permission_without_project() throws Except @Test public void fail_when_component_is_not_a_project() throws Exception { expectedException.expect(BadRequestException.class); - insertComponent(newFileDto(newProjectDto("project-uuid"), "file-uuid")); + insertComponent(newFileDto(newProjectDto("project-uuid"), null, "file-uuid")); insertGroup("sonar-administrators"); commit(); diff --git a/server/sonar-server/src/test/java/org/sonar/server/permission/ws/RemoveUserActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/permission/ws/RemoveUserActionTest.java index 1b6ceb69fece..bffdd63221e3 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/permission/ws/RemoveUserActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/permission/ws/RemoveUserActionTest.java @@ -163,7 +163,7 @@ public void fail_when_project_permission_without_permission() throws Exception { @Test public void fail_when_component_is_not_a_project() throws Exception { expectedException.expect(BadRequestException.class); - insertComponent(newFileDto(newProjectDto(), "file-uuid")); + insertComponent(newFileDto(newProjectDto(), null, "file-uuid")); ws.newPostRequest(CONTROLLER, ACTION) .setParam(PARAM_USER_LOGIN, "ray.bradbury") diff --git a/server/sonar-server/src/test/java/org/sonar/server/project/ws/BulkDeleteActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/project/ws/BulkDeleteActionTest.java index 9be6fa38854c..3e92da3db202 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/project/ws/BulkDeleteActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/project/ws/BulkDeleteActionTest.java @@ -181,7 +181,7 @@ public void fail_if_insufficient_privileges() throws Exception { @Test public void fail_if_scope_is_not_project() throws Exception { expectedException.expect(IllegalArgumentException.class); - dbClient.componentDao().insert(dbSession, ComponentTesting.newFileDto(ComponentTesting.newProjectDto(), "file-uuid")); + dbClient.componentDao().insert(dbSession, ComponentTesting.newFileDto(ComponentTesting.newProjectDto(), null, "file-uuid")); dbSession.commit(); ws.newPostRequest("api/projects", ACTION).setParam(PARAM_IDS, "file-uuid").execute(); diff --git a/server/sonar-server/src/test/java/org/sonar/server/project/ws/DeleteActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/project/ws/DeleteActionTest.java index 5f2485f3b2a9..4adf08593447 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/project/ws/DeleteActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/project/ws/DeleteActionTest.java @@ -200,7 +200,7 @@ public void fail_if_insufficient_privileges() throws Exception { @Test public void fail_if_scope_is_not_project() throws Exception { expectedException.expect(IllegalArgumentException.class); - dbClient.componentDao().insert(dbSession, ComponentTesting.newFileDto(ComponentTesting.newProjectDto(), "file-uuid")); + dbClient.componentDao().insert(dbSession, ComponentTesting.newFileDto(ComponentTesting.newProjectDto(), null, "file-uuid")); dbSession.commit(); newRequest().setParam(PARAM_ID, "file-uuid").execute(); diff --git a/server/sonar-server/src/test/java/org/sonar/server/source/ws/IndexActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/source/ws/IndexActionTest.java index e1bf541b4884..9ba327212b41 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/source/ws/IndexActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/source/ws/IndexActionTest.java @@ -64,7 +64,7 @@ public class IndexActionTest { WsTester tester; ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); @Before public void setUp() { diff --git a/server/sonar-server/src/test/java/org/sonar/server/source/ws/LinesActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/source/ws/LinesActionTest.java index cd5b0f96d051..d0e7fc2e4f37 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/source/ws/LinesActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/source/ws/LinesActionTest.java @@ -174,7 +174,7 @@ public void should_check_permission() throws Exception { private void newFile() { ComponentDto project = ComponentTesting.newProjectDto(PROJECT_UUID); - ComponentDto file = ComponentTesting.newFileDto(project, FILE_UUID).setKey(FILE_KEY); + ComponentDto file = ComponentTesting.newFileDto(project, null, FILE_UUID).setKey(FILE_KEY); componentDao.insert(dbTester.getSession(), project, file); dbTester.getSession().commit(); } diff --git a/server/sonar-server/src/test/java/org/sonar/server/source/ws/RawActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/source/ws/RawActionTest.java index 07712bf75d99..a8b7c0cbde45 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/source/ws/RawActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/source/ws/RawActionTest.java @@ -63,7 +63,7 @@ public class RawActionTest { WsTester tester; ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); @Before public void setUp() { diff --git a/server/sonar-server/src/test/java/org/sonar/server/source/ws/ScmActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/source/ws/ScmActionTest.java index a1fbb0f3b0b0..9d90512e3d8a 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/source/ws/ScmActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/source/ws/ScmActionTest.java @@ -182,7 +182,7 @@ public void fail_without_code_viewer_permission() throws Exception { private void initFile() { ComponentDto project = ComponentTesting.newProjectDto(PROJECT_UUID); - dbClient.componentDao().insert(dbTester.getSession(), project, ComponentTesting.newFileDto(project, FILE_UUID).setKey(FILE_KEY)); + dbClient.componentDao().insert(dbTester.getSession(), project, ComponentTesting.newFileDto(project, null, FILE_UUID).setKey(FILE_KEY)); dbTester.getSession().commit(); } diff --git a/server/sonar-server/src/test/java/org/sonar/server/source/ws/ShowActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/source/ws/ShowActionTest.java index 09664fd6f656..45482df88d4e 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/source/ws/ShowActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/source/ws/ShowActionTest.java @@ -65,7 +65,7 @@ public class ShowActionTest { ComponentDao componentDao; ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); @Before public void setUp() { diff --git a/server/sonar-server/src/test/java/org/sonar/server/test/ws/CoveredFilesActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/test/ws/CoveredFilesActionTest.java index e422141a9652..5f560053f1b0 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/test/ws/CoveredFilesActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/test/ws/CoveredFilesActionTest.java @@ -79,8 +79,8 @@ public void covered_files() { new CoveredFileDoc().setFileUuid(FILE_2_ID).setCoveredLines(Arrays.asList(1, 2, 3)))); when(dbClient.componentDao().selectByUuids(any(DbSession.class), anyList())).thenReturn( Arrays.asList( - newFileDto(newProjectDto(), FILE_1_ID).setKey("org.foo.Bar.java").setLongName("src/main/java/org/foo/Bar.java"), - newFileDto(newProjectDto(), FILE_2_ID).setKey("org.foo.File.java").setLongName("src/main/java/org/foo/File.java"))); + newFileDto(newProjectDto(), null, FILE_1_ID).setKey("org.foo.Bar.java").setLongName("src/main/java/org/foo/Bar.java"), + newFileDto(newProjectDto(), null, FILE_2_ID).setKey("org.foo.File.java").setLongName("src/main/java/org/foo/File.java"))); TestRequest request = ws.newRequest().setParam(TEST_ID, "test-uuid"); @@ -97,8 +97,8 @@ public void fail_when_test_uuid_is_unknown() { new CoveredFileDoc().setFileUuid(FILE_2_ID).setCoveredLines(Arrays.asList(1, 2, 3)))); when(dbClient.componentDao().selectByUuids(any(DbSession.class), anyList())).thenReturn( Arrays.asList( - newFileDto(newProjectDto(), FILE_1_ID).setKey("org.foo.Bar.java").setLongName("src/main/java/org/foo/Bar.java"), - newFileDto(newProjectDto(), FILE_2_ID).setKey("org.foo.File.java").setLongName("src/main/java/org/foo/File.java"))); + newFileDto(newProjectDto(), null, FILE_1_ID).setKey("org.foo.Bar.java").setLongName("src/main/java/org/foo/Bar.java"), + newFileDto(newProjectDto(), null, FILE_2_ID).setKey("org.foo.File.java").setLongName("src/main/java/org/foo/File.java"))); expectedException.expect(NotFoundException.class); expectedException.expectMessage("Test with id 'test-uuid' is not found"); diff --git a/server/sonar-server/src/test/java/org/sonar/server/ui/ws/ComponentNavigationActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/ui/ws/ComponentNavigationActionTest.java index ad7cb0c8ee8f..d9e0d745f66c 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/ui/ws/ComponentNavigationActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/ui/ws/ComponentNavigationActionTest.java @@ -319,7 +319,7 @@ public void bread_crumbs_on_several_levels() throws Exception { ComponentDto module = ComponentTesting.newModuleDto("bcde", project) .setKey("palap").setName("Palap"); ComponentDto directory = ComponentTesting.newDirectory(module, "src/main/xoo"); - ComponentDto file = ComponentTesting.newFileDto(directory, "cdef").setName("Source.xoo") + ComponentDto file = ComponentTesting.newFileDto(directory, directory, "cdef").setName("Source.xoo") .setKey("palap:src/main/xoo/Source.xoo") .setPath(directory.path()); dbClient.componentDao().insert(dbTester.getSession(), project, module, directory, file); diff --git a/server/sonar-server/src/test/java/org/sonar/server/user/ServerUserSessionTest.java b/server/sonar-server/src/test/java/org/sonar/server/user/ServerUserSessionTest.java index 59428c8fb8aa..2c99713694f5 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/user/ServerUserSessionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/user/ServerUserSessionTest.java @@ -19,14 +19,6 @@ */ package org.sonar.server.user; -import static org.assertj.core.api.Assertions.assertThat; -import static org.sonar.core.permission.GlobalPermissions.DASHBOARD_SHARING; -import static org.sonar.core.permission.GlobalPermissions.QUALITY_PROFILE_ADMIN; -import static org.sonar.core.permission.GlobalPermissions.SYSTEM_ADMIN; -import static org.sonar.db.user.UserTesting.newUserDto; -import static org.sonar.server.user.ServerUserSession.createForAnonymous; -import static org.sonar.server.user.ServerUserSession.createForUser; - import javax.annotation.Nullable; import org.junit.Before; import org.junit.Rule; @@ -46,6 +38,14 @@ import org.sonar.db.user.UserPermissionDto; import org.sonar.server.exceptions.ForbiddenException; +import static org.assertj.core.api.Assertions.assertThat; +import static org.sonar.core.permission.GlobalPermissions.DASHBOARD_SHARING; +import static org.sonar.core.permission.GlobalPermissions.QUALITY_PROFILE_ADMIN; +import static org.sonar.core.permission.GlobalPermissions.SYSTEM_ADMIN; +import static org.sonar.db.user.UserTesting.newUserDto; +import static org.sonar.server.user.ServerUserSession.createForAnonymous; +import static org.sonar.server.user.ServerUserSession.createForUser; + public class ServerUserSessionTest { static final String LOGIN = "marius"; @@ -71,7 +71,7 @@ public class ServerUserSessionTest { @Before public void setUp() throws Exception { project = componentDbTester.insertComponent(ComponentTesting.newProjectDto(PROJECT_UUID)); - file = componentDbTester.insertComponent(ComponentTesting.newFileDto(project, FILE_UUID).setKey(FILE_KEY)); + file = componentDbTester.insertComponent(ComponentTesting.newFileDto(project, null, FILE_UUID).setKey(FILE_KEY)); dbClient.userDao().insert(dbSession, userDto); dbSession.commit(); } @@ -162,7 +162,7 @@ public void check_component_key_permission_with_only_global_permission_ok() { @Test public void check_component_key_permission_ko() { ComponentDto project2 = componentDbTester.insertComponent(ComponentTesting.newProjectDto()); - ComponentDto file2 = componentDbTester.insertComponent(ComponentTesting.newFileDto(project2)); + ComponentDto file2 = componentDbTester.insertComponent(ComponentTesting.newFileDto(project2, null)); addProjectPermissions(project, UserRole.USER); UserSession session = newUserSession(userDto); @@ -190,7 +190,7 @@ public void check_component_uuid_permission_ko() { @Test public void check_component_key_permission_when_project_not_found() { ComponentDto project2 = componentDbTester.insertComponent(ComponentTesting.newProjectDto()); - ComponentDto file2 = componentDbTester.insertComponent(ComponentTesting.newFileDto(project2) + ComponentDto file2 = componentDbTester.insertComponent(ComponentTesting.newFileDto(project2, null) // Simulate file is linked to an invalid project .setProjectUuid("INVALID")); addProjectPermissions(project, UserRole.USER); diff --git a/server/sonar-server/src/test/java/org/sonar/server/view/index/ViewIndexerTest.java b/server/sonar-server/src/test/java/org/sonar/server/view/index/ViewIndexerTest.java index 9a0adb7f8391..11e30bec4766 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/view/index/ViewIndexerTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/view/index/ViewIndexerTest.java @@ -185,7 +185,7 @@ public void clear_views_lookup_cache_on_index_view_uuid() { private ComponentDto addProjectWithIssue(RuleDto rule) { ComponentDto project = ComponentTesting.newProjectDto(); - ComponentDto file = ComponentTesting.newFileDto(project); + ComponentDto file = ComponentTesting.newFileDto(project, null); dbClient.componentDao().insert(dbSession, project, file); dbClient.roleDao().insertGroupRole(dbSession, new GroupRoleDto().setRole(UserRole.USER).setGroupId(null).setResourceId(project.getId())); diff --git a/sonar-db/src/main/java/org/sonar/db/component/ComponentDto.java b/sonar-db/src/main/java/org/sonar/db/component/ComponentDto.java index b52484c66820..42354cd46acd 100644 --- a/sonar-db/src/main/java/org/sonar/db/component/ComponentDto.java +++ b/sonar-db/src/main/java/org/sonar/db/component/ComponentDto.java @@ -64,7 +64,7 @@ public class ComponentDto implements Component { * - on root module: UUID="1" UUID_PATH="." * - on module: UUID="2" UUID_PATH=".1." * - on directory: UUID="3" UUID_PATH=".1.2." - * - on file: UUID="4" UUID_PATH="1.2.3." + * - on file: UUID="4" UUID_PATH=".1.2.3." * - on view: UUID="5" UUID_PATH="." * - on sub-view: UUID="6" UUID_PATH=".5." * @@ -88,8 +88,8 @@ public class ComponentDto implements Component { * Badly named, it is not the root ! * - on root module: UUID="1" ROOT_UUID="1" * - on modules, whatever depth, value is the root module: UUID="2" ROOT_UUID="1" - * - on directory, value is the closed module: UUID="3" ROOT_UUID="2" - * - on file, value is the closed module: UUID="4" ROOT_UUID="2" + * - on directory, value is the closest module: UUID="3" ROOT_UUID="2" + * - on file, value is the closest module: UUID="4" ROOT_UUID="2" * - on view: UUID="5" ROOT_UUID="5" * - on sub-view: UUID="6" ROOT_UUID="5" * @since 6.0 diff --git a/sonar-db/src/test/java/org/sonar/db/component/ComponentDaoTest.java b/sonar-db/src/test/java/org/sonar/db/component/ComponentDaoTest.java index 70a3f9f2ddf2..394d2a241e90 100644 --- a/sonar-db/src/test/java/org/sonar/db/component/ComponentDaoTest.java +++ b/sonar-db/src/test/java/org/sonar/db/component/ComponentDaoTest.java @@ -846,11 +846,11 @@ public void selectChildren() { componentDb.insertProjectAndSnapshot(project); ComponentDto module = newModuleDto(MODULE_UUID, project); componentDb.insertComponent(module); - ComponentDto file1 = newFileDto(project, FILE_1_UUID).setKey("file-key-1").setName("File One"); + ComponentDto file1 = newFileDto(project, null, FILE_1_UUID).setKey("file-key-1").setName("File One"); componentDb.insertComponent(file1); - ComponentDto file2 = newFileDto(module, FILE_2_UUID).setKey("file-key-2").setName("File Two"); + ComponentDto file2 = newFileDto(module, null, FILE_2_UUID).setKey("file-key-2").setName("File Two"); componentDb.insertComponent(file2); - ComponentDto file3 = newFileDto(module, FILE_3_UUID).setKey("file-key-3").setName("File Three"); + ComponentDto file3 = newFileDto(module, null, FILE_3_UUID).setKey("file-key-3").setName("File Three"); componentDb.insertComponent(file3); db.commit(); componentDb.indexAllComponents(); @@ -923,7 +923,7 @@ public void selectChildren_with_pagination() { ComponentDto project = newProjectDto(PROJECT_UUID); componentDb.insertProjectAndSnapshot(project); for (int i = 1; i <= 9; i++) { - componentDb.insertComponent(newFileDto(project, "file-uuid-" + i)); + componentDb.insertComponent(newFileDto(project, null, "file-uuid-" + i)); } db.commit(); @@ -941,9 +941,9 @@ public void selectChildren_with_pagination() { public void selectChildren_ordered_by_file_path() { ComponentDto project = newProjectDto(PROJECT_UUID); componentDb.insertProjectAndSnapshot(project); - componentDb.insertComponent(newFileDto(project, "file-uuid-1").setName("file-name-1").setPath("3")); - componentDb.insertComponent(newFileDto(project, "file-uuid-2").setName("file-name-2").setPath("2")); - componentDb.insertComponent(newFileDto(project, "file-uuid-3").setName("file-name-3").setPath("1")); + componentDb.insertComponent(newFileDto(project, null, "file-uuid-1").setName("file-name-1").setPath("3")); + componentDb.insertComponent(newFileDto(project, null, "file-uuid-2").setName("file-name-2").setPath("2")); + componentDb.insertComponent(newFileDto(project, null, "file-uuid-3").setName("file-name-3").setPath("1")); db.commit(); componentDb.indexAllComponents(); @@ -1007,7 +1007,7 @@ public void selectParent() { componentDb.insertProjectAndSnapshot(project); ComponentDto module = newModuleDto(MODULE_UUID, project); componentDb.insertComponent(module); - ComponentDto file = newFileDto(module, FILE_1_UUID); + ComponentDto file = newFileDto(module, null, FILE_1_UUID); componentDb.insertComponent(file); db.commit(); @@ -1023,7 +1023,7 @@ public void selectAncestors() { componentDb.insertProjectAndSnapshot(project); ComponentDto module = newModuleDto(MODULE_UUID, project); componentDb.insertComponent(module); - ComponentDto file = newFileDto(module, FILE_1_UUID); + ComponentDto file = newFileDto(module, null, FILE_1_UUID); componentDb.insertComponent(file); db.commit(); @@ -1045,8 +1045,8 @@ public void selectDescendants() { ComponentDto project = newProjectDto(PROJECT_UUID); componentDb.insertProjectAndSnapshot(project); componentDb.insertComponent(newModuleDto("module-1-uuid", project)); - componentDb.insertComponent(newFileDto(project, "file-1-uuid")); - componentDb.insertComponent(newFileDto(project, "file-2-uuid")); + componentDb.insertComponent(newFileDto(project, null, "file-1-uuid")); + componentDb.insertComponent(newFileDto(project, null, "file-2-uuid")); db.commit(); componentDb.indexAllComponents(); @@ -1071,10 +1071,10 @@ public void selectDescendants_of_a_project_paginated_and_ordered() { ComponentDto project = newProjectDto(PROJECT_UUID).setKey("project-key"); componentDb.insertProjectAndSnapshot(project); componentDb.insertComponent(newModuleDto("module-1-uuid", project)); - componentDb.insertComponent(newFileDto(project, "file-uuid-1").setName("file-name-1")); - componentDb.insertComponent(newFileDto(project, "another-uuid")); + componentDb.insertComponent(newFileDto(project, null, "file-uuid-1").setName("file-name-1")); + componentDb.insertComponent(newFileDto(project, null, "another-uuid")); for (int i = 2; i <= 9; i++) { - componentDb.insertComponent(newFileDto(project, "file-uuid-" + i).setName("file-name-" + i)); + componentDb.insertComponent(newFileDto(project, null, "file-uuid-" + i).setName("file-name-" + i)); } db.commit(); componentDb.indexAllComponents(); diff --git a/sonar-db/src/test/java/org/sonar/db/component/ComponentKeyUpdaterDaoTest.java b/sonar-db/src/test/java/org/sonar/db/component/ComponentKeyUpdaterDaoTest.java index 518871a4d353..ede24716973f 100644 --- a/sonar-db/src/test/java/org/sonar/db/component/ComponentKeyUpdaterDaoTest.java +++ b/sonar-db/src/test/java/org/sonar/db/component/ComponentKeyUpdaterDaoTest.java @@ -110,7 +110,7 @@ public void shouldNotUpdateAllSubmodules() { public void fail_with_functional_exception_when_sub_component_key_is_longer_than_authorized() { ComponentDto project = newProjectDto("project-uuid").setKey("old-project-key"); componentDb.insertComponent(project); - componentDb.insertComponent(newFileDto(project).setKey("old-project-key:file")); + componentDb.insertComponent(newFileDto(project, null).setKey("old-project-key:file")); String newLongProjectKey = Strings.repeat("a", 400); thrown.expect(IllegalArgumentException.class); thrown.expectMessage("Component key length (405) is longer than the maximum authorized (400). '" + newLongProjectKey + ":file' was provided."); diff --git a/sonar-db/src/test/java/org/sonar/db/component/ComponentTesting.java b/sonar-db/src/test/java/org/sonar/db/component/ComponentTesting.java index 3d80147a92b0..2116b81f5d2f 100644 --- a/sonar-db/src/test/java/org/sonar/db/component/ComponentTesting.java +++ b/sonar-db/src/test/java/org/sonar/db/component/ComponentTesting.java @@ -20,6 +20,7 @@ package org.sonar.db.component; import java.util.Date; +import javax.annotation.Nullable; import org.sonar.api.resources.Qualifiers; import org.sonar.api.resources.Scopes; import org.sonar.core.util.Uuids; @@ -29,13 +30,13 @@ public class ComponentTesting { - public static ComponentDto newFileDto(ComponentDto subProjectOrProject) { - return newFileDto(subProjectOrProject, Uuids.create()); + public static ComponentDto newFileDto(ComponentDto subProjectOrProject, @Nullable ComponentDto directory) { + return newFileDto(subProjectOrProject, directory, Uuids.create()); } - public static ComponentDto newFileDto(ComponentDto module, String fileUuid) { + public static ComponentDto newFileDto(ComponentDto module, @Nullable ComponentDto directory, String fileUuid) { String path = "src/main/xoo/org/sonar/samples/File.xoo"; - return newChildComponent(fileUuid, module) + return newChildComponent(fileUuid, module, directory == null ? module : directory) .setKey("KEY_" + fileUuid) .setName("NAME_" + fileUuid) .setLongName(path) @@ -51,7 +52,7 @@ public static ComponentDto newDirectory(ComponentDto module, String path) { } public static ComponentDto newDirectory(ComponentDto module, String uuid, String path) { - return newChildComponent(uuid, module) + return newChildComponent(uuid, module, module) .setKey(!path.equals("/") ? module.getKey() + ":" + path : module.getKey() + ":/") .setName(path) .setLongName(path) @@ -61,7 +62,7 @@ public static ComponentDto newDirectory(ComponentDto module, String uuid, String } public static ComponentDto newSubView(ComponentDto viewOrSubView, String uuid, String key) { - return newChildComponent(uuid, viewOrSubView) + return newChildComponent(uuid, viewOrSubView, viewOrSubView) .setKey(key) .setName(key) .setLongName(key) @@ -70,7 +71,7 @@ public static ComponentDto newSubView(ComponentDto viewOrSubView, String uuid, S } public static ComponentDto newModuleDto(String uuid, ComponentDto parentModuleOrProject) { - return newChildComponent(uuid, parentModuleOrProject) + return newChildComponent(uuid, parentModuleOrProject, parentModuleOrProject) .setModuleUuidPath(parentModuleOrProject.moduleUuidPath() + uuid + UUID_PATH_SEPARATOR) .setKey("KEY_" + uuid) .setName("NAME_" + uuid) @@ -156,7 +157,7 @@ public static ComponentDto newView() { public static ComponentDto newProjectCopy(String uuid, ComponentDto project, ComponentDto view) { checkNotNull(project.getId(), "The project need to be persisted before creating this technical project."); - return newChildComponent(uuid, view) + return newChildComponent(uuid, view, view) .setUuid(uuid) .setKey(view.key() + project.key()) .setName(project.name()) @@ -170,7 +171,7 @@ public static ComponentDto newProjectCopy(String uuid, ComponentDto project, Com public static ComponentDto newDevProjectCopy(String uuid, ComponentDto project, ComponentDto developer) { checkNotNull(project.getId(), "The project need to be persisted before creating this technical project."); - return newChildComponent(uuid, developer) + return newChildComponent(uuid, developer, developer) .setUuid(uuid) .setKey(developer.key() + ":" + project.key()) .setName(project.name()) @@ -182,14 +183,14 @@ public static ComponentDto newDevProjectCopy(String uuid, ComponentDto project, .setLanguage(null); } - public static ComponentDto newChildComponent(String uuid, ComponentDto parent) { + public static ComponentDto newChildComponent(String uuid, ComponentDto moduleOrProject, ComponentDto parent) { return new ComponentDto() .setUuid(uuid) .setUuidPath(ComponentDto.formatUuidPathFromParent(parent)) - .setProjectUuid(parent.projectUuid()) - .setRootUuid(parent.uuid()) - .setModuleUuid(parent.uuid()) - .setModuleUuidPath(parent.moduleUuidPath()) + .setProjectUuid(moduleOrProject.projectUuid()) + .setRootUuid(moduleOrProject.uuid()) + .setModuleUuid(moduleOrProject.uuid()) + .setModuleUuidPath(moduleOrProject.moduleUuidPath()) .setCreatedAt(new Date()) .setEnabled(true); } diff --git a/sonar-db/src/test/java/org/sonar/db/issue/IssueDaoTest.java b/sonar-db/src/test/java/org/sonar/db/issue/IssueDaoTest.java index 9fe992fd17e0..82baa486078d 100644 --- a/sonar-db/src/test/java/org/sonar/db/issue/IssueDaoTest.java +++ b/sonar-db/src/test/java/org/sonar/db/issue/IssueDaoTest.java @@ -39,7 +39,7 @@ public class IssueDaoTest { public static final RuleDto RULE = RuleTesting.newXooX1(); public static final ComponentDto PROJECT = ComponentTesting.newProjectDto(); - public static final ComponentDto FILE = ComponentTesting.newFileDto(PROJECT); + public static final ComponentDto FILE = ComponentTesting.newFileDto(PROJECT, null); public static final String ISSUE_KEY1 = "I1"; public static final String ISSUE_KEY2 = "I2"; diff --git a/sonar-db/src/test/java/org/sonar/db/issue/IssueMapperTest.java b/sonar-db/src/test/java/org/sonar/db/issue/IssueMapperTest.java index 20aec627ab33..3b3e7b137667 100644 --- a/sonar-db/src/test/java/org/sonar/db/issue/IssueMapperTest.java +++ b/sonar-db/src/test/java/org/sonar/db/issue/IssueMapperTest.java @@ -48,9 +48,9 @@ public class IssueMapperTest { public void setUp() throws Exception { project = ComponentTesting.newProjectDto(); dbTester.getDbClient().componentDao().insert(dbSession, project); - file = ComponentTesting.newFileDto(project); + file = ComponentTesting.newFileDto(project, null); dbTester.getDbClient().componentDao().insert(dbSession, file); - file2 = ComponentTesting.newFileDto(project).setUuid("file2 uuid"); + file2 = ComponentTesting.newFileDto(project, null).setUuid("file2 uuid"); dbTester.getDbClient().componentDao().insert(dbSession, file2); rule = RuleTesting.newXooX1(); dbTester.getDbClient().ruleDao().insert(dbSession, rule);