[MESOS-3762] Refactor SSLTest fixture such that MesosTest can use the same helpers.

• Type: Task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.26.0
• Component/s: test
• Labels: mesosphere
• Sprint: Mesosphere Sprint 21
• Story Points: 3
• Assignee: Joseph Wu
• Reporter: Joseph Wu
• Shepherd: Joris Van Remoortere
• Created: 19/Oct/15 22:43
• Updated: 26/Nov/18 12:21
• Resolved: 26/Nov/18 12:21

Description

In order to write tests that exercise SSL with other components of Mesos, such as the HTTP scheduler library, we need to use the setup/teardown logic found in the SSLTest fixture.

Currently, the test fixtures have separate inheritance structures like this:

SSLTest <- ::testing::Test
MesosTest <- TemporaryDirectoryTest <- ::testing::Test

where ::testing::Test is a gtest class.

The plan is the following:

1. Change SSLTest to inherit from TemporaryDirectoryTest. This will require moving the setup (generation of keys and certs) from SetUpTestCase to SetUp. At the same time, some of the cleanup logic in the SSLTest will not be needed.
2. Move the logic of generating keys/certs into helpers, so that individual tests can call them when needed, much like MesosTest.
3. Write a child class of SSLTest which has the same functionality as the existing SSLTest, for use by the existing tests that rely on SSLTest or the RegistryClientTest.
4. Have MesosTest inherit from SSLTest (which might be renamed during the refactor). If Mesos is not compiled with --enable-ssl, then SSLTest could be #ifdef'd into any empty class.

The resulting structure should be like:

MesosTest <- SSLTest <- TemporaryDirectoryTest <- ::testing::Test
ChildOfSSLTest /

Issue Links

• blocks: MESOS-3753 Test the HTTP Scheduler library with SSL enabled
• breaks: MESOS-3847 Root tests for LinuxFilesystemIsolatorTest are broken
• relates to: MESOS-3848 Refactor Environment::mkdtemp into TemporaryDirectoryTest.