testdouble / testdouble/testdouble.js
Warn when arity of stubbing or verification doesn't match function.length
Open
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 1.4k
- Forks
- 140
- Avg merge
- 22h
- Merged PRs (30d)
- 1
Description
If I do a test double func:
var dog = {
bite: function (thing) {}
}
var woof = td.replace(dog, 'bite')
Then setting up a stubbing with a different arity should warn:
td.when(dog.woof('mailman', 'spritely')).thenReturn("UH OH") // arity mismatch warning
This warning should be able to be muted on a case-by-case basis since function .length is unreliable.
td.when(dog.woof('mailman', 'spritely'), {ignoreArity: true}).thenReturn("UH OH") // no warning
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing the td.when stubbing and verification entry points, then identify where the replaced function's length is available. Add an arity-mismatch warning and support the issue's ignoreArity option for individual cases. Done means mismatched arguments warn by default and the opt-out suppresses that warning.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- testing
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100