Rename build to test in CI. Fix failing tests after the language_id to language change.

fkdosilovic · fkdosilovic · commit 202ce73a11a2 · 2024-11-14T23:12:08.000+01:00
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -8,7 +8,7 @@ permissions:
   contents: read
 
 jobs:
-  build:
+  test:
     runs-on: ubuntu-latest
 
     steps:
@@ -28,5 +28,9 @@ jobs:
           JUDGE0_ATD_API_KEY: ${{ secrets.JUDGE0_ATD_API_KEY }}
           JUDGE0_RAPID_API_KEY: ${{ secrets.JUDGE0_RAPID_API_KEY }}
           JUDGE0_SULU_API_KEY: ${{ secrets.JUDGE0_SULU_API_KEY }}
+          JUDGE0_TEST_API_KEY: ${{ secrets.JUDGE0_TEST_API_KEY }}
+          JUDGE0_TEST_API_KEY_HEADER: ${{ secrets.JUDGE0_TEST_API_KEY_HEADER }}
+          JUDGE0_TEST_CE_ENDPOINT: ${{ secrets.JUDGE0_TEST_CE_ENDPOINT }}
+          JUDGE0_TEST_EXTRA_CE_ENDPOINT: ${{ secrets.JUDGE0_TEST_EXTRA_CE_ENDPOINT }}
         run: |
           pipenv run pytest -vv
diff --git a/src/judge0/clients.py b/src/judge0/clients.py
@@ -147,13 +147,7 @@ def create_submissions(self, submissions: list[Submission]) -> list[Submission]:
                     f"{submission.language}!"
                 )
 
-        submissions_body = [submission.to_dict() for submission in submissions]
-        # We have to resolve language_id because language_id can be Language
-        # enumeration.
-        for submission_body in submissions_body:
-            submission_body["language_id"] = self.resolve_language_id(
-                submission_body["language_id"]
-            )
+        submissions_body = [submission.to_dict(self) for submission in submissions]
 
         resp = requests.post(
             f"{self.endpoint}/submissions/batch",
diff --git a/tests/test_api.py b/tests/test_api.py
@@ -57,12 +57,12 @@ def test_resolve_client_empty_submissions_argument(submissions):
 def test_resolve_client_no_common_client_for_submissions():
     cpp_submission = Submission(
         source_code="",  # source code is not important in this test
-        language_id=LanguageAlias.CPP_GCC,
+        language=LanguageAlias.CPP_GCC,
     )
 
     py_submission = Submission(
         source_code="",  # source code is not important in this test
-        language_id=LanguageAlias.PYTHON_FOR_ML,
+        language=LanguageAlias.PYTHON_FOR_ML,
     )
 
     submissions = [cpp_submission, py_submission]
@@ -74,12 +74,12 @@ def test_resolve_client_no_common_client_for_submissions():
 def test_resolve_client_common_ce_client():
     cpp_submission = Submission(
         source_code="",  # source code is not important in this test
-        language_id=LanguageAlias.CPP_GCC,
+        language=LanguageAlias.CPP_GCC,
     )
 
     py_submission = Submission(
         source_code="",  # source code is not important in this test
-        language_id=LanguageAlias.PYTHON,
+        language=LanguageAlias.PYTHON,
     )
 
     submissions = [cpp_submission, py_submission]
@@ -90,12 +90,12 @@ def test_resolve_client_common_ce_client():
 def test_resolve_client_common_extra_ce_client():
     cpp_submission = Submission(
         source_code="",  # source code is not important in this test
-        language_id=LanguageAlias.CPP_CLANG,
+        language=LanguageAlias.CPP_CLANG,
     )
 
     py_submission = Submission(
         source_code="",  # source code is not important in this test
-        language_id=LanguageAlias.PYTHON_FOR_ML,
+        language=LanguageAlias.PYTHON_FOR_ML,
     )
 
     submissions = [cpp_submission, py_submission]
