11 Commits

Author	SHA1	Message	Date
Brennan Kinney	62fdcb05f5	chore: Remove redundant capability `SYS_PTRACE` (#2624) ... * chore: Remove `SYS_PTRACE` capability from docs and configs * chore: Remove `SYS_PTRACE` capability from tests Doesn't seem to be required. It was originally added when the original change detection feature PR apparently needed it to function.	2022-06-07 01:20:13 +12:00
Brennan Kinney	3b4f44e837	tests(fix): Adjust for local testing conditions (#2606) ... * tests(fix): Increase some timeouts Running tests locally via a VM these tests would fail sometimes due to the time from being queued and Amavis actually processing being roughly around 30 seconds. There should be no harm in raising this to 60 seconds, other than delaying a failure case which will ripple through other time sensitive tests. It's better to pass when functionality is actually correct but just needs a bit longer to complete. * tests(fix): Don't setup an invalid hostname During container startup `helpers/dns.sh` would panic with `hostname -f` failing. Dropping `--domainname` for this container is fine and does not affect the point of it's test. --- It's unclear why this does not occur in CI. Possibly changes within the docker daemon since as CI runs docker on Ubuntu 20.04? (2020). For clarity, this may be equivalent to setting a hostname of `domain.com.domain.com`, or `--hostname` value truncated the NIS domain (`--domainname`) of the same value. IIRC, it would still fail with both options using different values if `--hostname` was multi-label. I believe I've documented how non-deterministic these options can be across different environments. `--hostname` should be preferred. There doesn't seem to be any reason to actually need `--domainname` (which is NIS domain name, unrelated to the DNS domain name). We still need to properly investigate reworking our ENV support that `dns.sh` manages. --- Containers were also not removing themselves after failures either (missing teardown). Which would cause problems when running tests again. * chore: Normalize white-space Sets a consistent indent size of 2 spaces. Previously this varied a fair bit, sometimes with tabs or mixed tabs and spaces. Some formatting with blank lines. Easier to review with white-space in diff ignored. Some minor edits besides blank lines, but no change in functionality. * fix: `setup.sh` target container under test Some of the `setup.sh` commands did not specify the container which was problematic if another `docker-mailserver` container was running, causing test failures. This probably doesn't help with `test/no_container.bats`, but at least prevents `test/tests.bats` failing at this point.	2022-05-30 12:53:30 +12:00
Casper	628e902233	Remove unnecessary quotes from command substitutions (#2561)	2022-05-05 10:28:38 +02:00
Georg Lauterbach	21c218ac68	scripts: remove `DMS_DEBUG` (#2523) ... * remove DMS_DEBUG from tests * remove DMS_DEBUG from doc and scripts * updated issue template * re-add description about removal of DMS_DEBUG	2022-04-03 13:29:10 +02:00
Georg Lauterbach	321ae744fa	scripts: small refactorings (#2485) ... Co-authored-by: Casper <casperklein@users.noreply.github.com>	2022-03-17 11:24:30 +01:00
Casper	6d1c740310	tests: remove legacy functions / tests (#2434)	2022-03-02 14:55:17 +01:00
Georg Lauterbach	9ce719213b	first refactoring fr *.bats according to shellcheck	2020-11-05 19:15:28 +01:00
Martin Schulze	0453240098	Catch errors on config duplication command	2020-10-20 09:33:47 +02:00
Georg Lauterbach	3f5c518950	mostly `pwd` replacements and introduction of local keywords	2020-10-19 13:13:42 +02:00
Martin Schulze	5fef8a5b78	Give each container their own private config folder	2020-10-17 02:02:46 +02:00
Germain Masse	fa167a9e51	Test special-use folders are available in IMAP	2019-11-01 20:04:37 +00:00