Thread-safety : {@code LOGGED_ERRORS} est un + * {@link ConcurrentHashMap#newKeySet()} — aucun besoin de synchronized + * même si appelé depuis un thread loader pendant qu'un tick client est + * en flight.
*/ public static void resetLoggedErrors() { LOGGED_ERRORS.clear(); diff --git a/src/main/resources/assets/tiedup/animmodels/animations/context_stand_idle.json b/src/main/resources/assets/tiedup/animmodels/animations/context_stand_idle.json index 090fcb3..5d2efb5 100644 --- a/src/main/resources/assets/tiedup/animmodels/animations/context_stand_idle.json +++ b/src/main/resources/assets/tiedup/animmodels/animations/context_stand_idle.json @@ -1,7 +1,7 @@ { "_comment": "PLACEHOLDER Phase 2.7 — animation idle 2-keyframes identity (joueur statique, pas de balancement). Boucle 2 secondes. DOIT parser avec JsonAssetLoader.loadClipForAnimation(). À remplacer par un export Blender authored (respiration/balancement subtle) quand dispo. Voir docs/plans/rig/ASSETS_NEEDED.md section 2.", + "_comment_armature": "Pas de champ 'armature' ici — le loader ne lit pas ce champ au runtime. L'armature est résolue par le call site (voir DirectStaticAnimation ctor dans TiedUpAnimationRegistry.initStaticAnimations, qui passe TiedUpArmatures.BIPED au super).", "format": "ATTRIBUTES", - "armature": "tiedup:biped", "animation": [ { "name": "Root", diff --git a/src/main/resources/assets/tiedup/armatures/biped.json b/src/main/resources/assets/tiedup/armatures/biped.json index dff2563..5b02043 100644 --- a/src/main/resources/assets/tiedup/armatures/biped.json +++ b/src/main/resources/assets/tiedup/armatures/biped.json @@ -1,5 +1,5 @@ { - "_comment": "PLACEHOLDER Phase 2.7 — identity transforms (joints empilés à l'origine). Doit parser avec JsonAssetLoader.loadArmature() mais n'est PAS encore utilisé au runtime (TiedUpArmatures.BIPED est construit proceduralement en Java). À remplacer par un export Blender addon Antikythera-Studios quand dispo. Voir docs/plans/rig/ASSETS_NEEDED.md section 1.", + "_comment": "PLACEHOLDER FILE - NOT LOADED AT RUNTIME Phase 2.7. TiedUpArmatures.BIPED is built procedurally (voir TiedUpArmatures.buildBiped). This JSON is a reference for future Blender-authored biped.json (Phase 3+). Voir docs/plans/rig/ASSETS_NEEDED.md section 1.", "armature": { "armature_format": "ATTRIBUTES", "joints": [ diff --git a/src/test/java/com/tiedup/remake/rig/TiedUpAnimationRegistryTest.java b/src/test/java/com/tiedup/remake/rig/TiedUpAnimationRegistryTest.java new file mode 100644 index 0000000..8278534 --- /dev/null +++ b/src/test/java/com/tiedup/remake/rig/TiedUpAnimationRegistryTest.java @@ -0,0 +1,94 @@ +/* + * © 2026 TiedUp! Remake Contributors, distributed under GPLv3. + */ + +package com.tiedup.remake.rig; + +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertNotNull; +import static org.junit.jupiter.api.Assertions.assertSame; + +import net.minecraft.resources.ResourceLocation; + +import org.junit.jupiter.api.Test; + +import com.tiedup.remake.rig.anim.types.DirectStaticAnimation; + +/** + * Tests du registry {@link TiedUpAnimationRegistry}. + * + *Ces tests verrouillent :
+ *Les tests n'ont PAS besoin du runtime MC — {@code initStaticAnimations()} + * construit juste les instances {@link DirectStaticAnimation} (pas de + * {@code loadAnimation()} déclenché tant que {@code getAnimationClip()} n'est + * pas lu). La {@code ResourceLocation} créée via le constructor n'a pas besoin + * de Minecraft runtime non plus.
+ */ +class TiedUpAnimationRegistryTest { + + /** + * Appeler {@link TiedUpAnimationRegistry#initStaticAnimations()} deux fois + * de suite doit être un no-op sur le second appel — le champ + * {@code CONTEXT_STAND_IDLE} garde la même référence. + * + *Si l'init n'était pas idempotent, un double-call (scénario hot-reload + * dev, test runner partageant la JVM, etc.) recréerait l'instance et + * invaliderait tous les sites qui l'ont capturée en field final ou local. + * Le guard {@code if (CONTEXT_STAND_IDLE != null) return;} protège de ça — + * ce test s'assure qu'il n'est pas retiré en régression.
+ */ + @Test + void initStaticAnimationsIsIdempotent() { + TiedUpAnimationRegistry.initStaticAnimations(); + + assertNotNull(TiedUpAnimationRegistry.CONTEXT_STAND_IDLE, + "Après initStaticAnimations, CONTEXT_STAND_IDLE doit être non-null"); + + DirectStaticAnimation firstInstance = TiedUpAnimationRegistry.CONTEXT_STAND_IDLE; + + // Deuxième call — doit être no-op, pas recréer l'instance. + TiedUpAnimationRegistry.initStaticAnimations(); + + assertSame(firstInstance, TiedUpAnimationRegistry.CONTEXT_STAND_IDLE, + "Un second call à initStaticAnimations ne doit pas remplacer l'instance — " + + "le guard `if (CONTEXT_STAND_IDLE != null) return;` serait cassé sinon"); + } + + /** + * Le {@code registryName()} de {@code CONTEXT_STAND_IDLE} doit matcher + * {@code tiedup:context_stand_idle} — alignement avec le path du JSON + * asset {@code assets/tiedup/animmodels/animations/context_stand_idle.json}. + * + *Si le registry name dérive du path (via la résolution + * {@code resourceLocation = animmodels/animations/${registry.path}.json}), + * une désynchronisation code vs asset se traduit par un + * {@code AssetLoadingException} runtime. Ce test est un smoke check rapide + * au niveau de l'identité du registry name.
+ */ + @Test + void contextStandIdleHasRegistryName() { + TiedUpAnimationRegistry.initStaticAnimations(); + + DirectStaticAnimation anim = TiedUpAnimationRegistry.CONTEXT_STAND_IDLE; + assertNotNull(anim, "CONTEXT_STAND_IDLE doit être initialisé"); + + ResourceLocation expected = + ResourceLocation.fromNamespaceAndPath(TiedUpRigConstants.MODID, "context_stand_idle"); + + assertEquals(expected, anim.registryName(), + "CONTEXT_STAND_IDLE.registryName() doit être tiedup:context_stand_idle — " + + "désynchronisation avec assets/tiedup/animmodels/animations/context_stand_idle.json sinon"); + } +}