fix: correction des tests unitaires et ajout de nouveau test by gtolontop · Pull Request #1170 · ServerOpenMC/PluginV2

gtolontop · 2026-03-11T16:32:29Z

Fixes les tests unitaires qui échouaient avec ClassNotFoundException.

Problème

SpawnerExtractorListener importe dev.lone.itemsadder.api.CustomStack, une dépendance compileOnly absente du classpath de test. Le classloader de MockBukkit résout les références de classes de façon eager, ce qui provoque un NoClassDefFoundError lors du chargement du plugin en test.

Le listener était enregistré sans le guard !isUnitTestVersion(), contrairement à ItemsAddersListener qui a le même problème et est déjà correctement protégé.

Correction

Déplacement de new SpawnerExtractorListener() à l'intérieur du bloc !OMCPlugin.isUnitTestVersion(), cohérent avec le fix FancyNpcsHook du commit bd1ab12.

…Version

TabList utilise PacketAdapter de ProtocolLib (dépendance compileOnly), ce qui cause un ClassNotFoundException au lancement des tests MockBukkit.

MultiBlockManager.init() est appelé via loadWithItemsAdder() qui ne s'exécute pas en environnement de test. config reste null et provoque un NPE lors du onDisable.

Le BukkitRunnable (delay 0) s'exécute pendant le load du plugin, donc le MOTD capturé est déjà le custom. Avec 2 entrées dans motd.yml, Random peut piocher la même → assertNotEquals échoue 50% du temps. Fix: forcer un MOTD connu avant de tick.

Le test testTransferBalanceWithReasonRegistersTransaction échoue car la transaction est enregistrée via runTaskAsynchronously et performTicks ne garantit pas la fin des tâches async. Ajout de waitAsyncTasksFinished() pour attendre la complétion avant de vérifier.

iambibi · 2026-03-11T18:19:45Z

Ah tu as réussi

iambibi · 2026-03-11T18:20:21Z

Oui nirbose avait dit que les task en async faisait buger les tests ahaha j'ai pas pensé à te le dire

iambibi

j'attends la review de Nirbose, mais pour moi ça me va

gtolontop · 2026-03-11T20:19:57Z

Il faut que je push encore une dernière version pour fix la dernière issue et se sera good :)

gtolontop · 2026-03-11T20:20:43Z

Je suis entrain de les ajouter

Nouveaux tests: - DateUtils: conversion secondes/millis/ticks, isBefore - InputUtils: pluralize, edge cases null/empty/décimaux - PaddingUtils: centrage, débordement, padding impair - EnumUtils: match valide/invalide/null, valeur par défaut - MathUtils: interpolation de couleurs (lerpColor) - YmlUtils: deep copy de maps/listes imbriquées - Queue: ajout, suppression, éviction LRU, taille max - ColorUtils: conversions couleur/code/nom/matériau/RGB - RandomUtils: bornes int/double/float, cas limites - EconomyManager: formatage simplifié (k/M/B suffixes)

Ajout de waitAsyncTasksFinished() après performTicks() dans tous les tests Economy qui vérifient des transactions enregistrées en async.

gtolontop · 2026-03-11T20:39:47Z

Les tests on bloqué sur de l'ancien code je l'ai fix ça build ça devrais passer mais dcp y'a des test + les fix des résultats des test qui passais pas

iambibi · 2026-03-12T05:39:41Z

Rename ta branche + le nom de ta pr dcp

ltuffery

Ça ne paraît pas OK. J’ai juste cette remarque, puis je ferai la review finale.

ltuffery · 2026-03-12T11:02:52Z

src/main/java/fr/openmc/core/features/cube/multiblocks/MultiBlockManager.java

            list.add(map);
        }

+        if (config == null) return;


gtolontop · 2026-03-12T11:23:38Z

Je termine tous proprement quand je rentre chez moi et se sera good

gtolontop · 2026-03-12T11:24:04Z

et quel son les soucis ?

Évite d'itérer inutilement sur les multiblocks si config n'a pas été initialisé (init() appelé uniquement via loadWithItemsAdder).

gtolontop · 2026-03-12T18:43:33Z

Tous devrais être bon :)

PuppyTransGirl · 2026-03-12T19:02:00Z

src/main/java/fr/openmc/core/features/cube/multiblocks/MultiBlockManager.java

    }

    public static void save() {
+        if (config == null) return;


pourquoi mettre ce check ?

@ltuffery l'avait mis dans son commit, je l'ai juste déplacé au début du fichier pour éviter d'itérer pour rien. Il a ajouter le check (je pense) parce que MultiBlockManager.init() passe par loadWithItemsAdder() qui ne tourne pas en test, dcp config reste null et save() dans le onDisable() casse un NPE

Puis c'est vrai que si la config est nulle, ça va planter, je vois pas en quoi c'est dérangeant

C'est plus rapide j'ai juste bouger ta ligne

J'ai juste pas compris de quel commit tu parles

Surtout que même @ltuffery a demandé pourquoi tu l'as mit

src/test/java/fr/openmc/core/features/economy/EconomyFormattingTest.java

gtolontop · 2026-03-15T21:01:58Z

Vous voulez que je fasse des changements supplémentaire ?

iambibi · 2026-03-15T21:32:32Z

Bah je sais pas trop pour moi c'est OK, mais je crois que @ltuffery n'a pas encore fait sa review

gtolontop · 2026-03-15T21:46:38Z

Bah je sais pas trop pour moi c'est OK, mais je crois que @ltuffery n'a pas encore fait sa review

Okay hésitez :)
(si tu as d'autre issue a fix je peux les fix se soir je suis dispo)

iambibi · 2026-03-16T05:42:26Z

Ouais désolé si ça prends un peu de temps (regarde le nombre de pr que j'ai, la majorité c'est des une lignes rien passe, je crois qu'ils ont pas encore compris que ça a commencé 🤡).
En parallèle si tu veux encore contribuer mais tu sais pas quelle issue je peux t'en donner une

gtolontop added 6 commits March 11, 2026 17:32

fix: déplacement du SpawnerExtractorListener dans le guard isUnitTest…

38b8f1b

…Version

chore: logs verbeux pour diagnostiquer les tests unitaires

74b91e4

fix: ignorer TabList dans les tests unitaires

a5e6a04

TabList utilise PacketAdapter de ProtocolLib (dépendance compileOnly), ce qui cause un ClassNotFoundException au lancement des tests MockBukkit.

fix: null check dans MultiBlockManager.save()

81ed6e9

MultiBlockManager.init() est appelé via loadWithItemsAdder() qui ne s'exécute pas en environnement de test. config reste null et provoque un NPE lors du onDisable.

iambibi added the ✨ Fixes Quelque chose a été patch label Mar 11, 2026

iambibi added this to the 2.2.8 milestone Mar 11, 2026

iambibi assigned gtolontop Mar 11, 2026

iambibi requested a review from ltuffery March 11, 2026 18:41

iambibi reviewed Mar 11, 2026

View reviewed changes

gtolontop added 2 commits March 11, 2026 21:28

fix: waitAsyncTasksFinished sur tous les tests de transactions

5f5c6ec

Ajout de waitAsyncTasksFinished() après performTicks() dans tous les tests Economy qui vérifient des transactions enregistrées en async.

ltuffery reviewed Mar 12, 2026

View reviewed changes

fix: déplacer le null check au début de MultiBlockManager.save()

ffeacbd

Évite d'itérer inutilement sur les multiblocks si config n'a pas été initialisé (init() appelé uniquement via loadWithItemsAdder).

gtolontop changed the title ~~fix: Tests unitaires cassés à cause du SpawnerExtractorListener~~ fix: correction des tests unitaires et ajout de nouveau test Mar 12, 2026

PuppyTransGirl reviewed Mar 12, 2026

View reviewed changes

iambibi requested changes Mar 12, 2026

View reviewed changes

src/test/java/fr/openmc/core/features/economy/EconomyFormattingTest.java Show resolved Hide resolved

gtolontop marked this pull request as draft March 12, 2026 20:29

iambibi modified the milestones: 2.2.8, 2.2.9 Mar 14, 2026

Conversation

gtolontop commented Mar 11, 2026

Problème

Correction

Uh oh!

iambibi commented Mar 11, 2026

Uh oh!

iambibi commented Mar 11, 2026

Uh oh!

iambibi left a comment

Choose a reason for hiding this comment

Uh oh!

gtolontop commented Mar 11, 2026

Uh oh!

gtolontop commented Mar 11, 2026

Uh oh!

gtolontop commented Mar 11, 2026

Uh oh!

iambibi commented Mar 12, 2026

Uh oh!

ltuffery left a comment

Choose a reason for hiding this comment

Uh oh!

ltuffery Mar 12, 2026

Choose a reason for hiding this comment

Uh oh!

gtolontop commented Mar 12, 2026

Uh oh!

gtolontop commented Mar 12, 2026

Uh oh!

gtolontop commented Mar 12, 2026

Uh oh!

PuppyTransGirl Mar 12, 2026

Choose a reason for hiding this comment

Uh oh!

gtolontop Mar 12, 2026

Choose a reason for hiding this comment

Uh oh!

iambibi Mar 12, 2026

Choose a reason for hiding this comment

Uh oh!

gtolontop Mar 12, 2026

Choose a reason for hiding this comment

Uh oh!

PuppyTransGirl Mar 12, 2026

Choose a reason for hiding this comment

Uh oh!

PuppyTransGirl Mar 12, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

gtolontop commented Mar 15, 2026

Uh oh!

iambibi commented Mar 15, 2026

Uh oh!

gtolontop commented Mar 15, 2026

Uh oh!

iambibi commented Mar 16, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants