Skip to content

Commit

Permalink
Remove modules from Dockerfile which are already part of core modules (
Browse files Browse the repository at this point in the history
…#66)

See https://prosody.im/doc/modules.

Add test to check if log contains warnings

This fixes #63.
  • Loading branch information
SaraSmiseth authored Mar 4, 2024
1 parent 74c6451 commit 0e33f70
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
4 changes: 0 additions & 4 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -99,13 +99,9 @@ COPY *.bash /usr/local/bin/

RUN download-prosody-modules.bash \
&& docker-prosody-module-install.bash \
bookmarks `# XEP-0411: Bookmarks Conversion` \
carbons `# message carbons (XEP-0280)` \
cloud_notify `# XEP-0357: Push Notifications` \
csi `# client state indication (XEP-0352)` \
e2e_policy `# require end-2-end encryption` \
filter_chatstates `# disable "X is typing" type messages` \
smacks `# stream management (XEP-0198)` \
throttle_presence `# presence throttling in CSI` \
vcard_muc `# XEP-0153: vCard-Based Avatar (MUC)` \
&& rm -rf "/usr/src/prosody-modules"
Expand Down
5 changes: 5 additions & 0 deletions tests/tests.bats
Original file line number Diff line number Diff line change
Expand Up @@ -91,3 +91,8 @@ load 'bats/bats-assert/load'
run bash -c "sudo docker-compose exec $batsContainerName /bin/bash -c \"/entrypoint.bash check\" | grep 'deprecated' -A 3"
assert_failure
}

@test "Should not have warnings in log" {
run bash -c "sudo docker-compose logs $batsContainerName | grep -E \"warn\""
assert_failure
}

0 comments on commit 0e33f70

Please sign in to comment.