Skip to content

Commit 848e448

Browse files
committed
V1 cleanup tests (#3165)
* Fix resource type error message syntax * Update tests for alphabetic ordering
1 parent 8c45a89 commit 848e448

File tree

4 files changed

+6
-8
lines changed

4 files changed

+6
-8
lines changed

src/cfnlint/rules/resources/Type.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ def cfnresourcetype(
5050
("Custom::", "AWS::Serverless::")
5151
) and not resource_type.endswith("::MODULE"):
5252
yield ValidationError(
53-
f"Resource type `{resource_type}` does not exist in '{region}'",
53+
f"Resource type {resource_type!r} does not exist in {region!r}",
5454
path=deque(["Type"]),
5555
rule=self,
5656
)

test/unit/rules/resources/rds/test_db_cluster_engine_version.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ def validator():
3434
(
3535
{
3636
"Engine": "mysql",
37-
"EngineVersion": "5.7.37",
37+
"EngineVersion": "5.7.44",
3838
},
3939
[],
4040
),
@@ -74,8 +74,7 @@ def validator():
7474
[
7575
ValidationError(
7676
(
77-
"'foo' is not one of ['5.7.37', '5.7.38', '5.7.39', '5.7.40', "
78-
"'5.7.41', '5.7.42', '5.7.43', '5.7.44', '8.0.28', '8.0.31', "
77+
"'foo' is not one of ['5.7.44', '8.0.28', '8.0.31', "
7978
"'8.0.32', '8.0.33', '8.0.34', '8.0.35', '8.0.36']"
8079
),
8180
rule=DbClusterEngineVersion(),

test/unit/rules/resources/rds/test_db_instance_engine_version.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ def validator():
6363
"'custom-sqlserver-web', 'db2-ae', 'db2-se', 'mariadb', "
6464
"'mysql', 'oracle-ee', 'oracle-ee-cdb', 'oracle-se2', "
6565
"'oracle-se2-cdb', 'postgres', 'sqlserver-ee', "
66-
"'sqlserver-se', 'sqlserver-ex', 'sqlserver-web']"
66+
"'sqlserver-ex', 'sqlserver-se', 'sqlserver-web']"
6767
),
6868
rule=DbInstanceEngineVersion(),
6969
path=deque(["Engine"]),
@@ -79,8 +79,7 @@ def validator():
7979
[
8080
ValidationError(
8181
(
82-
"'foo' is not one of ['5.7.37', '5.7.38', '5.7.39', '5.7.40', "
83-
"'5.7.41', '5.7.42', '5.7.43', '5.7.44', '8.0.28', '8.0.31', "
82+
"'foo' is not one of ['5.7.44', '8.0.28', '8.0.31', "
8483
"'8.0.32', '8.0.33', '8.0.34', '8.0.35', '8.0.36']"
8584
),
8685
rule=DbInstanceEngineVersion(),

test/unit/rules/resources/test_type.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ def test_types_with_conditions(self, cfn):
5151
errors,
5252
[
5353
ValidationError(
54-
"Resource type `Foo::Bar::Type` does not exist in 'us-east-1'",
54+
"Resource type 'Foo::Bar::Type' does not exist in 'us-east-1'",
5555
path=deque(["Type"]),
5656
schema_path=deque([]),
5757
rule=Type(),

0 commit comments

Comments
 (0)