OpenSearch/qa/os
William Brafford 2311bc62f7
Filter empty lines from "docker ls" response (#52081)
* Filter empty lines from docker ls response

In order to cut down on test time, our docker/vagrant tests build the
docker image outside of the vagrant VM. When we get around to launching
the Vagrant VM, we mount that already-built docker image to a known
location. At that point, we need to load the docker image. But we only
want to load it once. As we're running tests, we use "docker ls" to
check whether the local image is loaded for use. Empty output from the
particular ls invocation means no image is loaded.

There was a bug in how we checked this. In Java, splitting an empty
string will yield an array containing one empty string. So when we're
counting the output from the docker ls command, we need to filter out
empty lines in order to proceed to loading the image for docker tests.
2020-02-14 11:39:13 -05:00
..
bats Migrate SysV init tests from bats to java packaging (#51077) (#51498) 2020-02-10 17:41:33 -05:00
centos-6 Rename vagrant project to os (#45509) (#45530) 2019-08-14 10:30:41 -07:00
centos-7 Add Docker packaging tests on 7.x (#48857) 2019-11-05 15:17:59 +00:00
debian-8 Rename vagrant project to os (#45509) (#45530) 2019-08-14 10:30:41 -07:00
debian-9 Add Docker packaging tests on 7.x (#48857) 2019-11-05 15:17:59 +00:00
fedora-28 Add Docker packaging tests on 7.x (#48857) 2019-11-05 15:17:59 +00:00
fedora-29 Add Docker packaging tests on 7.x (#48857) 2019-11-05 15:17:59 +00:00
oel-6 Rename vagrant project to os (#45509) (#45530) 2019-08-14 10:30:41 -07:00
oel-7 Rename vagrant project to os (#45509) (#45530) 2019-08-14 10:30:41 -07:00
opensuse-42 Rename vagrant project to os (#45509) (#45530) 2019-08-14 10:30:41 -07:00
sles-12 Rename vagrant project to os (#45509) (#45530) 2019-08-14 10:30:41 -07:00
src/test/java/org/elasticsearch/packaging Filter empty lines from "docker ls" response (#52081) 2020-02-14 11:39:13 -05:00
ubuntu-1604 Add Docker packaging tests on 7.x (#48857) 2019-11-05 15:17:59 +00:00
ubuntu-1804 Add Docker packaging tests on 7.x (#48857) 2019-11-05 15:17:59 +00:00
windows-2012r2 Separate distro tests to be per distribution (#45565) 2019-08-20 13:12:15 -07:00
windows-2016 Separate distro tests to be per distribution (#45565) 2019-08-20 13:12:15 -07:00
README.md Rename vagrant project to os (#45509) (#45530) 2019-08-14 10:30:41 -07:00
build.gradle Migrate SysV init tests from bats to java packaging (#51077) (#51498) 2020-02-10 17:41:33 -05:00

README.md

packaging tests

This project contains tests that verify the distributions we build work correctly on the operating systems we support. They're intended to cover the steps a user would take when installing and configuring an Elasticsearch distribution. They're not intended to have significant coverage of the behavior of Elasticsearch's features.

There are two types of tests in this project. The old tests live in src/test/ and are written in Bats, which is a flavor of bash scripts that run as unit tests. These tests are deprecated because Bats is unmaintained and cannot run on Windows.

The new tests live in src/main/ and are written in Java. Like the old tests, this project's tests are run inside the VM, not on your host. All new packaging tests should be added to this set of tests if possible.

Running these tests

See the section in TESTING.asciidoc

Adding a new test class

When gradle runs the packaging tests on a VM, it runs the full suite by default. To add a test class to the suite, add its class to the @SuiteClasses annotation in PackagingTests.java. If a test class is added to the project but not to this annotation, it will not run in CI jobs. The test classes are run in the order they are listed in the annotation.

Choosing which distributions to test

Distributions are represented by enum values which know if they are compatible with the platform the tests are currently running on. To skip a test if the distribution it's using isn't compatible with the current platform, put this assumption in your test method or in a @Before method

assumeTrue(distribution.packaging.compatible);

Similarly if you write a test that is intended only for particular platforms, you can make an assumption using the constants and methods in Platforms.java

assumeTrue("only run on windows", Platforms.WINDOWS);

assumeTrue("only run if using systemd", Platforms.isSystemd());

Writing a test that covers multiple distributions

It seems like the way to do this that makes it the most straightforward to run and reproduce specific test cases is to create a test case class with an abstract method that provides the distribution

public abstract class MyTestCase {
  @Test
  public void myTest() { /* do something with the value of #distribution() */ }
  abstract Distribution distribution();
}

and then for each distribution you want to test, create a subclass

public class MyTestDefaultTar extends MyTestCase {
  @Override
  Distribution distribution() { return Distribution.DEFAULT_TAR; }
}

That way when a test fails the user gets told explicitly that MyTestDefaultTar failed, and to reproduce it they should run that class. See ArchiveTestCase and its children for an example of this.

Running external commands

In general it's probably best to avoid running external commands when a good Java alternative exists. For example most filesystem operations can be done with the java.nio.file APIs. For those that aren't, use an instance of Shell

This class runs scripts in either bash with the bash -c <script> syntax, or in powershell with the powershell.exe -Command <script> syntax.

Shell sh = new Shell();

// equivalent to `bash -c 'echo $foo; echo $bar'`
sh.bash("echo $foo; echo $bar");

// equivalent to `powershell.exe -Command 'Write-Host $foo; Write-Host $bar'`
sh.powershell("Write-Host $foo; Write-Host $bar");

Notes about powershell

Powershell scripts for the most part have backwards compatibility with legacy cmd.exe commands and their syntax. Most of the commands you'll want to use in powershell are Cmdlets which generally don't have a one-to-one mapping with an executable file.

When writing powershell commands in this project it's worth testing them by hand, as sometimes when a script can't be interpreted correctly it will fail silently.