Fix test #211

Activity

1. helmut-hoffer-von-ankershoffen

   assigned

   Copilot

   on Oct 27, 2025
2. github-actions commented on Oct 27, 2025

   

   github-actionsBot

   on Oct 27, 2025 – with GitHub Actions

   Last edited by github-actions

   Contributor

   More actions

   Claude finished @helmut-hoffer-von-ankershoffen's task —— View job • claude/issue-211-20251027-0603 • Create PR ➔

   ---

   Fixing platform e2e test ✅

   Fixed the missing argument issue in tests/aignostics/platform/e2e_test.py.

   Tasks

   • Locate the function and understand its signature
   • Find where it's being called incorrectly
   • Fix the function call
   • Run linting to verify
   • Commit and push changes
   • Provide PR link

   Changes Made

   • File: tests/aignostics/platform/e2e_test.py (line 302)
   • Issue: _get_three_spots_payload_for_test() was called without required expires_seconds argument
   • Fix: Added expires_seconds=TEST_APPLICATION_SUBMIT_AND_FIND_DEADLINE_SECONDS + 60 * 5
   • Pattern: Consistent with other test calls in the same file (lines 351-353, 377-379)

   Verification

   All linting checks passed:

   • ✅ Ruff formatting
   • ✅ Ruff linting
   • ✅ PyRight type checking
   • ✅ MyPy type checking

   --- • Branch
3. github-actions

   added a commit that references this issue on Oct 27, 2025

   fix: add missing expires_seconds argument to _get_three_spots_payload…

   ...

   This issue will close once commit 1ff8ec1 is merged into the 'main' branch.

   1ff8ec1
4. helmut-hoffer-von-ankershoffen

   closed this as completedin #213on Oct 27, 2025
5. helmut-hoffer-von-ankershoffen

   added 3 commits that reference this issue on Oct 27, 2025

   fix: add missing expires_seconds argument to _get_three_spots_payload…

   ...

   This issue will close once commit d62a27f is merged into the 'main' branch.

   d62a27f

   fix: add missing expires_seconds argument to _get_three_spots_payload…

   ...

   This issue will close once commit 78b4b63 is merged into the 'main' branch.

   78b4b63

   fix: add missing expires_seconds argument to _get_three_spots_payload…

   ...

   This issue will close once commit 81e0c79 is merged into the 'main' branch.

   81e0c79