Skip to content

Commit 7d0bacf

Browse files
harden Docker image, add monthly rebuilds, run xt tests in CI (NicTool#342)
Migrate 18_interaction.t and 20_permission.t from Test.pm to Test::More so prove can parse the TAP output. Run all four xt test files (5,996 tests) inside the CI container on every PR. Add apt-get upgrade to the Dockerfile so images pick up security patches. Add a monthly cron rebuild (1st of month, 6 AM UTC) to keep production images current between code changes. Co-authored-by: Moses Ingersoll <258583966+burning-bush-dev@users.noreply.github.com> Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 4089728 commit 7d0bacf

5 files changed

Lines changed: 182 additions & 178 deletions

File tree

.github/workflows/ci.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -125,6 +125,9 @@ jobs:
125125
- name: Run server tests
126126
run: docker exec ci make -C /workspace/server test
127127

128+
- name: Run extended permission & delegation tests
129+
run: docker exec -w /workspace/server ci bash -c 'prove -v xt/*.t'
130+
128131
- name: Failure diagnostics
129132
if: failure()
130133
run: |

.github/workflows/docker.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
name: Docker
22

33
on:
4+
schedule:
5+
- cron: '0 6 1 * *' # 6 AM UTC on the 1st of each month
46
pull_request:
57
types: [opened, synchronize, reopened]
68
paths:
@@ -64,6 +66,9 @@ jobs:
6466
- name: Run client tests
6567
run: docker compose -f dist/docker/docker-compose.yml exec -T web make -C /usr/local/nictool/client test
6668

69+
- name: Run extended permission & delegation tests
70+
run: docker compose -f dist/docker/docker-compose.yml exec -T web bash -c 'cd /usr/local/nictool/server && prove -v xt/*.t'
71+
6772
- name: Failure diagnostics
6873
if: failure()
6974
run: |

dist/docker/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ FROM debian:bookworm AS base
44

55
ENV DEBIAN_FRONTEND=noninteractive LANG=en_US.UTF-8 LC_ALL=C.UTF-8
66

7-
RUN apt-get -q update && apt-get install -qy \
7+
RUN apt-get -q update && apt-get upgrade -qy && apt-get install -qy \
88
perl \
99
cpanminus \
1010
build-essential \

0 commit comments

Comments
 (0)