Separate ListingByTag into group & subscription level

azure-mgmt-resources/src/main/java/com/microsoft/azure/management/resources/GenericResources.java

@@ -9,7 +9,7 @@
 import com.microsoft.azure.management.apigeneration.Fluent;
 import com.microsoft.azure.management.resources.fluentcore.arm.collection.SupportsGettingById;
 import com.microsoft.azure.management.resources.fluentcore.arm.collection.SupportsListingByGroup;
-import com.microsoft.azure.management.resources.fluentcore.arm.collection.SupportsListingByTag;
+import com.microsoft.azure.management.resources.fluentcore.arm.collection.SupportsListingInGroupByTag;
 import com.microsoft.azure.management.resources.fluentcore.collection.SupportsCreating;
 import com.microsoft.azure.management.resources.fluentcore.collection.SupportsListing;
 
@@ -22,7 +22,7 @@
 public interface GenericResources extends
         SupportsListing<GenericResource>,
         SupportsListingByGroup<GenericResource>,
-        SupportsListingByTag<GenericResource>,
+        SupportsListingInGroupByTag<GenericResource>,
         SupportsGettingById<GenericResource>,
         SupportsCreating<GenericResource.DefinitionStages.Blank> {
     /**

azure-mgmt-resources/src/main/java/com/microsoft/azure/management/resources/ResourceGroups.java

@@ -12,13 +12,15 @@
 import com.microsoft.azure.management.resources.fluentcore.collection.SupportsCreating;
 import com.microsoft.azure.management.resources.fluentcore.collection.SupportsDeleting;
 import com.microsoft.azure.management.resources.fluentcore.collection.SupportsListing;
+import com.microsoft.azure.management.resources.fluentcore.collection.SupportsListingByTag;
 
 /**
  * Entry point to resource group management API.
  */
 @Fluent
 public interface ResourceGroups extends
         SupportsListing<ResourceGroup>,
+        SupportsListingByTag<ResourceGroup>,
         SupportsGettingByName<ResourceGroup>,
         SupportsCreating<ResourceGroup.DefinitionStages.Blank>,
         SupportsDeleting,

azure-mgmt-resources/src/main/java/com/microsoft/azure/management/resources/fluentcore/arm/collection/SupportsListingInGroupByTag.java

@@ -0,0 +1,31 @@
+/**
+ * Copyright (c) Microsoft Corporation. All rights reserved.
+ * Licensed under the MIT License. See License.txt in the project root for
+ * license information.
+ */
+
+package com.microsoft.azure.management.resources.fluentcore.arm.collection;
+
+import com.microsoft.azure.PagedList;
+import com.microsoft.azure.management.apigeneration.LangDefinition;
+import com.microsoft.azure.management.apigeneration.LangDefinition.MethodConversion;
+
+/**
+ * Provides access to listing Azure resources of a specific type based on their tag.
+ * <p>
+ * (Note: this interface is not intended to be implemented by user code)
+ *
+ * @param <T> the fluent type of the resource
+ */
+@LangDefinition(ContainerName = "CollectionActions", MethodConversionType = MethodConversion.OnlyMethod)
+public interface SupportsListingInGroupByTag<T> {
+    /**
+     * Lists all the resources with the specified tag.
+     *
+     * @param resourceGroupName the name of the resource group
+     * @param tagName tag's name as the key
+     * @param tagValue tag's value
+     * @return list of resources
+     */
+    PagedList<T> listByTag(String resourceGroupName, String tagName, String tagValue);
+}

azure-mgmt-resources/src/main/java/com/microsoft/azure/management/resources/fluentcore/arm/collection/SupportsListingByTag.java → azure-mgmt-resources/src/main/java/com/microsoft/azure/management/resources/fluentcore/collection/SupportsListingByTag.java

@@ -4,7 +4,7 @@
  * license information.
  */
 
-package com.microsoft.azure.management.resources.fluentcore.arm.collection;
+package com.microsoft.azure.management.resources.fluentcore.collection;
 
 import com.microsoft.azure.PagedList;
 import com.microsoft.azure.management.apigeneration.LangDefinition;
@@ -22,10 +22,9 @@ public interface SupportsListingByTag<T> {
     /**
      * Lists all the resources with the specified tag.
      *
-     * @param resourceGroupName the name of the resource group
      * @param tagName tag's name as the key
      * @param tagValue tag's value
      * @return list of resources
      */
-    PagedList<T> listByTag(String resourceGroupName, String tagName, String tagValue);
+    PagedList<T> listByTag(String tagName, String tagValue);
 }

azure-mgmt-resources/src/main/java/com/microsoft/azure/management/resources/implementation/ResourceGroupsImpl.java

@@ -36,6 +36,21 @@ public PagedList<ResourceGroup> list() {
         return wrapList(client.list());
     }
 
+    @Override
+    public PagedList<ResourceGroup> listByTag(String tagName, String tagValue) {
+        if (tagName == null) {
+            throw new IllegalArgumentException("tagName == null");
+        }
+        String odataFilter;
+        if (tagValue == null) {
+            odataFilter = String.format("tagname eq '%s'", tagName);
+        } else {
+            odataFilter = String.format("tagname eq '%s' and tagvalue eq '%s'", tagName, tagValue);
+        }
+        return wrapList(client.list(
+                odataFilter, null));
+    }
+
     @Override
     public ResourceGroupImpl getByName(String name) {
         return wrapModel(client.get(name));

azure-mgmt-resources/src/test/java/com/microsoft/azure/management/resources/ResourceGroupsTests.java

@@ -26,7 +26,7 @@ public void canCreateResourceGroup() throws Exception {
                 .create();
         // List
         ResourceGroup groupResult = null;
-        for (ResourceGroup rg : resourceGroups.list()) {
+        for (ResourceGroup rg : resourceGroups.listByTag("department", "finance")) {
             if (rg.name().equals(rgName)) {
                 groupResult = rg;
                 break;