From c8fc49387968e95b29cc80f606af914746533eb9 Mon Sep 17 00:00:00 2001 From: Igor Kuznetsov Date: Sat, 5 Jul 2025 16:53:28 +0400 Subject: [PATCH 1/4] Add tests for safety gymnasium --- .github/workflows/unit-tests-docker.yml | 2 -- tests/functional/test_env.py | 2 +- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/.github/workflows/unit-tests-docker.yml b/.github/workflows/unit-tests-docker.yml index 3158532..c3156c6 100644 --- a/.github/workflows/unit-tests-docker.yml +++ b/.github/workflows/unit-tests-docker.yml @@ -1,7 +1,5 @@ name: Unit Tests on: - push: - branches: [ "master", "develop"] pull_request: branches: [ "master", "develop"] diff --git a/tests/functional/test_env.py b/tests/functional/test_env.py index 2cf2ec2..1facf87 100644 --- a/tests/functional/test_env.py +++ b/tests/functional/test_env.py @@ -39,7 +39,7 @@ ] -env_names: list[str] = dm_control_envs # + safety_envs +env_names: list[str] = dm_control_envs + safety_envs @pytest.mark.parametrize("env_name", env_names) From 11df8343d6d03a4e3ee751f8a9fd558550f1df9d Mon Sep 17 00:00:00 2001 From: Igor Kuznetsov Date: Sat, 5 Jul 2025 17:07:27 +0400 Subject: [PATCH 2/4] Install SafetyGymnasium into Dockerfile --- Dockerfile | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/Dockerfile b/Dockerfile index 029e0f1..688df96 100644 --- a/Dockerfile +++ b/Dockerfile @@ -4,6 +4,13 @@ WORKDIR /app RUN pip install --no-cache-dir --upgrade pip +# Install SafetyGymansium from external lib +RUN wget https://github.com/PKU-Alignment/safety-gymnasium/archive/refs/heads/main.zip && \ + unzip main.zip && \ + cd safety-gymnasium-main && \ + pip install . && \ + cd .. + COPY . . RUN pip install --no-cache-dir . && pip install pytest pytest-cov From 3d9e5dec585e8ed63e442d7263d2a543738ddf27 Mon Sep 17 00:00:00 2001 From: Igor Kuznetsov Date: Sat, 5 Jul 2025 17:13:48 +0400 Subject: [PATCH 3/4] Change tests coverage in CI --- .github/workflows/unit-tests-docker.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/unit-tests-docker.yml b/.github/workflows/unit-tests-docker.yml index c3156c6..16fae70 100644 --- a/.github/workflows/unit-tests-docker.yml +++ b/.github/workflows/unit-tests-docker.yml @@ -20,7 +20,7 @@ jobs: - name: Extract coverage run: | docker run --rm -v $(pwd):/host oprl sh -c " - pytest --cov=oprl --cov-report=xml && + pytest tests/funcational --cov=oprl --cov-report=xml && cp coverage.xml /host/ " From c47a2b79905d21bc29647632a05f91717b9870b6 Mon Sep 17 00:00:00 2001 From: Igor Kuznetsov Date: Sat, 5 Jul 2025 17:17:59 +0400 Subject: [PATCH 4/4] Fix typo --- .github/workflows/unit-tests-docker.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/unit-tests-docker.yml b/.github/workflows/unit-tests-docker.yml index 16fae70..7adeafd 100644 --- a/.github/workflows/unit-tests-docker.yml +++ b/.github/workflows/unit-tests-docker.yml @@ -20,7 +20,7 @@ jobs: - name: Extract coverage run: | docker run --rm -v $(pwd):/host oprl sh -c " - pytest tests/funcational --cov=oprl --cov-report=xml && + pytest tests/functional --cov=oprl --cov-report=xml && cp coverage.xml /host/ "