FlexMeasures / flexmeasures / 30483630671

83%

main: 83%

push

github

fix: address round-2 Copilot findings (count_annotations bug, docstrings, exclude overwrite)

- GenericAsset.count_annotations() called .count() directly on the Select
  returned by query_asset_annotations(), which has no such method (a
  pre-existing bug, surfaced by this PR's Query->Select type-hint fix).
  Now executes via db.session.scalar(select(func.count()).select_from(...)).
  Added a regression test.
- AssetChartOpenAPISchema/AssetChartDataOpenAPISchema.__init__ overwrote
  kwargs["exclude"] instead of extending it; now appends to any
  caller-supplied excludes.
- dev/sensors.py docstrings still mentioned legacy underscored names
  (include_data, chart_type, most_recent_beliefs_only) for fields now
  canonically hyphenated; updated to the hyphenated spelling with a
  legacy-alias note, matching the start/end/prior docs already updated.

The OpenAPI spec's info.version ("1.0.0") flagged by Copilot predates
this PR (confirmed unchanged since before any of these commits) and is
unrelated to this diff -- no action taken.

Signed-off-by: F.N. Claessen <claessen@seita.nl>

2 of 4 new or added lines in 2 files covered. (50.0%)

16147 of 19440 relevant lines covered (83.06%)

0.83 hits per line