OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

Add GitHub Action for pytest

Open rbren opened this issue 1 year ago • 6 comments

rbren avatar Mar 30 '24 16:03 rbren

I think tests are failing because we're missing pytest in the pipenv file.

enyst avatar Apr 01 '24 19:04 enyst

Good catch! 👀

rbren avatar Apr 02 '24 03:04 rbren

This seems relevant: https://stackoverflow.com/questions/54012510/pipenv-shell-in-jenkins-return-the-issue-termios-error-25-inappropriate-ioct#54012557

enyst avatar Apr 02 '24 12:04 enyst

this is shockingly hard 😭

rbren avatar Apr 02 '24 22:04 rbren

It works! Okay, it fails, but it starts tests.

Just in time for the switch to poetry. 😅

Edit: https://github.com/OpenDevin/OpenDevin/pull/620/commits/c0aaddb86963ee864c1803191319aaf5d9962172

enyst avatar Apr 03 '24 00:04 enyst

@rbren https://github.com/dorbanianas/OpenDevin/actions/runs/8532623697/job/23374074085 you can check this the branch is ready and it passes the tests

anasdorbani avatar Apr 03 '24 03:04 anasdorbani

I guess now the main checks the make build and also run tests so we need to think about more regression tests to have

anasdorbani avatar Apr 05 '24 02:04 anasdorbani

it's working

rbren avatar Apr 05 '24 18:04 rbren

it's working

you made it 😁

anasdorbani avatar Apr 05 '24 22:04 anasdorbani