rspec
/
rspec-mocks
/
1033
/
3
98%
master: 63%
master: 63%
|
Ran
|
Files
38
|
Run time
–
|
Badge
Embed ▾
README BADGES
|
coverage: 97.214% (-0.5%) from 97.711%
push
travis-ci
myronmarstonRefactor to improve perf. - Don't get the original method handle for a test double (it doesn't have an original method as its not a partial mock). - Leverage InstanceMethodStasher; since it has to stash the method anyway, it can store it so we don't have to waste effort getting it. - Delay getting the original_method until the last possible moment: right before we redefine it with a new definition. As a side benefit, this change makes it so that we no longer "leak" odd obfuscated aliased methods onto partial mock objects. This is a good thing. I ran benchmarks of this commits vs the prior (unoptimized fix) vs what we originally had in master using this shell command: time (for i in {1..20}; do bin/rspec -fp --order default; done) Here are the totals: - this commit: 39.070 seconds - prior unoptimized fix (c864e0a53): 39.936 - master (80d80d919): 37.783
1710 of 1759 relevant lines covered (97.21%)
121.95 hits per line
Source Files on job 1033.3 (1.9.3)
| Coverage | ∆ | File | Lines | Relevant | Covered | Missed | Hits/Line |
|---|