Skip to content

Commit

Permalink
Do not use Function.prototype.bind on it.only
Browse files Browse the repository at this point in the history
  • Loading branch information
btmills committed Jan 1, 2021
1 parent c3fda44 commit 3a195d1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions designs/2020-rule-tester-only/README.md
Expand Up @@ -46,8 +46,8 @@ To allow using `RuleTester` with a custom test framework other than Mocha, paral
1. Add an `itOnly` `set` accessor that sets `RuleTester[IT_ONLY]`.
1. Add an `itOnly` `get` accessor.
1. If `RuleTester[IT_ONLY]` is set, return it.
2. If global `it` and `it.only` are functions, return `Function.bind.call(it.only, it)`.
3. Throw an error:
2. If global `it` and `it.only` are functions, return `it.only`. *Note: Do not return `Function.bind.call(it.only, it)` because this does not work with Mocha. If `this` is important, we could return a wrapper function.*
1. Throw an error:
1. If either `RuleTester[DESCRIBE]` or `RuleTester[IT]` is customized, recommend setting a custom `RuleTester.itOnly`.
2. If global `it` is a function, the current test framework does not support `only`.
3. Otherwise recommend installing a test framework like Mocha so that `only` can be used.
Expand Down

0 comments on commit 3a195d1

Please sign in to comment.