Add test for EBS volume creation without size (#12227)

nik-localstack · web-flow · commit 8b5cedd6fea4 · 2025-02-24T11:18:25.000+01:00
diff --git a/tests/aws/services/ec2/test_ec2.py b/tests/aws/services/ec2/test_ec2.py
@@ -831,3 +831,10 @@ def test_create_specific_vpc_id(account_id, region_name, create_vpc, set_resourc
 
     vpc = create_vpc(cidr_block=cidr_block)
     assert vpc["Vpc"]["VpcId"] == f"vpc-{custom_id}"
+
+
+@markers.aws.validated
+def test_raise_create_volume_without_size(snapshot, aws_client):
+    with pytest.raises(ClientError) as e:
+        aws_client.ec2.create_volume(AvailabilityZone="eu-central-1a")
+    snapshot.match("request-missing-size", e.value.response)
diff --git a/tests/aws/services/ec2/test_ec2.snapshot.json b/tests/aws/services/ec2/test_ec2.snapshot.json
@@ -320,5 +320,20 @@
         }
       }
     }
+  },
+  "tests/aws/services/ec2/test_ec2.py::test_raise_create_volume_without_size": {
+    "recorded-date": "04-02-2025, 12:53:29",
+    "recorded-content": {
+      "request-missing-size": {
+        "Error": {
+          "Code": "MissingParameter",
+          "Message": "The request must contain the parameter size/snapshot"
+        },
+        "ResponseMetadata": {
+          "HTTPHeaders": {},
+          "HTTPStatusCode": 400
+        }
+      }
+    }
   }
 }
diff --git a/tests/aws/services/ec2/test_ec2.validation.json b/tests/aws/services/ec2/test_ec2.validation.json
@@ -13,5 +13,8 @@
   },
   "tests/aws/services/ec2/test_ec2.py::TestEc2Integrations::test_vcp_peering_difference_regions": {
     "last_validated_date": "2024-06-07T21:28:25+00:00"
+  },
+  "tests/aws/services/ec2/test_ec2.py::test_raise_create_volume_without_size": {
+    "last_validated_date": "2025-02-04T12:53:29+00:00"
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -320,5 +320,20 @@`
`320`	`320`	`}`
`321`	`321`	`}`
`322`	`322`	`}`
	`323`	`+ },`
	`324`	`+ "tests/aws/services/ec2/test_ec2.py::test_raise_create_volume_without_size": {`
	`325`	`+ "recorded-date": "04-02-2025, 12:53:29",`
	`326`	`+ "recorded-content": {`
	`327`	`+ "request-missing-size": {`
	`328`	`+ "Error": {`
	`329`	`+ "Code": "MissingParameter",`
	`330`	`+ "Message": "The request must contain the parameter size/snapshot"`
	`331`	`+ },`
	`332`	`+ "ResponseMetadata": {`
	`333`	`+ "HTTPHeaders": {},`
	`334`	`+ "HTTPStatusCode": 400`
	`335`	`+ }`
	`336`	`+ }`
	`337`	`+ }`
`323`	`338`	`}`
`324`	`339`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,5 +13,8 @@`
`13`	`13`	`},`
`14`	`14`	`"tests/aws/services/ec2/test_ec2.py::TestEc2Integrations::test_vcp_peering_difference_regions": {`
`15`	`15`	`"last_validated_date": "2024-06-07T21:28:25+00:00"`
	`16`	`+ },`
	`17`	`+ "tests/aws/services/ec2/test_ec2.py::test_raise_create_volume_without_size": {`
	`18`	`+ "last_validated_date": "2025-02-04T12:53:29+00:00"`
`16`	`19`	`}`
`17`	`20`	`}`