MatrixAI / Polykey-CLI

Polykey CLI - Open Source Decentralized Secret Sharing System for Zero Trust Workflows
https://polykey.com
GNU General Public License v3.0
6 stars 3 forks source link

Fix Docker integration test to use `status.waitFor` #237

Open brynblack opened 3 months ago

brynblack commented 3 months ago

Specification

As a stopgap for fixing the CI jobs we had to remove the usage of status.waitFor in the Docker integration tests. The reason for this is despite fixing how the filesystems were mounted for testing, any file changes inside of the container were not propagated back to the host filesystem. This means we could not check the status for any state change inside the test.

We need to figure out how to properly mount the filesystem between the host and the container filesystem to make sure that changes are propagated between both, and we need to update the test to use status.waitFor.

Additional context

Tasks

  1. Work out how to properly bind mount
  2. Fix up the tests
linear[bot] commented 3 months ago

ENG-354 Fix Docker integration test to use `status.waitFor`