5
69%
develop: 74%

Ran 07 Nov 2018 05:40PM UTC

Files 393

Run time 2min

Badge

Embed ▾

Committed 07 Nov 2018 05:14PM UTC coverage: 60.837% (-0.02%) from 60.857%

Job # TEST_AIIDA_BACKEND=django TEST_TYPE="tests"

Build Type

Pull #2113

travis-ci

Committed by

web-flow

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.

Pull Request Pull Request #2113: Upgrade the dependency requirement of SqlAlchemy

Run Details

22506 of 36994 relevant lines covered (60.84%)

0.61 hits per line

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

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

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

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

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

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

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

README BADGES
x