TykTechnologies / tyk / 3172 / 2

Ran 04 Oct 2017 05:42PM UTC

Files 84

Run time 7s

Badge

Embed ▾

Committed 04 Oct 2017 05:39PM UTC coverage: 45.946% (-0.04%) from 45.984%

Job # LATEST_GO=true

Build Type

push

travis-ci

Committed by

buger

Commit Message

all: clean up and simplify a bunch of code

First, we know that in the versions map, the key will always be equal to
value.Name. The docs don't explicitly state this, but other parts of the
code such as elsewhere in the gateway and the dashboard use the two
interchangeably.

Do the same in the API definition loader. It was caching both the key
and value in the request's context. Cache only the value. We only ever
needed to cache the version key here, and it won't be useful in the
future, so remove the machinery too.

While at it, collapse some identical switch cases and make
ParseDefinition take a reader, since a string is not necessary.

Finally, remove a test API definition file I recently introduced by
mistake.

Coverage Stats

6098 of 13272 relevant lines covered (45.95%)

0.5 hits per line

TykTechnologies / tyk / 3172 / 2

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3172.2 (LATEST_GO=true)

TykTechnologies / tyk / 3172 / 2

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3172.2 (LATEST_GO=true)

README BADGES
x