Skip to content

Commit

Permalink
Merge pull request #1592 from private-octopus/picohttp_t_comments
Browse files Browse the repository at this point in the history
Always trace picohhtp_t tests
  • Loading branch information
huitema authored Dec 8, 2023
2 parents fead7cb + 5624fbf commit ea3390d
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .github/workflows/ci-asan-ubsan.yml
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,8 @@ jobs:
else
echo "No leaks detected in picoquic_ct"
fi
./picohttp_ct -n -r 1>http_ct.txt 2>sanity.txt || QUICHTTPRESULT=$?
#./picohttp_ct -n -r 1>http_ct.txt 2>sanity.txt || QUICHTTPRESULT=$?
./picohttp_ct -n -r 2>sanity.txt || QUICHTTPRESULT=$?
echo "running picohttp_ct returns <$QUICHTTPRESULT> "
cat sanity.txt
if [ ! -z ${QUICHTTPRESULT} ]; then
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/ci-tests-no-fusion.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ jobs:
make test && QUICRESULT=$?
cd ..
if [[ ${QUICRESULT} == 0 ]]; then exit 0; fi;
cat /home/runner/work/picoquic/picoquic/build/Testing/Temporary/LastTest.log
exit 1
#TODO: reset the test once openssl issue is understood.
Expand Down
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -22,3 +22,4 @@ Release
*/*.ilk
*.b
/picoquic/qlogger.c
/picohttp_t/h3-m-www
1 change: 1 addition & 0 deletions picoquictest/webtransport_test.c
Original file line number Diff line number Diff line change
Expand Up @@ -124,6 +124,7 @@ static int picowt_baton_test_one(
ret = 1;
}
else {
h3zero_cb->no_print = 1;
picoquic_set_callback(test_ctx->cnx_client, h3zero_callback, h3zero_cb);
/* Initialize the callback context. First, create a bidir stream */
wt_baton_ctx_init(&baton_ctx, h3zero_cb, NULL, NULL);
Expand Down

0 comments on commit ea3390d

Please sign in to comment.