diff options
author | Julien Lancelot <julien.lancelot@sonarsource.com> | 2019-02-01 10:42:59 +0100 |
---|---|---|
committer | sonartech <sonartech@sonarsource.com> | 2019-02-11 09:11:43 +0100 |
commit | ec6a2c374ee80c327a5e9d0c7e1cd20c23d18fc0 (patch) | |
tree | 5ef7254477ae02037aff92567420158650515946 /server | |
parent | cc7ee64201bf5dece53fdbcb3a986c4206e0bf78 (diff) | |
download | sonarqube-ec6a2c374ee80c327a5e9d0c7e1cd20c23d18fc0.tar.gz sonarqube-ec6a2c374ee80c327a5e9d0c7e1cd20c23d18fc0.zip |
SONAR-11650 Remove usage of FILE_SOURCES#DATA_TYPE
Diffstat (limited to 'server')
10 files changed, 7 insertions, 59 deletions
diff --git a/server/sonar-db-dao/src/main/java/org/sonar/db/source/FileSourceDao.java b/server/sonar-db-dao/src/main/java/org/sonar/db/source/FileSourceDao.java index 3a1567ccb3c..1085b2c59e9 100644 --- a/server/sonar-db-dao/src/main/java/org/sonar/db/source/FileSourceDao.java +++ b/server/sonar-db-dao/src/main/java/org/sonar/db/source/FileSourceDao.java @@ -59,7 +59,7 @@ public class FileSourceDao implements Dao { PreparedStatement pstmt = null; ResultSet rs = null; try { - pstmt = connection.prepareStatement("SELECT line_hashes FROM file_sources WHERE file_uuid=? AND data_type='SOURCE'"); + pstmt = connection.prepareStatement("SELECT line_hashes FROM file_sources WHERE file_uuid=?"); pstmt.setString(1, fileUuid); rs = pstmt.executeQuery(); if (rs.next()) { @@ -93,7 +93,7 @@ public class FileSourceDao implements Dao { ResultSet rs = null; Reader reader = null; try { - pstmt = connection.prepareStatement("SELECT line_hashes FROM file_sources WHERE file_uuid=? AND data_type='SOURCE'"); + pstmt = connection.prepareStatement("SELECT line_hashes FROM file_sources WHERE file_uuid=?"); pstmt.setString(1, fileUuid); rs = pstmt.executeQuery(); if (rs.next()) { diff --git a/server/sonar-db-dao/src/main/java/org/sonar/db/source/FileSourceMapper.java b/server/sonar-db-dao/src/main/java/org/sonar/db/source/FileSourceMapper.java index 0da4f1980d9..98de827933f 100644 --- a/server/sonar-db-dao/src/main/java/org/sonar/db/source/FileSourceMapper.java +++ b/server/sonar-db-dao/src/main/java/org/sonar/db/source/FileSourceMapper.java @@ -27,7 +27,7 @@ import org.apache.ibatis.session.ResultHandler; public interface FileSourceMapper { - List<FileSourceDto> selectHashesForProject(@Param("projectUuid") String projectUuid, @Param("dataType") String dataType); + List<FileSourceDto> selectHashesForProject(@Param("projectUuid") String projectUuid); @CheckForNull FileSourceDto selectByFileUuid(@Param("fileUuid") String fileUuid); diff --git a/server/sonar-db-dao/src/main/protobuf/db-file-sources.proto b/server/sonar-db-dao/src/main/protobuf/db-file-sources.proto index bad663a92d0..5c073555fe3 100644 --- a/server/sonar-db-dao/src/main/protobuf/db-file-sources.proto +++ b/server/sonar-db-dao/src/main/protobuf/db-file-sources.proto @@ -74,24 +74,3 @@ message Data { repeated Line lines = 1; } -message Test { - optional string uuid = 1; - optional string name = 2; - optional TestStatus status = 3; - optional int64 execution_time_ms = 4; - optional string stacktrace = 5; - optional string msg = 6; - repeated CoveredFile covered_file = 7; - - message CoveredFile { - optional string file_uuid = 1; - repeated int32 covered_line = 2 [packed = true]; - } - - enum TestStatus { - OK = 1; - FAILURE = 2; - ERROR = 3; - SKIPPED = 4; - } -} diff --git a/server/sonar-db-dao/src/main/resources/org/sonar/db/component/ComponentMapper.xml b/server/sonar-db-dao/src/main/resources/org/sonar/db/component/ComponentMapper.xml index 40ab818d93a..fc0a1a5cf37 100644 --- a/server/sonar-db-dao/src/main/resources/org/sonar/db/component/ComponentMapper.xml +++ b/server/sonar-db-dao/src/main/resources/org/sonar/db/component/ComponentMapper.xml @@ -226,7 +226,6 @@ and p.scope='FIL' INNER JOIN file_sources fs ON fs.file_uuid=p.uuid - and fs.data_type='SOURCE' where root.uuid=#{projectUuid,jdbcType=VARCHAR} </select> @@ -241,7 +240,6 @@ FROM projects p INNER JOIN file_sources fs ON fs.file_uuid=p.uuid - and fs.data_type='SOURCE' <include refid="modulesTreeQuery"/> </select> @@ -590,7 +588,6 @@ from projects p inner join file_sources fs on fs.file_uuid = p.uuid - and fs.data_type = 'SOURCE' where p.project_uuid = #{projectUuid,jdbcType=VARCHAR} and p.enabled = ${_true} diff --git a/server/sonar-db-dao/src/main/resources/org/sonar/db/source/FileSourceMapper.xml b/server/sonar-db-dao/src/main/resources/org/sonar/db/source/FileSourceMapper.xml index b45f1f1f669..7e017a68b8f 100644 --- a/server/sonar-db-dao/src/main/resources/org/sonar/db/source/FileSourceMapper.xml +++ b/server/sonar-db-dao/src/main/resources/org/sonar/db/source/FileSourceMapper.xml @@ -22,7 +22,6 @@ file_sources where file_uuid = #{fileUuid,jdbcType=VARCHAR} - and data_type = 'SOURCE' </select> <select id="selectHashesForProject" parameterType="map" resultType="org.sonar.db.source.FileSourceDto"> @@ -33,11 +32,10 @@ src_hash as srcHash, revision, updated_at as updatedAt - from + from file_sources where project_uuid = #{projectUuid,jdbcType=VARCHAR} - and data_type = 'SOURCE' </select> <select id="scrollLineHashes" parameterType="map" resultType="org.sonar.db.source.LineHashesWithUuidDto" fetchSize="${_scrollFetchSize}" resultSetType="FORWARD_ONLY"> @@ -48,7 +46,6 @@ from projects p inner join file_sources fs on fs.file_uuid = p.uuid - and fs.data_type = 'SOURCE' where p.uuid in <foreach collection="fileUuids" item="fileUuid" open="(" close=")" separator=","> @@ -64,7 +61,6 @@ file_sources WHERE file_uuid = #{fileUuid,jdbcType=VARCHAR} - and data_type = 'SOURCE' </select> <insert id="insert" parameterType="org.sonar.db.source.FileSourceDto" useGeneratedKeys="false"> @@ -80,7 +76,6 @@ line_count, data_hash, src_hash, - data_type, revision ) values @@ -95,7 +90,6 @@ #{lineCount,jdbcType=INTEGER}, #{dataHash,jdbcType=VARCHAR}, #{srcHash,jdbcType=VARCHAR}, - 'SOURCE', #{revision,jdbcType=VARCHAR} ) </insert> diff --git a/server/sonar-db-dao/src/test/java/org/sonar/db/source/FileSourceDaoTest.java b/server/sonar-db-dao/src/test/java/org/sonar/db/source/FileSourceDaoTest.java index 0524b9e7234..d62bb2757c9 100644 --- a/server/sonar-db-dao/src/test/java/org/sonar/db/source/FileSourceDaoTest.java +++ b/server/sonar-db-dao/src/test/java/org/sonar/db/source/FileSourceDaoTest.java @@ -97,16 +97,6 @@ public class FileSourceDaoTest { } @Test - public void no_line_hashes_when_only_test_data() { - dbTester.prepareDbUnit(getClass(), "no_line_hashes_when_only_test_data.xml"); - - ReaderToStringConsumer fn = new ReaderToStringConsumer(); - underTest.readLineHashesStream(dbSession, "FILE1_UUID", fn); - - assertThat(fn.result).isNull(); - } - - @Test public void insert() { FileSourceDto expected = new FileSourceDto() .setProjectUuid("PRJ_UUID") diff --git a/server/sonar-db-dao/src/test/resources/org/sonar/db/purge/PurgeDaoTest/shouldDeleteProject.xml b/server/sonar-db-dao/src/test/resources/org/sonar/db/purge/PurgeDaoTest/shouldDeleteProject.xml index f0e0ed49218..11c383b63fd 100644 --- a/server/sonar-db-dao/src/test/resources/org/sonar/db/purge/PurgeDaoTest/shouldDeleteProject.xml +++ b/server/sonar-db-dao/src/test/resources/org/sonar/db/purge/PurgeDaoTest/shouldDeleteProject.xml @@ -224,6 +224,5 @@ line_count="0" data_hash="321654987" created_at="123456789" - updated_at="123456789" - data_type="SOURCE"/> + updated_at="123456789"/> </dataset> diff --git a/server/sonar-db-dao/src/test/resources/org/sonar/db/source/FileSourceDaoTest/no_line_hashes_when_only_test_data.xml b/server/sonar-db-dao/src/test/resources/org/sonar/db/source/FileSourceDaoTest/no_line_hashes_when_only_test_data.xml deleted file mode 100644 index 01bc65ac204..00000000000 --- a/server/sonar-db-dao/src/test/resources/org/sonar/db/source/FileSourceDaoTest/no_line_hashes_when_only_test_data.xml +++ /dev/null @@ -1,11 +0,0 @@ -<dataset> - - <file_sources id="101" project_uuid="PRJ_UUID" file_uuid="FILE1_UUID" - binary_data="abcde" data_hash="[null]" - line_hashes="[null]" - line_count="0" - src_hash="[null]" - line_hashes_version="[null]" - created_at="1500000000000" updated_at="1500000000000" data_type="TEST" /> - -</dataset> diff --git a/server/sonar-db-dao/src/test/resources/org/sonar/db/source/FileSourceDaoTest/shared.xml b/server/sonar-db-dao/src/test/resources/org/sonar/db/source/FileSourceDaoTest/shared.xml index 77d242162c6..53556dae846 100644 --- a/server/sonar-db-dao/src/test/resources/org/sonar/db/source/FileSourceDaoTest/shared.xml +++ b/server/sonar-db-dao/src/test/resources/org/sonar/db/source/FileSourceDaoTest/shared.xml @@ -5,7 +5,7 @@ line_hashes="ABC\nDEF\nGHI" line_count="3" src_hash="FILE_HASH" revision="123456789" - created_at="1500000000000" updated_at="1500000000000" data_type="SOURCE" + created_at="1500000000000" updated_at="1500000000000" line_hashes_version="[null]"/> </dataset> diff --git a/server/sonar-db-dao/src/test/resources/org/sonar/db/source/FileSourceDaoTest/update-result.xml b/server/sonar-db-dao/src/test/resources/org/sonar/db/source/FileSourceDaoTest/update-result.xml index e51106a4630..139656474e2 100644 --- a/server/sonar-db-dao/src/test/resources/org/sonar/db/source/FileSourceDaoTest/update-result.xml +++ b/server/sonar-db-dao/src/test/resources/org/sonar/db/source/FileSourceDaoTest/update-result.xml @@ -6,7 +6,7 @@ line_hashes="NEW_LINE_HASHES" line_count="1" src_hash="NEW_FILE_HASH" revision="987654321" - created_at="1500000000000" updated_at="1500000000002" data_type="SOURCE" + created_at="1500000000000" updated_at="1500000000002" line_hashes_version="1" /> |