From 62b87a34284c4f72480be8c0a878dbe7cca36c20 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 4 Dec 2025 06:55:28 +0000 Subject: [PATCH 1/2] Initial plan From c906906efca14e18cfd18886c95e7190c1bc91f7 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 4 Dec 2025 07:04:26 +0000 Subject: [PATCH 2/2] Fix typos in tools folder Co-authored-by: dingmeng-xue <56333063+dingmeng-xue@users.noreply.github.com> --- .../Az.Tools.Predictor.Test/ParameterSetTests.cs | 6 +++--- .../Telemetry/HistoryTelemetryData.cs | 2 +- .../FormatPs1XmlGenerator/Reflector.cs | 2 +- .../NetcoreTests/Resources/DeploymentTests.ps1 | 2 +- tools/PublishModules.psm1 | 2 +- .../BreakingChangeAnalyzer/CmdletMetadataHelper.cs | 2 +- tools/TestFx/Mocks/MockContext.cs | 2 +- .../PermissiveRecordMatcherWithApiExclusion.cs | 6 +++--- .../PermissiveRecordMatcherWithResourceApiExclusion.cs | 6 +++--- tools/TestFx/TestManager.cs | 10 +++++----- tools/TestFx/TestModelExtensions.cs | 8 ++++---- 11 files changed, 24 insertions(+), 24 deletions(-) diff --git a/tools/Az.Tools.Predictor/Az.Tools.Predictor.Test/ParameterSetTests.cs b/tools/Az.Tools.Predictor/Az.Tools.Predictor.Test/ParameterSetTests.cs index 9c2a8167555a..a32c5dc7bd87 100644 --- a/tools/Az.Tools.Predictor/Az.Tools.Predictor.Test/ParameterSetTests.cs +++ b/tools/Az.Tools.Predictor/Az.Tools.Predictor.Test/ParameterSetTests.cs @@ -134,7 +134,7 @@ public void VerifySwitchParameterAtTheEnd() } /// - /// Verify that the switch parameter in the middel can be parsed. + /// Verify that the switch parameter in the middle can be parsed. /// [Fact] public void VerifySwitchParameterInMiddle() @@ -270,7 +270,7 @@ public void VerifyExcessPositionalParameters() } /// - /// Verify that a positioinal parameter is followed by a switch parameter. + /// Verify that a positional parameter is followed by a switch parameter. /// [Fact] public void VerifyPositionalParametersFollowedBySwitchParameters() @@ -307,7 +307,7 @@ public void VerifyPositionalParametersFollowedByNamedParameters() } /// - /// Verify that we throw the exception when the positional paramters are after th named ones. + /// Verify that we throw the exception when the positional parameters are after the named ones. /// [Fact] public void VerifyPositionalParametersAfterNamedParameters() diff --git a/tools/Az.Tools.Predictor/Az.Tools.Predictor/Telemetry/HistoryTelemetryData.cs b/tools/Az.Tools.Predictor/Az.Tools.Predictor/Telemetry/HistoryTelemetryData.cs index 39df2372bb67..d30fbd92cde3 100644 --- a/tools/Az.Tools.Predictor/Az.Tools.Predictor/Telemetry/HistoryTelemetryData.cs +++ b/tools/Az.Tools.Predictor/Az.Tools.Predictor/Telemetry/HistoryTelemetryData.cs @@ -46,7 +46,7 @@ public sealed class HistoryTelemetryData : ITelemetryData public string Command { get; } /// - /// Gets whether the commdn line ran successfully. + /// Gets whether the command line ran successfully. /// public bool Success { get; } diff --git a/tools/FormatPs1XmlGenerator/FormatPs1XmlGenerator/Reflector.cs b/tools/FormatPs1XmlGenerator/FormatPs1XmlGenerator/Reflector.cs index 1a872eeb7977..d10dfa6199c9 100644 --- a/tools/FormatPs1XmlGenerator/FormatPs1XmlGenerator/Reflector.cs +++ b/tools/FormatPs1XmlGenerator/FormatPs1XmlGenerator/Reflector.cs @@ -48,7 +48,7 @@ public IList GetCmdletTypes(string assemblyPath, string[] cmdlets, out str { if (string.IsNullOrEmpty(assemblyPath)) throw new ArgumentNullException(nameof(assemblyPath)); - // Every module has dependecy on the Az.Accounts module + // Every module has dependency on the Az.Accounts module // First, try to find the Az.Accounts module in the 'artifacts' directory Assembly Resoler(AssemblyLoadContext context, AssemblyName name) { diff --git a/tools/InstallationTests/NetcoreTests/Resources/DeploymentTests.ps1 b/tools/InstallationTests/NetcoreTests/Resources/DeploymentTests.ps1 index 0839c26ec843..fd6e09e13f7c 100644 --- a/tools/InstallationTests/NetcoreTests/Resources/DeploymentTests.ps1 +++ b/tools/InstallationTests/NetcoreTests/Resources/DeploymentTests.ps1 @@ -266,7 +266,7 @@ function Test-NewDeploymentWithKeyVaultReference .SYNOPSIS Tests deployment with template file with complex parameters. #> -function Test-NewDeploymentWithComplexPramaters +function Test-NewDeploymentWithComplexParameters { # Setup $rgname = Get-ResourceGroupName diff --git a/tools/PublishModules.psm1 b/tools/PublishModules.psm1 index 60fc5da0b008..f107994aa5b0 100644 --- a/tools/PublishModules.psm1 +++ b/tools/PublishModules.psm1 @@ -480,7 +480,7 @@ function Save-PackageLocally { $ModulePaths = $env:PSModulePath -split ';' $DestinationModulePath = [System.IO.Path]::Combine($ModulePaths[0], $ModuleName, $RequiredVersion) Expand-Archive -Path $NupkgFilePath -DestinationPath $DestinationModulePath -Force - Write-Output "Downloaded the package sucessfully" + Write-Output "Downloaded the package successfully" } } } diff --git a/tools/StaticAnalysis/BreakingChangeAnalyzer/CmdletMetadataHelper.cs b/tools/StaticAnalysis/BreakingChangeAnalyzer/CmdletMetadataHelper.cs index ac0211701f21..7b5883fb5799 100644 --- a/tools/StaticAnalysis/BreakingChangeAnalyzer/CmdletMetadataHelper.cs +++ b/tools/StaticAnalysis/BreakingChangeAnalyzer/CmdletMetadataHelper.cs @@ -107,7 +107,7 @@ public void CompareCmdletMetadata( // Go through all of the breaking change checks for parameters using the ParameterMetadataHelper _parameterMetadataHelper.CompareParameterMetadata(oldCmdlet, oldCmdlet.Parameters, newCmdlet.Parameters, issueLogger); - // Go through all of the breaking change checks for parameter sets using the ParmaterSetMetadataHelper + // Go through all of the breaking change checks for parameter sets using the ParameterSetMetadataHelper _parameterSetMetadataHelper.CompareParameterSetMetadata(oldCmdlet, oldCmdlet.ParameterSets, newCmdlet.ParameterSets, issueLogger); } // If the cmdlet cannot be found, log an issue diff --git a/tools/TestFx/Mocks/MockContext.cs b/tools/TestFx/Mocks/MockContext.cs index 9d132f7ad3c1..ef7323f1d2ab 100644 --- a/tools/TestFx/Mocks/MockContext.cs +++ b/tools/TestFx/Mocks/MockContext.cs @@ -275,7 +275,7 @@ public void Stop() if (HttpMockServer.Mode == HttpRecorderMode.Record) { - // this check should be removed once we make the optimizatoin default + // this check should be removed once we make the optimization default if (OptimizeTestRecordingFile) { RecordedFiles procRecFile = new RecordedFiles(recordedFilePath); diff --git a/tools/TestFx/Recorder/PermissiveRecordMatcherWithApiExclusion.cs b/tools/TestFx/Recorder/PermissiveRecordMatcherWithApiExclusion.cs index ea13d805520e..74d1b1e890b0 100644 --- a/tools/TestFx/Recorder/PermissiveRecordMatcherWithApiExclusion.cs +++ b/tools/TestFx/Recorder/PermissiveRecordMatcherWithApiExclusion.cs @@ -62,11 +62,11 @@ public virtual string GetMatchingKey(System.Net.Http.HttpRequestMessage request) var agent = request.Headers.FirstOrDefault(h => h.Key.Equals("User-Agent")); if (agent.Key != null) { - foreach (var userAgnet in _userAgentsToIgnore) + foreach (var userAgent in _userAgentsToIgnore) { - if (agent.Value.Any(v => v.StartsWith(userAgnet.Key, StringComparison.OrdinalIgnoreCase))) + if (agent.Value.Any(v => v.StartsWith(userAgent.Key, StringComparison.OrdinalIgnoreCase))) { - path = RemoveOrReplaceApiVersion(path, userAgnet.Value); + path = RemoveOrReplaceApiVersion(path, userAgent.Value); break; } } diff --git a/tools/TestFx/Recorder/PermissiveRecordMatcherWithResourceApiExclusion.cs b/tools/TestFx/Recorder/PermissiveRecordMatcherWithResourceApiExclusion.cs index 26b0c6e5b022..fbebb227b8d8 100644 --- a/tools/TestFx/Recorder/PermissiveRecordMatcherWithResourceApiExclusion.cs +++ b/tools/TestFx/Recorder/PermissiveRecordMatcherWithResourceApiExclusion.cs @@ -76,11 +76,11 @@ public virtual string GetMatchingKey(System.Net.Http.HttpRequestMessage request) var agent = request.Headers.FirstOrDefault(h => h.Key.Equals("User-Agent")); if (agent.Key != null) { - foreach (var userAgnet in _userAgentsToIgnore) + foreach (var userAgent in _userAgentsToIgnore) { - if (agent.Value.Any(v => v.StartsWith(userAgnet.Key, StringComparison.OrdinalIgnoreCase))) + if (agent.Value.Any(v => v.StartsWith(userAgent.Key, StringComparison.OrdinalIgnoreCase))) { - requestUri = RemoveOrReplaceApiVersion(requestUri, userAgnet.Value); + requestUri = RemoveOrReplaceApiVersion(requestUri, userAgent.Value); break; } } diff --git a/tools/TestFx/TestManager.cs b/tools/TestFx/TestManager.cs index 8073570e73fb..170281190298 100644 --- a/tools/TestFx/TestManager.cs +++ b/tools/TestFx/TestManager.cs @@ -104,7 +104,7 @@ public ITestRunnerFactory WithProjectSubfolderForTests(string folderName) } /// - /// Overrided default script name, which by convension is the cs test class name with ps1 extension. + /// Overrides default script name, which by convention is the cs test class name with ps1 extension. /// /// /// self @@ -140,7 +140,7 @@ public ITestRunnerFactory WithNewRmModules(Func /// Adds extra RM modules in addition to the RMProfileModule and RMResourceModule, - /// witch are added in the constructor. + /// which are added in the constructor. /// /// /// @@ -152,10 +152,10 @@ public ITestRunnerFactory WithExtraRmModules(Func - /// Set new argumets for the mock server record matcher + /// Set new arguments for the mock server record matcher /// /// Dictionary [userAgent:apiVersion] to ignore - /// Dictionary [resouceProvider:apiVersion] to match + /// Dictionary [resourceProvider:apiVersion] to match /// public ITestRunnerFactory WithNewRecordMatcherArguments(Dictionary userAgentsToIgnore, Dictionary resourceProviders, bool ignoreResourceClient = true) { @@ -184,7 +184,7 @@ public ITestRunnerFactory WithCleanupAction(Action cleanupAction) } /// - /// Sets a new HttpMockServer.Matcher implementation. By defauls it's PermissiveRecordMatcherWithApiExclusion + /// Sets a new HttpMockServer.Matcher implementation. By default it's PermissiveRecordMatcherWithApiExclusion /// /// delegate /// self diff --git a/tools/TestFx/TestModelExtensions.cs b/tools/TestFx/TestModelExtensions.cs index 80165761cd97..fd04cfb723f2 100644 --- a/tools/TestFx/TestModelExtensions.cs +++ b/tools/TestFx/TestModelExtensions.cs @@ -24,7 +24,7 @@ public static class TestModelExtensions /// Check that the extended properties in the two models are equal /// /// This model - /// The model being comparied to this model + /// The model being compared to this model /// true if the models are equal, or false otherwise public static bool CheckExtensionsEqual(this IExtensibleModel model, IExtensibleModel other) { @@ -47,7 +47,7 @@ public static bool IsEqual(this IAzureTenant baseTenant, IAzureTenant other) /// Check if this account equals another account /// /// The base account for comparison - /// The accoutn to compare to + /// The account to compare to /// true if the elements of both accounts are equal, otherwise false public static bool IsEqual(this IAzureAccount baseAccount, IAzureAccount other) { @@ -63,7 +63,7 @@ public static bool IsEqual(this IAzureAccount baseAccount, IAzureAccount other) /// /// The base subscription for comparison /// The subscription to compare to - /// True if the elements of both subscriptiosn are equal, otherwise false + /// True if the elements of both subscriptions are equal, otherwise false public static bool IsEqual(this IAzureSubscription baseSub, IAzureSubscription other) { return (baseSub == null && other == null) || (baseSub.CheckExtensionsEqual(other) @@ -77,7 +77,7 @@ public static bool IsEqual(this IAzureSubscription baseSub, IAzureSubscription o /// /// this environment /// The environment to compare to - /// Treu fi the elements of the environment are equal, otherwise false + /// True if the elements of the environment are equal, otherwise false public static bool IsEqual(this IAzureEnvironment environment, IAzureEnvironment other) { return (environment == null && other == null)