End to end test workflow #30

Brainicism · 2024-09-15T03:02:46Z

EDIT by @grqz: pinned info:
examples:

RH's not invoked https://github.com/Brainicism/bgutil-ytdlp-pot-provider/actions/runs/10869148543/job/30159954418
Passing: https://github.com/Brainicism/bgutil-ytdlp-pot-provider/actions/runs/10870654853/job/30163532307
HTTP Server not transpiling: https://github.com/Brainicism/bgutil-ytdlp-pot-provider/actions/runs/10869223803/job/30160127984
HTTP Server terminates midway after being pinged: https://github.com/Brainicism/bgutil-ytdlp-pot-provider/actions/runs/10869309089/job/30160329617

Note

some of them may not be up-to-date

.github/workflows/test.yml

…vider into e2e-test

.github/workflows/test.yml

- show `waiting for the server to be up` only once - reduce timeout - sleep less per loop

wrong: this commit contains intentional issues or reserved a bug from previous commits intentionally in order to test the ci

.github/workflows/test.yml

grqz · 2024-09-15T23:27:20Z

why is https://github.com/Brainicism/bgutil-ytdlp-pot-provider/actions/runs/10875000151/job/30172911791 failing?

Brainicism · 2024-09-15T23:34:32Z

why is https://github.com/Brainicism/bgutil-ytdlp-pot-provider/actions/runs/10875000151/job/30172911791 failing?

not sure..

grqz · 2024-09-16T00:23:14Z

i see. it seems to be because we're giving the response of function ping to jq. we should still keep the echo command out. probably before the timeout. but not sure why it succeeded on the latest commit. maybe the server was starting up too fast that the body is not run at all(?)

grqz

LGTM

End-to-end test workflow

927adc6

Brainicism force-pushed the e2e-test branch from e15b387 to 927adc6 Compare September 15, 2024 03:04

Chmod

591bb45

This comment was marked as resolved.

Sign in to view

Brainicism changed the title ~~E2e test~~ End to end test workflow Sep 15, 2024

Brainicism added 2 commits September 14, 2024 20:26

Capture stderr

46f382e

fix 2>&1 position

d37e6a0

grqz reviewed Sep 15, 2024

View reviewed changes

.github/workflows/test.yml Outdated Show resolved Hide resolved

grqz reviewed Sep 15, 2024

View reviewed changes

.github/workflows/test.yml Outdated Show resolved Hide resolved

cd ..

801f146

grqz reviewed Sep 15, 2024

View reviewed changes

.github/workflows/test.yml Outdated Show resolved Hide resolved

grqz and others added 3 commits September 15, 2024 15:31

Update .github/workflows/test.yml

50c7555

wait for server up

d4afa2d

Merge branch 'e2e-test' of github.com:Brainicism/bgutil-ytdlp-pot-pro…

2586ff5

…vider into e2e-test

grqz reviewed Sep 15, 2024

View reviewed changes

.github/workflows/test.yml Outdated Show resolved Hide resolved

grqz reviewed Sep 15, 2024

View reviewed changes

.github/workflows/test.yml Outdated Show resolved Hide resolved

grqz reviewed Sep 15, 2024

View reviewed changes

.github/workflows/test.yml Outdated Show resolved Hide resolved

grqz added 13 commits September 15, 2024 15:56

move lint and format check to test.yml

50d8576

be consistent: search for corresponding msg

375254b

use github-styled error output

f24b523

- show `waiting for the server to be up` only once - reduce timeout - sleep less per loop

[wrong] test

7731931

[wrong] fix gh error msg

a196a52

[wrong] dont report file path

fd5d643

wrong: this commit contains intentional issues or reserved a bug from previous commits intentionally in order to test the ci

[wrong] fill required field title

5f5f2b9

wrong: this commit contains intentional issues or reserved a bug from previous commits intentionally in order to test the ci

[wrong] test

45a6f67

wrong: this commit contains intentional issues or reserved a bug from previous commits intentionally in order to test the ci

[wrong] test: dont exit with 1

dbb6d1a

wrong: this commit contains intentional issues or reserved a bug from previous commits intentionally in order to test the ci

[wrong] handle non-zero exit code from ytdlp

c761f42

wrong: this commit contains intentional issues or reserved a bug from previous commits intentionally in order to test the ci

[wrong] write error titles

8b61f0f

wrong: this commit contains intentional issues or reserved a bug from previous commits intentionally in order to test the ci

[wrong] format ping result with jq

6472d51

wrong: this commit contains intentional issues or reserved a bug from previous commits intentionally in order to test the ci

[wrong] format ping result with jq

3edd410

wrong: this commit contains intentional issues or reserved a bug from previous commits intentionally in order to test the ci

grqz reviewed Sep 15, 2024

View reviewed changes

.github/workflows/test.yml Outdated Show resolved Hide resolved

.github/workflows/test.yml Outdated Show resolved Hide resolved

grqz and others added 8 commits September 16, 2024 01:02

Update .github/workflows/test.yml

0ca4221

Update .github/workflows/test.yml

be5e87a

Use docker for server method

b5521e5

@commander-js/extra-typings as non-dev dependecy

14f4618

remove ..

6d7e2eb

Seperate script/server method into different jobs

5d4f212

Remove if always in server mode test

e165688

Remove release.yml from pull requests

c4c9b9a

grqz reviewed Sep 15, 2024

View reviewed changes

.github/workflows/test.yml Outdated Show resolved Hide resolved

Docker stop instead of killing pid

01843cc

grqz reviewed Sep 15, 2024

View reviewed changes

.github/workflows/test.yml Outdated Show resolved Hide resolved

grqz reviewed Sep 15, 2024

View reviewed changes

.github/workflows/test.yml Outdated Show resolved Hide resolved

grqz reviewed Sep 15, 2024

View reviewed changes

.github/workflows/test.yml Show resolved Hide resolved

Brainicism added 2 commits September 15, 2024 16:03

Code review changes

c3fef69

Load: true

5273546

grqz added 8 commits September 16, 2024 15:14

fiix jq error

2c94d0c

use colours in jq

95659d5

add colour to yt-dlp; use env; cleanup

0d14de1

fix port

8807255

use numbers in env

08cd278

separate lines in setup script; cleanup

5fd95f8

quote args

cd49c82

use 0.4

cb94051

grqz approved these changes Sep 16, 2024

View reviewed changes

Brainicism merged commit be847ed into master Sep 16, 2024
3 checks passed

Brainicism deleted the e2e-test branch December 25, 2024 06:28

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

End to end test workflow #30

End to end test workflow #30

Brainicism commented Sep 15, 2024 •

edited by grqz

Loading

This comment was marked as resolved.

grqz commented Sep 15, 2024

Brainicism commented Sep 15, 2024

grqz commented Sep 16, 2024 •

edited

Loading

grqz left a comment

End to end test workflow #30

End to end test workflow #30

Conversation

Brainicism commented Sep 15, 2024 • edited by grqz Loading

This comment was marked as resolved.

grqz commented Sep 15, 2024

Brainicism commented Sep 15, 2024

grqz commented Sep 16, 2024 • edited Loading

grqz left a comment

Choose a reason for hiding this comment

Brainicism commented Sep 15, 2024 •

edited by grqz

Loading

grqz commented Sep 16, 2024 •

edited

Loading