langgenius / langgenius/dify

too many patch in test

Open
#37,827 3 comments 1 reaction 0 assignees View on GitHub
project#dify
Dominant language
TypeScript
Stars
156k
Forks
24.6k
Avg merge
22h 9m
Merged PRs (30d)
610

Description

there are sooo many patch in the unit test / integration test, I think it's not good.

@dosu any idea?

Contributor guide

Open the contributing guide

Research direction

The issue names unit and integration tests but provides no files or failing case. Start by locating the tests that use patch and establish which usage is considered unnecessary, then confirm the scope and completion criteria with maintainers. Done is an agreed, tested reduction in unnecessary patching.

Written by the indexing model from the issue text.

Assessment

Domain
testing
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.