Skip to content

Commit 35dddc2

Browse files
committed
remove uneeded tests
1 parent 29b83a0 commit 35dddc2

File tree

1 file changed

+0
-33
lines changed

1 file changed

+0
-33
lines changed

sdk/test/Services/S3/UnitTests/Custom/ResponseMapperTests.cs

Lines changed: 0 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -236,15 +236,6 @@ public void MapPutObjectResponse_NullValues_HandledCorrectly()
236236
}
237237
}
238238

239-
[TestMethod]
240-
[TestCategory("S3")]
241-
public void ValidatePutObjectResponseDefinitionCompleteness()
242-
{
243-
ValidateResponseDefinitionCompleteness<PutObjectResponse>(
244-
new[] { "Definition", "UploadResponse", "PutObjectResponse" },
245-
"PutObjectResponse");
246-
}
247-
248239
[TestMethod]
249240
[TestCategory("S3")]
250241
public void ValidateTransferUtilityUploadResponseDefinitionCompleteness()
@@ -262,19 +253,6 @@ public void ValidateCompleteMultipartUploadResponseConversionCompleteness()
262253
new[] { "Conversion", "CompleteMultipartResponse", "UploadResponse" },
263254
"TransferUtilityUploadResponse");
264255
}
265-
266-
267-
[TestMethod]
268-
[TestCategory("S3")]
269-
public void ValidateGetObjectResponseDefinitionCompleteness()
270-
{
271-
ValidateResponseDefinitionCompleteness<GetObjectResponse>(
272-
new[] { "Definition", "DownloadResponse", "GetObjectResponse" },
273-
"GetObjectResponse",
274-
// GetObjectResponse has some properties returned in the Headers property for legacy reasons which is different
275-
// than the model, so we check the Headers as well.
276-
GetHeadersCollectionProperties);
277-
}
278256

279257
// Uncomment for DOTNET-8277
280258

@@ -476,16 +454,5 @@ private static void ValidateResponseDefinitionCompleteness<TResponse>(
476454
$"Please verify they exist in the response class{additionalContext}.");
477455
}
478456
}
479-
480-
/// <summary>
481-
/// Gets properties from HeadersCollection that should be included in validation
482-
/// </summary>
483-
private static IEnumerable<string> GetHeadersCollectionProperties()
484-
{
485-
return typeof(HeadersCollection)
486-
.GetProperties(BindingFlags.Public | BindingFlags.Instance)
487-
.Where(p => p.CanRead && p.Name != "Count" && p.Name != "Keys") // Exclude collection metadata properties
488-
.Select(p => p.Name);
489-
}
490457
}
491458
}

0 commit comments

Comments
 (0)