Prefer IReadOnlyCollection<> or IReadOnlyList<> over IEnumerable<>

Closes wixtoolset/issues#6422

3 files changed, 5 insertions, 4 deletions

diff --git a/src/test/Example.Extension/ExamplePreprocessorExtensionAndCommandLine.cs b/src/test/Example.Extension/ExamplePreprocessorExtensionAndCommandLine.cs
index 49f68de5..7244798a 100644
--- a/src/test/Example.Extension/ExamplePreprocessorExtensionAndCommandLine.cs
+++ b/src/test/Example.Extension/ExamplePreprocessorExtensionAndCommandLine.cs
@@ -12,7 +12,7 @@ namespace Example.Extension
     {
         private string exampleValueFromCommandLine;
 
-        public IEnumerable<ExtensionCommandLineSwitch> CommandLineSwitches => throw new NotImplementedException();
+        public IReadOnlyCollection<ExtensionCommandLineSwitch> CommandLineSwitches => throw new NotImplementedException();
 
         public ExamplePreprocessorExtensionAndCommandLine()
         {
diff --git a/src/test/Example.Extension/ExampleWindowsInstallerBackendExtension.cs b/src/test/Example.Extension/ExampleWindowsInstallerBackendExtension.cs
index 87b7855c..afccc56f 100644
--- a/src/test/Example.Extension/ExampleWindowsInstallerBackendExtension.cs
+++ b/src/test/Example.Extension/ExampleWindowsInstallerBackendExtension.cs
@@ -9,7 +9,7 @@ namespace Example.Extension
 
     internal class ExampleWindowsInstallerBackendExtension : BaseWindowsInstallerBackendBinderExtension
     {
-        public override IEnumerable<TableDefinition> TableDefinitions => ExampleTableDefinitions.All;
+        public override IReadOnlyCollection<TableDefinition> TableDefinitions => ExampleTableDefinitions.All;
 
         public override bool TryProcessSymbol(IntermediateSection section, IntermediateSymbol symbol, WindowsInstallerData output, TableDefinitionCollection tableDefinitions)
         {
diff --git a/src/test/WixToolsetTest.CoreIntegration/LinkerFixture.cs b/src/test/WixToolsetTest.CoreIntegration/LinkerFixture.cs
index 676d7d87..cfe4d3f1 100644
--- a/src/test/WixToolsetTest.CoreIntegration/LinkerFixture.cs
+++ b/src/test/WixToolsetTest.CoreIntegration/LinkerFixture.cs
@@ -3,6 +3,7 @@
 
 namespace WixToolsetTest.CoreIntegration
 {
+    using System;
     using System.IO;
     using System.Linq;
     using WixBuildTools.TestSupport;
@@ -29,8 +30,8 @@ namespace WixToolsetTest.CoreIntegration
 
             var creator = serviceProvider.GetService<ISymbolDefinitionCreator>();
             var context = serviceProvider.GetService<ILinkContext>();
-            context.Extensions = Enumerable.Empty<WixToolset.Extensibility.ILinkerExtension>();
-            context.ExtensionData = Enumerable.Empty<WixToolset.Extensibility.IExtensionData>();
+            context.Extensions = Array.Empty<WixToolset.Extensibility.ILinkerExtension>();
+            context.ExtensionData = Array.Empty<WixToolset.Extensibility.IExtensionData>();
             context.Intermediates = new[] { intermediate1, intermediate2 };
             context.SymbolDefinitionCreator = creator;