launchpad-dev team mailing list archive

[Michael Nelson <michael.nelson@xxxxxxxxxxxxx>]

On Wed, Sep 22, 2010 at 12:09 AM, Robert Collins
<robertc@xxxxxxxxxxxxxxxxx> wrote:
> On Wed, Sep 22, 2010 at 1:15 AM, Michael Nelson
> <michael.nelson@xxxxxxxxxxxxx> wrote:
>> On Tue, Sep 21, 2010 at 8:41 AM, Martin Pool <mbp@xxxxxxxxxxxxx> wrote:
>>> On 18 September 2010 11:01, Robert Collins <robert.collins@xxxxxxxxxxxxx> wrote:
>>>> Just noticed while discussing stuff with Maris that feature flags are
>>>> being used in a very odd way.
>>>
>>> Just for easier future reference than grepping mail, this is described
>>> in <https://code.launchpad.net/~mbp/launchpad/flags-webapp/+merge/32967>
>>> lands, this will be in the lib.lp.services.features docstring.  At the
>>> moment it's in a docfile under that directory.   If anything else is
>>> unclear please do ask.
>>
>> The only thing that was unclear for me was how to switch the feature
>> on for testing when using getFeatureFlag(). I ended up with the
>> following helper... is there a better way to do this?
>
> yes, see the existing tests that do it ;)

... which is what I did... just the wrong one (I based it off
lp.services.features.tests.test_flags). The memcache ones (and
librarian ones) are clearer. Thanks.