Skip to content

Commit 2c8fb45

Browse files
authored
refactor: fix build warnings in test projects (#1203)
Fix build warnings in test projects.
1 parent d2ae767 commit 2c8fb45

File tree

6 files changed

+22
-0
lines changed

6 files changed

+22
-0
lines changed

tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/MockDirectoryInfoTests.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -518,7 +518,9 @@ public void Directory_exists_after_creation_with_security()
518518
var directoryInfo = fileSystem.DirectoryInfo.New(XFS.Path(@"c:\abc"));
519519

520520
// Act
521+
#pragma warning disable CA1416
521522
directoryInfo.Create(new DirectorySecurity());
523+
#pragma warning restore CA1416
522524

523525
// Assert
524526
Assert.That(directoryInfo.Exists, Is.True);

tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/MockDirectoryTests.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2085,7 +2085,9 @@ public void MockDirectory_GetAccessControl_ShouldThrowExceptionOnDirectoryNotFou
20852085
var fileSystem = new MockFileSystem();
20862086

20872087
// Act
2088+
#pragma warning disable CA1416
20882089
Assert.Throws<DirectoryNotFoundException>(() => fileSystem.Directory.GetAccessControl(XFS.Path(@"c:\foo")));
2090+
#pragma warning restore CA1416
20892091
}
20902092

20912093
[Test]

tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/MockFileArgumentPathTests.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,10 @@ private static IEnumerable<Action<IFile>> GetFileSystemActionsForArgumentNullExc
5151
yield return fs => fs.SetLastAccessTimeUtc((string)null, DateTime.Now);
5252
yield return fs => fs.SetLastWriteTime((string)null, DateTime.Now);
5353
yield return fs => fs.SetLastWriteTimeUtc((string)null, DateTime.Now);
54+
#pragma warning disable CA1416
5455
yield return fs => fs.Decrypt(null);
5556
yield return fs => fs.Encrypt(null);
57+
#pragma warning restore CA1416
5658
}
5759

5860
[TestCaseSource(nameof(GetFileSystemActionsForArgumentNullException))]

tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/MockFileInfoTests.cs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -499,7 +499,10 @@ public void MockFileInfo_OpenRead_ShouldReturnByteContentOfFile()
499499
byte[] result = new byte[2];
500500
using (var stream = fileInfo.OpenRead())
501501
{
502+
#pragma warning disable CA2022
503+
// ReSharper disable once MustUseReturnValue
502504
stream.Read(result, 0, 2);
505+
#pragma warning restore CA2022
503506
}
504507

505508
Assert.That(result, Is.EqualTo(new byte[] { 1, 2 }));

tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/MockFileOpenTests.cs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -247,7 +247,9 @@ public void MockFile_Read_ShouldRetainCreationTimeAndUpdateLastAccessTime()
247247
var fi = fs.FileInfo.New(filepath);
248248
var stream = fi.OpenRead();
249249
var buffer = new byte[16];
250+
#pragma warning disable CA2022
250251
stream.Read(buffer, 0, buffer.Length);
252+
#pragma warning restore CA2022
251253
fi.Refresh();
252254
// Assert
253255
Assert.That(fi.CreationTime, Is.EqualTo(creationTime));
@@ -270,7 +272,12 @@ public async Task MockFile_ReadAsync_ShouldRetainCreationTimeAndUpdateLastAccess
270272
var fi = fs.FileInfo.New(filepath);
271273
var stream = fi.OpenRead();
272274
var buffer = new byte[16];
275+
#pragma warning disable CA1835
276+
#pragma warning disable CA2022
277+
// ReSharper disable once MustUseReturnValue
273278
await stream.ReadAsync(buffer, 0, buffer.Length);
279+
#pragma warning restore CA2022
280+
#pragma warning restore CA1835
274281
fi.Refresh();
275282
// Assert
276283
Assert.That(fi.CreationTime, Is.EqualTo(creationTime));

tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/MockFileTests.cs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -605,7 +605,9 @@ public void MockFile_Encrypt_ShouldSetEncryptedAttribute()
605605
});
606606

607607
// Act
608+
#pragma warning disable CA1416
608609
fileSystem.File.Encrypt(filePath);
610+
#pragma warning restore CA1416
609611
var attributes = fileSystem.File.GetAttributes(filePath);
610612

611613
// Assert
@@ -623,10 +625,14 @@ public void MockFile_Decrypt_ShouldRemoveEncryptedAttribute()
623625
{
624626
{filePath, fileData }
625627
});
628+
#pragma warning disable CA1416
626629
fileSystem.File.Encrypt(filePath);
630+
#pragma warning restore CA1416
627631

628632
// Act
633+
#pragma warning disable CA1416
629634
fileSystem.File.Decrypt(filePath);
635+
#pragma warning restore CA1416
630636
var attributes = fileSystem.File.GetAttributes(filePath);
631637

632638
// Assert

0 commit comments

Comments
 (0)