Skip to content

Commit a7205d5

Browse files
committed
Make sure correct results were pusblished.
Signed-off-by: Jay Zhang <jiz@microsoft.com>
1 parent 83219cb commit a7205d5

File tree

4 files changed

+25
-47
lines changed

4 files changed

+25
-47
lines changed

.github/workflows/keras_application_test_core.yml

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -154,15 +154,10 @@ jobs:
154154
pull-requests: write
155155
if: always()
156156
steps:
157-
- name: Download Artifacts
158-
uses: actions/download-artifact@v3
159-
with:
160-
path: artifacts
161-
162157
- name: Publish Test Results
163158
uses: EnricoMi/publish-unit-test-result-action@v2
164159
with:
165-
files: "artifacts/**/*.xml"
160+
files: "junit/test-results.xml"
166161

167162
publish-test-results-tf-15:
168163
name: "Publish Tests Results(TF15) to Github"
@@ -173,12 +168,7 @@ jobs:
173168
pull-requests: write
174169
if: always()
175170
steps:
176-
- name: Download Artifacts
177-
uses: actions/download-artifact@v3
178-
with:
179-
path: artifacts
180-
181171
- name: Publish Test Results
182172
uses: EnricoMi/publish-unit-test-result-action@v2
183173
with:
184-
files: "artifacts/**/*.xml"
174+
files: "junit/test-results.xml"

.github/workflows/keras_unit_test_core.yml

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -113,15 +113,10 @@ jobs:
113113
pull-requests: write
114114
if: always()
115115
steps:
116-
- name: Download Artifacts
117-
uses: actions/download-artifact@v3
118-
with:
119-
path: artifacts
120-
121116
- name: Publish Test Results
122117
uses: EnricoMi/publish-unit-test-result-action@v2
123118
with:
124-
files: "artifacts/**/*.xml"
119+
files: "junit/test-results.xml"
125120

126121
publish-test-results-tf-15:
127122
name: "Publish Tests Results(TF15) to Github"
@@ -132,12 +127,7 @@ jobs:
132127
pull-requests: write
133128
if: always()
134129
steps:
135-
- name: Download Artifacts
136-
uses: actions/download-artifact@v3
137-
with:
138-
path: artifacts
139-
140130
- name: Publish Test Results
141131
uses: EnricoMi/publish-unit-test-result-action@v2
142132
with:
143-
files: "artifacts/**/*.xml"
133+
files: "junit/test-results.xml"

.github/workflows/pretrained_model_tests_core.yml

Lines changed: 18 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -47,22 +47,23 @@ jobs:
4747
# TODO: fix unity model path
4848
# python tests/run_pretrained_models.py --backend $CI_ONNX_BACKEND --opset $CI_ONNX_OPSET --config tests/unity.yaml || status=$?
4949
python tests/run_pretrained_models.py --backend onnxruntime --opset ${{ matrix.opset_version }} --skip_tf_tests False --skip_tflite_tests False --skip_tfjs_tests True --config tests/run_pretrained_models.yaml || status=$?
50+
ls *.xml
5051
51-
publish-test-results:
52-
name: "Publish Tests Results to Github"
53-
needs: run_tests
54-
runs-on: ubuntu-latest
55-
permissions:
56-
checks: write
57-
pull-requests: write
58-
if: always()
59-
steps:
60-
- name: Download Artifacts
61-
uses: actions/download-artifact@v3
62-
with:
63-
path: artifacts
52+
# publish-test-results:
53+
# name: "Publish Tests Results to Github"
54+
# needs: run_tests
55+
# runs-on: ubuntu-latest
56+
# permissions:
57+
# checks: write
58+
# pull-requests: write
59+
# if: always()
60+
# steps:
61+
# - name: Download Artifacts
62+
# uses: actions/download-artifact@v3
63+
# with:
64+
# path: artifacts
6465

65-
- name: Publish Test Results
66-
uses: EnricoMi/publish-unit-test-result-action@v2
67-
with:
68-
files: "artifacts/**/*.xml"
66+
# - name: Publish Test Results
67+
# uses: EnricoMi/publish-unit-test-result-action@v2
68+
# with:
69+
# files: "**/pytest*.xml"

.github/workflows/unit_tests_core.yml

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ jobs:
5252
export TF2ONNX_SKIP_TFJS_TESTS=True
5353
export TF2ONNX_SKIP_TF_TESTS=False
5454
python -m pytest --cov=tf2onnx --cov-report=term --disable-pytest-warnings -r s tests --cov-append
55+
ls
5556
5657
- name: Run unit_test (Windows)
5758
if: runner.os == 'Windows'
@@ -62,6 +63,7 @@ jobs:
6263
set TF2ONNX_SKIP_TFJS_TESTS=True
6364
set TF2ONNX_SKIP_TF_TESTS=False
6465
python -m pytest --cov=tf2onnx --cov-report=term --disable-pytest-warnings -r s tests --cov-append
66+
ls
6567
6668
publish-test-results:
6769
name: "Publish Tests Results to Github"
@@ -72,12 +74,7 @@ jobs:
7274
pull-requests: write
7375
if: always()
7476
steps:
75-
- name: Download Artifacts
76-
uses: actions/download-artifact@v3
77-
with:
78-
path: artifacts
79-
8077
- name: Publish Test Results
8178
uses: EnricoMi/publish-unit-test-result-action@v2
8279
with:
83-
files: "artifacts/**/*.xml"
80+
files: "**/test-results.xml"

0 commit comments

Comments
 (0)