Skip to content

Commit 277a291

Browse files
fixes
1 parent 9db7937 commit 277a291

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

docs/changelog.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ See [upgrade notes][1] for helpful information when upgrading from previous vers
3535
- Check that replica locations are in allowed regions by @BernieWhite.
3636
[#3441](https://github.com/Azure/PSRule.Rules.Azure/issues/3441)
3737
- Azure Cache for Redis:
38-
- Check that legacy Azure Cache for Redis instances are migrated to Azure Managed Redis by @BenjaminEngeset.
38+
- Check for legacy Azure Cache for Redis instances by @BenjaminEngeset.
3939
[#3605](https://github.com/Azure/PSRule.Rules.Azure/issues/3605)
4040
- Managed Instance for Apache Cassandra:
4141
- Check that Managed Instance for Apache Cassandra clusters have availability zones enabled by @BenjaminEngeset.

tests/PSRule.Rules.Azure.Tests/Azure.Redis.Tests.ps1

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -357,13 +357,12 @@ Describe 'Azure.Redis' -Tag 'Redis' {
357357
It 'Azure.Redis.MigrateAMR' {
358358
$filteredResult = $result | Where-Object { $_.RuleName -eq 'Azure.Redis.MigrateAMR' };
359359

360-
# Fail - All Azure Cache for Redis instances should fail this rule
360+
# Fail - all instances should fail
361361
$ruleResult = @($filteredResult | Where-Object { $_.Outcome -eq 'Fail' });
362362
$ruleResult | Should -Not -BeNullOrEmpty;
363363
$ruleResult.Length | Should -Be 12;
364364
$ruleResult.TargetName | Should -BeIn 'redis-A', 'redis-B', 'redis-C', 'redis-D', 'redis-E', 'redis-F', 'redis-G', 'redis-H', 'redis-I', 'redis-J', 'redis-Q', 'redis-R';
365365

366-
# Verify the reason message
367366
$ruleResult[0].Reason | Should -BeExactly "Azure Cache for Redis is being retired. Migrate to Azure Managed Redis.";
368367
}
369368
}

0 commit comments

Comments
 (0)