Skip to content

Added more tests#2052

Open
jessevz wants to merge 3 commits intodevfrom
improve-tests
Open

Added more tests#2052
jessevz wants to merge 3 commits intodevfrom
improve-tests

Conversation

@jessevz
Copy link
Copy Markdown
Contributor

@jessevz jessevz commented Apr 29, 2026

hashtopolis/python-hashtopolis#2 should be merged first otherwise tests cannot pass

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds additional APIv2 CI tests, primarily focused on ACL/list visibility checks and introducing coverage for the ApiToken (JWT API key) resource.

Changes:

  • Added ApiToken test object creation support and a new ApiToken test suite (including token-field behavior checks).
  • Introduced a shared ACL list-visibility helper plus multiple test_acl cases across existing model tests.
  • Cleaned up a couple of unused imports in existing tests.

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
ci/apiv2/utils.py Adds ApiToken test-object creation, restricted-user helper, and a reusable ACL list-visibility assertion helper.
ci/apiv2/testfiles/apitoken/create_apitoken_001.json Adds JSON payload fixture for creating an ApiToken in tests.
ci/apiv2/test_apitoken.py New test suite for ApiToken CRUD/behavior (token returned only on create, ACL list behavior, etc.).
ci/apiv2/test_taskwrapper.py Adds ACL list test for TaskWrapper.
ci/apiv2/test_task.py Adds ACL list test for Task.
ci/apiv2/test_hashlist.py Adds ACL list test for Hashlist.
ci/apiv2/test_file.py Adds ACL list test for File.
ci/apiv2/test_agentstat.py Adds ACL list test for AgentStat.
ci/apiv2/test_agentassignment.py Removes unused import and adds ACL list test for AgentAssignment.
ci/apiv2/test_agent.py Adds ACL list test for Agent.
ci/apiv2/test_pretask.py Removes unused import.
ci/apiv2/test_hash.py Removes unused import.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ci/apiv2/utils.py Outdated
Comment thread ci/apiv2/test_apitoken.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants