2
0%
master: 0%

Ran 16 Sep 2017 03:57PM UTC

Files 167

Run time 6s

Badge

Embed ▾

Committed 16 Sep 2017 03:56PM UTC coverage: 70.31% (+0.004%) from 70.306%

Job # TOX_ENV=py27-cdh-airflow_backend_sqlite

Build Type

push

travis-ci

Committed by Bolke de Bruin

Commit Message

[AIRFLOW-1177] Fix Variable.setdefault w/existing JSON

Previously due to a logic error if you attempt to
use
`Variable.setdefault()` with
`deserialize_json=True` and the value
already existed it would die with:

    ...
        my = Variable.setdefault('regions', ['uk'],
deserialize_json=True)
      File "/usr/local/lib/python3.5/site-
packages/airflow/models.py", line 3623, in
setdefault
        return json.loads(obj.val)
    AttributeError: 'str' object has no attribute
'val'

The problem was that the `Variable.get()` call was
returning the value,
not a variable object.

Closes #2540 from ashb/variable-setdefault-json

Coverage Stats

8177 of 11630 relevant lines covered (70.31%)

0.7 hits per line

airbnb / airflow / 7704 / 2
0%
master: 0%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 7704.2 (TOX_ENV=py27-cdh-airflow_backend_sqlite)

airbnb / airflow / 7704 / 2 0% master: 0%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 7704.2 (TOX_ENV=py27-cdh-airflow_backend_sqlite)

airbnb / airflow / 7704 / 2
0%
master: 0%

README BADGES
x