RedTurtle / eve-sqlalchemy / 112
97%

DEFAULT BRANCH: master
Ran
Jobs 6
Files 19
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
112

Pull #47

travis-ci

goneri 
embedded table: use DOMAIN to look up the resource fields

This patch resolve two issues:
- it's now possible to have a resource field that is a relation to
  another relation, even if the name doesn't match. For example,
  product_stable and product_devel are two one-to-one field with a
  relationship to product
- when we associate a foreign resource, we let
  eve.utils.embedded_document() decide what field should be import in
  the embedded resource. Previously, all the fields of the embedded
  resource were imported, with the thread of potential endless loops.
  For example, a any-to-one relation called
  `foo` with an associate to `bar` that we pull with embedded.
  `bar` has got a `foo_collection` with a link to `foo`. Previously, the
  `foo_collection` filed was automatically added in the list of field of `bar`.
  This was enough to create a resource loop in `lookup_foreign_resource`.
- eve.utils.embedded_document() was confused because it was try to
  resolve itself embedded resource that were already pulled by
  eve_sqlalchemy.utils.sqla_object_to_dict(). Because of that, all the
  embedd resources were prefixed with a ID_FIELD in the final JSON.
  We now only retrieve the ID_FIELD value, as expected by
  embedded_document()
- the patch, with https://github.com/nicolaiarocci/eve/pull/656 will allow
  the use of sub-embedded document
Pull Request #47: embedded table: use DOMAIN to look up the resource fields

2486 of 2558 relevant lines covered (97.19%)

5.83 hits per line

Jobs
ID Job ID Ran Files Coverage
1 112.1 (TOX_ENV=py26) 19
97.19
 Travis Job 112.1
2 112.2 (TOX_ENV=py27) 19
97.19
 Travis Job 112.2
3 112.3 (TOX_ENV=py33) 19
97.19
 Travis Job 112.3
4 112.4 (TOX_ENV=py34) 19
97.19
 Travis Job 112.4
5 112.5 (TOX_ENV=pypy) 19
97.19
 Travis Job 112.5
6 112.6 (TOX_ENV=pep8) 19
97.19
 Travis Job 112.6
Source Files on build 112
Coverage ∆ File Lines Relevant Covered Missed Hits/Line