8920
70%
master: 97%

Ran 30 Oct 2015 02:54PM UTC

Jobs 1

Files 88

Run time 13s

Badge

Embed ▾

pending completion

Build # 8920

Build Type

push

travis-ci

Committed by

seanh

Commit Message

Add AnnotationHTMLPresenter

This splits h.api.models.Annotation into two classes:

1. h.api.models.Annotation, which contains only model code and no presentation
   code.

   This _does_ contain code for papering over the variability of our annotation
   data. For example it has a .uri property that always returns a unicode
   string even if the underlying annotation dict has no "uri" key or has a key
   whose value isn't a string.

2. h.AnnotationHTMLPresenter, a wrapper class for h.api.models.Annotation that:

   a. Overrides some properties with HTML-specific behaviour.
      For example it overrides .uri to provide an HTML-escaped copy.

   b. Adds some new HTML-specific properties. For example it adds a
      .document_link property that returns an escaped, HTML-formatted link
      for the annotation's document.

Many Annotation tests are now moved to AnnotationHTMLPresenter tests.

feed/atom.py, feeds/rss.py and groups/views.py are updated to use
AnnotationHTMLPresenter.

Tests for these modules need to be updated to mock AnnotationHTMLPresenter
behaviour, which is unfortunately quite awkward and requires a mock class.

Run Details

2606 of 3748 relevant lines covered (69.53%)

0.7 hits per line

Uncovered Existing Lines

Lines	Coverage	∆	File
4	100.0		h/annotation_html_presenter.py

Jobs

ID	Job ID	Ran	Files	Coverage
1	8920.1	30 Oct 2015 02:54PM UTC	0	69.53	Travis Job 8920.1

Source Files on build 8920

Detailed source file information is not available for this build.

hypothesis / h / 8920
70%
master: 97%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 8920

hypothesis / h / 8920 70% master: 97%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 8920

hypothesis / h / 8920
70%
master: 97%

README BADGES
x