13196
66%
master: 66%

Ran 12 Nov 2020 11:50PM UTC

Jobs 2

Files 114

Run time 48s

Badge

Embed ▾

Committed 12 Nov 2020 11:36PM UTC coverage: 65.336% (+0.01%) from 65.324%

Build # 13196

Build Type

push

travis-pro

Committed by

web-flow

Commit Message

Simplify sa.NewOrder transaction (#5177)

This change refactors the sa.NewOrder method to be more readable.
Previously, the outer method both modified and returned the request
object, a confusion violation of abstraction. Simultaneously, the inner
transaction function modified its input order object, but returned the
original request object which it hadn't modified.

Now, the NewOrder method does not modify its input, and instead
returns an all-new Order object. Additionally, the inner transaction
function does not modify its input, and returns an order model with
relevant fields set.

Part of #5166

Run Details

13292 of 20344 relevant lines covered (65.34%)

1.46 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
13	13196.13 (TESTFLAGS="--coverage" CONTAINER="netaccess")	12 Nov 2020 11:50PM UTC	0	65.34	Travis Job 13196.13
14	13196.14 (TESTFLAGS="--coverage" CONTAINER="netaccess")	12 Nov 2020 11:51PM UTC	0	65.34	Travis Job 13196.14

Source Files on build 13196

Detailed source file information is not available for this build.

letsencrypt / boulder / 13196
66%
master: 66%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 13196

letsencrypt / boulder / 13196 66% master: 66%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 13196

letsencrypt / boulder / 13196
66%
master: 66%

README BADGES
x