Skip to content

Commit 4409d36

Browse files
committed
Fixed Tests
1 parent 6af7574 commit 4409d36

File tree

2 files changed

+2
-9
lines changed

2 files changed

+2
-9
lines changed

System.IO.Abstractions.TestingHelpers.Tests/MockFileStreamTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,6 @@ public void MockFileStream_Constructor_ReadTypeNotWritable()
8282
[TestCase(FileShare.None, MockFileStream.StreamType.WRITE)]
8383
[TestCase(FileShare.Read, MockFileStream.StreamType.WRITE)]
8484
[TestCase(FileShare.Write, MockFileStream.StreamType.READ)]
85-
[TestCase(FileShare.Write, MockFileStream.StreamType.WRITE)]
8685
public void MockFileStream_Constructor_Insufficient_FileShare_Throws_Exception(FileShare allowedFileShare, MockFileStream.StreamType streamType)
8786
{
8887
var filePath = @"C:\locked.txt";
@@ -97,6 +96,7 @@ public void MockFileStream_Constructor_Insufficient_FileShare_Throws_Exception(F
9796

9897
[Test]
9998
[TestCase(FileShare.Read, MockFileStream.StreamType.READ)]
99+
[TestCase(FileShare.Read | FileShare.Write, MockFileStream.StreamType.APPEND)]
100100
[TestCase(FileShare.ReadWrite, MockFileStream.StreamType.READ)]
101101
[TestCase(FileShare.ReadWrite, MockFileStream.StreamType.WRITE)]
102102
public void MockFileStream_Constructor_Sufficient_FileShare_Does_Not_Throw_Exception(FileShare allowedFileShare, MockFileStream.StreamType streamType)

System.IO.Abstractions.TestingHelpers/MockFileStream.cs

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ public MockFileStream(
4343
if (mockFileDataAccessor.FileExists(path))
4444
{
4545
var fileData = mockFileDataAccessor.GetFile(path);
46-
CheckFileAccess(path, streamType, fileData);
46+
fileData.CheckFileAccess(path, streamType != StreamType.READ ? FileAccess.Write : FileAccess.Read);
4747

4848
/* only way to make an expandable MemoryStream that starts with a particular content */
4949
var data = fileData.Contents;
@@ -67,13 +67,6 @@ public MockFileStream(
6767
canWrite = streamType != StreamType.READ;
6868
}
6969

70-
private static void CheckFileAccess(string path, StreamType streamType, MockFileData fileData)
71-
{
72-
fileData.CheckFileAccess(path, FileAccess.Read);
73-
if (streamType != StreamType.READ)
74-
fileData.CheckFileAccess(path, FileAccess.Write);
75-
}
76-
7770
public override bool CanWrite => canWrite;
7871

7972
#if NET40

0 commit comments

Comments
 (0)