HomePhabricator

add "update" mode to Diffusion coverage Conduit

Description

add "update" mode to Diffusion coverage Conduit

Summary:
This diff adds a new mode argument to the Diffusion Conduit API with two options:

  • "overwrite": the default, maintains the current behavior of deleting all coverage in the specified branch before uploading the new coverage
  • "update": does not delete old coverage, but will overwrite previous coverage information if it's for the same file and commit

DiffusionRequest::loadCoverage already loads a file's coverage from the
latest available commit, so uploading coverage for different files in different
commits with "update" will result in seeing the latest uploaded coverage in
Diffusion.

Test Plan: manual local verification

Reviewers: epriestley, Blessed Reviewers

Reviewed By: epriestley, Blessed Reviewers

Subscribers: Korvin

Differential Revision: https://secure.phabricator.com/D14428

Details

Provenance
ddfisherAuthored on
ddfisherPushed on Nov 10 2015, 12:52 AM
Reviewer
Blessed Reviewers
Differential Revision
D14428: add "update" mode to Diffusion coverage Conduit
Parents
rP7fd6704fb5c7: Add a crumb to blog on Phame posts
Branches
Unknown
Tags
Unknown
Build Status
Buildable 8742
Build 10159: Run Core Tests