1237 commits

This branch

This branch

All branches

Author	SHA1	Message	Date
Winni Neessen	466c2892bf	Add AttachReadSeeker tests for Unix and general cases ... Introduce tests for the AttachReadSeeker method, detailing scenarios for general use and Unix-specific paths. Ensure file handling and error conditions are properly validated to maintain robustness.	2024-10-28 09:34:05 +01:00
Winni Neessen	e7e0fe03bb	Remove redundant OS version from CI matrix ... This commit eliminates the duplication of the '14.1' OS version in the CI workflow matrix configuration. Simplifying the matrix helps in avoiding redundant test runs and ensures a clearer CI process.	2024-10-27 21:00:22 +01:00
Winni Neessen	c0c4049964	Enable Unix write tests and fix OS version matrix. ... Added a test environment variable to control Unix write tests and included their setup in the test function. Also corrected the OS version matrix by uncommenting '14.0' and '13.4'.	2024-10-27 20:59:24 +01:00
Winni Neessen	6a43cf4aaf	Rename and update tests for UNIX-specific functionality ... Renamed `msg_nowin_test.go` to `msg_unix_test.go` to better reflect its purpose. Updated the file attachment test to use `/dev/mem` for Unix environments instead of a temporary file with restricted permissions, ensuring compatibility with continuous integration environments.	2024-10-27 20:46:16 +01:00
Winni Neessen	e74adb8b90	Update CI configuration for FreeBSD ... Commented out older OS versions and added no copyback option. Adjusted the go test run step to include workspace directory change.	2024-10-27 18:22:22 +01:00
Winni Neessen	9682755e25	Delete unused .cirrus.yml configuration file ... The .cirrus.yml file was removed as it is no longer needed for our CI/CD processes. This helps clean up the repository and avoids confusion over unused configurations.	2024-10-27 18:03:09 +01:00
Winni Neessen	8fb05a33ff	Update tempfile path in unit tests ... Changed tempfile creation path to "testdata/tmp" for better test organization and to ensure temporary files are stored within the testdata directory. This change affects the TestMsg_AttachFile_unixOnly tests.	2024-10-27 18:02:37 +01:00
Winni Neessen	7b9df7de47	Remove harden-runner step from CI workflow ... The harden-runner step has been removed from the FreeBSD testing matrix in the CI workflow. This change simplifies the workflow and removes an additional security auditing step.	2024-10-27 17:55:12 +01:00
Winni Neessen	bdffa22ad8	Remove duplicate Go test step from CI workflow ... Eliminated a redundant "Run go test" step from the GitHub Actions CI configuration. This helps streamline the workflow and avoid unnecessary repetitions in the CI process.	2024-10-27 17:50:36 +01:00
Winni Neessen	89f29b241e	Add FreeBSD testing to CI workflow ... Included FreeBSD versions 13.4, 14.0, and 14.1 in the CI pipeline. Configured the workflow to harden the runner, checkout code, and run tests on the FreeBSD virtual machine.	2024-10-27 17:49:04 +01:00
Winni Neessen	eed3dec7d6	Clean up redundant error checks in msg_nowin_test.go ... Removed unnecessary assertions for specific error types in test cases. This streamlines the tests, maintaining focus on the primary failure conditions without checking the error type.	2024-10-27 16:19:16 +01:00
Winni Neessen	3be41b1aea	Fix chmod notation to use octal literal syntax ... Updated the chmod call in msg_nowin_test.go to use the consistent and proper octal literal syntax (0o000) instead of 0000. This change improves code readability and aligns with Go convention.	2024-10-27 16:10:46 +01:00
Winni Neessen	f7cfe5289a	Remove obsolete tests and add error handling test ... Removed commented-out tests to clean up the codebase. Added a new test to verify behavior when the file is closed early during attachment, ensuring robustness.	2024-10-27 16:08:50 +01:00
Winni Neessen	946ad294ce	Add tests for Msg attachment handling ... Introduce new unit tests for verifying the `AttachReader` method and enhancing error handling in attachment functions. These tests cover scenarios for success, file permissions, and early file closure errors.	2024-10-27 15:54:39 +01:00
Winni Neessen	0cf636ee9b	Add test for AttachFile method and change Skip to Log ... Replace t.Skip with t.Log to avoid skipping tests and ensure they log their status. Introduce a comprehensive test for the AttachFile method, covering cases with existing, non-existent files, and using options.	2024-10-27 15:28:42 +01:00
Winni Neessen	84f9d0583d	Refactor buffer initialization to use bytes.NewBuffer ... Replaced direct bytes.Buffer{} initialization with bytes.NewBuffer(nil) for better consistency and potential performance improvements. This refactor affects both the HTML and plain text alternative writers.	2024-10-27 15:17:30 +01:00
Winni Neessen	254dc81706	Add tests for AddAlternativeTextTemplate method ... Include various scenarios like default, with existing body string, nil template, and invalid template. These tests help in validating the addition of alternative text templates to messages.	2024-10-27 15:16:49 +01:00
Winni Neessen	e00ddda3a3	Fix test expectation and add HTML template tests ... Corrected the expected number of parts in an existing test from 1 to 2. Additionally, introduced tests for adding HTML templates to message bodies, including cases with a valid template, a body string, a nil template, and an invalid template.	2024-10-27 15:13:57 +01:00
Winni Neessen	602f8a6e29	Add comprehensive tests for message alternatives ... Introduced tests for AddAlternativeString and AddAlternativeWriter methods, including variations for charset, encoding, content description, and simultaneous usages. Ensure accurate handling of content types, encodings, and body strings. This enhances coverage and reliability of message formatting features.	2024-10-27 15:10:01 +01:00
Winni Neessen	4db66696a6	Add tests for SetBodyTextTemplate method in msg_test.go ... Implemented thorough unit tests to validate the SetBodyTextTemplate method in msg_test.go. These tests cover default behavior, handling of nil templates, and invalid templates, ensuring robustness and reliability.	2024-10-27 14:56:34 +01:00
Winni Neessen	f8caa5599b	Refactor buffer creation in msg.go ... Replace `bytes.Buffer{}` with `bytes.NewBuffer(nil)` for buffer instantiation to improve clarity and consistency. This change simplifies the buffer creation process and aligns it with common Go practices.	2024-10-27 14:55:51 +01:00
Winni Neessen	35ffc95102	Add tests for SetBodyWriter and SetBodyHTMLTemplate ... Introduced new tests to validate the functionality of `SetBodyWriter` with `WithPartEncoding` and `WithPartContentDescription`. Also added comprehensive tests for `SetBodyHTMLTemplate`, covering default behavior and error cases.	2024-10-27 14:50:14 +01:00
Winni Neessen	432e21f162	Refactor buffer initialization in message template execution ... Change buffer initialization from an empty `bytes.Buffer` to `bytes.NewBuffer(nil)` to streamline buffer allocation. This adjustment ensures better flexibility and aligns with best practices in handling buffer slices.	2024-10-27 14:33:00 +01:00
Winni Neessen	0aa81d724b	Reorganize and cleanup msg_test.go ... Removed redundant and commented-out test functions to improve code readability and maintainability. Moved the essential helper functions (checkAddrHeader and checkGenHeader) to the end to keep the structure organized.	2024-10-27 14:20:44 +01:00
Winni Neessen	78ee1a2a81	Fix timeout message in client_test.go ... Updated the skip message to accurately reflect the timeout reason when closing the test server connection. This improves the clarity of the error reporting in tests.	2024-10-27 12:11:48 +01:00
Winni Neessen	b510d2654c	Handle timeout errors in test client cleanup ... Added a check in the client cleanup function to skip the test if the client connection fails due to a timeout. This ensures that network-related timeouts do not cause test failures, providing more reliable test outcomes.	2024-10-27 12:10:51 +01:00
Winni Neessen	056ec60734	Remove newline characters in test data strings ... Trim newline characters from test attachment and embed strings. Updated assertions to compare strings without trailing newlines for consistency.	2024-10-27 12:05:18 +01:00
Winni Neessen	cb5ac8b0e2	Suppress inspection for deprecated method usage in test ... Added GoLand directive to suppress deprecation warnings for SetAttachements method in msg_test.go. This method is already fully tested by SetAttachments and the test is marked to be skipped.	2024-10-27 11:32:15 +01:00
Winni Neessen	6376f29190	Add ContentType tests and simplify error messages ... Introduced new test cases for ContentType values to ensure proper handling. Additionally, simplified multiple error messages in existing tests to enhance readability and maintain consistency.	2024-10-27 11:27:38 +01:00
Winni Neessen	9a01629c47	Add unit tests for msg embed setters and unsetters ... Introduce tests for `SetEmbeds` with single, multiple, and no files. Add `UnsetAllEmbeds` and `UnsetAllParts` tests to ensure embeds and attachments are removed correctly.	2024-10-27 10:49:02 +01:00
Winni Neessen	2dad9d36b2	Add embed and attachment tests ... Introduced tests for embedding files and managing attachments in messages. Validated behavior for setting and unsetting attachments and retrieving embedded files with various scenarios.	2024-10-27 10:42:32 +01:00
Winni Neessen	c8c7d18ba9	Add test attachment with license information ... Introduced a new test attachment file to the testdata directory. Included a corresponding license file to specify copyright and licensing details.	2024-10-27 10:30:20 +01:00
Winni Neessen	472a5a6454	Add tests for SetAttachments in msg_test.go ... Implemented comprehensive tests for SetAttachments including scenarios with single, multiple, and no files. Additionally, deprecated the SetAttachements function and noted it as fully tested by SetAttachments.	2024-10-27 10:25:53 +01:00
Winni Neessen	f2619737e8	Add tests for Msg GetBoundary method ... Introduce unit tests for the GetBoundary method in the Msg type. Ensure proper functionality for messages with and without a boundary set.	2024-10-27 10:13:14 +01:00
Winni Neessen	b7ca41af81	Remove redundant TestMsg_SetAttachments ... Eliminated the TestMsg_SetAttachments function from the test suite as it duplicates functionality covered by other tests. Removing this redundant code improves maintainability and clarity.	2024-10-27 10:09:56 +01:00
Winni Neessen	66e25d82d3	Add test for GetAttachments method with no attachment ... Introduced a new test case to check `GetAttachments` when no attachments are present in the message object. Removed the redundant `TestMsg_GetAttachments` function to simplify the test suite.	2024-10-27 10:09:25 +01:00
Winni Neessen	babf7b9780	Add tests for GetAttachments method in Msg ... This commit introduces unit tests for the GetAttachments method in the Msg struct to ensure it correctly handles both single and multiple attachments. The tests verify the name and content of the attachments to validate the expected behavior.	2024-10-27 10:07:05 +01:00
Winni Neessen	43f9ffa3af	Add tests for GetGenHeader and GetParts methods in Msg ... Introduce comprehensive tests for the GetGenHeader and GetParts methods. These tests cover single and multiple values for headers, body parts retrieval, and edge cases such as nil values, ensuring robustness and correctness.	2024-10-27 09:58:22 +01:00
Winni Neessen	e808e0b972	Add comprehensive unit tests for email address getters ... Introduced unit tests for GetFrom, GetFromString, GetTo, GetToString, GetCc, GetCcString, GetBcc, and GetBccString methods. These tests cover scenarios with single, multiple, and no addresses, ensuring correct functionality of email address retrieval methods.	2024-10-26 23:20:23 +02:00
Winni Neessen	22f56a0143	Add tests for GetAddrHeaderString in msg_test.go ... Add unit tests for the GetAddrHeaderString function to verify correct behavior for various header types and address counts. These tests ensure the function correctly handles single and multiple addresses, as well as scenarios with no addresses.	2024-10-26 22:57:47 +02:00
Winni Neessen	d16ae61f64	Add test for GetAddrHeader with multiple valid addresses ... Introduce a new test to validate the behavior of GetAddrHeader for handling multiple addresses (to, cc, bcc). This test ensures that the function correctly processes and returns the expected addresses in various headers.	2024-10-26 22:53:30 +02:00
Winni Neessen	5e5bcef696	Add tests for Msg_GetAddrHeader ... Introduce unit tests for the GetAddrHeader function in the Msg struct. These tests cover various scenarios, including valid addresses and handling of headers like "from," "to," "cc," and "bcc."	2024-10-26 22:50:31 +02:00
Winni Neessen	7b600534ea	Add tests for Msg.GetRecipients method ... Implemented comprehensive tests for the Msg.GetRecipients method, covering various scenarios such as recipients in "to", "cc", "bcc" fields and combinations of these. Ensured correct handling of each case and validation against expected recipient counts and specific addresses.	2024-10-26 22:31:13 +02:00
Winni Neessen	842d4373f2	Add missing newline at EOF ... Ensured the file ends with a newline to comply with best practices. This change improves code readability and consistency with other files in the repository.	2024-10-26 22:14:56 +02:00
Winni Neessen	5c2831c331	Add tests for MDN address formatting and GetSender logic ... Implemented tests for `RequestMDNToFormat`, `RequestMDNAddTo`, and `RequestMDNAddToFormat` methods to ensure proper address handling. Added comprehensive `GetSender` method tests to verify sender retrieval in various scenarios. This enhances coverage and robustness of email message handling functionality.	2024-10-26 22:13:51 +02:00
Winni Neessen	4fe9022815	Refactor RequestMDNAddTo ... Simplify the address handling for the "Disposition-Notification-To" header in msg.go. The code now directly reassigns the previously fetched addresses and ensures appending the new address effectively. This improves code readability and correctness in updating the header.	2024-10-26 22:13:37 +02:00
Winni Neessen	4f97cd8261	Initialize genHeader in RequestMDNTo method ... Add checks to initialize `genHeader` if it is nil in the RequestMDNTo method to prevent potential nil map assignment errors. The newly implemented tests showed that this method was never actually working and the old test was inefficient to identify this.	2024-10-26 21:33:43 +02:00
Winni Neessen	9e51dba82a	Add new tests for Msg methods ... Introduce tests for SetImportance, SetOrganization, SetUserAgent, and IsDelivered methods in the Msg type. These additions ensure the correct behavior and robustness of the email handling functionality.	2024-10-26 21:07:48 +02:00
Winni Neessen	cf117d320b	Add nil check in testMessage helper function ... Added a conditional check to ensure 'NewMsg()' does not return nil in the 'testMessage' helper function. This update will prevent potential nil pointer dereferences and improve the robustness of the test.	2024-10-26 21:07:04 +02:00
Winni Neessen	42c63791ef	Move delivery status update after writer close check ... Relocated the `isDelivered` flag update to occur after the writer's close method validation. This ensures that the message delivery status is only marked as true if no errors arise during the writer close process. This showed up in the new test cases that covered errors on closing the DATA channel. This can already happen before the first byte is sent to the server. Therefore isDelivered should be set after a successful close.	2024-10-26 21:06:21 +02:00