changeset 2303:45d5c99c88bb

Remove unused HostVMArguments class. Reviewed-by: jerboaa Review-thread: http://icedtea.classpath.org/pipermail/thermostat/2016-May/018885.html
author Jie Kang <jkang@redhat.com>
date Fri, 20 May 2016 09:38:17 -0400
parents 3bc74bccf849
children 846831e69547
files client/cli/src/main/java/com/redhat/thermostat/client/cli/HostVMArguments.java common/core/src/test/java/com/redhat/thermostat/common/cli/HostVMArgumentsTest.java
diffstat 2 files changed, 0 insertions(+), 127 deletions(-) [+]
line wrap: on
line diff
--- a/client/cli/src/main/java/com/redhat/thermostat/client/cli/HostVMArguments.java	Thu May 19 18:55:05 2016 +0200
+++ /dev/null	Thu Jan 01 00:00:00 1970 +0000
@@ -1,84 +0,0 @@
-/*
- * Copyright 2012-2016 Red Hat, Inc.
- *
- * This file is part of Thermostat.
- *
- * Thermostat is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published
- * by the Free Software Foundation; either version 2, or (at your
- * option) any later version.
- *
- * Thermostat is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with Thermostat; see the file COPYING.  If not see
- * <http://www.gnu.org/licenses/>.
- *
- * Linking this code with other modules is making a combined work
- * based on this code.  Thus, the terms and conditions of the GNU
- * General Public License cover the whole combination.
- *
- * As a special exception, the copyright holders of this code give
- * you permission to link this code with independent modules to
- * produce an executable, regardless of the license terms of these
- * independent modules, and to copy and distribute the resulting
- * executable under terms of your choice, provided that you also
- * meet, for each linked independent module, the terms and conditions
- * of the license of that module.  An independent module is a module
- * which is not derived from or based on this code.  If you modify
- * this code, you may extend this exception to your version of the
- * library, but you are not obligated to do so.  If you do not wish
- * to do so, delete this exception statement from your version.
- */
-
-package com.redhat.thermostat.client.cli;
-
-import com.redhat.thermostat.client.cli.internal.LocaleResources;
-import com.redhat.thermostat.common.cli.Arguments;
-import com.redhat.thermostat.common.cli.CommandException;
-import com.redhat.thermostat.shared.locale.Translate;
-import com.redhat.thermostat.storage.core.HostRef;
-import com.redhat.thermostat.storage.core.VmRef;
-
-public class HostVMArguments {
-
-    private static final Translate<LocaleResources> tr = LocaleResources.createLocalizer();
-
-    private HostRef host;
-    private VmRef vm;
-
-    public HostVMArguments(Arguments args) throws CommandException {
-        this(args, true, true);
-    }
-
-    public HostVMArguments(Arguments args, boolean hostRequired, boolean vmRequired) throws CommandException {
-        String hostId = args.getArgument(Arguments.HOST_ID_ARGUMENT);
-        String vmId = args.getArgument(VmArgument.ARGUMENT_NAME);
-        if (hostRequired && hostId == null) {
-            throw new CommandException(tr.localize(LocaleResources.HOSTID_REQUIRED));
-        } else if (hostId == null) {
-            host = null;
-        } else {
-            host = new HostRef(hostId, "dummy");
-        }
-        if (vmId == null && vmRequired) {
-            throw new CommandException(tr.localize(LocaleResources.VMID_REQUIRED));
-        } else if (vmId == null) {
-            vm = null;
-        } else {
-            vm = new VmRef(host, vmId, -1, "dummy");
-        }
-    }
-
-    public HostRef getHost() {
-        return host;
-    }
-
-    public VmRef getVM() {
-        return vm;
-    }
-}
-
--- a/common/core/src/test/java/com/redhat/thermostat/common/cli/HostVMArgumentsTest.java	Thu May 19 18:55:05 2016 +0200
+++ /dev/null	Thu Jan 01 00:00:00 1970 +0000
@@ -1,43 +0,0 @@
-/*
- * Copyright 2012-2016 Red Hat, Inc.
- *
- * This file is part of Thermostat.
- *
- * Thermostat is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published
- * by the Free Software Foundation; either version 2, or (at your
- * option) any later version.
- *
- * Thermostat is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with Thermostat; see the file COPYING.  If not see
- * <http://www.gnu.org/licenses/>.
- *
- * Linking this code with other modules is making a combined work
- * based on this code.  Thus, the terms and conditions of the GNU
- * General Public License cover the whole combination.
- *
- * As a special exception, the copyright holders of this code give
- * you permission to link this code with independent modules to
- * produce an executable, regardless of the license terms of these
- * independent modules, and to copy and distribute the resulting
- * executable under terms of your choice, provided that you also
- * meet, for each linked independent module, the terms and conditions
- * of the license of that module.  An independent module is a module
- * which is not derived from or based on this code.  If you modify
- * this code, you may extend this exception to your version of the
- * library, but you are not obligated to do so.  If you do not wish
- * to do so, delete this exception statement from your version.
- */
-
-package com.redhat.thermostat.common.cli;
-
-public class HostVMArgumentsTest {
-
-    // TODO make a test that checks the argument parsing behaviour.
-}
-