Edgewall Software
Modify

Opened 12 years ago

Last modified 10 years ago

#762 new defect

Code Coverage and Unit Test Charts do not play along with Git

Reported by: andreas.schroeder@… Owned by:
Priority: minor Milestone: 0.6.1
Component: Trac plugin Version: dev
Keywords: Cc: spam.martin.schulze@…
Operating System: Linux

Description

The "Code Coverage" and "Unit Tests" charts on the "Build Configuration" view remain empty when the repository in use is a Git repository.

This is because the json-data that is returned for plotting in the charts use the revision as x value, and hence with Git are of the sort:

{"data":[{"data":[
  ["fcbb2ef18ca2ac9625282fd927f2b0eeee0128a5",6955],
  ["3aab25b1fcf1ff60cfcc6fcb08a0a263ef66865e",6955],
  ["8a6ee8a6e9d6cfd64a613ec9fa71fea831c05358",6955],
  (...)

and the plot() function in the chrome/bitten/jquery.flot.js script uses these tags as x values. According to Hodgestar, the root cause of the issue can be found

"at line 49 of bitten/report/testing.py (and some related bits later): tests.append([rev, 0, 0, 0])"

Attachments (4)

empty_charts.png (4.8 KB) - added by andreas.schroeder@… 12 years ago.
The empty coverage and unit test charts I get when using Git
bitten-git-charts.patch (6.8 KB) - added by andreas.schroeder@… 12 years ago.
A patch that should fix the bug.
bitten-git-charts.2.patch (11.0 KB) - added by andreas.schroeder@… 12 years ago.
fixed errors in lint and adjusted tests.
bitten-git-charts-labels.patch (112.5 KB) - added by andreas.schroeder@… 12 years ago.
Second patch proposal, includes flot patch proposal for rotated labels. Patch is against Revision 1010.

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by andreas.schroeder@…

The empty coverage and unit test charts I get when using Git

comment:1 Changed 12 years ago by anonymous

This is the image I get:

The empty coverage and unit test charts I get when using Git

Changed 12 years ago by andreas.schroeder@…

A patch that should fix the bug.

Changed 12 years ago by andreas.schroeder@…

fixed errors in lint and adjusted tests.

comment:2 Changed 12 years ago by andreas.schroeder@…

please look into the second patch, the first one had errors, and the unit tests didn't pass.

Changed 12 years ago by andreas.schroeder@…

Second patch proposal, includes flot patch proposal for rotated labels. Patch is against Revision 1010.

comment:3 Changed 10 years ago by Martin Schulze <spam.martin.schulze@…>

  • Cc spam.martin.schulze@… added

I see that this problem still persists. What is the current status on this one?

Add Comment

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The ticket will remain with no owner.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.