From 8d4d2215052252318721b7b85c63333b95158a12 Mon Sep 17 00:00:00 2001 From: Scott Beddall Date: Thu, 1 Aug 2024 17:51:36 -0700 Subject: [PATCH] remove a test that doesn't have a point if we don't error on sanitizer removal --- .../AdminTests.cs | 26 ------------------- 1 file changed, 26 deletions(-) diff --git a/tools/test-proxy/Azure.Sdk.Tools.TestProxy.Tests/AdminTests.cs b/tools/test-proxy/Azure.Sdk.Tools.TestProxy.Tests/AdminTests.cs index 0553b16bcb5..cbb80b4e769 100644 --- a/tools/test-proxy/Azure.Sdk.Tools.TestProxy.Tests/AdminTests.cs +++ b/tools/test-proxy/Azure.Sdk.Tools.TestProxy.Tests/AdminTests.cs @@ -938,32 +938,6 @@ public async Task TestAddSanitizerContinuesWithTwoRequiredParams() Assert.Equal(targetString, actualTargetString); } - [Fact] - public async Task RemoveSanitizerErrorsForInvalidIdOnRecording() - { - RecordingHandler testRecordingHandler = new RecordingHandler(Directory.GetCurrentDirectory()); - var httpContext = new DefaultHttpContext(); - await testRecordingHandler.StartPlaybackAsync("Test.RecordEntries/oauth_request_with_variables.json", httpContext.Response); - var recordingId = httpContext.Response.Headers["x-recording-id"]; - httpContext.Request.Headers["x-recording-id"] = recordingId; - httpContext.Response.Body = new MemoryStream(); - var controller = new Admin(testRecordingHandler, _nullLogger) - { - ControllerContext = new ControllerContext() - { - HttpContext = httpContext - } - }; - - var testSet = new RemoveSanitizerList() { Sanitizers = new List() { "0" } }; - - var assertion = await Assert.ThrowsAsync( - async () => await controller.RemoveSanitizers(testSet) - ); - - Assert.Contains("Unable to remove 1 sanitizer. Detailed list follows: \nThe requested sanitizer for removal \"0\" is not active on recording/playback with id", assertion.Message); - } - [Fact] public async Task RemoveSanitizersSilentlyAcceptsInvalidSanitizer() {