Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Finish unit tests for review controller and started for event controller #15955

Open
wants to merge 23 commits into
base: dev
Choose a base branch
from
Open

Finish unit tests for review controller and started for event controller #15955

wants to merge 23 commits into from
+405 −66

Conversation

iursevla
Copy link
Contributor

@iursevla iursevla commented Jan 12, 2025 •
edited
Loading

Proposed change

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code
  • Documentation Update

Additional information

  • This PR continues the unit tests efforts for the HTTP API

Checklist

  • The code change is tested and works locally.
  • Local tests pass. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • The code has been formatted using Ruff (ruff format frigate)

@iursevla
Started unit tests for the review controller
7746eb1
@iursevla
Revert "Started unit tests for the review controller"
137f4e2 
This reverts commit 7746eb1.
@iursevla
Merge remote-tracking branch 'upstream/dev' into dev
edabe94
@iursevla
Merge remote-tracking branch 'upstream/dev' into dev
1d4de12
@iursevla
Merge remote-tracking branch 'upstream/dev' into dev
6e6fc21
@iursevla
Merge remote-tracking branch 'upstream/dev' into dev
7f69e59
@iursevla
Merge remote-tracking branch 'upstream/dev' into dev
4a9d21c
@iursevla
Merge remote-tracking branch 'upstream/dev' into dev
d256344
@iursevla
Merge remote-tracking branch 'upstream/dev' into dev
126e3bd
@iursevla
Merge remote-tracking branch 'upstream/dev' into dev
3193651
@iursevla
Merge remote-tracking branch 'upstream/dev' into dev
a43af33
@iursevla
Merge remote-tracking branch 'upstream/dev' into dev
4810a18
@iursevla
Merge remote-tracking branch 'upstream/dev' into dev
fdbc55e
@iursevla
Started unit tests for GET /review/activity/motion Endpoint
cce41cb
@iursevla
Started unit tests for GET /review/event/{event_id} Endpoint
5487058
@iursevla
Continued unit tests for GET /review/event/{event_id} Endpoint
ea364ac
@iursevla
Continued unit tests for GET /review/{event_id} Endpoint
142fb82
@iursevla
Continued unit tests for GET /review/{review_id} Endpoint
056d722
@iursevla
Added unit tests for GET /review/{review_id}/viewed Endpoint
bd746cd
@iursevla
Added unit tests for GET /stats Endpoint
8966497
@iursevla
Added unit tests for GET /events Endpoint
3aaf671
@iursevla
Updated unit tests for GET /events Endpoint
713f70a
@netlify Netlify
Copy link

netlify bot commented Jan 12, 2025 •
edited
Loading

✅ Deploy Preview for frigate-docs canceled.

Name Link
🔨 Latest commit 2aeca7d
🔍 Latest deploy log https://app.netlify.com/sites/frigate-docs/deploys/67843f9e57fec40008340c0b

@iursevla iursevla changed the title Review controller unit tests and started event.py Review controller unit tests and started unit tests for event.py Jan 12, 2025
@iursevla iursevla changed the title Review controller unit tests and started unit tests for event.py Finish unit tests for review controller and started for event controller Jan 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@iursevla