test: fix and add client

[IHtDzenda]

No description provided.

[JerrySB03]

I'm not an idiot

[IHtDzenda]

Meow

[Copilot]

Pull Request Overview

This PR removes legacy encryption tests, adds client messaging tests, and updates visibility on the Client class.

• Deleted outdated CryptTests to focus on client functionality
• Added ClientTests for establishing connections and sending messages
• Made Client class public and removed encryption/decryption methods from Program.cs

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File	Description
tests/CryptTests.cs	Removed encryption-related tests
tests/ClientTests.cs	Added tests for client messaging functionality
sutor-aes/Program.cs	Removed encryption/decryption methods from Program
sutor-aes/Client.cs	Made Client class public

Comments suppressed due to low confidence (2)

sutor-aes/Program.cs:15

• The encryption and decryption methods were removed, which breaks the library’s core crypto functionality. If this removal is unintentional, restore these methods or relocate them to a dedicated crypto service class.

public static string DecryptString(byte[] cipherText, byte[] key, byte[] iv)

tests/ClientTests.cs:8

• The test class is named ProgramTests but lives in ClientTests.cs and tests the Client behavior. Rename it to ClientTests for consistency.

public class ProgramTests

[Copilot]

_key and _iv are initialized in Setup but never used in this test. Consider removing them to simplify the test fixture.

Suggested change

	private byte[] _key;
	private byte[] _iv;
	[SetUp]
	public void Setup()
	{
	_key = new byte[32];
	Random random = new Random();
	random.NextBytes(_key);
	_iv = new byte[16];
	random.NextBytes(_iv);
	// Removed unused variables _key and _iv
	[SetUp]
	public void Setup()
	{
	// Removed initialization of unused variables _key and _iv

[Copilot]

The variable recieved is misspelled; correct it to received.

Suggested change

	bool recieved = false;
	alice.OnMessage((string msg) => recieved = msg == originalText);
	bob.SendMessage(alice, originalText);
	Assert.That(recieved);
	bool received = false;
	alice.OnMessage((string msg) => received = msg == originalText);
	bob.SendMessage(alice, originalText);
	Assert.That(received);

[Copilot]

This assertion lacks a constraint and will not compile under NUnit. Use Assert.That(received, Is.True) or Assert.IsTrue(received) to explicitly check the boolean.

Suggested change

	Assert.That(recieved);
	Assert.IsTrue(recieved);

[JerrySB03]

Copilot is an idiot, this works