5
69%
develop: 74%

Ran 07 Nov 2018 05:25PM UTC

Files 393

Run time 2min

Badge

Embed ▾

Committed 07 Nov 2018 05:25PM UTC coverage: 60.867%. Remained the same

Job # TEST_AIIDA_BACKEND=django TEST_TYPE="tests"

Build Type

push

travis-ci

Committed by

sphuber

Commit Message

Wrap `flag_modified` to unwrap `ModelWrapper` instances

The new backend wraps the database model instances in the `ModelWrapper`
utility class, which will take care of timely flushing and updating the
model instance upon changes. Across the code, model instances are passed
to the `flag_modified` functionn of SqlAlchemy that mark a certain key
on an instance as modified. However, sometimes instead of the actual
model instance being passed, the wrapped version is passed. This used to
be ignored for `sqlalchemy==1.0.19` but in `sqlalchemy==1.2.12` to which
we have updated, an `InvalidRequestError` is raised.

To overcome this, we define our own `flag_modified` function that wraps
the one of SqlAlchemy and will unwrap any wrapped instances that are
passed to it.

Run Details

22517 of 36994 relevant lines covered (60.87%)

0.61 hits per line

aiidateam / aiida_core / 4375 / 5
69%
develop: 74%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4375.5 (TEST_AIIDA_BACKEND=django TEST_TYPE="tests")

aiidateam / aiida_core / 4375 / 5 69% develop: 74%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4375.5 (TEST_AIIDA_BACKEND=django TEST_TYPE="tests")

aiidateam / aiida_core / 4375 / 5
69%
develop: 74%

README BADGES
x