5
0%
master: 0%

Ran 16 Sep 2017 03:56PM UTC

Files 167

Run time 6s

Badge

Embed ▾

Committed 16 Sep 2017 03:56PM UTC coverage: 69.931% (+0.003%) from 69.928%

Job # TOX_ENV=py34-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

8133 of 11630 relevant lines covered (69.93%)

0.7 hits per line

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

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 7704.5 (TOX_ENV=py34-cdh-airflow_backend_sqlite)

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

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 7704.5 (TOX_ENV=py34-cdh-airflow_backend_sqlite)

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

README BADGES
x