3
97%
master: 97%

Ran 26 Mar 2018 01:50PM UTC

Files 1192

Run time 41s

Badge

Embed ▾

Committed 26 Mar 2018 01:50PM UTC coverage: 50.125%. First build

Job # ACTION=gulp GULPTASK=test-app

Build Type

Pull #3076

travis-ci

Committed by Sean Hammond

Commit Message

Copy transform_links() from elastic.py to storage

Duplicate the Document._transform_links() method from elastic.py in
storage.py, and add unit tests and a docstring for it.

Nothing calls the function except tests, yet.

This function is going to be needed to transform the "document" sub
objects that the client posts to the API when creating or updating an
annotation into DocumentURI objects.

The function is duplicated to avoid making changes to the legacy code in
elastic.py, and because elastic.py will be getting deleted to the
duplication doesn't matter too much.

Top-level in storage.py is probably not the right place for this
function, but just put it there for now.

Only minimal changes were made to the function from elastic.py to get
the code working in its new home where it no longer has `self`:

- Replace self with a document dict argument
- Replace self.claimaint with a unicode argument
- Replace self.created with a datetime argument
- Replace self.updated with a datetime argument

Pull Request Pull Request #3076:

Run Details

45443 of 90659 relevant lines covered (50.13%)

0.5 hits per line

hypothesis / h / 11155 / 3
97%
master: 97%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 11155.3 (ACTION=gulp GULPTASK=test-app)

hypothesis / h / 11155 / 3 97% master: 97%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 11155.3 (ACTION=gulp GULPTASK=test-app)

hypothesis / h / 11155 / 3
97%
master: 97%

README BADGES
x