1
100%
master: 90%

Ran 25 Sep 2017 09:30AM UTC

Files 12

Run time 0s

Badge

Embed ▾

Committed 25 Sep 2017 09:12AM UTC coverage: 100.0%. First build

Job # TRAVIS=yes

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%)

1.0 hits per line

ottogroup / palladium / 236 / 1
100%
master: 90%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 236.1 (TRAVIS=yes)

ottogroup / palladium / 236 / 1 100% master: 90%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 236.1 (TRAVIS=yes)

ottogroup / palladium / 236 / 1
100%
master: 90%

README BADGES
x