Skip to content

Commit b551347

Browse files
committed
Cloudflare changed their status code
1 parent 754c8db commit b551347

3 files changed

Lines changed: 16 additions & 11 deletions

File tree

tests/Cloudflare.NET.Tests/Fixtures/CloudflareApiTestFixture.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@ public async Task InitializeAsync()
144144
// This ensures validation completes before any tests run, even with parallelization.
145145
PermissionValidationRunner.InitializeAccountValidation(ApiTokensApi, _settings.AccountId);
146146
await PermissionValidationRunner.EnsureAccountValidationAsync();
147-
147+
148148
// NOTE: Uncomment if you want to skip the test if user validation hasn't run yet.
149149
/*Skip.If(
150150
!PermissionValidationState.UserValidationCompleted,

tests/Cloudflare.NET.Tests/IntegrationTests/AccountSubscriptionsApiIntegrationTests.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -281,7 +281,7 @@ await act.Should()
281281

282282
/// <summary>I11: Verifies that invalid rate plan returns API error.</summary>
283283
[IntegrationTest]
284-
public async Task CreateAccountSubscriptionAsync_InvalidRatePlan_ThrowsNotFound()
284+
public async Task CreateAccountSubscriptionAsync_InvalidRatePlan_ThrowsBadRequest()
285285
{
286286
// Arrange
287287
var accountId = _settings.AccountId;
@@ -291,10 +291,10 @@ public async Task CreateAccountSubscriptionAsync_InvalidRatePlan_ThrowsNotFound(
291291
// Act
292292
var act = () => _sut.CreateAccountSubscriptionAsync(accountId, request);
293293

294-
// Assert - Per Cloudflare API: Invalid rate plan references return 404 Not Found
294+
// Assert - Invalid rate plan references return 400 Bad Request with error code 7501.
295295
await act.Should()
296296
.ThrowAsync<HttpRequestException>()
297-
.Where(ex => ex.StatusCode == System.Net.HttpStatusCode.NotFound);
297+
.Where(ex => ex.StatusCode == System.Net.HttpStatusCode.BadRequest);
298298
}
299299

300300
/// <summary>I12: Verifies that malformed subscription ID returns API error.</summary>

tests/Cloudflare.NET.Tests/IntegrationTests/ZoneSubscriptionsApiIntegrationTests.cs

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -268,14 +268,19 @@ await act.Should()
268268
.Where(ex => ex.StatusCode == System.Net.HttpStatusCode.NotFound);
269269
}
270270

271-
/// <summary>I12: Verifies that create subscription with invalid rate plan returns 404.</summary>
271+
/// <summary>I12: Verifies that create subscription with invalid rate plan returns 400 Bad Request.</summary>
272272
/// <remarks>
273-
/// Per Cloudflare API: POST with non-existent rate plan returns 404 Not Found.
274-
/// Error code 1298: "Review the rate plan ID and try again. Could not find the rate plan."
275-
/// https://developers.cloudflare.com/api/resources/zones/subresources/subscriptions/methods/create/
273+
/// <para>
274+
/// Per Cloudflare API: POST with non-existent rate plan returns 400 Bad Request with error code 7501.
275+
/// Error message: "unknown or deprecated rate plan: 'invalid-rate-plan-that-does-not-exist'"
276+
/// </para>
277+
/// <para>
278+
/// <b>Note:</b> The documentation previously indicated 404, but current behavior returns 400.
279+
/// If this test fails with 404, Cloudflare may have reverted to the documented behavior.
280+
/// </para>
276281
/// </remarks>
277282
[IntegrationTest]
278-
public async Task CreateZoneSubscriptionAsync_InvalidRatePlan_ThrowsNotFound()
283+
public async Task CreateZoneSubscriptionAsync_InvalidRatePlan_ThrowsBadRequest()
279284
{
280285
// Arrange
281286
var zoneId = _settings.ZoneId;
@@ -285,10 +290,10 @@ public async Task CreateZoneSubscriptionAsync_InvalidRatePlan_ThrowsNotFound()
285290
// Act
286291
var act = () => _sut.CreateZoneSubscriptionAsync(zoneId, request);
287292

288-
// Assert - Non-existent rate plan returns 404 Not Found
293+
// Assert - Invalid rate plan references return 400 Bad Request with error code 7501.
289294
await act.Should()
290295
.ThrowAsync<HttpRequestException>()
291-
.Where(ex => ex.StatusCode == System.Net.HttpStatusCode.NotFound);
296+
.Where(ex => ex.StatusCode == System.Net.HttpStatusCode.BadRequest);
292297
}
293298

294299
/// <summary>I13: Verifies that update subscription on non-existent zone returns 404.</summary>

0 commit comments

Comments
 (0)