aboutsummaryrefslogtreecommitdiffstats
path: root/server
diff options
context:
space:
mode:
authorJulien Lancelot <julien.lancelot@sonarsource.com>2019-02-01 10:42:59 +0100
committersonartech <sonartech@sonarsource.com>2019-02-11 09:11:43 +0100
commitec6a2c374ee80c327a5e9d0c7e1cd20c23d18fc0 (patch)
tree5ef7254477ae02037aff92567420158650515946 /server
parentcc7ee64201bf5dece53fdbcb3a986c4206e0bf78 (diff)
downloadsonarqube-ec6a2c374ee80c327a5e9d0c7e1cd20c23d18fc0.tar.gz
sonarqube-ec6a2c374ee80c327a5e9d0c7e1cd20c23d18fc0.zip
SONAR-11650 Remove usage of FILE_SOURCES#DATA_TYPE
Diffstat (limited to 'server')
-rw-r--r--server/sonar-db-dao/src/main/java/org/sonar/db/source/FileSourceDao.java4
-rw-r--r--server/sonar-db-dao/src/main/java/org/sonar/db/source/FileSourceMapper.java2
-rw-r--r--server/sonar-db-dao/src/main/protobuf/db-file-sources.proto21
-rw-r--r--server/sonar-db-dao/src/main/resources/org/sonar/db/component/ComponentMapper.xml3
-rw-r--r--server/sonar-db-dao/src/main/resources/org/sonar/db/source/FileSourceMapper.xml8
-rw-r--r--server/sonar-db-dao/src/test/java/org/sonar/db/source/FileSourceDaoTest.java10
-rw-r--r--server/sonar-db-dao/src/test/resources/org/sonar/db/purge/PurgeDaoTest/shouldDeleteProject.xml3
-rw-r--r--server/sonar-db-dao/src/test/resources/org/sonar/db/source/FileSourceDaoTest/no_line_hashes_when_only_test_data.xml11
-rw-r--r--server/sonar-db-dao/src/test/resources/org/sonar/db/source/FileSourceDaoTest/shared.xml2
-rw-r--r--server/sonar-db-dao/src/test/resources/org/sonar/db/source/FileSourceDaoTest/update-result.xml2
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" />