Add toInclude

[grug]

What

Add toInclude matcher.

Why

Closes #41

Notes

Housekeeping

• Unit tests
• Documentation is up to date
• No additional lint warnings
• Add yourself to contributors list (yarn contributor)

[mattphillips]

Looks good @grug just need to rebase master back into your branch 😄

[codecov-io]

Codecov Report

Merging #85 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #85   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          48     50    +2     
  Lines         216    225    +9     
  Branches       30     31    +1     
=====================================
+ Hits          216    225    +9

Impacted Files	Coverage Δ
src/matchers/toInclude/predicate.js	100% <100%> (ø)
src/matchers/toInclude/index.js	100% <100%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3cfaab4...c67e404. Read the comment docs.

[grug]

Awesome. I think I've rebased that correctly. Let me know if you need anything else for this PR to be merged :)

[mattphillips]

Just noticed this function isn't returning the message if you remove the curly braces it should fix this snapshot:

exports[`.toInclude .not.toInclude fails when a string does have a given substring 1`] = `"<red>No message was specified for this matcher.</>"`;

const passMessage = (actual, expected) => () =>
  matcherHint('.not.toInclude') +
    '\n\n' +
    'Expected string to not include:\n' +
    `  ${printExpected(expected)}\n` +
    'Received:\n' +
    `  ${printReceived(actual)}`;

[grug]

Yup I'll fix that now

[grug]

Okay - I've fixed up the changes you've suggested :) let me know if I should do anything else before this can be merged.

[mattphillips]

Good job @grug thanks!