refactor: extract ACL related functionality

src/System.IO.Abstractions.TestingHelpers/FileSystemExtensibility.cs

@@ -0,0 +1,32 @@
+using System.Collections.Generic;
+
+namespace System.IO.Abstractions.TestingHelpers
+{
+    internal class FileSystemExtensibility
+    {
+        private readonly Dictionary<string, object> _metadata = new();
+
+        public virtual void StoreMetadata(string key, object value)
+        {
+            _metadata[key] = value;
+        }
+
+        public virtual object RetrieveMetadata(string key)
+        {
+            if (_metadata.TryGetValue(key, out object value))
+            {
+                return value;
+            }
+
+            return default;
+        }
+
+        internal void CloneFrom(FileSystemExtensibility template)
+        {
+            foreach (var item in template._metadata)
+            {
+                _metadata[item.Key] = item.Value;
+            }
+        }
+    }
+}

src/System.IO.Abstractions.TestingHelpers/MockDirectory.cs

@@ -1,9 +1,6 @@
 using System.Collections.Generic;
-using System.ComponentModel;
 using System.Globalization;
 using System.Linq;
-using System.Runtime.Versioning;
-using System.Security.AccessControl;
 using System.Text.RegularExpressions;
 
 namespace System.IO.Abstractions.TestingHelpers
@@ -39,17 +36,10 @@ public class MockDirectory : DirectoryBase
         /// <inheritdoc />
         public override IDirectoryInfo CreateDirectory(string path)
         {
-            return CreateDirectoryInternal(path, null);
+            return CreateDirectoryInternal(path);
         }
 
-
-        /// <inheritdoc />
-        public override IDirectoryInfo CreateDirectory(string path, DirectorySecurity directorySecurity)
-        {
-            return CreateDirectoryInternal(path, directorySecurity);
-        }
-
-        private IDirectoryInfo CreateDirectoryInternal(string path, DirectorySecurity directorySecurity)
+        private IDirectoryInfo CreateDirectoryInternal(string path)
         {
             if (path == null)
             {
@@ -80,11 +70,6 @@ private IDirectoryInfo CreateDirectoryInternal(string path, DirectorySecurity di
 
             var created = new MockDirectoryInfo(mockFileDataAccessor, path);
 
-            if (directorySecurity != null)
-            {
-                created.SetAccessControl(directorySecurity);
-            }
-
             return created;
         }
 
@@ -173,31 +158,6 @@ public override bool Exists(string path)
         }
 
 
-        /// <inheritdoc />
-        [SupportedOSPlatform("windows")]
-        public override DirectorySecurity GetAccessControl(string path)
-        {
-            mockFileDataAccessor.PathVerifier.IsLegalAbsoluteOrRelative(path, "path");
-            path = path.TrimSlashes();
-
-            if (!mockFileDataAccessor.Directory.Exists(path))
-            {
-                throw CommonExceptions.CouldNotFindPartOfPath(path);
-            }
-
-            var directoryData = (MockDirectoryData)mockFileDataAccessor.GetFile(path);
-            return directoryData.AccessControl;
-        }
-
-
-        /// <inheritdoc />
-        [SupportedOSPlatform("windows")]
-        public override DirectorySecurity GetAccessControl(string path, AccessControlSections includeSections)
-        {
-            return GetAccessControl(path);
-        }
-
-
         /// <inheritdoc />
         public override DateTime GetCreationTime(string path)
         {
@@ -559,22 +519,6 @@ public override IFileSystemInfo ResolveLinkTarget(string linkPath, bool returnFi
 
 #endif
 
-        /// <inheritdoc />
-        [SupportedOSPlatform("windows")]
-        public override void SetAccessControl(string path, DirectorySecurity directorySecurity)
-        {
-            mockFileDataAccessor.PathVerifier.IsLegalAbsoluteOrRelative(path, "path");
-            path = path.TrimSlashes();
-
-            if (!mockFileDataAccessor.Directory.Exists(path))
-            {
-                throw CommonExceptions.CouldNotFindPartOfPath(path);
-            }
-
-            var directoryData = (MockDirectoryData)mockFileDataAccessor.GetFile(path);
-            directoryData.AccessControl = directorySecurity;
-        }
-
         /// <inheritdoc />
         public override void SetCreationTime(string path, DateTime creationTime)
         {

src/System.IO.Abstractions.TestingHelpers/MockDirectoryData.cs

@@ -7,10 +7,6 @@ namespace System.IO.Abstractions.TestingHelpers
     [Serializable]
     public class MockDirectoryData : MockFileData
     {
-
-        [NonSerialized]
-        private DirectorySecurity accessControl;
-
         /// <inheritdoc />
         public MockDirectoryData() : base(string.Empty)
         {
@@ -25,9 +21,16 @@ public new DirectorySecurity AccessControl
             {
                 // DirectorySecurity's constructor will throw PlatformNotSupportedException on non-Windows platform, so we initialize it in lazy way.
                 // This let's us use this class as long as we don't use AccessControl property.
-                return accessControl ?? (accessControl = new DirectorySecurity());
+                var value = Extensibility.RetrieveMetadata("AccessControl:DirectorySecurity");
+                if (value is not DirectorySecurity directorySecurity)
+                {
+                    directorySecurity = new DirectorySecurity();
+                    Extensibility.StoreMetadata("AccessControl:DirectorySecurity", directorySecurity);
+                }
+
+                return directorySecurity;
             }
-            set { accessControl = value; }
+            set { Extensibility.StoreMetadata("AccessControl:DirectorySecurity", value); }
         }
     }
 }

src/System.IO.Abstractions.TestingHelpers/MockDirectoryInfo.cs

@@ -1,5 +1,6 @@
 using System.Collections.Generic;
 using System.Linq;
+using System.Runtime.Versioning;
 using System.Security.AccessControl;
 
 namespace System.IO.Abstractions.TestingHelpers
@@ -8,7 +9,7 @@ namespace System.IO.Abstractions.TestingHelpers
 
     /// <inheritdoc />
     [Serializable]
-    public class MockDirectoryInfo : DirectoryInfoBase
+    public class MockDirectoryInfo : DirectoryInfoBase, IFileSystemAclSupport
     {
         private readonly IMockFileDataAccessor mockFileDataAccessor;
         private readonly string directoryPath;
@@ -26,6 +27,15 @@ public MockDirectoryInfo(IMockFileDataAccessor mockFileDataAccessor, string dire
         {
             this.mockFileDataAccessor = mockFileDataAccessor ?? throw new ArgumentNullException(nameof(mockFileDataAccessor));
 
+            if (directoryPath== null)
+            {
+                throw new ArgumentNullException("path", StringResources.Manager.GetString("VALUE_CANNOT_BE_NULL"));
+            }
+            if (directoryPath.Trim() == string.Empty)
+            {
+                throw CommonExceptions.PathIsNotOfALegalForm("path");
+            }
+
             originalPath = directoryPath;
             directoryPath = mockFileDataAccessor.Path.GetFullPath(directoryPath);
 
@@ -179,14 +189,7 @@ public override void Create()
             mockFileDataAccessor.Directory.CreateDirectory(FullName);
             refreshOnNextRead = true;
         }
-
-        /// <inheritdoc />
-        public override void Create(DirectorySecurity directorySecurity)
-        {
-            mockFileDataAccessor.Directory.CreateDirectory(FullName, directorySecurity);
-            refreshOnNextRead = true;
-        }
-
+
         /// <inheritdoc />
         public override IDirectoryInfo CreateSubdirectory(string path)
         {
@@ -277,19 +280,7 @@ public override IEnumerable<IFileSystemInfo> EnumerateFileSystemInfos(string sea
             return GetFileSystemInfos(searchPattern, enumerationOptions);
         }
 #endif
-
-        /// <inheritdoc />
-        public override DirectorySecurity GetAccessControl()
-        {
-            return mockFileDataAccessor.Directory.GetAccessControl(directoryPath);
-        }
-
-        /// <inheritdoc />
-        public override DirectorySecurity GetAccessControl(AccessControlSections includeSections)
-        {
-            return mockFileDataAccessor.Directory.GetAccessControl(directoryPath, includeSections);
-        }
-
+
         /// <inheritdoc />
         public override IDirectoryInfo[] GetDirectories()
         {
@@ -388,13 +379,7 @@ public override void MoveTo(string destDirName)
         {
             mockFileDataAccessor.Directory.Move(directoryPath, destDirName);
         }
-
-        /// <inheritdoc />
-        public override void SetAccessControl(DirectorySecurity directorySecurity)
-        {
-            mockFileDataAccessor.Directory.SetAccessControl(directoryPath, directorySecurity);
-        }
-
+
         /// <inheritdoc />
         public override IDirectoryInfo Parent
         {
@@ -435,5 +420,32 @@ public override string ToString()
         {
             return originalPath;
         }
+
+        /// <inheritdoc cref="IFileSystemAclSupport.GetAccessControl()" />
+        [SupportedOSPlatform("windows")]
+        public object GetAccessControl()
+        {
+            return GetMockDirectoryData().AccessControl;
+        }
+
+        /// <inheritdoc cref="IFileSystemAclSupport.GetAccessControl(IFileSystemAclSupport.AccessControlSections)" />
+        [SupportedOSPlatform("windows")]
+        public object GetAccessControl(IFileSystemAclSupport.AccessControlSections includeSections)
+        {
+            return GetMockDirectoryData().AccessControl;
+        }
+
+        /// <inheritdoc cref="IFileSystemAclSupport.SetAccessControl(object)" />
+        [SupportedOSPlatform("windows")]
+        public void SetAccessControl(object value)
+        {
+            GetMockDirectoryData().AccessControl = value as DirectorySecurity;
+        }
+
+        private MockDirectoryData GetMockDirectoryData()
+        {
+            return mockFileDataAccessor.GetFile(directoryPath) as MockDirectoryData
+                ?? throw CommonExceptions.CouldNotFindPartOfPath(directoryPath);
+        }
     }
 }

src/System.IO.Abstractions.TestingHelpers/MockFile.cs

@@ -1,8 +1,6 @@
 using System.Collections.Generic;
 using System.Globalization;
 using System.Linq;
-using System.Runtime.Versioning;
-using System.Security.AccessControl;
 using System.Text;
 
 namespace System.IO.Abstractions.TestingHelpers
@@ -268,28 +266,6 @@ public override bool Exists(string path)
             return false;
         }
 
-        /// <inheritdoc />
-        [SupportedOSPlatform("windows")]
-        public override FileSecurity GetAccessControl(string path)
-        {
-            mockFileDataAccessor.PathVerifier.IsLegalAbsoluteOrRelative(path, "path");
-
-            if (!mockFileDataAccessor.FileExists(path))
-            {
-                throw CommonExceptions.FileNotFound(path);
-            }
-
-            var fileData = mockFileDataAccessor.GetFile(path);
-            return fileData.AccessControl;
-        }
-
-        /// <inheritdoc />
-        [SupportedOSPlatform("windows")]
-        public override FileSecurity GetAccessControl(string path, AccessControlSections includeSections)
-        {
-            return GetAccessControl(path);
-        }
-
         /// <summary>
         /// Gets the <see cref="FileAttributes"/> of the file on the path.
         /// </summary>
@@ -732,22 +708,6 @@ public override IFileSystemInfo ResolveLinkTarget(string linkPath, bool returnFi
         }
 #endif
 
-        /// <inheritdoc />
-        [SupportedOSPlatform("windows")]
-        public override void SetAccessControl(string path, FileSecurity fileSecurity)
-        {
-            mockFileDataAccessor.PathVerifier.IsLegalAbsoluteOrRelative(path, "path");
-
-            if (!mockFileDataAccessor.FileExists(path))
-            {
-                throw CommonExceptions.FileNotFound(path);
-            }
-
-            var fileData = mockFileDataAccessor.GetFile(path);
-            mockFileDataAccessor.AdjustTimes(fileData, TimeAdjustments.LastAccessTime);
-            fileData.AccessControl = fileSecurity;
-        }
-
         /// <inheritdoc />
         public override void SetAttributes(string path, FileAttributes fileAttributes)
         {

src/System.IO.Abstractions.TestingHelpers/MockFileData.cs

@@ -34,10 +34,13 @@ public class MockFileData
         public static readonly DateTimeOffset DefaultDateTimeOffset = new DateTime(1601, 01, 01, 00, 00, 00, DateTimeKind.Utc);
 
         /// <summary>
-        /// The access control of the <see cref="MockFileData"/>.
+        /// The extensibility of the <see cref="MockFileData"/>.
         /// </summary>
+        internal FileSystemExtensibility Extensibility
+            => extensibility;
+
         [NonSerialized]
-        private FileSecurity accessControl;
+        private FileSystemExtensibility extensibility = new FileSystemExtensibility();
 
         /// <summary>
         /// Gets a value indicating whether the <see cref="MockFileData"/> is a directory or not.
@@ -98,8 +101,8 @@ public MockFileData(MockFileData template)
             {
                 throw new ArgumentNullException(nameof(template));
             }
-
-            accessControl = template.accessControl;
+            
+            extensibility.CloneFrom(template.extensibility);
             Attributes = template.Attributes;
             Contents = template.Contents.ToArray();
             CreationTime = template.CreationTime;
@@ -179,7 +182,7 @@ public DateTimeOffset LastWriteTime
         public FileAttributes Attributes { get; set; } = FileAttributes.Normal;
 
         /// <summary>
-        /// Gets or sets <see cref="FileSecurity"/> of the <see cref="MockFileData"/>. This is the object that is returned for this <see cref="MockFileData"/> when calling <see cref="FileBase.GetAccessControl(string)"/>.
+        /// Gets or sets <see cref="FileSecurity"/> of the <see cref="MockFileData"/>.
         /// </summary>
         [SupportedOSPlatform("windows")]
         public FileSecurity AccessControl
@@ -188,9 +191,16 @@ public FileSecurity AccessControl
             {
                 // FileSecurity's constructor will throw PlatformNotSupportedException on non-Windows platform, so we initialize it in lazy way.
                 // This let's us use this class as long as we don't use AccessControl property.
-                return accessControl ?? (accessControl = new FileSecurity());
+                var value = Extensibility.RetrieveMetadata("AccessControl:FileSecurity");
+                if (value is not FileSecurity fileSecurity)
+                {
+                    fileSecurity = new FileSecurity();
+                    Extensibility.StoreMetadata("AccessControl:FileSecurity", fileSecurity);
+                }
+
+                return fileSecurity;
             }
-            set { accessControl = value; }
+            set { Extensibility.StoreMetadata("AccessControl:FileSecurity", value); }
         }
 
         /// <summary>