Files
Copilot 41053cb80f Replace Assert.IsTrue with CollectionAssert.AreEqual for collection comparisons (#1730)
* Initial plan

* Refactor collection assertions to use CollectionAssert.AreEqual

Replace Assert.IsTrue(xxx.IsEqualTo(yyy)) and Assert.IsTrue(xxx.SequenceEqual(yyy))
with CollectionAssert.AreEqual(expected, actual) across 55 test files

Co-authored-by: Rob-Hague <5132141+Rob-Hague@users.noreply.github.com>

* Fix argument order in CollectionAssert.AreEqual and remove unnecessary using directives

- Fixed argument order in KeyExchangeDhGroupExchangeReplyTest.cs (expected first, actual second)
- Fixed argument order in KeyExchangeInitMessageTest.cs (expected first, actual second)
- Removed unnecessary 'using System.Linq' directives from all affected test files

Co-authored-by: Rob-Hague <5132141+Rob-Hague@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Rob-Hague <5132141+Rob-Hague@users.noreply.github.com>
Co-authored-by: Rob Hague <rob.hague00@gmail.com>
2026-06-26 17:44:36 +02:00

122 lines
4.6 KiB
C#

using System;
using System.Collections.Generic;
using System.Globalization;
using System.Text;
using Microsoft.VisualStudio.TestTools.UnitTesting;
using Renci.SshNet.Common;
using Renci.SshNet.Sftp;
using Renci.SshNet.Sftp.Requests;
using Renci.SshNet.Sftp.Responses;
namespace Renci.SshNet.Tests.Classes.Sftp.Requests.ExtendedRequests
{
[TestClass]
public class PosixRenameRequestTest
{
private uint _protocolVersion;
private uint _requestId;
private string _name;
private string _oldPath;
private byte[] _oldPathBytes;
private string _newPath;
private byte[] _newPathBytes;
private byte[] _nameBytes;
private Encoding _encoding;
[TestInitialize]
public void Init()
{
var random = new Random();
_encoding = Encoding.Unicode;
_protocolVersion = (uint)random.Next(0, int.MaxValue);
_requestId = (uint)random.Next(0, int.MaxValue);
_oldPath = random.Next().ToString(CultureInfo.InvariantCulture);
_oldPathBytes = _encoding.GetBytes(_oldPath);
_newPath = random.Next().ToString(CultureInfo.InvariantCulture);
_newPathBytes = _encoding.GetBytes(_newPath);
_name = "posix-rename@openssh.com";
_nameBytes = Encoding.UTF8.GetBytes(_name);
}
[TestMethod]
public void Constructor()
{
var request = new PosixRenameRequest(_protocolVersion, _requestId, _oldPath, _newPath, _encoding, null);
Assert.AreSame(_encoding, request.Encoding);
Assert.AreEqual(_name, request.Name);
Assert.AreEqual(_newPath, request.NewPath);
Assert.AreEqual(_oldPath, request.OldPath);
Assert.AreEqual(_protocolVersion, request.ProtocolVersion);
Assert.AreEqual(_requestId, request.RequestId);
Assert.AreEqual(SftpMessageTypes.Extended, request.SftpMessageType);
}
[TestMethod]
public void Complete_SftpStatusResponse()
{
IList<SftpStatusResponse> statusActionInvocations = new List<SftpStatusResponse>();
Action<SftpStatusResponse> statusAction = statusActionInvocations.Add;
var statusResponse = new SftpStatusResponse(_protocolVersion);
var request = new PosixRenameRequest(_protocolVersion, _requestId, _oldPath, _newPath, _encoding, statusAction);
request.Complete(statusResponse);
Assert.HasCount(1, statusActionInvocations);
Assert.AreSame(statusResponse, statusActionInvocations[0]);
}
[TestMethod]
public void GetBytes()
{
var request = new PosixRenameRequest(_protocolVersion, _requestId, _oldPath, _newPath, _encoding, null);
var bytes = request.GetBytes();
var expectedBytesLength = 0;
expectedBytesLength += 4; // Length
expectedBytesLength += 1; // Type
expectedBytesLength += 4; // RequestId
expectedBytesLength += 4; // Name length
expectedBytesLength += _nameBytes.Length; // Name
expectedBytesLength += 4; // OldPath length
expectedBytesLength += _oldPathBytes.Length; // OldPath
expectedBytesLength += 4; // NewPath length
expectedBytesLength += _newPathBytes.Length; // NewPath
Assert.HasCount(expectedBytesLength, bytes);
var sshDataStream = new SshDataStream(bytes);
Assert.AreEqual((uint)bytes.Length - 4, sshDataStream.ReadUInt32());
Assert.AreEqual((byte)SftpMessageTypes.Extended, sshDataStream.ReadByte());
Assert.AreEqual(_requestId, sshDataStream.ReadUInt32());
Assert.AreEqual((uint)_nameBytes.Length, sshDataStream.ReadUInt32());
var actualNameBytes = new byte[_nameBytes.Length];
_ = sshDataStream.Read(actualNameBytes, 0, actualNameBytes.Length);
CollectionAssert.AreEqual(_nameBytes, actualNameBytes);
Assert.AreEqual((uint)_oldPathBytes.Length, sshDataStream.ReadUInt32());
var actualOldPath = new byte[_oldPathBytes.Length];
_ = sshDataStream.Read(actualOldPath, 0, actualOldPath.Length);
CollectionAssert.AreEqual(_oldPathBytes, actualOldPath);
Assert.AreEqual((uint)_newPathBytes.Length, sshDataStream.ReadUInt32());
var actualNewPath = new byte[_newPathBytes.Length];
_ = sshDataStream.Read(actualNewPath, 0, actualNewPath.Length);
CollectionAssert.AreEqual(_newPathBytes, actualNewPath);
Assert.IsTrue(sshDataStream.IsEndOfData);
}
}
}