ts-mockito icon indicating copy to clipboard operation
ts-mockito copied to clipboard

Updated README.md to include object compare

Open designxtek opened this issue 6 years ago • 9 comments

In reference to https://github.com/NagRock/ts-mockito/issues/93 on verifying calls with objects

designxtek avatar Mar 12 '18 21:03 designxtek

What do you think about separate section in readme.md about matchers?

NagRock avatar Mar 13 '18 07:03 NagRock

Yeah I could create a separate section. I'm not too familiar with it but I can start something.

designxtek avatar Mar 13 '18 16:03 designxtek

Would be great

NagRock avatar Mar 13 '18 16:03 NagRock

Added a new section. Let me know if there is any other changes needed. Thanks

designxtek avatar Mar 13 '18 17:03 designxtek

Codecov Report

Merging #94 into master will increase coverage by 0.31%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #94      +/-   ##
==========================================
+ Coverage   94.58%   94.89%   +0.31%     
==========================================
  Files          35       34       -1     
  Lines         609      607       -2     
  Branches       69       69              
==========================================
  Hits          576      576              
+ Misses         24       22       -2     
  Partials        9        9
Impacted Files Coverage Δ
src/stub/MethodStub.ts

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 93d6625...996b81f. Read the comment docs.

codecov-io avatar Mar 13 '18 17:03 codecov-io

@NagRock I just started using this library yesterday so i'm a newbie but wondering about PR's like this which pass all tests and were introduced over a year ago. Is this library still being supported? Just a bit worried I may have gotten to the party too late 😆

yankeeinlondon avatar Jul 15 '19 15:07 yankeeinlondon

@NagRock Can we please merge this, I think a lot of people will go through this issue(and would help to save a lot of time).

Thanks.

avneesh91 avatar Jan 23 '20 06:01 avneesh91

Good to merge!

henrikra avatar Apr 12 '21 08:04 henrikra