mixandClaude Opus 5.5
6023feb405
test / test (push) Canceled after 0s
The last stage of the 2026-10-12 audit; no cut. Doc drift: architecture.md (the sender check in § OpenDKIM, the limit: row's 48 hours, the inbound log and filter-list routes and who may open them, migrations to 0011 and every table, KitPage as production data, Reload configuration as a restore writes, the panel's roles counted), schema-migrations.md (tables and columns since v2), guide.md (a delivery's lines go at the next prune; INBOUND_JOURNAL_MILTER_SOCKET moved to architecture.md's internal list), development.md (the env guard, the deferral the e2e proves, its fixed sleeps), roadmap.md. Comments say what the code does now; TestBaselineIsTheOnlyMigration is TestChainStartsAtTheBaseline; the milter package doc names its three milters. Dead code gone: view.Facts, dmarc HostedSuggestion and ValidateHostedAddress, app UpdateMode (each with the test that alone kept it), the method checks the method-bound routes made dead; Settings.WithResult calls resultFlash. staticcheck 2026.1 in test.yml on both modules, clean: the e2e package builds without its tests now (a test and two constants moved out of non-test files), three findings fixed. The test moved, TestImageBuildPreservesLogrotateMode, had sat in a non-test file and never run; run, it failed on the entrypoint (docker run without a hostname) and now asks with --entrypoint stat. The test gaps no step named, each closed: follow() with mail.log gone past blindMin and made again; CheckRestore with a manifest that is not JSON; the origin check with a port; every /server/ pattern in web.go through server() and in the routes table (it found GET /server/health/fragment missing from the table - guarded, not tested); L9 as the owner decided it on 2026-10-12, the comment on senderSession.Headers and the test passing an h that holds the earlier messages' From:. Plan stage 6 ticked; roadmap audit row done, 81/81. Tested on the test server: go vet + go test of 27 packages, make e2e (all pass, 513 s) on this tree; staticcheck locally on both modules. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
64 lines
1.9 KiB
YAML
64 lines
1.9 KiB
YAML
name: test
|
|
|
|
on:
|
|
push:
|
|
branches: [main]
|
|
pull_request:
|
|
|
|
jobs:
|
|
test:
|
|
runs-on: ubuntu-latest
|
|
steps:
|
|
- uses: actions/checkout@v4
|
|
with:
|
|
# design-first.sh reads each commit's parent and the template history.
|
|
fetch-depth: 0
|
|
|
|
- name: Design first
|
|
env:
|
|
BASE: ${{ github.event.pull_request.base.sha || github.event.before }}
|
|
HEAD: ${{ github.event.pull_request.head.sha || github.sha }}
|
|
run: bash .github/scripts/design-first.sh "$BASE" "$HEAD"
|
|
|
|
- uses: actions/setup-go@v5
|
|
with:
|
|
go-version: "1.26"
|
|
cache: true
|
|
cache-dependency-path: |
|
|
go.sum
|
|
test/e2e/go.sum
|
|
|
|
- name: Format
|
|
run: |
|
|
fmt="$(gofmt -l .)"
|
|
if [ -n "$fmt" ]; then
|
|
echo "$fmt"
|
|
echo "gofmt needed on the files above; run 'gofmt -w .'"
|
|
exit 1
|
|
fi
|
|
|
|
- name: Vet
|
|
run: go vet ./...
|
|
|
|
# Unused code, dead branches and the usual slips go vet does not look
|
|
# for. The version is pinned: a new release adds checks, and a check
|
|
# that appears under a push is a red build nobody asked for. 2026.1 is
|
|
# the release for Go 1.26. test/e2e is a module of its own, so ./...
|
|
# skips it; it is checked in its own step below.
|
|
- name: Staticcheck
|
|
run: go run honnef.co/go/tools/cmd/staticcheck@2026.1 ./...
|
|
|
|
- name: Test
|
|
run: go test ./...
|
|
|
|
# The e2e module runs only in release.yml, after a release is published;
|
|
# a type error there would first show up then. Compile it here without
|
|
# running it: `go test -run` would still run TestMain, which brings up
|
|
# the whole stand.
|
|
- name: Compile the e2e
|
|
working-directory: test/e2e
|
|
run: |
|
|
go vet ./...
|
|
go test -c -o /dev/null .
|
|
go run honnef.co/go/tools/cmd/staticcheck@2026.1 ./...
|