Skip to content

Commit acc3cae

Browse files
authored
Remove "reinstall" packaging tests (#37851)
The packaging tests currently have a test which installs elasticsearch, removes it, modifies ownership of /etc/elasticsearch, and reinstalls. It then checks that the /etc/elasticsearch directory has ownership that the package expects. But the recursive change touches files not owned by the package. In the past this worked because we did a recursive ownership change within the package postinst. However, that was recently removed, and thus this test no longer makes sense.
1 parent a644bc0 commit acc3cae

File tree

2 files changed

+0
-89
lines changed

2 files changed

+0
-89
lines changed

qa/vagrant/src/test/resources/packaging/tests/90_reinstall.bats

Lines changed: 0 additions & 78 deletions
This file was deleted.

qa/vagrant/src/test/resources/packaging/utils/utils.bash

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -233,17 +233,6 @@ assert_output() {
233233
echo "$output" | grep -E "$1"
234234
}
235235

236-
assert_recursive_ownership() {
237-
local directory=$1
238-
local user=$2
239-
local group=$3
240-
241-
realuser=$(find $directory -printf "%u\n" | sort | uniq)
242-
[ "$realuser" = "$user" ]
243-
realgroup=$(find $directory -printf "%g\n" | sort | uniq)
244-
[ "$realgroup" = "$group" ]
245-
}
246-
247236
# Deletes everything before running a test file
248237
clean_before_test() {
249238

0 commit comments

Comments
 (0)