Skip to content

Commit 2d10d22

Browse files
committed
Minor style issues on list accounts operation
1 parent 5a2cef9 commit 2d10d22

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

src/ResourceManager/AzureBatch/Commands.Batch/Accounts/Helpers.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ public static string FormatTagsTable(Hashtable[] tags)
162162
/// <param name="account">The account to filter on.</param>
163163
/// <param name="tag">The tag to filter on.</param>
164164
/// <returns>Whether or not the account's tags match with the given tag</returns>
165-
public static bool FilterAccounts(AccountResource account, Hashtable tag)
165+
public static bool MatchesTag(AccountResource account, Hashtable tag)
166166
{
167167
if (tag != null && tag.Count >= 1)
168168
{

src/ResourceManager/AzureBatch/Commands.Batch/Models/BatchClient.Accounts.cs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,9 @@ public virtual BatchAccountContext ListKeys(string resourceGroupName, string acc
157157
? BatchManagementClient.Account.List()
158158
: BatchManagementClient.Account.ListByResourceGroup(resourceGroupName);
159159

160-
var batchAccountContexts = ListAllAccounts(response).Where(acct => Helpers.FilterAccounts(acct, tag)).
160+
var batchAccountContexts =
161+
ListAllAccounts(response).
162+
Where(acct => Helpers.MatchesTag(acct, tag)).
161163
Select(resource => BatchAccountContext.ConvertAccountResourceToNewAccountContext(resource));
162164

163165
return batchAccountContexts;

0 commit comments

Comments
 (0)