11155
97%

Ran 26 Mar 2018 01:50PM UTC

Jobs 1

Files 1192

Run time 45s

Badge

Embed ▾

pending completion

Build # 11155

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

Jobs

ID	Job ID	Ran	Files	Coverage
3	11155.3 (ACTION=gulp GULPTASK=test-app)	26 Mar 2018 01:50PM UTC	0	50.13	Travis Job 11155.3

Source Files on build 11155

Detailed source file information is not available for this build.

hypothesis / h / 11155
97%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 11155

hypothesis / h / 11155 97%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 11155

hypothesis / h / 11155
97%

README BADGES
x