From: Aron Roberts <aron@socrates.berkeley.edu>
Date: Thu, 2 Dec 2010 16:07:39 +0000 (+0000)
Subject: CSPACE-139: Temporarily commented out @Test annotations in BlobAuthRefsTest and Media... 
X-Git-Url: https://git.aero2k.de/?a=commitdiff_plain;h=221506a952be45f88e508164a789b3a8ceb9c347;p=tmp%2Fjakarta-migration.git

CSPACE-139: Temporarily commented out @Test annotations in BlobAuthRefsTest and MediaAuthRefsTest classes until payloads for these two record types are reflected in these tests, and it's determined that either or both have at least one authref field.
---

diff --git a/services/blob/client/src/test/java/org/collectionspace/services/client/test/BlobAuthRefsTest.java b/services/blob/client/src/test/java/org/collectionspace/services/client/test/BlobAuthRefsTest.java
index fc10b00d8..ffd99b31f 100644
--- a/services/blob/client/src/test/java/org/collectionspace/services/client/test/BlobAuthRefsTest.java
+++ b/services/blob/client/src/test/java/org/collectionspace/services/client/test/BlobAuthRefsTest.java
@@ -151,7 +151,13 @@ public class BlobAuthRefsTest extends BaseServiceTest {
         return extractId(res);
     }
 
-    @Test(dataProvider = "testName", dataProviderClass = AbstractServiceTestImpl.class, dependsOnMethods = {"createWithAuthRefs"})
+    // @Test annotation commented out by Aron 2010-12-02 until blob payload is set to the
+    // actual payload - it currently appears to be cloned from another record type - and
+    // it's determined that this payload has at least one authority field.
+    //
+    // If there are no authref fields in the payload, we might remove this test class outright.
+    //
+    // @Test(dataProvider = "testName", dataProviderClass = AbstractServiceTestImpl.class, dependsOnMethods = {"createWithAuthRefs"})
     public void readAndCheckAuthRefs(String testName) throws Exception {
         logger.debug(testBanner(testName, CLASS_NAME));
         testSetup(STATUS_OK, ServiceRequestType.READ);
diff --git a/services/media/client/src/test/java/org/collectionspace/services/client/test/MediaAuthRefsTest.java b/services/media/client/src/test/java/org/collectionspace/services/client/test/MediaAuthRefsTest.java
index 5be02297d..25b227519 100644
--- a/services/media/client/src/test/java/org/collectionspace/services/client/test/MediaAuthRefsTest.java
+++ b/services/media/client/src/test/java/org/collectionspace/services/client/test/MediaAuthRefsTest.java
@@ -151,7 +151,14 @@ public class MediaAuthRefsTest extends BaseServiceTest {
         return extractId(res);
     }
 
-    @Test(dataProvider = "testName", dataProviderClass = AbstractServiceTestImpl.class, dependsOnMethods = {"createWithAuthRefs"})
+    // @Test annotation commented out by Aron 2010-12-02 until media payload is set to the
+    // actual payload - it currently appears to be cloned from another record type - and
+    // it's determined that this payload has at least one authority field.
+    //
+    // When that happens, this test class will also need to be revised accordingly to
+    // reflect the actual names and number of authref fields in that payload.
+    //
+    // @Test(dataProvider = "testName", dataProviderClass = AbstractServiceTestImpl.class, dependsOnMethods = {"createWithAuthRefs"})
     public void readAndCheckAuthRefs(String testName) throws Exception {
         logger.debug(testBanner(testName, CLASS_NAME));
         testSetup(STATUS_OK, ServiceRequestType.READ);