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.
This commit is contained in:
parent
a644bc095c
commit
acc3cae40c
|
@ -1,78 +0,0 @@
|
|||
#!/usr/bin/env bats
|
||||
|
||||
# Tests upgrading elasticsearch from a previous version with the deb or rpm
|
||||
# packages. Just uses a single node cluster on the current machine rather than
|
||||
# fancy rolling restarts.
|
||||
|
||||
# WARNING: This testing file must be executed as root and can
|
||||
# dramatically change your system. It should only be executed
|
||||
# in a throw-away VM like those made by the Vagrantfile at
|
||||
# the root of the Elasticsearch source code. This should
|
||||
# cause the script to fail if it is executed any other way:
|
||||
[ -f /etc/is_vagrant_vm ] || {
|
||||
>&2 echo "must be run on a vagrant VM"
|
||||
exit 1
|
||||
}
|
||||
|
||||
# The test case can be executed with the Bash Automated
|
||||
# Testing System tool available at https://github.com/sstephenson/bats
|
||||
# Thanks to Sam Stephenson!
|
||||
|
||||
# Licensed to Elasticsearch under one or more contributor
|
||||
# license agreements. See the NOTICE file distributed with
|
||||
# this work for additional information regarding copyright
|
||||
# ownership. Elasticsearch licenses this file to you under
|
||||
# the Apache License, Version 2.0 (the "License"); you may
|
||||
# not use this file except in compliance with the License.
|
||||
# You may obtain a copy of the License at
|
||||
#
|
||||
# http://www.apache.org/licenses/LICENSE-2.0
|
||||
#
|
||||
# Unless required by applicable law or agreed to in writing,
|
||||
# software distributed under the License is distributed on an
|
||||
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
|
||||
# KIND, either express or implied. See the License for the
|
||||
# specific language governing permissions and limitations
|
||||
# under the License.
|
||||
|
||||
# Load test utilities
|
||||
load $BATS_UTILS/utils.bash
|
||||
load $BATS_UTILS/packages.bash
|
||||
|
||||
# Cleans everything for the 1st execution
|
||||
setup() {
|
||||
skip_not_dpkg_or_rpm
|
||||
export PACKAGE_NAME="elasticsearch-oss"
|
||||
}
|
||||
|
||||
@test "[REINSTALL] install" {
|
||||
clean_before_test
|
||||
install_package
|
||||
}
|
||||
|
||||
@test "[REINSTALL] purge elasticsearch" {
|
||||
purge_elasticsearch
|
||||
}
|
||||
|
||||
@test "[REINSTALL] chown directories" {
|
||||
# to simulate the loss of ownership
|
||||
if [ -d /var/lib/elasticsearch ]; then
|
||||
sudo chown -R root:root /var/lib/elasticsearch
|
||||
fi
|
||||
if [ -d "/var/log/elasticsearch" ]; then
|
||||
sudo chown -R root:root /var/log/elasticsearch
|
||||
fi
|
||||
if [ -d /etc/elasticsearch ]; then
|
||||
sudo chown -R root:root /etc/elasticsearch
|
||||
fi
|
||||
}
|
||||
|
||||
@test "[REINSTALL] reinstall elasticsearch" {
|
||||
install_package
|
||||
}
|
||||
|
||||
@test "[REINSTALL] check ownership" {
|
||||
assert_recursive_ownership /var/lib/elasticsearch elasticsearch elasticsearch
|
||||
assert_recursive_ownership /var/log/elasticsearch elasticsearch elasticsearch
|
||||
assert_recursive_ownership /etc/elasticsearch root elasticsearch
|
||||
}
|
|
@ -233,17 +233,6 @@ assert_output() {
|
|||
echo "$output" | grep -E "$1"
|
||||
}
|
||||
|
||||
assert_recursive_ownership() {
|
||||
local directory=$1
|
||||
local user=$2
|
||||
local group=$3
|
||||
|
||||
realuser=$(find $directory -printf "%u\n" | sort | uniq)
|
||||
[ "$realuser" = "$user" ]
|
||||
realgroup=$(find $directory -printf "%g\n" | sort | uniq)
|
||||
[ "$realgroup" = "$group" ]
|
||||
}
|
||||
|
||||
# Deletes everything before running a test file
|
||||
clean_before_test() {
|
||||
|
||||
|
|
Loading…
Reference in New Issue