dataforgoodfr / dataforgoodfr/Coordonnees

[Coordo PY] Chore: Create Test Suite for indicators

Open
#49 0 comments 0 reactions 1 assignee View on GitHub

@OlivierCoen is already working on this.

Since Mar 26, 2026.

coordo-py P2
Dominant language
Python
Stars
6
Forks
0
PR merge metrics
No merged PRs in 30d

Description

Objective

As a Lead Dev, when new changes are performed concerning indicator computation, I want to ensure no regression has been introduced.

Context

Coordo-py lib is growing in complexity and we don't have any test suite for now.
We have a complex Parser that translates expressions in a custom Lark Grammar into SQL queries, that is very error prone and not stable yet.
The parser is not stable yet and is going to evolve as we define new indicators for the Seed and All4Trees project, thus we must ensure that all future evolutions will not break anything that was working before.

What to Test :
  • All basic agregate functions : avg, sum, list, etc.
  • All custom functions calls: unique, centroid, percentile, shannon, gini
  • Arithmetic operators
  • Comparison operators
  • Conditional operators
  • Operations over floats and or integers
  • Aggregate function over conditions

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.