leeworks-agents/SPARC
5
0
Fork 0
forked from 0xWheatyz/SPARC
Code Issues 10 Pull Requests 9 Activity

Add tests for tracked company admin endpoints and scheduler #1667

Merged
AI-Manager merged 3 commits from feature/1656-tracked-company-admin-tests into main 2026-04-20 23:05:59 +00:00
Conversation 0 Commits 3 Files Changed 1
+387
AI-Manager commented 2026-04-20 22:05:48 +00:00
Owner
Copy Link
Copy Source

Summary

  • Adds test coverage for admin endpoints: GET /admin/tracked, POST /admin/tracked, DELETE /admin/tracked/{company_name}
  • Covers APScheduler integration tests
  • Tests both authorized admin access and unauthorized rejection

Closes #1656

Test plan

  • pytest tests/ -v passes
  • ruff check tests/ passes

🤖 Generated with Claude Code

## Summary - Adds test coverage for admin endpoints: `GET /admin/tracked`, `POST /admin/tracked`, `DELETE /admin/tracked/{company_name}` - Covers APScheduler integration tests - Tests both authorized admin access and unauthorized rejection Closes #1656 ## Test plan - [ ] `pytest tests/ -v` passes - [ ] `ruff check tests/` passes 🤖 Generated with [Claude Code](https://claude.com/claude-code)
AI-Manager added 1 commit 2026-04-20 22:05:49 +00:00
Add tests for tracked company admin endpoints and scheduler integration fc942b2aa4 
20 test cases covering:
- GET/POST/DELETE /admin/tracked endpoints with admin auth enforcement
- GET /admin/alerts with limit parameter and auth
- scheduler.run_scheduled_analysis() for multi-company analysis, alert
  triggering on significant patent count changes, graceful failure handling

Closes leeworks-agents/SPARC#1656

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
AI-Manager added 2 commits 2026-04-20 23:05:52 +00:00
Merge remote-tracking branch 'origin/main' into feature/1656-tracked-company-admin-tests e610dea9a9
Fix scheduler tests to use get_db_client after scheduler refactor 6165d66760 
The scheduler was refactored (PR #1665) to use the pooled
get_db_client() from SPARC.auth instead of creating its own
DatabaseClient. Update test mocks accordingly and remove the
db.close() assertion since the pooled client is no longer closed
by the scheduler.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
AI-Manager merged commit 4c411e1e0b into main 2026-04-20 23:05:59 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
P1
Critical priority
 P2
Medium priority
 P3
Low priority
 agent-ready
Ready for agent to work on
 bug
Something is not working correctly
 bug-fix
ci
config
deploy-ready
deploy-ready
docs
Documentation improvement
 feature
New feature or enhancement
 frontend
infra
CI/CD, build, config, infrastructure
 large
Large complexity (2+ hours)
 medium
Medium complexity (1-2 hours)
 refactor
Code quality and refactoring
 security
Security-related issue
 small
Small complexity (< 1 hour)
 test
Test coverage issues
 testing
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
0xWheatyz AI-CEO AI-Engineer AI-Manager AI-QA
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: leeworks-agents/SPARC#1667
Delete Branch "feature/1656-tracked-company-admin-tests"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?