236
100%
master: 90%

Ran 25 Sep 2017 09:30AM UTC

Jobs 2

Files 12

Run time 2s

Badge

Embed ▾

pending completion

Build # 236

Build Type

push

travis-ci

Committed by Daniel Nouri

Commit Message

No longer disallow recursive call to get_config (reverts e390cbf)

It turns out that we make use of recursive `get_config` calls in our
own code: `CachedUpdatePersister` initializes with
`self.update_cache()`, which is decorated with
`PluggableDecorator('update_model_decorators')`, which calls
`get_config` to get a list of strings pointing to decorator functions.

In any event, it may be a good idea to call `self.update_cache()` from
a thread so to not block initialization.  However,
`PluggableDecorator` likely has other legitimate uses, so this isn't a
fix for the problem at hand.

The caveat still applies that if one calls `get_config` from component
code, one must be aware that the part of config we want to refer to
may not be initialized yet.  Best way to work around this is to either
not rely on `__factory__` and friends at all (PluggableDecorator does
that), or to make that part of the configuration a sub-entry to the
component's own configuration, if possible.

Run Details

1199 of 1199 relevant lines covered (100.0%)

2.0 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	236.1 (TRAVIS=yes)	25 Sep 2017 09:30AM UTC	0	100.0	Travis Job 236.1
2	236.2 (TRAVIS=yes)	25 Sep 2017 09:30AM UTC	0	100.0	Travis Job 236.2

Source Files on build 236

Detailed source file information is not available for this build.

ottogroup / palladium / 236
100%
master: 90%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 236

ottogroup / palladium / 236 100% master: 90%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 236

ottogroup / palladium / 236
100%
master: 90%

README BADGES
x