SONAR-7675 InternalPropertiesImpl must commit

diff --git a/server/sonar-server/src/main/java/org/sonar/server/property/InternalPropertiesImpl.java b/server/sonar-server/src/main/java/org/sonar/server/property/InternalPropertiesImpl.java
index f99002d9dee20b3bafc28397e0da73989147c269..3881c0949dfea5bbbaf191d334af027785a1fa0d 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/property/InternalPropertiesImpl.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/property/InternalPropertiesImpl.java
@@ -55,6 +55,7 @@ public class InternalPropertiesImpl implements InternalProperties {
       } else {
         dbClient.internalPropertiesDao().save(dbSession, propertyKey, value);
       }
+      dbSession.commit();
     }
   }
 

diff --git a/server/sonar-server/src/test/java/org/sonar/server/property/InternalPropertiesImplTest.java b/server/sonar-server/src/test/java/org/sonar/server/property/InternalPropertiesImplTest.java
index 20c371ba256602fd5e119ee9ad39c5dd61fdafc4..7abec84773eaf9404f4bedc3a4a011d38d5d6ec3 100644 (file)
--- a/server/sonar-server/src/test/java/org/sonar/server/property/InternalPropertiesImplTest.java
+++ b/server/sonar-server/src/test/java/org/sonar/server/property/InternalPropertiesImplTest.java
@@ -94,6 +94,7 @@ public class InternalPropertiesImplTest {
     underTest.write(SOME_KEY, null);
 
     verify(internalPropertiesDao).saveAsEmpty(dbSession, SOME_KEY);
+    verify(dbSession).commit();
   }
 
   @Test
@@ -101,6 +102,7 @@ public class InternalPropertiesImplTest {
     underTest.write(SOME_KEY, EMPTY_STRING);
 
     verify(internalPropertiesDao).saveAsEmpty(dbSession, SOME_KEY);
+    verify(dbSession).commit();
   }
 
   @Test
@@ -108,6 +110,7 @@ public class InternalPropertiesImplTest {
     underTest.write(SOME_KEY, SOME_VALUE);
 
     verify(internalPropertiesDao).save(dbSession, SOME_KEY, SOME_VALUE);
+    verify(dbSession).commit();
   }
 
   private void expectKeyNullOrEmptyIAE() {