intridea / omniauth / 479
94%

DEFAULT BRANCH: master
Ran
Jobs 14
Files 9
Run time 13min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
479

Pull #774

travis-ci

schneems 
Minimize object creation on middleware call.

This patch isn't nearly as invasive as #773. Also it was tested against itself, not in a full Rails app. I'm contributing the perf scripts I wrote in another PR.

Notice that by memorizing methods, not only are we reducing objects created in omniauth, but also in hashie due to fewer method calls to various omniauth options.


## Bench

Before patch

```
Calculating -------------------------------------
                 ips       470 i/100ms
-------------------------------------------------
                 ips     4877.1 (±10.1%) i/s -      24440 in   5.061663s
2.1.2  ~/documents/projects/omniauth (master)
$ be rake perf:ips
Calculating -------------------------------------
                 ips       467 i/100ms
-------------------------------------------------
                 ips     4840.9 (±14.5%) i/s -      23817 in   5.068649s
2.1.2  ~/documents/projects/omniauth (master)
$ be rake perf:ips
Calculating -------------------------------------
                 ips       465 i/100ms
-------------------------------------------------
                 ips     4799.5 (±11.8%) i/s -      23715 in   5.019191s
```

After Patch

```
$ be rake perf:ips
Calculating -------------------------------------
                 ips       631 i/100ms
-------------------------------------------------
                 ips     6269.1 (±11.8%) i/s -      30919 in   5.014849s
2.1.2  ~/documents/projects/omniauth (master)
$ be rake perf:ips
Calculating -------------------------------------
                 ips       615 i/100ms
-------------------------------------------------
                 ips     6207.8 (±10.3%) i/s -      31365 in   5.106172s
2.1.2  ~/documents/projects/omniauth (master)
$ be rake perf:ips
Calculating -------------------------------------
                 ips       522 i/100ms
-------------------------------------------------
                 ips     6155.8 (±9.3%) i/s -      30798 in   5.048455s
```



## M... (continued)
Pull Request #774:

511 of 544 relevant lines covered (93.93%)

213.98 hits per line

Jobs
ID Job ID Ran Files Coverage
2 479.2 (1.9.3, Gemfile) 0
93.1
 Travis Job 479.2
3 479.3 (2.0.0, Gemfile) 0
93.1
 Travis Job 479.3
4 479.4 (2.1, Gemfile) 0
93.1
 Travis Job 479.4
6 479.6 (jruby-19mode, Gemfile) 0
93.28
 Travis Job 479.6
7 479.7 (jruby-head, Gemfile) 0
93.24
 Travis Job 479.7
8 479.8 (rbx-2, Gemfile) 0
98.39
 Travis Job 479.8
9 479.9 (ruby-head, Gemfile) 0
93.1
 Travis Job 479.9
11 479.11 (1.9.3, Gemfile.rack-1.3.x) 0
93.51
 Travis Job 479.11
12 479.12 (2.0.0, Gemfile.rack-1.3.x) 0
93.51
 Travis Job 479.12
13 479.13 (2.1, Gemfile.rack-1.3.x) 0
93.51
 Travis Job 479.13
15 479.15 (jruby-19mode, Gemfile.rack-1.3.x) 0
93.68
 Travis Job 479.15
16 479.16 (jruby-head, Gemfile.rack-1.3.x) 0
93.64
 Travis Job 479.16
17 479.17 (rbx-2, Gemfile.rack-1.3.x) 0
98.8
 Travis Job 479.17
18 479.18 (ruby-head, Gemfile.rack-1.3.x) 0
93.51
 Travis Job 479.18
Source Files on build 479
Detailed source file information is not available for this build.