CoNexDat / CoNexDat/LaNet-vi

Add tests for visualization/ and metrics/, then raise coverage threshold to 60%

Open
#11 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
8
Forks
0
Avg merge
6h 10m
Merged PRs (30d)
16

Description

## Context

CI enforces `--cov-fail-under=40` (`pyproject.toml`), while actual coverage sits around 55%. The gap is almost entirely in `src/lanet_vi/visualization/` (layout, matplotlib renderer) and `src/lanet_vi/metrics/`, which have no direct tests.

## Plan

1. Add tests for `visualization/` — layout coordinates for a small known graph, renderer smoke tests on a headless `Agg` backend, legend content (this would also catch #10).
2. Add tests for `metrics/`.
3. Raise `--cov-fail-under` to 60 once the suite is there. Do not raise the threshold first — that just makes CI red without adding coverage.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Contributor guide

Open the contributing guide

Research direction

Start with the existing test layout and the --cov-fail-under=40 setting in pyproject.toml, then inspect src/lanet_vi/visualization/ and src/lanet_vi/metrics/. Run the current test suite with coverage to establish the baseline. Done means direct tests cover the listed layout, headless renderer, legend, and metrics behavior, and CI passes with the threshold raised to 60%.

Written by the indexing model from the issue text.

Assessment

Tech stack
matplotlib, python
Domain
data-visualization, testing
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.