Skip to content

Commit cbe952e

Browse files
lianhaoashahba
andauthored
Fail CI manifest test if response content is not expected (#1145)
Signed-off-by: Lianhao Lu <[email protected]> Co-authored-by: Abolfazl Shahbazi <[email protected]>
1 parent 3b1a9fe commit cbe952e

File tree

6 files changed

+8
-2
lines changed

6 files changed

+8
-2
lines changed

CodeGen/tests/test_manifest_on_gaudi.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ function validate_codegen() {
5151

5252
if [ $status == false ]; then
5353
echo "Response check failed, please check the logs in artifacts!"
54+
exit 1
5455
else
5556
echo "Response check succeed!"
5657
fi

CodeGen/tests/test_manifest_on_xeon.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ function validate_codegen() {
5151

5252
if [ $status == false ]; then
5353
echo "Response check failed, please check the logs in artifacts!"
54+
exit 1
5455
else
5556
echo "Response check succeed!"
5657
fi

CodeTrans/tests/test_manifest_on_gaudi.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ function validate_codetrans() {
5252

5353
if [ $status == false ]; then
5454
echo "Response check failed, please check the logs in artifacts!"
55+
exit 1
5556
else
5657
echo "Response check succeed!"
5758
fi

CodeTrans/tests/test_manifest_on_xeon.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ function validate_codetrans() {
5252

5353
if [ $status == false ]; then
5454
echo "Response check failed, please check the logs in artifacts!"
55+
exit 1
5556
else
5657
echo "Response check succeed!"
5758
fi

DocSum/tests/test_manifest_on_gaudi.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,12 +46,13 @@ function validate_docsum() {
4646
echo "Checking response results, make sure the output is reasonable. "
4747
local status=false
4848
if [[ -f $LOGFILE ]] && \
49-
[[ $(grep -c "versatile toolkit" $LOGFILE) != 0 ]]; then
49+
[[ $(grep -c "embedding" $LOGFILE) != 0 ]]; then
5050
status=true
5151
fi
5252

5353
if [ $status == false ]; then
5454
echo "Response check failed, please check the logs in artifacts!"
55+
exit 1
5556
else
5657
echo "Response check succeed!"
5758
fi

DocSum/tests/test_manifest_on_xeon.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,12 +46,13 @@ function validate_docsum() {
4646
echo "Checking response results, make sure the output is reasonable. "
4747
local status=false
4848
if [[ -f $LOGFILE ]] && \
49-
[[ $(grep -c "versatile toolkit" $LOGFILE) != 0 ]]; then
49+
[[ $(grep -c "embedding" $LOGFILE) != 0 ]]; then
5050
status=true
5151
fi
5252

5353
if [ $status == false ]; then
5454
echo "Response check failed, please check the logs in artifacts!"
55+
exit 1
5556
else
5657
echo "Response check succeed!"
5758
fi

0 commit comments

Comments
 (0)