Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

End to end test workflow #30

Merged
merged 77 commits into from
Sep 16, 2024
Merged
Changes from 1 commit
Commits
Show all changes
77 commits
Select commit Hold shift + click to select a range
927adc6
End-to-end test workflow
Brainicism Sep 15, 2024
591bb45
Chmod
Brainicism Sep 15, 2024
46f382e
Capture stderr
Brainicism Sep 15, 2024
d37e6a0
fix 2>&1 position
Brainicism Sep 15, 2024
801f146
cd ..
Brainicism Sep 15, 2024
50c7555
Update .github/workflows/test.yml
grqz Sep 15, 2024
d4afa2d
wait for server up
Brainicism Sep 15, 2024
2586ff5
Merge branch 'e2e-test' of github.com:Brainicism/bgutil-ytdlp-pot-pro…
Brainicism Sep 15, 2024
50d8576
move `lint and format check` to test.yml
grqz Sep 15, 2024
375254b
be consistent: search for corresponding msg
grqz Sep 15, 2024
f24b523
use github-styled error output
grqz Sep 15, 2024
7731931
[wrong] test
grqz Sep 15, 2024
a196a52
[wrong] fix gh error msg
grqz Sep 15, 2024
fd5d643
[wrong] dont report file path
grqz Sep 15, 2024
5f5f2b9
[wrong] fill required field `title`
grqz Sep 15, 2024
45a6f67
[wrong] test
grqz Sep 15, 2024
dbb6d1a
[wrong] test: dont exit with 1
grqz Sep 15, 2024
c761f42
[wrong] handle non-zero exit code from ytdlp
grqz Sep 15, 2024
8b61f0f
[wrong] write error titles
grqz Sep 15, 2024
6472d51
[wrong] format ping result with jq
grqz Sep 15, 2024
3edd410
[wrong] format ping result with jq
grqz Sep 15, 2024
6fbe394
[wrong] display source when yt-dlp fails
grqz Sep 15, 2024
9bee51e
[wrong] test github styled output: ::error
grqz Sep 15, 2024
6a248a4
[wrong] add if:always()
grqz Sep 15, 2024
338a52f
[wrong] exit 1, try omitting line/column
grqz Sep 15, 2024
37eebb5
[wrong] remove line/column
grqz Sep 15, 2024
5b4dfa5
[wrong] remove redundant logging, code formatting
grqz Sep 15, 2024
fc67dd9
[wrong] correct one intentional issue
grqz Sep 15, 2024
e3f12e0
[wrong] fix: path doesnt exist
grqz Sep 15, 2024
a4a49dd
[wrong] group logs, support multi-line logs
grqz Sep 15, 2024
d8c9254
[correct] remove intentional raise
grqz Sep 15, 2024
2d1e6dd
dont use default path: interferes with HTTP test
grqz Sep 15, 2024
6af7872
capture server log
grqz Sep 15, 2024
1892ede
remove several unnecessary `always()`
grqz Sep 15, 2024
f0ddaac
[wrong:for testing] add timer for http server
grqz Sep 15, 2024
ff2c4ba
try to fix trailing spaces
grqz Sep 15, 2024
9b94c4a
try to fix trailing spaces
grqz Sep 15, 2024
44a6a18
[wrong] test ci
grqz Sep 15, 2024
535a213
remove intentional `raise`s
grqz Sep 15, 2024
755e888
[wrong] test with problematic server
grqz Sep 15, 2024
94e9ddb
[wrong:server] exit when timeout reached
grqz Sep 15, 2024
e54276b
[wrong:server] kill the process after being pinged
grqz Sep 15, 2024
bdde8c1
check if process is running before killing
grqz Sep 15, 2024
8d82fce
fix var name
grqz Sep 15, 2024
40a6840
fix everything wrong
grqz Sep 15, 2024
0a768bb
--no-simulate
grqz Sep 15, 2024
d00f5b5
fix: display ping response properly
grqz Sep 15, 2024
8924261
fix: `tee` output
grqz Sep 15, 2024
abc3c43
export ping rresp path
grqz Sep 15, 2024
189e52b
override PING_RESP_PATH instead
grqz Sep 15, 2024
4266e84
dont use tmp file, capture output instead
grqz Sep 15, 2024
3321f5a
use echo for strings
grqz Sep 15, 2024
fefe0c1
dont fmt with jq
grqz Sep 15, 2024
8150f2b
retry
grqz Sep 15, 2024
315a315
debug
grqz Sep 15, 2024
c845dc4
wrong var name
grqz Sep 15, 2024
4feecbc
use jq to parse resp
grqz Sep 15, 2024
714326a
use a larger video
grqz Sep 15, 2024
0ca4221
Update .github/workflows/test.yml
grqz Sep 15, 2024
be5e87a
Update .github/workflows/test.yml
grqz Sep 15, 2024
b5521e5
Use docker for server method
Brainicism Sep 15, 2024
14f4618
@commander-js/extra-typings as non-dev dependecy
Brainicism Sep 15, 2024
6d7e2eb
remove ..
Brainicism Sep 15, 2024
5d4f212
Seperate script/server method into different jobs
Brainicism Sep 15, 2024
e165688
Remove if always in server mode test
Brainicism Sep 15, 2024
c4c9b9a
Remove release.yml from pull requests
Brainicism Sep 15, 2024
01843cc
Docker stop instead of killing pid
Brainicism Sep 15, 2024
c3fef69
Code review changes
Brainicism Sep 15, 2024
5273546
Load: true
Brainicism Sep 15, 2024
2c94d0c
fiix jq error
grqz Sep 16, 2024
95659d5
use colours in jq
grqz Sep 16, 2024
0d14de1
add colour to yt-dlp; use env; cleanup
grqz Sep 16, 2024
8807255
fix port
grqz Sep 16, 2024
08cd278
use numbers in env
grqz Sep 16, 2024
5fd95f8
separate lines in setup script; cleanup
grqz Sep 16, 2024
cd49c82
quote args
grqz Sep 16, 2024
cb94051
use 0.4
grqz Sep 16, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,10 @@ jobs:
cd server/
node build/main.js&
cd ..
until curl --silent --fail http://127.0.0.1:4416/ping; do
echo "Waiting for server to be up..."
grqz marked this conversation as resolved.
Show resolved Hide resolved
sleep 5
grqz marked this conversation as resolved.
Show resolved Hide resolved
done
script_response=$(./yt-dlp -vF --print-traffic BaW_jenozKc --extractor-args "youtube:getpot_bgutil_baseurl=http://127.0.0.1:4416 2>&1")
grqz marked this conversation as resolved.
Show resolved Hide resolved
grqz marked this conversation as resolved.
Show resolved Hide resolved
echo $script_response
if [[ "$script_response" != *"BgUtilHTTPPot: Generating POT via HTTP server"* ]]; then
Expand Down