Merge branch 'xen_tests_fix'

This closes #293
This commit is contained in:
wilderrodrigues 2015-05-23 17:17:22 +02:00
commit d7854791b2
69 changed files with 74 additions and 73 deletions

View File

@ -98,7 +98,7 @@ import com.cloud.agent.resource.virtualnetwork.VirtualRoutingResource;
import com.cloud.exception.InternalErrorException;
import com.cloud.host.Host.Type;
import com.cloud.hypervisor.Hypervisor.HypervisorType;
import com.cloud.hypervisor.xenserver.resource.wrapper.CitrixRequestWrapper;
import com.cloud.hypervisor.xenserver.resource.wrapper.citrix.CitrixRequestWrapper;
import com.cloud.network.Networks;
import com.cloud.network.Networks.BroadcastDomainType;
import com.cloud.network.Networks.TrafficType;
@ -4940,4 +4940,4 @@ public abstract class CitrixResourceBase implements ServerResource, HypervisorRe
}
}
}
}
}

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.Set;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.Set;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import com.cloud.agent.api.Answer;
import com.cloud.agent.api.Command;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import com.cloud.agent.api.Answer;
import com.cloud.agent.api.CheckHealthAnswer;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.List;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import com.cloud.agent.api.Answer;
import com.cloud.agent.api.CheckOnHostAnswer;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.HashMap;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.io.BufferedReader;
import java.io.IOException;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.HashMap;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.LinkedHashSet;
import java.util.List;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.ArrayList;
import java.util.List;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.Set;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.Set;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import com.cloud.agent.api.Answer;
import com.cloud.agent.api.GetVmDiskStatsAnswer;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.ArrayList;
import java.util.HashMap;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.Set;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.Iterator;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.Set;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import com.cloud.agent.api.Answer;
import com.cloud.agent.api.ModifySshKeysCommand;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.HashMap;
import java.util.Map;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import com.cloud.agent.api.Answer;
import com.cloud.agent.api.routing.NetworkElementCommand;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import com.cloud.agent.api.Answer;
import com.cloud.agent.api.NetworkRulesSystemVmCommand;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import com.cloud.agent.api.Answer;
import com.cloud.agent.api.NetworkRulesVmSecondaryIpCommand;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;
import org.apache.xmlrpc.XmlRpcException;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;
import org.apache.xmlrpc.XmlRpcException;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;
import org.apache.xmlrpc.XmlRpcException;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;
import org.apache.xmlrpc.XmlRpcException;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import com.cloud.agent.api.Answer;
import com.cloud.agent.api.PerformanceMonitorAnswer;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import com.cloud.agent.api.Answer;
import com.cloud.agent.api.PingTestCommand;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.Set;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.net.URI;
import java.util.HashMap;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;
import org.apache.xmlrpc.XmlRpcException;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.Set;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.Set;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import com.cloud.agent.api.Answer;
import com.cloud.agent.api.RebootCommand;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.Hashtable;
import java.util.Set;
@ -44,7 +44,7 @@ public class CitrixRequestWrapper extends RequestWrapper {
instance = new CitrixRequestWrapper();
}
Reflections baseWrappers = new Reflections("com.cloud.hypervisor.xenserver.resource.wrapper");
Reflections baseWrappers = new Reflections("com.cloud.hypervisor.xenserver.resource.wrapper.citrix");
@SuppressWarnings("rawtypes")
Set<Class<? extends CommandWrapper>> baseSet = baseWrappers.getSubTypesOf(CommandWrapper.class);
@ -121,4 +121,4 @@ public class CitrixRequestWrapper extends RequestWrapper {
return commandWrapper.execute(command, serverResource);
}
}
}

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.HashMap;
import java.util.List;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.Iterator;
import java.util.Set;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.log4j.Logger;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.Map;
import java.util.Set;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.HashMap;
import java.util.List;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.ArrayList;
import java.util.HashMap;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import org.apache.cloudstack.storage.command.StorageSubSystemCommand;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.util.Set;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import com.cloud.agent.api.Answer;
import com.cloud.agent.api.UpdateHostPasswordCommand;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import java.net.URI;

View File

@ -17,7 +17,7 @@
// under the License.
//
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import com.cloud.agent.api.Answer;
import com.cloud.agent.api.Command;

View File

@ -14,7 +14,7 @@
// KIND, either express or implied. See the License for the
// specific language governing permissions and limitations
// under the License.
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
@ -138,6 +138,7 @@ import com.xensource.xenapi.Pool;
import com.xensource.xenapi.Types.BadServerResponse;
import com.xensource.xenapi.Types.XenAPIException;
@RunWith(MockitoJUnitRunner.class)
public class CitrixRequestWrapperTest {
@ -1838,4 +1839,4 @@ class NotAValidCommand extends Command {
return false;
}
}
}

View File

@ -14,7 +14,7 @@
// KIND, either express or implied. See the License for the
// specific language governing permissions and limitations
// under the License.
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertNotNull;

View File

@ -14,7 +14,7 @@
// KIND, either express or implied. See the License for the
// specific language governing permissions and limitations
// under the License.
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertNotNull;

View File

@ -14,7 +14,7 @@
// KIND, either express or implied. See the License for the
// specific language governing permissions and limitations
// under the License.
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertNotNull;

View File

@ -14,7 +14,7 @@
// KIND, either express or implied. See the License for the
// specific language governing permissions and limitations
// under the License.
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertNotNull;

View File

@ -14,7 +14,7 @@
// KIND, either express or implied. See the License for the
// specific language governing permissions and limitations
// under the License.
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertNotNull;

View File

@ -14,7 +14,7 @@
// KIND, either express or implied. See the License for the
// specific language governing permissions and limitations
// under the License.
package com.cloud.hypervisor.xenserver.resource.wrapper;
package com.cloud.hypervisor.xenserver.resource.wrapper.citrix;
import static org.junit.Assert.assertTrue;