Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix NsJail Tests #93

Merged
merged 7 commits into from
Mar 8, 2021
Merged

Fix NsJail Tests #93

merged 7 commits into from
Mar 8, 2021

Commits on Feb 5, 2021

  1. CI: fix test step not failing when tests fail

    Generating the report in the same step resulted in the report exit code
    overriding the exit code of the test runner.
    MarkKoz committed Feb 5, 2021
    Configuration menu
    Copy the full SHA
    e12e403 View commit details
    Browse the repository at this point in the history
  2. Suppress NsJail info logs during tests

    The logs clutter up the test output way too much.
    MarkKoz committed Feb 5, 2021
    Configuration menu
    Copy the full SHA
    cfe57d2 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    a8b9c33 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    a1b5e5b View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    2240dde View commit details
    Browse the repository at this point in the history

Commits on Mar 8, 2021

  1. Docker: improve caching & install numpy in container

    CI was building the image twice: once with dev dependencies and again
    without. Separating the pipenv command into separate layers allows the
    second build in CI to take advantage of the cache for the base
    dependencies that it will share across both builds.
    
    Install numpy along with the dev dependencies within the container.
    Previously it was installed in CI only, but this meant extra work for
    those running tests locally.
    
    Install numpy to the correct site.
    MarkKoz committed Mar 8, 2021
    Configuration menu
    Copy the full SHA
    3f474ff View commit details
    Browse the repository at this point in the history
  2. Merge master into bug/tests/nsjail

    The branch needs the fixes from #94 to make the tests pass.
    MarkKoz committed Mar 8, 2021
    Configuration menu
    Copy the full SHA
    46fc728 View commit details
    Browse the repository at this point in the history