100880 set 3 changes (#4599)

microsoft · Sep 5, 2024 · 14494b0 · 14494b0
1 parent e1d65a3
commit 14494b0
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/src/Microsoft.Health.Fhir.Core/Features/Search/TypedElementSearchIndexer.cs b/src/Microsoft.Health.Fhir.Core/Features/Search/TypedElementSearchIndexer.cs
@@ -75,7 +75,7 @@ public IReadOnlyCollection<SearchIndexEntry> Extract(ResourceElement resource)
             // This allow to resolve %resource FhirPath to provided value.
             context.Resource = resource.Instance;
 
-            IEnumerable<SearchParameterInfo> searchParameters = _searchParameterDefinitionManager.GetSearchParameters(resource.InstanceType);
+            var searchParameters = _searchParameterDefinitionManager.GetSearchParameters(resource.InstanceType).ToList();
 
             foreach (SearchParameterInfo searchParameter in searchParameters)
             {
@@ -261,7 +261,7 @@ private List<ISearchValue> ExtractSearchValues(
                     continue;
                 }
 
-                IEnumerable<ISearchValue> searchValues = converter.ConvertTo(extractedValue);
+                var searchValues = converter.ConvertTo(extractedValue).ToList();
 
                 if (searchValues != null)
                 {

diff --git a/src/Microsoft.Health.Fhir.Core/Features/Validation/ResourceNotValidException.cs b/src/Microsoft.Health.Fhir.Core/Features/Validation/ResourceNotValidException.cs
@@ -13,7 +13,7 @@ namespace Microsoft.Health.Fhir.Core.Features.Validation
 {
     public class ResourceNotValidException : FhirException
     {
-        public ResourceNotValidException(IEnumerable<OperationOutcomeIssue> validationFailures)
+        public ResourceNotValidException(IReadOnlyCollection<OperationOutcomeIssue> validationFailures)
         {
             EnsureArg.IsNotNull(validationFailures, nameof(validationFailures));
 
@@ -23,7 +23,7 @@ public ResourceNotValidException(IEnumerable<OperationOutcomeIssue> validationFa
             }
         }
 
-        public ResourceNotValidException(IEnumerable<ValidationFailure> validationFailures)
+        public ResourceNotValidException(IReadOnlyCollection<ValidationFailure> validationFailures)
         {
             EnsureArg.IsNotNull(validationFailures, nameof(validationFailures));
 

diff --git a/src/Microsoft.Health.Fhir.SchemaManager/MutuallyExclusiveOptionValidator.cs b/src/Microsoft.Health.Fhir.SchemaManager/MutuallyExclusiveOptionValidator.cs
@@ -17,7 +17,7 @@ public static class MutuallyExclusiveOptionValidator
     /// <param name="commandResult">The result of the command being run</param>
     /// <param name="mutuallyExclusiveOptions">The list of mutually exclusive options</param>
     /// <param name="validationErrorMessage">The message to show if only one of the option is not present</param>
-    public static void Validate(CommandResult commandResult, IEnumerable<Option> mutuallyExclusiveOptions, string validationErrorMessage)
+    public static void Validate(CommandResult commandResult, IReadOnlyCollection<Option> mutuallyExclusiveOptions, string validationErrorMessage)
     {
         EnsureArg.IsNotNull(commandResult, nameof(commandResult));
         EnsureArg.IsNotNull(mutuallyExclusiveOptions, nameof(mutuallyExclusiveOptions));

diff --git a/...lth.Fhir.Shared.Api.UnitTests/Features/Filters/ValidateContentTypeFilterAttributeTests.cs b/...lth.Fhir.Shared.Api.UnitTests/Features/Filters/ValidateContentTypeFilterAttributeTests.cs
@@ -381,7 +381,7 @@ public async Task GivenARequestWithAValidAcceptHeaderAndFormatOverride_WhenSetti
             Assert.Equal(applicationXml, context.HttpContext.Response.ContentType);
         }
 
-        private ValidateFormatParametersAttribute CreateFilter(IEnumerable<TextOutputFormatter> formatters = null)
+        private ValidateFormatParametersAttribute CreateFilter(IReadOnlyCollection<TextOutputFormatter> formatters = null)
         {
             if (formatters == null)
             {