Opened 10 years ago

Closed 10 years ago

#46 closed defect (fixed)

fix c++ coverage tests in regards to include headers

Reported by: jreed Owned by:
Priority: low Milestone:
Component: Unclassified Version:
Keywords: tests Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity:
Sub-Project: Feature Depending on Ticket:
Estimated Difficulty: Add Hours to Ticket:
Total Hours: Internal?:

Description (last modified by jreed)

The google tests are using a mix of headers from the include symlinks and the target headers, so the results are inconsistent. Some have complete coverage from the symlinks but low coverage from the target, while others have low coverage from the symlink. (If it was consistent, I would just exclude one or the other from the coverage report.)

(This was a coverage report regression after the includes were used from include symlink tree.)

Two ideas:

  • remove include symlink tree and move the files to src/lib/FOO/cpp/FOO/BAR.h and point the includes back to FOO/BAR.h
  • remove include symlink tree and move the files to src/lib/cpp/FOO/BAR.h and point the includes back to FOO/BAR.h (so one source tree per language as jelte proposes).

See [bind10-dev] unit tests and include symlink tree

Subtickets

Change History (2)

comment:1 Changed 10 years ago by jreed

  • Description modified (diff)

comment:2 Changed 10 years ago by jreed

  • Resolution set to fixed
  • Status changed from new to closed

The symlink trees were removed and the src was restructured. This problem should be gone.

Note: See TracTickets for help on using tickets.