dgtlmoon / dgtlmoon/changedetection.io

[feature] improve test coverage for `tags` and `notification test`

Open
#3,032 1 comment 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
34.1k
Forks
2k
Avg merge
23h 38m
Merged PRs (30d)
67

Description

Based on my analysis of the codebase, here are some endpoints that appear to have less test coverage:

1. Several conditions blueprint endpoints:
- /conditions//verify-condition-single-rule route **(fixed)**
2. Settings blueprint endpoints:
- /settings/reset-api-key
3. Many of the tag-related endpoints:
- /tags/mute/
- /tags/unlink/
- /tags/edit/ (GET and POST) (sort of related https://github.com/dgtlmoon/changedetection.io/issues/2942 )
4. Price data follower endpoints:
- Appears to have less comprehensive tests
5. The /api/share-url endpoint
6. Notification testing endpoints:
- /notification/send-test/

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the existing tests and handlers for /tags/mute/, /tags/unlink/, /tags/edit/, /settings/reset-api-key, /api/share-url, and /notification/send-test/. Add coverage for the untested behavior across the selected endpoints, including GET and POST for tag editing, and confirm the test suite passes.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend, testing-qa
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.