You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ro...@apache.org on 2018/04/19 18:58:42 UTC
[cloudstack] branch master updated: server: Remote empty VPN test
class (#2587)
This is an automated email from the ASF dual-hosted git repository.
rohit pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack.git
The following commit(s) were added to refs/heads/master by this push:
new fbf07fa server: Remote empty VPN test class (#2587)
fbf07fa is described below
commit fbf07fa56599380294f9dce0c49875c5ff7578f5
Author: Rafael Weingärtner <ra...@gmail.com>
AuthorDate: Thu Apr 19 15:58:38 2018 -0300
server: Remote empty VPN test class (#2587)
I found this empty test while working on other PRs. Empty/ignored tests do not help us. I am removing it. In the future, if we manage to improve these classes, we can work on unit test cases for them.
---
server/pom.xml | 1 -
.../vpn/MockRemoteAccessVPNServiceProvider.java | 71 --------------------
.../com/cloud/network/vpn/RemoteAccessVpnTest.java | 76 ----------------------
3 files changed, 148 deletions(-)
diff --git a/server/pom.xml b/server/pom.xml
index 41668ec..cd99780 100644
--- a/server/pom.xml
+++ b/server/pom.xml
@@ -183,7 +183,6 @@
<exclude>com/cloud/storage/dao/*</exclude>
<exclude>com/cloud/vm/dao/*</exclude>
<exclude>com/cloud/api/ListPerfTest.java</exclude>
- <exclude>com/cloud/network/vpn/RemoteAccessVpnTest.java</exclude>
<exclude>com/cloud/network/security/SecurityGroupManagerImpl2Test.java</exclude>
<exclude>com/cloud/network/security/SecurityGroupManagerImpl2Test.java</exclude>
<exclude>com/cloud/vpc/VpcTestConfiguration.java</exclude>
diff --git a/server/src/test/java/com/cloud/network/vpn/MockRemoteAccessVPNServiceProvider.java b/server/src/test/java/com/cloud/network/vpn/MockRemoteAccessVPNServiceProvider.java
deleted file mode 100644
index 0156afb..0000000
--- a/server/src/test/java/com/cloud/network/vpn/MockRemoteAccessVPNServiceProvider.java
+++ /dev/null
@@ -1,71 +0,0 @@
-// Licensed to the Apache Software Foundation (ASF) under one
-// or more contributor license agreements. See the NOTICE file
-// distributed with this work for additional information
-// regarding copyright ownership. The ASF licenses this file
-// to you under the Apache License, Version 2.0 (the
-// "License"); you may not use this file except in compliance
-// with the License. You may obtain a copy of the License at
-//
-// http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing,
-// software distributed under the License is distributed on an
-// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
-// KIND, either express or implied. See the License for the
-// specific language governing permissions and limitations
-// under the License.
-package com.cloud.network.vpn;
-
-import java.util.List;
-import java.util.Map;
-
-import javax.naming.ConfigurationException;
-
-import com.cloud.exception.ResourceUnavailableException;
-import com.cloud.network.RemoteAccessVpn;
-import com.cloud.network.VpnUser;
-import com.cloud.network.element.RemoteAccessVPNServiceProvider;
-import com.cloud.utils.component.ManagerBase;
-
-public class MockRemoteAccessVPNServiceProvider extends ManagerBase implements RemoteAccessVPNServiceProvider {
-
- @Override
- public boolean configure(String name, Map<String, Object> params) throws ConfigurationException {
- return true;
- }
-
- @Override
- public String getName() {
- return "MockRemoteAccessVPNServiceProvider";
- }
-
- @Override
- public boolean start() {
- return true;
- }
-
- @Override
- public boolean stop() {
- // TODO Auto-generated method stub
- return false;
- }
-
- @Override
- public String[] applyVpnUsers(RemoteAccessVpn vpn, List<? extends VpnUser> users) throws ResourceUnavailableException {
- // TODO Auto-generated method stub
- return null;
- }
-
- @Override
- public boolean startVpn(RemoteAccessVpn vpn) throws ResourceUnavailableException {
- // TODO Auto-generated method stub
- return false;
- }
-
- @Override
- public boolean stopVpn(RemoteAccessVpn vpn) throws ResourceUnavailableException {
- // TODO Auto-generated method stub
- return false;
- }
-
-}
diff --git a/server/src/test/java/com/cloud/network/vpn/RemoteAccessVpnTest.java b/server/src/test/java/com/cloud/network/vpn/RemoteAccessVpnTest.java
deleted file mode 100644
index 58315e6..0000000
--- a/server/src/test/java/com/cloud/network/vpn/RemoteAccessVpnTest.java
+++ /dev/null
@@ -1,76 +0,0 @@
-// Licensed to the Apache Software Foundation (ASF) under one
-// or more contributor license agreements. See the NOTICE file
-// distributed with this work for additional information
-// regarding copyright ownership. The ASF licenses this file
-// to you under the Apache License, Version 2.0 (the
-// "License"); you may not use this file except in compliance
-// with the License. You may obtain a copy of the License at
-//
-// http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing,
-// software distributed under the License is distributed on an
-// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
-// KIND, either express or implied. See the License for the
-// specific language governing permissions and limitations
-// under the License.
-package com.cloud.network.vpn;
-
-import org.apache.log4j.Logger;
-import org.junit.After;
-import org.junit.Test;
-
-public class RemoteAccessVpnTest {
- private final static Logger s_logger = Logger.getLogger(RemoteAccessVpnTest.class);
-
-// private static void addDaos(MockComponentLocator locator) {
-// locator.addDao("AccountDao", AccountDaoImpl.class);
-// locator.addDao("VpnUserDao", VpnUserDaoImpl.class);
-// locator.addDao("FirewallRulesDao", FirewallRulesDaoImpl.class);
-// locator.addDao("IPAddressDao", IPAddressDaoImpl.class);
-// locator.addDao("DomainDao", DomainDaoImpl.class);
-// locator.addDao("UsageEventDao", UsageEventDaoImpl.class);
-// locator.addDao("RemoteAccessVpnDao", RemoteAccessVpnDaoImpl.class);
-// locator.addDao("ConfigurationDao", ConfigurationDaoImpl.class);
-//
-// }
-//
-// private static void addManagers(MockComponentLocator locator) {
-// locator.addManager("AccountManager", MockAccountManagerImpl.class);
-// locator.addManager("DomainManager", MockDomainManagerImpl.class);
-// locator.addManager("NetworkManager", MockNetworkManagerImpl.class);
-// locator.addManager("NetworkModel", MockNetworkModelImpl.class);
-// locator.addManager("RulesManager", MockRulesManagerImpl.class);
-// locator.addManager("FirewallManager", MockFirewallManagerImpl.class);
-// }
-
-// @Before
-// public void setUp() {
-// locator = new MockComponentLocator("management-server");
-// addDaos(locator);
-// addManagers(locator);
-// s_logger.info("Finished setUp");
-// }
-
- @After
- public void tearDown() throws Exception {
- }
-
- @Test
- public void testInjected() throws Exception {
-// List<Pair<String, Class<? extends RemoteAccessVPNServiceProvider>>> list =
-// new ArrayList<Pair<String, Class<? extends RemoteAccessVPNServiceProvider>>>();
-// list.add(new Pair<String, Class<? extends RemoteAccessVPNServiceProvider>>("RemoteAccessVPNServiceProvider", MockRemoteAccessVPNServiceProvider.class));
-// locator.addAdapterChain(RemoteAccessVPNServiceProvider.class, list);
-// s_logger.info("Finished add adapter");
-// locator.makeActive(new DefaultInterceptorLibrary());
-// s_logger.info("Finished make active");
-// RemoteAccessVpnManagerImpl vpnMgr = ComponentLocator.inject(RemoteAccessVpnManagerImpl.class);
-// s_logger.info("Finished inject");
-// Assert.assertTrue(vpnMgr.configure("RemoteAccessVpnMgr",new HashMap<String, Object>()) );
-// Assert.assertTrue(vpnMgr.start());
-// int numProviders = vpnMgr.getRemoteAccessVPNServiceProviders().size();
-// Assert.assertTrue(numProviders > 0);
- }
-
-}
--
To stop receiving notification emails like this one, please contact
rohit@apache.org.