stripe / stripe-go / 2809
82%
master: 66%

LAST BUILD BRANCH: latest-codegen-beta
DEFAULT BRANCH: master
Ran
Jobs 5
Files 85
Run time 4min
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
2809

push

travis-ci

brandur 
Push parameter encoding into `BackendConfiguration.Call`

Now that we have a little more consistency throughout the library, here
we modify `BackendConfiguration.Call` (temporarily renamed to `Call2`
while we refactor) so that it can take parameter structs directly, then
encode them. This allows us to remove this common boilerplate from every
API call throughout the entire library:

``` go
if params != nil {
    commonParams = &params.Params
    body = &form.Values{}
    form.AppendTo(body, params)
}
```

Temporarily though, only the `charge/` package has been converted over
to show what it looks like before we convert everything.

There is a little bit of type trickiness that allows this to happen, and
which requires a somewhat advanced understanding of some Go language
semantics. We define a `ParamsContainer` interface as this:

``` go
type ParamsContainer interface {
    GetParams() *Params
}
```

Then on the `Params` struct itself, we define an implementation:

``` go
func (p *Params) GetParams() *Params {
    return p
}
```

In Go, whenever a struct is embedded in another struct, the host
inherits the function implementation of the struct which was embedded.
Because every parameter struct in the library embeds `Params`, they all
get a `GetParams` implementation and thus implement `ParamsContainer`
automatically:

``` go
type ChargeParams struct {
    Params `form:"*"`
    ...
}
```

This allows us to take a `ParamsContainer` from `Call`.

Most of this applies similarly for `ListParams`, and similarly `iter.go`
also gets a `Query2` and `GetIter2` to keep everything compiling while
we refactor everything. I did a little more refactoring in `iter.go` as
well just because there was quite a few very bad names in there. If this
patch lands, I'll probably refactor even more of it in a follow up.

3933 of 4746 relevant lines covered (82.87%)

121.22 hits per line

Jobs
ID Job ID Ran Files Coverage
1 2809.1 85
82.87
 Travis Job 2809.1
2 2809.2 85
82.87
 Travis Job 2809.2
3 2809.3 85
82.87
 Travis Job 2809.3
4 2809.4 85
82.87
 Travis Job 2809.4
5 2809.5 85
82.87
 Travis Job 2809.5
Source Files on build 2809