Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

test coverage for net.openhft.chronicle.queue.impl.single.StoreTailer #821

Open
JerryShea opened this issue Apr 7, 2021 · 3 comments
Open

test coverage for net.openhft.chronicle.queue.impl.single.StoreTailer #821

JerryShea opened this issue Apr 7, 2021 · 3 comments
Labels
review
Milestone
x.22

Comments

@JerryShea
Copy link
Contributor

There are a few obvious gaps here - of particular concern is gaps in readingDocument0 and associated methods
@JerryShea JerryShea assigned nicktindall and unassigned JerryShea Jun 7, 2021
@peter-lawrey peter-lawrey added this to the x.22 milestone Jun 16, 2021
@JerryShea
Copy link
Contributor Author

Actually something is broken as coverage is reporting as 0% on https://sonarcloud.io/project/overview?id=OpenHFT_Chronicle-Queue

@minborg
Copy link
Contributor

minborg commented Apr 6, 2022 •
edited
Loading

These libraries have broken coverage: CQ, CTF, CV , and JLBH:

image

@JerryShea
Copy link
Contributor Author

JerryShea commented May 3, 2022 •
edited
Loading

Thanks @nicktindall for fixing up sonar! Coverage for this class is now up to 80%.

However, given the importance of this class, I think we have to leave this issue open.

@JerryShea JerryShea removed the wontfix label May 16, 2022
@tgd tgd added the review label Jun 3, 2024
@yevgenp yevgenp mentioned this issue Jun 20, 2024
Merged
@JerryShea JerryShea mentioned this issue Jul 2, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
review
Projects
None yet
Milestone
x.22
Development

When branches are created from issues, their pull requests are automatically linked.

feature/821-test-coverage-for-single-store-tailer OpenHFT/Chronicle-Queue
6 participants
@tgd @nicktindall @peter-lawrey @RobAustin @minborg @JerryShea