/*
 * 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 org.apache.geode.security;

import static org.apache.geode.distributed.ConfigurationProperties.SECURITY_MANAGER;
import static org.apache.geode.security.SecurityTestUtil.assertNotAuthorized;
import static org.apache.geode.security.SecurityTestUtil.createClientCache;
import static org.apache.geode.security.SecurityTestUtil.createProxyRegion;

import org.junit.Rule;
import org.junit.Test;
import org.junit.experimental.categories.Category;

import org.apache.geode.cache.Region;
import org.apache.geode.cache.RegionShortcut;
import org.apache.geode.cache.client.ClientCache;
import org.apache.geode.cache.client.Pool;
import org.apache.geode.cache.client.PoolManager;
import org.apache.geode.cache.query.CqAttributes;
import org.apache.geode.cache.query.CqAttributesFactory;
import org.apache.geode.cache.query.CqQuery;
import org.apache.geode.cache.query.QueryService;
import org.apache.geode.test.dunit.VM;
import org.apache.geode.test.dunit.rules.ClusterStartupRule;
import org.apache.geode.test.junit.categories.DistributedTest;
import org.apache.geode.test.junit.categories.SecurityTest;
import org.apache.geode.test.junit.rules.ServerStarterRule;

@Category({DistributedTest.class, SecurityTest.class})
public class ClientCQAuthDUnitTest {

  @Rule
  public ClusterStartupRule startupRule = new ClusterStartupRule();
  private static String REGION_NAME = "AuthRegion";
  private final VM client1 = startupRule.getVM(1);
  private final VM client2 = startupRule.getVM(2);
  private final VM client3 = startupRule.getVM(3);

  @Rule
  public ServerStarterRule server = new ServerStarterRule()
      .withProperty(SECURITY_MANAGER, SimpleTestSecurityManager.class.getName())
      .withRegion(RegionShortcut.REPLICATE, REGION_NAME);

  @Test
  public void verifyCQPermissions() {
    String query = "select * from /AuthRegion";
    int serverPort = server.getPort();

    // client has no permission whatsoever
    client1.invoke(() -> {
      ClientCache cache = createClientCache("test", "test", serverPort);
      final Region region = createProxyRegion(cache, REGION_NAME);
      Pool pool = PoolManager.find(region);
      QueryService qs = pool.getQueryService();
      CqAttributes cqa = new CqAttributesFactory().create();

      // Create the CqQuery (this is on the client side)
      CqQuery cq = qs.newCq("CQ1", query, cqa);

      assertNotAuthorized(cq::execute, "DATA:READ:AuthRegion");
      assertNotAuthorized(cq::executeWithInitialResults, "DATA:READ:AuthRegion");
      assertNotAuthorized(qs::getAllDurableCqsFromServer, "CLUSTER:READ");
    });

    // client2 has part of the permission
    client2.invoke(() -> {
      ClientCache cache = createClientCache("clusterManageQuery", "clusterManageQuery", serverPort);
      final Region region = createProxyRegion(cache, REGION_NAME);
      Pool pool = PoolManager.find(region);
      QueryService qs = pool.getQueryService();
      CqAttributes cqa = new CqAttributesFactory().create();

      // Create the CqQuery (this is on the client side)
      CqQuery cq = qs.newCq("CQ1", query, cqa);
      assertNotAuthorized(cq::execute, "DATA:READ:AuthRegion");
      assertNotAuthorized(cq::executeWithInitialResults, "DATA:READ:AuthRegion");
      cq.close();
    });

    // client3 has all the permissions
    client3.invoke(() -> {
      ClientCache cache = createClientCache("clusterManageQuery,dataRead",
          "clusterManageQuery,dataRead", serverPort);
      Region region = createProxyRegion(cache, REGION_NAME);
      Pool pool = PoolManager.find(region);
      QueryService qs = pool.getQueryService();
      CqAttributes cqa = new CqAttributesFactory().create();

      // Create the CqQuery
      CqQuery cq = qs.newCq("CQ1", query, cqa);
      cq.execute();
      cq.stop();
    });
  }
}

@@ -783,17 +783,16 @@
 se.%0A */%0A
-%0A
 package 
@@ -1135,16 +1135,41 @@
 egion;%0A%0A
+import org.junit.Before;%0A
 import o
@@ -2068,84 +2068,8 @@
  %7B%0A%0A
-  @Rule%0A  public ClusterStartupRule startupRule = new ClusterStartupRule();%0A
   pr
@@ -2081,16 +2081,22 @@
  static 
+final 
 String R
@@ -2122,16 +2122,17 @@
 egion%22;%0A
+%0A
   privat
@@ -2125,38 +2125,32 @@
 on%22;%0A%0A  private 
-final 
 VM client1 = sta
@@ -2147,31 +2147,30 @@
 ent1
- = startupRule.getVM(1)
+;%0A  private VM client2
 ;%0A  
@@ -2169,38 +2169,32 @@
 ent2;%0A  private 
-final 
 VM client2 = sta
@@ -2190,64 +2190,74 @@
 ient
-2 = startupRule.getVM(2);%0A  private final VM client3 = s
+3;%0A%0A  @Rule%0A  public ClusterStartupRule startupRule = new ClusterS
 tart
@@ -2266,16 +2266,9 @@
 Rule
-.getVM(3
+(
 );%0A%0A
@@ -2475,16 +2475,163 @@
 NAME);%0A%0A
+  @Before%0A  public void setUp() %7B%0A    client1 = startupRule.getVM(1);%0A    client2 = startupRule.getVM(2);%0A    client3 = startupRule.getVM(3);%0A  %7D%0A%0A
   @Test%0A

Fix ClientCQAuthDUnitTest by invoking getVM during setUp