Skip to content

Commit

Permalink
Set OMP thrust method and capture streams in tests (#935)
Browse files Browse the repository at this point in the history
This commit makes 2 minor changes to the CI configuration to re-enable
building and running tests with the thrust OMP CPU backend. This was lost
in the migration to GHA and means we're not testing the thrust
simulation methods in CI which is a coverage hole. This fixes that
oversight. At the same time this commit adds the
QISKIT_TEST_CAPTURE_STREAMS env variable. Since Qiskit/qiskit#3982
merged the base test class has had support for capturing STDOUT, STDERR,
and python logging as part of the test runs, which is especially
important for our CI configuration where we run tests in parallel. This
just enables this option so we'll have captured output streams
associated with each test method.
  • Loading branch information
mtreinish authored Sep 18, 2020
1 parent 6a4c217 commit 44a3f8a
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -159,6 +159,9 @@ jobs:
matrix:
python-version: [3.6, 3.7, 3.8]
os: ["macOS-latest", "ubuntu-latest", "windows-latest"]
env:
AER_THRUST_BACKEND: OMP
QISKIT_TEST_CAPTURE_STREAMS: 1
steps:
- uses: actions/checkout@v2
- name: Set up Python ${{ matrix.python-version }}
Expand Down

0 comments on commit 44a3f8a

Please sign in to comment.