langgenius / langgenius/dify

merge dup functions in unit test

Open
#41,535 4 comments 1 reaction 0 assignees View on GitHub
good first issue
Dominant language
TypeScript
Stars
156k
Forks
24.6k
Avg merge
22h 9m
Merged PRs (30d)
610

Description

now we have sooo many `def _app(...) -> App` and similar functions, merge them will reduce LOT of code.

@dosu give me a full file list and how to fix them.

Contributor guide

Open the contributing guide

Research direction

No files, tests, or entry points are named. Start by inventorying the unit-test files containing duplicate _app(...) and similar helpers, then compare their signatures and behavior. Done means an agreed full file list, a consistent consolidation plan, and unit tests that still pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
testing
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.