dssg / dssg/triage

Add unit test for collate groups deprecation warning

Open
#906 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Jupyter Notebook
Stars
201
Forks
62
PR merge metrics
No merged PRs in 30d

Description

Including the groups key in the feature config was hitting a bug, so we should probably add a unit test to ensure this still works when only specifying `entity_id` (but raises a deprecation warning), even though we're moving away from supporting specifying this key generally.

Contributor guide

Open the contributing guide

Research direction

No file or existing test is named. Start by locating the feature-config handling and current tests for the groups and entity_id keys; reproduce the configuration using only entity_id and confirm the deprecation warning is emitted while the behavior still works. Done means the regression is covered by a unit test.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
testing
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.