• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

opentrials / api / 436 / 1
94%
master: 92%

Build:
Build:
LAST BUILD BRANCH: v1.1.1
DEFAULT BRANCH: master
Ran 08 Sep 2016 11:37AM UTC
Files 25
Run time 1s
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

08 Sep 2016 11:35AM UTC coverage: 93.696%. Remained the same
436.1

push

travis-ci

vitorbaptista
[#387] Add entity File that belongs to many Documents

This extracts the attributes related to the actual file location and its text
from the `documents` table to a new `files` table. The reasoning behind it is
that multiple documents can (and do) reference the same file, so this
extraction will allow us to keep a single entry in our database in those cases.

I haven't removed the columns from the `documents` table yet because I want to
populate the `files` table with our current contents before that. After we run
this migration and populate the `files` table, we can add another migration to
remove the then redudant columns.

I also changed to use the "documentcloud_id" instead of its URL so we can
delete files from DocumentCloud as needed (e.g. when they're updated). This
isn't ideal, but we can generate a document's URL based on its ID.

This is an internal refactoring, meaning that the API response didn't change at
all.

opentrials/opentrials#387

327 of 349 relevant lines covered (93.7%)

31.01 hits per line

Source Files on job 436.1
  • Tree
  • List 0
  • Changed 0
  • Source Changed 0
  • Coverage Changed 0
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 436
  • Travis Job 436.1
  • 518d2b48 on github
  • Prev Job for on add-documents-files (#434.1)
  • Next Job for on add-documents-files (#439.1)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc