Skip to content

Commit 8ed31b7

Browse files
committed
Move docker tests to separate workflow
- do not run them on PR builds - see #596
1 parent 53ef207 commit 8ed31b7

File tree

2 files changed

+19
-15
lines changed

2 files changed

+19
-15
lines changed

.github/workflows/dockertests.yml

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
name: Dockertests
2+
3+
on:
4+
[push]
5+
6+
jobs:
7+
dockertests:
8+
runs-on: ubuntu-latest
9+
strategy:
10+
fail-fast: false
11+
matrix:
12+
docker-image: [centos, debian, fedora, ubuntu]
13+
steps:
14+
- uses: actions/checkout@v2
15+
- name: Setup docker container
16+
run: |
17+
docker build -t pyfakefs -f $GITHUB_WORKSPACE/.github/workflows/dockerfiles/Dockerfile_${{ matrix.docker-image }} . --build-arg github_repo=$GITHUB_REPOSITORY --build-arg github_branch=$(basename $GITHUB_REF)
18+
- name: Run tests
19+
run: docker run -t pyfakefs

.github/workflows/pythonpackage.yml

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -95,18 +95,3 @@ jobs:
9595
python -m pyfakefs.tests.performance_test
9696
fi
9797
shell: bash
98-
99-
# setting up up docker containers currently fails
100-
# dockertests:
101-
# runs-on: ubuntu-latest
102-
# strategy:
103-
# fail-fast: false
104-
# matrix:
105-
# docker-image: [centos, debian, fedora, ubuntu]
106-
# steps:
107-
# - uses: actions/checkout@v2
108-
# - name: Setup docker container
109-
# run: |
110-
# docker build -t pyfakefs -f $GITHUB_WORKSPACE/.github/workflows/dockerfiles/Dockerfile_${{ matrix.docker-image }} . --build-arg github_repo=$GITHUB_REPOSITORY --build-arg github_branch=$(basename $GITHUB_REF)
111-
# - name: Run tests
112-
# run: docker run -t pyfakefs

0 commit comments

Comments
 (0)