Author: [email protected]
Date: Fri Jan 27 14:45:14 2012
New Revision: 2017
Log:
AMDATU-508: Made API a bit more strict to ensure the correct methods are used.
Ensure that this method only gets used by other APIs, instead of directly in
tests.
Modified:
trunk/amdatu-core/itest/base/src/main/java/org/amdatu/core/itest/base/Fixture.java
Modified:
trunk/amdatu-core/itest/base/src/main/java/org/amdatu/core/itest/base/Fixture.java
==============================================================================
---
trunk/amdatu-core/itest/base/src/main/java/org/amdatu/core/itest/base/Fixture.java
(original)
+++
trunk/amdatu-core/itest/base/src/main/java/org/amdatu/core/itest/base/Fixture.java
Fri Jan 27 14:45:14 2012
@@ -42,6 +42,10 @@
* @return the provisioning option, never <code>null</code>.
*/
public static Option provision(ProvisionedBundle... includes) {
+ if (includes == null || includes.length == 0) {
+ throw new IllegalArgumentException("Fixture#provision() should have
at least one bundle to provision. Incorrect API-call?");
+ }
+
List<ProvisionOption<?>> provisionOptions = new
ArrayList<ProvisionOption<?>>();
for (ProvisionedBundle b : includes) {
provisionOptions.add(b.getProvisionOption());
@@ -61,8 +65,7 @@
*/
public static void configure(TestContext testContext, ProvisionedConfig...
includes) throws IOException {
if (includes == null || includes.length == 0) {
- // Nothing to do...
- return;
+ throw new IllegalArgumentException("Fixture#configure() should
have at least one configuration to provision. Incorrect API-call?");
}
try {
for (ProvisionedConfig c : includes) {
_______________________________________________
Amdatu-commits mailing list
[email protected]
http://lists.amdatu.org/mailman/listinfo/amdatu-commits
