GoEddie / GoEddie/SQLCover

Cobertura updates/adherence

Open
#79 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
TSQL
Stars
69
Forks
55
PR merge metrics
No merged PRs in 30d

Description

Currently SQLCover doesn't populate the `filename` field in its Cobertura report. Per the [DTD](https://github.com/cobertura/cobertura/blob/master/cobertura/src/site/htdocs/xml/coverage-03.dtd), this is a required field.

Since many downstream tools that consume Cobertura will assume this field has a value, I think it is better to put *something* rather than nothing, to make a stronger attempt to adhere to the format and increase output ingestion compatibility.

There is also a [0.4 DTD](https://github.com/cobertura/cobertura/tree/master/cobertura/src/site/htdocs/xml) available vs the 0.3 in use currently.

Contributor guide

No contributing guide indexed for this repository

Research direction

Compare SQLCover's Cobertura output with the referenced cobertura/src/site/htdocs/xml/coverage-03.dtd and the available 0.4 DTD. Locate the Cobertura report generation code, determine an appropriate filename value, and verify that the generated report includes it and remains compatible with the selected DTD.

Written by the indexing model from the issue text.

Assessment

Tech stack
sql
Domain
testing
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.