/*
 * Licensed 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 org.apache.karaf.cellar.shell.group;

import org.apache.karaf.cellar.core.Group;
import org.apache.karaf.cellar.core.Node;
import org.apache.karaf.cellar.core.control.ManageGroupAction;
import org.apache.karaf.shell.commands.Argument;
import org.apache.karaf.shell.commands.Command;

import java.util.LinkedList;
import java.util.List;

@Command(scope = "cluster", name = "group-delete", description = "Delete a cluster group.")
public class GroupDeleteCommand extends GroupSupport {

    @Argument(index = 0, name = "group", description = "The cluster group name.", required = true, multiValued = false)
    String groupName;

    @Override
    protected Object doExecute() throws Exception {
        Group group = groupManager.findGroupByName(groupName);
        if (group == null) {
            System.err.println("Cluster group " + groupName + " doesn't exist.");
            return null;
        }
        ClassLoader originalClassLoader = Thread.currentThread().getContextClassLoader();
        try {
            Thread.currentThread().setContextClassLoader(getClass().getClassLoader());
            List<String> nodes = new LinkedList<String>();
            if (group.getNodes() != null && !group.getNodes().isEmpty()) {
                for (Node n : group.getNodes()) {
                    nodes.add(n.getId());
                }
                doExecute(ManageGroupAction.QUIT, group.getName(), null, nodes);
            }

            groupManager.deleteGroup(group.getName());
        } finally {
            Thread.currentThread().setContextClassLoader(originalClassLoader);
        }
        return null;
    }

}

@@ -1259,16 +1259,53 @@
 ption %7B%0A
+        // check if the group exists%0A
         
@@ -1465,17 +1465,256 @@
 't exist
-.
+%22);%0A            return null;%0A        %7D%0A%0A        // check if the group doesn't contain nodes%0A        if (group.getNodes() != null && !group.getNodes().isEmpty()) %7B%0A            System.err.println(%22Cluster group %22 + groupName  + %22 is not empty
 %22);%0A    
@@ -1736,32 +1736,33 @@
 null;%0A        %7D%0A
+%0A
         ClassLoa

Check if a group is empty (no node) before deleting