Add e2e tests #70

vwxyzjn · 2021-10-22T16:33:33Z

This PR adds end-to-end tests to make sure the CleanRL scripts run without errors. The test case is basically running

subprocess.run("python cleanrl/ppo.py --total-timesteps 1000", shell=True, check=True)

[ ] only trigger the run when changes to the source code happen.

cleanrl/ddpg_continuous_action.py

cleanrl/sac_continuous_action.py

cleanrl/td3_continuous_action.py

tests/test_atari.py

yooceii · 2021-10-26T05:13:13Z

tests/test_classic_control.py

+
+def test_ppo():
+    subprocess.run(
+        "python cleanrl/ppo.py --num-envs 1 --num-steps 64 --total-timesteps 256",


tests/test_pybullet.py

vwxyzjn force-pushed the e2e-test branch from f27b969 to adc2c86 Compare October 22, 2021 17:40

vwxyzjn requested review from dosssman and yooceii October 22, 2021 18:16

vwxyzjn added 8 commits October 22, 2021 14:29

Add e2e tests

c0c82be

update tests

01ecd20

test

6c01d05

update pytest

5659c2f

test atari

e1b7c18

update ci

cf9cc26

add continuous action tests

d652a29

quick fix

38e0aea

vwxyzjn force-pushed the e2e-test branch from 51b60cd to 38e0aea Compare October 22, 2021 18:30

update test cases to save compute

1cc117c

yooceii reviewed Oct 26, 2021

View reviewed changes

revert changes

708ffa2

yooceii approved these changes Oct 27, 2021

View reviewed changes

vwxyzjn merged commit 87d4e87 into master Oct 27, 2021

vwxyzjn deleted the e2e-test branch October 27, 2021 02:27

Provide feedback