make check messages more detailed

This commit is contained in:
danikov 2012-02-16 17:30:17 +00:00
parent f7c35b7e61
commit 639b4202fb
1 changed files with 13 additions and 4 deletions

View File

@ -18,7 +18,10 @@
*/ */
package org.jclouds.vcloud.director.v1_5.domain; package org.jclouds.vcloud.director.v1_5.domain;
import static org.testng.Assert.*; import static org.testng.Assert.assertEquals;
import static org.testng.Assert.assertNotNull;
import static org.testng.Assert.assertTrue;
import static org.testng.Assert.fail;
import java.net.URI; import java.net.URI;
import java.util.Set; import java.util.Set;
@ -87,7 +90,9 @@ public class Checks {
} }
public static void checkType(String type) { public static void checkType(String type) {
assertTrue(VCloudDirectorMediaType.ALL.contains(type), "The Type must be a valid media type"); assertTrue(VCloudDirectorMediaType.ALL.contains(type),
String.format("The Type (%s) must be a valid media type - %s", type,
Iterables.toString(VCloudDirectorMediaType.ALL)));
} }
// NOTE this does not currently check anything // NOTE this does not currently check anything
@ -104,7 +109,9 @@ public class Checks {
public static void checkLink(Link link) { public static void checkLink(Link link) {
// Check required fields // Check required fields
assertNotNull(link.getRel(), "The Rel attribute of a Link must be set"); assertNotNull(link.getRel(), "The Rel attribute of a Link must be set");
assertTrue(Link.Rel.ALL.contains(link.getRel()), "The Rel attribute of a Link must be one of the allowed list"); assertTrue(Link.Rel.ALL.contains(link.getRel()),
String.format("The Rel attribute (%s) of a Link must be one of the allowed list - %s",
link.getRel(), Iterables.toString(Link.Rel.ALL)));
// Check parent type // Check parent type
checkReferenceType(link); checkReferenceType(link);
@ -113,7 +120,9 @@ public class Checks {
public static void checkTask(Task task) { public static void checkTask(Task task) {
// Check required fields // Check required fields
assertNotNull(task.getStatus(), "The Status attribute of a Task must be set"); assertNotNull(task.getStatus(), "The Status attribute of a Task must be set");
assertTrue(Task.Status.ALL.contains(task.getStatus()), "The Status of a Task must be one of the allowed list"); assertTrue(Task.Status.ALL.contains(task.getStatus().toString()),
String.format("The Status of a Task (%s) must be one of the allowed list - %s",
task.getStatus().toString(), Iterables.toString(Task.Status.ALL)));
// Check optional fields // Check optional fields
// NOTE operation cannot be checked // NOTE operation cannot be checked