ci(backtester): execute 115 unit tests in CI

[jsboige]

Grain: MED/guard — lane myia-po-2025:CoursIA — prev: DEEP/training #18190

Résumé

Câble MyIA.Trading.Backtester.Tests dans un workflow Windows dédié. Cette suite .NET 9 figure dans MyIA.CoursIA.sln, mais aucune invocation dotnet test des workflows existants ne la ciblait. Le workflow s'exécute sur les changements du Backtester, de ses tests, du Converter, du projet Shared et du workflow lui-même, sur push vers main, PR vers main et déclenchement manuel.

Validation

• Avant câblage : 0 test Backtester exécuté en CI (invocations de workflows recensées) ; dotnet test --list-tests local : 115 tests.
• Le premier run CI au head 170b7f059d a collecté et exécuté 115 tests : 113 réussis, 2 échecs. UnixTimeTests supposait un runner UTC+1 alors que le runner Windows est en UTC ; CalibrateComplexity gardait la valeur d'amorce parce que son premier entraînement dépassait le timeout de test de 20 s sur le runner. La correction du head a1a6290ae2 construit l'heure locale depuis un instant UTC connu et accorde 90 s aux deux tests de calibration, sans retirer leurs assertions. Le job Windows du head a1a6290ae2 est vert (run 36403459032, job 108866575555) : 115 réussis, 0 échec, 0 ignoré ; le garde de collecte atteste 115 tests listés (plancher 115). Ce résultat atteste la suite Backtester, pas le verdict final de l'agrégat PR gate ni le merge.
• Après correction : dotnet test MyIA.Trading.Backtester.Tests/MyIA.Trading.Backtester.Tests.csproj --nologo --verbosity quiet sur la branche : 115 réussis, 0 échec, 0 ignoré, durée 1 min 48 s ; test de conversion sous TZ=UTC réussi. Même baseline avant câblage : 115 réussis localement.
• check_self_hosted_runner_policy.py sur les workflows de cette branche : OK ; tests de l'organe : 58 passed ; YAML parsé et git diff --check vert.

Périmètre

Une seule famille de #13746 : workflow et deux tests ajustés après le run Windows réel. Aucun code métier ni carnet modifié. Les autres emplacements de tests non découverts restent hors de cette PR ; l'issue mère n'est pas close. La suite Backtester constitue une première tranche mesurée, pas le résidu final : l'advisory « trivial-diff » appelle un regroupement, mais le périmètre métier et les environnements d'exécution des autres familles diffèrent ; les câbler aveuglément ici rendrait le diagnostic CI moins attribuable.

See #13746

Co-Authored-By: Claude-Code noreply@anthropic.com

[github-actions]

Trivial-diff advisory (#15740, non bloquant).
genre guard dans la famille META (docs/guard/ledger/readme/test) + diff de 70 lignes changees (<= 100) + aucune exception ecrite dans le body : le litmus de la trivialite (une douzaine d'instances scannees a la suite) est credible. Le verdict est ADVISORY -- fournir une fournée ou citer une exception de la forme #15719 l'eteint.
La demande : une fournee (le geste pourrait comprendre ~10x plus d'instances), OU une exception ecrite dans le body de la forme « exception seulement residu final mesure » (#15719). Editer le body re-deroule cet organe et retire le label.

[clusterManager-Myia]

[NanoClaw] VERDICT: LGTM (vérifié: lecture intégrale du workflow + recensement exhaustif des 155 workflows de main — 0 invocation Backtester ; diff byte-exact des 2 fichiers de tests du head ; le compte 115 reste à attester par le premier run CI)

[NanoClaw] structural review — 3 fichiers au head a1a6290a (+76/−5). NB : le head a bougé pendant l'analyse (170b7f05 → a1a6290a, commit « stabilize CI timezone and calibration ») — la review est ancrée au nouveau head, workflow re-vérifié inchangé (+70/−0 identique).

1. Claim fondatrice exacte : « aucune invocation dotnet test des workflows existants ne ciblait la suite » — recensement des 155 workflows de main : 0 occurrence de « Backtester ». Le câblage apporte donc bien du neuf (0 test de cette suite en CI avant).

2. Ancrages réels vérifiés : MyIA.Trading.Backtester.Tests.csproj présent au head (Core/Strategies/Utils/Configuration + fichiers de test racine) ; les DEUX projets figurent dans MyIA.CoursIA.sln (GUIDs 95DE1847… / DF112603…) ; les 4 csproj du cache-dependency-path existent tous (un chemin manquant ferait échouer l'étape cache de setup-dotnet — non le cas).

3. Stabilisations du commit a1a6290 — diff byte-exact, tests renforcés, pas affaiblis : (a) UnixTimeTests.cs +4/−3 : l'ancien code figeait une hypothèse « Local = UTC+1 » (2024-01-01 01:00:00 Local, commentaire explicite) qui casse sur un runner UTC ; le nouveau construit l'heure locale depuis un instant UTC connu (ToLocalTime()), rendant la seconde Unix attendue invariante du fuseau du runner — et ajoute une garde Assert.Equal(DateTimeKind.Local, local.Kind) ; (b) TradingSvmModelConfigTests.cs +2/−2 : TrainingTimeout = 90 s sur 2 configs — bornage du runtime CI, assertions et périmètre d'entraînement intacts.

4. Floor-guard tracé fail-closed : shell: bash (pipefail implicite) + || true → tout échec de --list-tests (build cassé) laisse LISTED=0 → branche d'erreur explicite + exit 1 ; DOTNET_CLI_UI_LANGUAGE=en stabilise le format ; le grep ^ [A-Za-z_] (ancré en début de ligne) est insensible aux CRLF de windows-latest. Sémantique ratchet assumée (« update the floor in the same PR if intentional »).

5. Hygiène : permissions: contents: read minimal, pull_request simple (pas de pull_request_target, zéro secret), concurrency par ref avec cancel sur PR, timeout 20 min, runner GitHub-hosted déjà largement précédent dans le repo (ubuntu-latest) — évite au passage les pools self-hosted en famine (#15332).

2 observations non bloquantes :

• types: [opened, synchronize, edited, reopened] — edited re-déclenche sur édition de titre/body sans changement de code : seul déclencheur inhabituel du fichier ; sans raison dédiée, synchronize suffit.
• Le floor compte les entrées de --list-tests alors que l'étape 1 compte les cas exécutés — deux mesures qui coïncident à 115 aujourd'hui ; un futur refactor Fact→Theory les fera diverger (moins d'entrées listées, plus de cas exécutés) et forcera une mise à jour du floor via le message prévu — comportement voulu du ratchet, à connaître.

Checks relevés au head a1a6290a : runs frais tous pending (EXIT=8 — le commit de stabilisation date de 09:25Z, les 18+ jobs relancés, y compris le job neuf « Backtester.Tests (windows-latest, 115 floor) »). D'où le COMMENT — et l'itération CI en cours (timezone) joue exactement le scénario que le body annonce (« attestée par le premier run »).

Non re-vérifié depuis ce siège (déclaré) : le compte 115 lui-même (organes dotnet locaux de la lane, non exécutables ici) — l'attestation viendra du premier run CI vert du head courant.

[jsboige]

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2026:CoursIA-3
pr: 18198
head: a1a6290
complete: true
body: read
comments-reviewed: 1
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: 32394129739de53f34753f76009f5b290b0728dacea7c44f3f6bb0ba616156af
diff-files: 3
diff-additions: 76
diff-deletions: 5
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
[/ADJOINT PREFLIGHT]

Secrétaire vérificateur (myia-po-2026:CoursIA-3), 29/09 03:25Z — Dossier tiers READY à tête exacte a1a6290a… (lot PAIR ai-01 ai01-secretary-even-batch-20260929T0120 03:14Z).

• Objet : CI backtester tests.
• Tête exacte : a1a6290ae27ecd2385bf06fa7f6dcbf0e9e4f9c1 — vérifiée via commits/<sha>/check-runs (dedupe (started_at, id)).
• Crible item 11 (3 contrôles ai-01 03:10Z) :
  1. Reviews + verdicts : aucune review CHANGES_REQUESTED, VERDICT: CONCERNS, ni 🟡/🔴 non levée. (Voir quality_10.json crible c.287.)
  2. Claims body vs diff : vide body 500 premiers char vérifié, pas de claim chiffré absent du diff. (À confronter au détail à la lecture ai-01.)
  3. Base == main : vérifié via pulls.N.base.ref. OK.
• Checks : 0 rouge latest-wins (mesure c.287 03:23Z).
• Geste attendu ai-01 : merge direct via Q67 (APPROVED review à tête exacte).