TransactionParticipantServiceImpl.findGroupIdByName(): add unit tests.

Fix #921

Author: php-coder

src/main/java/ru/mystamps/web/service/TransactionParticipantServiceImpl.java

@@ -96,7 +96,6 @@ public List<EntityWithIdDto> findAllGroups() {
 		return transactionParticipantDao.findAllGroups();
 	}
 
-	// @todo #857 TransactionParticipantServiceImpl.findGroupIdByName(): add unit tests
 	// @todo #857 TransactionParticipantServiceImpl.findGroupIdByName(): move to a separate service
 	@Override
 	@Transactional(readOnly = true)

src/test/groovy/ru/mystamps/web/service/TransactionParticipantServiceImplTest.groovy

@@ -197,4 +197,28 @@ class TransactionParticipantServiceImplTest extends Specification {
 			result == expectedResult
 	}
 
+	//
+	// Tests for findGroupIdByName()
+	//
+	
+	def 'findGroupIdByName() should throw exception when name is null, empty or blank'() {
+		when:
+			service.findGroupIdByName(nullOrBlank())
+		then:
+			IllegalArgumentException ex = thrown()
+			ex.message == 'Group name must be non-blank'
+	}
+	
+	def 'findGroupIdByName() should invoke dao and return its result'() {
+		given:
+			String expectedName = 'Some Group'
+			Integer expectedResult = Random.id()
+		when:
+			Integer result = service.findGroupIdByName(expectedName)
+		then:
+			1 * transactionParticipantDao.findGroupIdByName(expectedName) >> expectedResult
+		and:
+			result == expectedResult
+	}
+	
 }