1338 commits

This branch

This branch

All branches

Author	SHA1	Message	Date
Winni Neessen	425a190eb1	fumpt'ed formatting	2024-10-25 11:36:05 +02:00
Winni Neessen	64aeb683ba	Format test cases for consistency ... Reformat test cases in `client_test.go` for better readability and consistency by converting single-line struct definitions into multi-line blocks. This change improves code maintainability and aligns the formatting across similar tests.	2024-10-25 11:35:25 +02:00
Winni Neessen	1dba76948f	Simplify test descriptions in msg_test.go ... Shortened the test case descriptions for better clarity and readability. Added a new test function 'TestMsg_Encoding' to verify encoding outputs.	2024-10-25 11:24:35 +02:00
Winni Neessen	120c2efd08	Refactor test cases with shared test data ... Consolidated repetitive test data into shared variables to improve code maintainability and readability. This modification also helps in reducing redundancy and streamlining future updates to test data.	2024-10-25 11:18:09 +02:00
Winni Neessen	c4946af3ab	Refactor msg_test.go to streamline tests and reduce redundancy ... Reorganized and condensed test cases for the NewMsg method by merging individual tests and removing obsolete functions. This improves maintainability and reduces duplication in the test suite.	2024-10-25 11:08:11 +02:00
Winni Neessen	64cfbf9e46	Rename test functions and add new header cases. ... Renamed test functions from `TestHeader_String` to `TestHeader_Stringer` and `TestAddrHeader_String` to `TestAddrHeader_Stringer` for consistency. Additionally, added new header cases such as "Content-Description" and "X-Auto-Response-Suppress" to improve test coverage.	2024-10-25 09:53:49 +02:00
Winni Neessen	c58d52e49a	Refactor TestImportance_StringFuncs in header_test.go ... Separated test cases for String, NumString, and XPrioString methods of the Importance object into distinct sub-tests. Improved readability and maintainability by grouping similar assertions together.	2024-10-25 09:48:00 +02:00
Winni Neessen	eebbaa2513	Refactor and reintegrate content type tests in file_test.go ... Reintegrates previously commented out tests for file content types, ensuring they are part of a structured t.Run block. Enhances readability and maintains functionality by checking the presence and correctness of attachments in a more concise manner.	2024-10-25 09:40:15 +02:00
Winni Neessen	8353b4b255	Follow upstream for HELO during Quit bug ... I reported the bug I fixed in 74fa3f6f62 to Go upstream. They fixed simpler by just ignoring the error (See: https://go.dev/cl/622476). We follow this patch accordingly. The upstream test has been adopted as well.	2024-10-25 09:33:45 +02:00
Winni Neessen	9834c6508d	Refactor file_test.go to use subtests ... Consolidate multiple test functions into a single TestFile function using subtests. This improves test organization and enhances readability. Added a new test case for testing file name attachment.	2024-10-24 17:09:55 +02:00
Winni Neessen	75e035c783	Add test cases for various EML parsing scenarios ... Introduced a series of test cases for validating EML parsing against different edge cases, including invalid headers, content types, and transfer encodings. Ensured both valid and invalid EML strings are covered to thoroughly test the robustness of the parser.	2024-10-24 16:42:11 +02:00
Winni Neessen	769783f037	Refactor error handling in eml parser ... Removed redundant error checking in address parsing as netmail.ParseAddressList already performs necessary checks. Added a default error return for unsupported content disposition types to improve robustness.	2024-10-24 16:42:00 +02:00
Winni Neessen	9f1e1976fe	Fix assignment in error handling for EML parsing function ... Correct the variable assignment in the `if err` statement to ensure proper error handling. This change eliminates a potential bug where the wrong variable might be used.	2024-10-24 14:49:24 +02:00
Winni Neessen	887e3cd768	Add EML parsing with new tests and examples ... Introduce new EML files with valid and invalid examples. Implement tests for EML parsing from readers and files, checking for both successful parsing and expected failures on invalid inputs.	2024-10-24 13:36:03 +02:00
Winni Neessen	127cfdf2bc	Fix error variable declaration in eml.go ... The error variable declaration has been corrected from "if err := parseEML(...)" to "if err = parseEML(...)". This change ensures consistency with the rest of the error handling code in the file.	2024-10-24 13:20:09 +02:00
Winni Neessen	7ed23bf01b	Remove outdated client test cases ... Removed obsolete and redundant client test cases that were no longer relevant. This cleanup improves code maintainability and readability by eliminating excessive, unused test methods.	2024-10-24 12:53:37 +02:00
Winni Neessen	0310527eb5	Completed client.go tests ... We've now covered 96% of all code. Everything else is not testable for us at this point.	2024-10-24 12:25:13 +02:00
Winni Neessen	1399a3331a	Refactor and extend client email tests ... Refactor existing email sending tests by organizing multiple edge cases and adding robust test coverage. This includes adding checks for invalid sender/recipient addresses, handling DSN support, and ensuring proper client server interactions during failures like DATA init, DATA close, and MAIL FROM.	2024-10-24 12:03:56 +02:00
Winni Neessen	45ebcb95b3	Remove redundant connection check in send function. ... The connection check is performed in the c.Reset call just before the c.checkconn call, making this redundant. Removing it simplifies the code and eliminates unnecessary error handling for connection status. This change helps improve code maintainability.	2024-10-24 12:02:57 +02:00
Winni Neessen	1519522e5d	Reduce sleep duration in client tests ... Decreased sleep time from 300ms to 30ms across multiple tests to improve test execution speed. Added a new test `TestClient_sendSingleMsg` to connect and send an email message, ensuring the robustness of the sending functionality.	2024-10-24 10:50:17 +02:00
Winni Neessen	3bf1992cab	Improve test conciseness and concurrency handling ... Simplified repeated message initialization by introducing a helper function `testMessage(t)`. Enhanced existing tests by adding robust concurrency tests and refined the structure of email sending scenarios.	2024-10-24 10:45:05 +02:00
Winni Neessen	4a8ac76636	Add nil check for smtpClient in checkConn function ... Previously, if smtpClient was nil, the checkConn function would not handle that case. This update ensures that an appropriate error is returned when smtpClient is nil, enhancing the robustness of the client connection checks.	2024-10-24 10:44:40 +02:00
Winni Neessen	5e3ebcc1a6	Remove redundant connection check in auth function ... The checkConn call in the auth function was redundant because the connection is already managed appropriately elsewhere. Removing this unnecessary check simplifies the code and avoids potential duplication of error handling.	2024-10-24 10:12:57 +02:00
Winni Neessen	040289cea4	Remove hardcoded test credentials and add new auth tests. ... Replaced hardcoded SMTP credentials with generic placeholders for improved security. Added new test cases to handle unsupported authentication methods and connections without TLS.	2024-10-24 10:12:43 +02:00
Winni Neessen	2a2176d700	Add tests for various SMTP authentication methods ... Implemented new test cases for different SMTP authentication methods including PLAIN, LOGIN, XOAUTH2, and various SCRAM mechanisms. These tests ensure that the client can correctly handle both successful and failing authentication scenarios, as well as unsupported authentication types.	2024-10-24 09:59:31 +02:00
Winni Neessen	e442419c18	Remove redundant connection check in tls function ... The `tls` function in `client.go` no longer checks for an active connection before executing. This simplifies the code since the connection check is either redundant or already handled elsewhere in the flow.	2024-10-24 09:19:08 +02:00
Winni Neessen	28dc629674	Refactor and expand client dial-and-send tests ... Renamed TestClient_DialAndSend to TestClient_DialAndSendWithContext and reorganized message setup. Added multiple test cases to cover different failure points in the DialAndSendWithContext method.	2024-10-24 00:50:16 +02:00
Winni Neessen	84ca70083a	Add test for DialAndSend functionality ... Introduce a new test `TestClient_DialAndSend` to validate the process of dialing and sending an email using the SMTP client. This includes setting up a mock SMTP server and verifying the process from message creation to sending.	2024-10-24 00:30:57 +02:00
Winni Neessen	06f6fd3692	Add client reset functionality tests ... Introduce tests for the client reset functionality, including scenarios to test successful reset, reset on a disconnected client, and reset with server failure. This ensures robustness and reliability of the client reset feature under various conditions.	2024-10-24 00:22:33 +02:00
Winni Neessen	2710250baa	Add *testing.T to simpleSMTPServer and logging improvements ... Pass `*testing.T` to `simpleSMTPServer` for enhanced test logging and helper methods. This allows better integration with the testing framework, converting standard log outputs to `t.Logf` for improved test diagnostics and error reporting.	2024-10-24 00:11:58 +02:00
Winni Neessen	7f3cd8dc38	Merge branch 'main' into feature/overhaul-tests	2024-10-23 23:52:13 +02:00
Winni Neessen	cf1246d9ea	Remove redundant DialWithContext test cases ... Deleted two test functions for DialWithContext that tested invalid HELO and authentication scenarios. These tests were deemed redundant as the error handling for these cases is covered elsewhere.	2024-10-23 23:34:13 +02:00
Winni Neessen	c63b8b124e	Add STARTTLS and SSL test cases for SMTP client ... Extended test cases to include scenarios for STARTTLS and SSL connections. This includes handling TLS configurations, testing certificate handling, and tests for various authentication methods under TLS.	2024-10-23 23:33:33 +02:00
Winni Neessen	563ccbab4a	Fix typo in comment within Quit function ... Corrected grammar in a comment to enhance code readability and maintain consistency. This change does not affect the functionality of the `Quit` method.	2024-10-23 22:26:08 +02:00
Winni Neessen	ea57644a8e	Add debug logging to client creation in tests ... Introduce `WithDebugLog()` in client creation for enhanced logging during tests. Correct handling of certain SMTP command errors by replacing `return` with `break` in switch cases for proper loop continuation.	2024-10-23 22:20:32 +02:00
Winni Neessen	74fa3f6f62	Fix Quit handling when initial HELO fails ... Ensure QUIT command can be sent even if initial HELO fails. Added a check to skip retrying HELO if it already failed, allowing for proper closing of the connection. This prevents potential hangs or errors during connection termination.	2024-10-23 21:54:06 +02:00
Winni Neessen	572751ac10	Add test for invalid HELO handling in SMTP client ... Introduce a new test case to ensure the SMTP client fails gracefully when an invalid HELO command is used. This includes validating error handling and maintaining the client connection integrity. Also, optimize EHLO/HELO command handling by enhancing syntax checking and error response generation.	2024-10-23 18:20:52 +02:00
Winni Neessen	d281f838d4	Add integration tests for invalid host and HELO failure ... Add test cases to validate client behavior on connecting with an invalid host and encountering a HELO failure. This helps ensure the client handles these error scenarios correctly, maintaining robustness and reliability.	2024-10-23 18:08:30 +02:00
Winni Neessen	0db1383940	Refactor buffer initialization in client tests ... Replaced &bytes.Buffer{} with bytes.NewBuffer(nil) for buffer initialization in various client tests. This change ensures more idiomatic and consistent creation of byte buffers throughout the test cases.	2024-10-23 17:58:08 +02:00
Winni Neessen	21184e60b9	Switch to using bytes.NewBuffer(nil) in tests ... Refactored the test code to initialize buffers using bytes.NewBuffer(nil) instead of &bytes.Buffer{}. This change ensures a consistent and idiomatic initialization of byte buffers throughout the test cases.	2024-10-23 17:57:09 +02:00
Winni Neessen	12695385e8	Refactor SMTP server test setup to use serverProps struct ... Consolidate server configuration properties into a new serverProps struct for better code clarity and future extensibility. This change involves updating simpleSMTPServer and test cases to use the new struct, allowing more control over server behavior during tests.	2024-10-23 17:55:31 +02:00
Winni Neessen	8a6cd2b448	Add and update client tests ... Reintroduces and enhances several client tests including `SetLogAuthData`, `Close`, `DialWithContext`, and others. Deprecated and non-functional tests are removed, and a new log parsing method is added for enhanced logging validation.	2024-10-23 17:34:23 +02:00
Winni Neessen	ae7160ddba	Refactor SMTP Auth unit test with table-driven approach ... Replaced the single test case for setting custom SMTP authentication with a table-driven approach. This refactor improves test coverage by including multiple authentication methods such as CRAM-MD5, LOGIN, PLAIN, SCRAM, and XOAUTH2.	2024-10-23 16:46:18 +02:00
Winni Neessen	d4dc212dd3	Refactor client tests and add SetSMTPAuthCustom test ... Removed outdated tests for TLSConfig, Username, Password, and SMTPAuth that were commented out. Added a new test for the SetSMTPAuthCustom method, specifically for PLAIN authentication, ensuring proper behavior and type checks.	2024-10-23 16:14:05 +02:00
Winni Neessen	17cb590a45	Add test for Client.SetSMTPAuth function ... Introduce a new test to validate the functionality of the Client.SetSMTPAuth method. This test covers various SMTP authentication types and ensures that the method correctly sets the expected authentication type while also verifying the override behavior for different custom and default authentications.	2024-10-23 15:44:21 +02:00
Winni Neessen	cec7e38332	Merge pull request #345 from wneessen/dependabot/github_actions/github/codeql-action-3.27.0 ... Bump github/codeql-action from 3.26.13 to 3.27.0	2024-10-23 15:32:09 +02:00
Winni Neessen	c946f74ad2	Add unit tests for Client TLS, Username, and Password methods ... Introduced unit tests for Client's SetTLSConfig, SetUsername, and SetPassword methods. The tests cover various scenarios, including setting valid configurations, handling nil inputs, and overriding previous settings. This improves our test coverage and ensures the reliability of these methods.	2024-10-23 15:31:00 +02:00
dependabot[bot]	9ad77012e3	Bump github/codeql-action from 3.26.13 to 3.27.0 ... Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.26.13 to 3.27.0. - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](f779452ac5...662472033e) --- updated-dependencies: - dependency-name: github/codeql-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>	2024-10-23 13:16:56 +00:00
Winni Neessen	68bc5dde72	Add comprehensive tests for Client TLS and debugging features ... Implemented detailed test cases for various TLS policies and settings, port configurations, and SSL options. Also added tests for debugging functionalities of the Client, enabling robust validation of different scenarios.	2024-10-23 15:15:59 +02:00
Winni Neessen	3efd2b529f	Set fallbackPort to 0 in SetTLSPortPolicy ... The default fallback port should be reset to 0 when setting the TLS port policy. This ensures the fallbackPort does not retain an incorrect value when switching policies.	2024-10-23 14:39:28 +02:00