From 02b63129b7d9dcd0c6a6bda8fa2a9b269459bb03 Mon Sep 17 00:00:00 2001
From: Sean Flanigan <sflaniga@redhat.com>
Date: Fri, 12 Jun 2015 12:41:24 +1000
Subject: [PATCH] Ignore slow tests

---
 .../src/test/java/org/zanata/SlowTest.java    |  2 ++
 .../rest/service/ResourceUtilsJpaTest.java    |  2 +-
 .../CopyTransServiceImplPerformanceTest.java  |  3 +--
 .../impl/CopyTransServiceImplTest.java        |  1 +
 .../service/impl/DocumentServiceImplTest.java | 20 +++++++++----------
 5 files changed, 15 insertions(+), 13 deletions(-)

diff --git a/zanata-war/src/test/java/org/zanata/SlowTest.java b/zanata-war/src/test/java/org/zanata/SlowTest.java
index 19dca87c52..21e325965b 100644
--- a/zanata-war/src/test/java/org/zanata/SlowTest.java
+++ b/zanata-war/src/test/java/org/zanata/SlowTest.java
@@ -26,6 +26,8 @@
 import java.lang.annotation.Target;
 
 /**
+ * (Currently not handled by test framework: please use
+ * <code>@Ignore("slow test")</code> as well.)
  * Marker annotation to identify tests that might be potentially slow to run.
  *
  * @author Carlos Munoz <a
diff --git a/zanata-war/src/test/java/org/zanata/rest/service/ResourceUtilsJpaTest.java b/zanata-war/src/test/java/org/zanata/rest/service/ResourceUtilsJpaTest.java
index 551ff0537d..7b9e915c73 100644
--- a/zanata-war/src/test/java/org/zanata/rest/service/ResourceUtilsJpaTest.java
+++ b/zanata-war/src/test/java/org/zanata/rest/service/ResourceUtilsJpaTest.java
@@ -69,7 +69,7 @@ public void transferFromResourceMetadata() {
         // TODO check the results in 'to'
     }
 
-    @Ignore
+    @Ignore("slow test")
     // This should be executed manually in IDE
     @Test
     @SlowTest
diff --git a/zanata-war/src/test/java/org/zanata/service/impl/CopyTransServiceImplPerformanceTest.java b/zanata-war/src/test/java/org/zanata/service/impl/CopyTransServiceImplPerformanceTest.java
index e3eece8ef4..057ab2ee46 100644
--- a/zanata-war/src/test/java/org/zanata/service/impl/CopyTransServiceImplPerformanceTest.java
+++ b/zanata-war/src/test/java/org/zanata/service/impl/CopyTransServiceImplPerformanceTest.java
@@ -42,7 +42,6 @@
 import org.junit.Test;
 import org.junit.rules.ExternalResource;
 import org.junit.rules.TestRule;
-import org.junit.runner.RunWith;
 import org.zanata.PerformanceProfiling;
 import org.zanata.SlowTest;
 import org.zanata.common.ContentState;
@@ -311,8 +310,8 @@ private static EntityMakerBuilder getEntityMakerBuilder() {
                         FixedValueMaker.EMPTY_STRING_MAKER);
     }
 
+    @Ignore("slow test")
     @Test
-    @Ignore
     @SlowTest
     @PerformanceProfiling
     public void testCopyTransForDocument() throws Exception {
diff --git a/zanata-war/src/test/java/org/zanata/service/impl/CopyTransServiceImplTest.java b/zanata-war/src/test/java/org/zanata/service/impl/CopyTransServiceImplTest.java
index 5624323d7a..21786e27d4 100644
--- a/zanata-war/src/test/java/org/zanata/service/impl/CopyTransServiceImplTest.java
+++ b/zanata-war/src/test/java/org/zanata/service/impl/CopyTransServiceImplTest.java
@@ -141,6 +141,7 @@ public static Object[][] copyTransParams() {
         return val;
     }
 
+    @Ignore("slow test")
     @Test
     @UseDataProvider("copyTransParams")
     @SlowTest
diff --git a/zanata-war/src/test/java/org/zanata/service/impl/DocumentServiceImplTest.java b/zanata-war/src/test/java/org/zanata/service/impl/DocumentServiceImplTest.java
index 855e06bd31..a4c5854d20 100644
--- a/zanata-war/src/test/java/org/zanata/service/impl/DocumentServiceImplTest.java
+++ b/zanata-war/src/test/java/org/zanata/service/impl/DocumentServiceImplTest.java
@@ -21,8 +21,11 @@
 
 package org.zanata.service.impl;
 
+import static org.mockito.Matchers.any;
 import static org.mockito.Matchers.anyInt;
 import static org.mockito.Matchers.anyString;
+import static org.mockito.Mockito.doNothing;
+import static org.mockito.Mockito.doReturn;
 import static org.mockito.Mockito.never;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
@@ -31,6 +34,7 @@
 
 import org.junit.Before;
 import org.junit.Test;
+import org.mockito.Matchers;
 import org.mockito.Mock;
 import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
@@ -70,6 +74,7 @@ public class DocumentServiceImplTest {
     private Messages msgs;
 
     private DocumentServiceImpl documentService;
+    private DocumentServiceImpl spyService;
 
     private Long docId = 1L, versionId = 1L, tfId = 1L;
     private LocaleId localeId = LocaleId.DE;
@@ -87,6 +92,11 @@ public void setup() {
         documentService = new DocumentServiceImpl();
         documentService.init(projectIterationDAO, documentDAO,
             translationStateCacheImpl, msgs);
+        // TODO spy should only be needed for legacy code
+        spyService = Mockito.spy(documentService);
+        // avoid triggering HTTP requests in a unit test:
+        doNothing().when(spyService).publishDocumentMilestoneEvent(
+                any(WebHook.class), any(DocumentMilestoneEvent.class));
 
         HProjectIteration version = Mockito.mock(HProjectIteration.class);
         HProject project = Mockito.mock(HProject.class);
@@ -107,8 +117,6 @@ public void setup() {
 
     @Test
     public void documentMilestoneEventTranslatedTest() {
-        DocumentServiceImpl spyService = Mockito.spy(documentService);
-
         WordStatistic stats = new WordStatistic(0, 0, 0, 10, 0);
         when(translationStateCacheImpl.getDocumentStatistics(docId, localeId))
             .thenReturn(stats);
@@ -129,8 +137,6 @@ public void documentMilestoneEventTranslatedTest() {
 
     @Test
     public void documentMilestoneEventTranslatedNot100Test() {
-        DocumentServiceImpl spyService = Mockito.spy(documentService);
-
         WordStatistic stats = new WordStatistic(0, 1, 0, 9, 0);
         when(translationStateCacheImpl.getDocumentStatistics(docId, localeId))
             .thenReturn(stats);
@@ -151,8 +157,6 @@ public void documentMilestoneEventTranslatedNot100Test() {
 
     @Test
     public void documentMilestoneEventApprovedTest() {
-        DocumentServiceImpl spyService = Mockito.spy(documentService);
-
         WordStatistic stats = new WordStatistic(10, 0, 0, 0, 0);
         when(translationStateCacheImpl.getDocumentStatistics(docId, localeId))
             .thenReturn(stats);
@@ -172,8 +176,6 @@ public void documentMilestoneEventApprovedTest() {
 
     @Test
     public void documentMilestoneEventApprovedNot100Test() {
-        DocumentServiceImpl spyService = Mockito.spy(documentService);
-
         WordStatistic stats = new WordStatistic(9, 0, 0, 1, 0);
         when(translationStateCacheImpl.getDocumentStatistics(docId, localeId))
             .thenReturn(stats);
@@ -194,7 +196,6 @@ public void documentMilestoneEventApprovedNot100Test() {
 
     @Test
     public void documentMilestoneEventSameStateTest1() {
-        DocumentServiceImpl spyService = Mockito.spy(documentService);
         WordStatistic stats = new WordStatistic(10, 0, 0, 0, 0);
         when(translationStateCacheImpl.getDocumentStatistics(docId, localeId))
             .thenReturn(stats);
@@ -215,7 +216,6 @@ public void documentMilestoneEventSameStateTest1() {
 
     @Test
     public void documentMilestoneEventSameStateTest2() {
-        DocumentServiceImpl spyService = Mockito.spy(documentService);
         WordStatistic stats = new WordStatistic(0, 0, 0, 10, 0);
 
         when(translationStateCacheImpl.getDocumentStatistics(docId, localeId))