Paths

Table of Contentst

Differential D8667

Add coverage reports to py.test runner
ClosedPublic
Actions

Authored by zeeg on Apr 2 2014, 1:16 AM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rARCb8e42614559b: Add coverage reports to py.test runner

Summary

Of note this now forces the pytest-cov plugin to be installed unless they turn it off.

Test Plan

../arcanist/bin/arc unit

COVERAGE REPORT
     97%     changes/models/test.py
    100%     tests/changes/api/serializer/models/test_testcase.py
    100%     changes/api/serializer/models/testcase.py
 UNIT OKAY  No unit test failures.
Updated an existing Differential revision:
        Revision URI: https://tails.corp.dropbox.com/D43387

Included changes:
  M       changes/api/serializer/models/testcase.py
  M       changes/models/test.py
  M       tests/changes/api/serializer/models/test_testcase.py

Diff Detail

Repository

rARC Arcanist

Branch

pytest-cover

Lint

Lint Passed

Unit

Tests Passed

Event Timeline

zeeg updated this revision to Diff 20554.Apr 2 2014, 1:16 AM

zeeg retitled this revision from to Add coverage reports to py.test runner.

zeeg updated this object.

zeeg edited the test plan for this revision. (Show Details)

zeeg added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers.Apr 2 2014, 1:16 AM

Herald added subscribers: Korvin, epriestley.

See also IRC. If there's no way to do anything but coverage.xml, this is fine. You shouldn't need to unlink the temp file.

src/unit/engine/PytestTestEngine.php
15	Why are we putting this at the project root? That seems really messy/gross? Can we just use a temp file like `$junit_tmp`?
17–18	Use `Filesystem::remove($file)`. But this should not be necessary with the above change.

Rely on coverage external binary so we can specify a tmp file for xml

do something about csprintf that I dont understand

epriestley accepted this revision.Apr 2 2014, 6:31 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Apr 2 2014, 6:31 PM

Closed by commit rARCb8e42614559b (authored by @zeeg, committed by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

unit/

engine/

PytestTestEngine.php

125 lines

Diff 20554

View Options

src/unit/engine/PytestTestEngine.php

Add coverage reports to py.test runnerClosedPublicActions