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.

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

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 :)

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

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 · 2026-03-16T09:07:12Z

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

mdrrrr t'inquiète pas de soucis, mais yep si ta des issue je peux check

ltuffery · 2026-03-16T10:04:12Z

@gtolontop c'est ready for review cette PR ? Car actuellement tu la mis draft

gtolontop · 2026-03-16T11:19:19Z

@gtolontop c'est ready for review cette PR ? Car actuellement tu la mis draft

Yep je la met

iambibi · 2026-03-16T11:46:50Z

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

mdrrrr t'inquiète pas de soucis, mais yep si ta des issue je peux check

La liaison Minecraft-Discord
#455
Ou si tu veux t'amuser tu peux faire une feature de bingo qui est actif pdt les week ends
#661

iambibi · 2026-03-16T11:47:38Z

Après si tu veux pas faire de features comme tu veux mais ça a peu de valeur de fixer des bugs, fin c'est cool mais le joueur va pas réellement le remarquer ahah

gtolontop · 2026-03-16T12:08:50Z

Je joue pas vraiment j'ai pas trop d'idée donc si vous voulez des truc dite le moi et je vous le ferez sans soucis mais faut me dire se que vous voulez, feature etc

gtolontop · 2026-03-16T12:09:05Z

Je vais check les deux issue que tu as send merci

iambibi · 2026-03-16T13:03:57Z

Je joue pas vraiment j'ai pas trop d'idée donc si vous voulez des truc dite le moi et je vous le ferez sans soucis mais faut me dire se que vous voulez, feature etc

Faudra qd même jouer pour tester ta feature hein

iambibi · 2026-03-16T13:04:46Z

Pour la liaison discord tu peux check le repo Plugin (de la v1), il contenait un système qui faisait la liaison discord, (avec juste une interaction base de donnée)

gtolontop · 2026-03-16T14:29:52Z

Je joue pas vraiment j'ai pas trop d'idée donc si vous voulez des truc dite le moi et je vous le ferez sans soucis mais faut me dire se que vous voulez, feature etc

Faudra qd même jouer pour tester ta feature hein

Oui je sais mdrrr je disais juste je joue pas en tant que joueur xd et c'est pour ça que j'ai pas forcément des idées de se qu'il faudrait ajouter

gtolontop · 2026-03-16T14:30:12Z

Pour la liaison discord tu peux check le repo Plugin (de la v1), il contenait un système qui faisait la liaison discord, (avec juste une interaction base de donnée)

Je vais check j'en avais fais un à l'époque

iambibi · 2026-03-16T14:44:05Z

Pour la liaison discord tu peux check le repo Plugin (de la v1), il contenait un système qui faisait la liaison discord, (avec juste une interaction base de donnée)

Je vais check j'en avais fais un à l'époque

N'implemente pas un bot discord dans le plugin ahah
#456 (comme cette personne)
Conseil que j'avais donné au mec la :
Donc sur la V1 on avait un système où le plugin mettait un lien dans la DB et ensuite dans une periode précise si le joueur ne fait pas la commande /link sur discord alors on supprime le lien dans le db. Et le bot a lui aussi acces a la db donc il peut regarder si le code existe et apres il mets dans la db un lien uuid minecraft et id discord

Exemple : LinkerApi https://github.com/ServerOpenMC/Plugin/blob/main/src/main/java/fr/communaywen/core/utils/LinkerAPI.java Commande /link https://github.com/ServerOpenMC/Plugin/blob/main/src/main/java/fr/communaywen/core/commands/link/LinkCommand.java

gtolontop · 2026-03-16T15:41:39Z

Pour la liaison discord tu peux check le repo Plugin (de la v1), il contenait un système qui faisait la liaison discord, (avec juste une interaction base de donnée)

Je vais check j'en avais fais un à l'époque

N'implemente pas un bot discord dans le plugin ahah #456 (comme cette personne) Conseil que j'avais donné au mec la : Donc sur la V1 on avait un système où le plugin mettait un lien dans la DB et ensuite dans une periode précise si le joueur ne fait pas la commande /link sur discord alors on supprime le lien dans le db. Et le bot a lui aussi acces a la db donc il peut regarder si le code existe et apres il mets dans la db un lien uuid minecraft et id discord

Exemple : LinkerApi https://github.com/ServerOpenMC/Plugin/blob/main/src/main/java/fr/communaywen/core/utils/LinkerAPI.java Commande /link https://github.com/ServerOpenMC/Plugin/blob/main/src/main/java/fr/communaywen/core/commands/link/LinkCommand.java

mdrrrrrrrrrrrrrrr j'avais pas prévue de foutre un bot 😂

PuppyTransGirl

la plupart sont les mêmes reviews mais à des endroits différents

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

src/test/java/fr/openmc/core/utils/InputUtilsTest.java

PuppyTransGirl · 2026-03-16T18:29:08Z

src/test/java/fr/openmc/core/utils/MotdUtilsTest.java

        String motd = getComponentContent(server.motd());

-        new MotdUtils();
        server.getScheduler().performTicks(12001L);


est-ce que le performTicks est encore utile vu que tu changes le motd instantanément ?

build.gradle

iambibi · 2026-03-18T10:54:30Z

tu as des reviews @gtolontop si tu avais pas vu

- Suppression des performTicks redondants avant waitAsyncTasksFinished - Ajout de tests NaN, Infinity, -Infinity dans InputUtilsTest - Suppression du bloc testLogging de debug dans build.gradle - Fix locale EconomyFormattingTest pour éviter les échecs liés au séparateur décimal

iambibi · 2026-03-19T16:05:21Z

Re review nécessaire par les auteurs des changements, ensuite on pourra la merge

ltuffery

LGFM

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

src/main/java/fr/openmc/core/features/cube/multiblocks/MultiBlockManager.java Outdated Show resolved Hide resolved

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

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

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

gtolontop marked this pull request as ready for review March 16, 2026 11:19

PuppyTransGirl requested changes Mar 16, 2026

View reviewed changes

ltuffery approved these changes Mar 20, 2026

View reviewed changes

iambibi merged commit 8aa0787 into ServerOpenMC:master Mar 20, 2026
3 checks passed

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!

Uh oh!

gtolontop commented Mar 12, 2026

Uh oh!

gtolontop commented Mar 12, 2026

Uh oh!

gtolontop commented Mar 12, 2026

Uh oh!

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!

gtolontop commented Mar 16, 2026

Uh oh!

ltuffery commented Mar 16, 2026

Uh oh!

gtolontop commented Mar 16, 2026

Uh oh!

iambibi commented Mar 16, 2026

Uh oh!

iambibi commented Mar 16, 2026

Uh oh!

gtolontop commented Mar 16, 2026

Uh oh!

gtolontop commented Mar 16, 2026

Uh oh!

iambibi commented Mar 16, 2026

Uh oh!

iambibi commented Mar 16, 2026

Uh oh!

gtolontop commented Mar 16, 2026

Uh oh!

gtolontop commented Mar 16, 2026

Uh oh!

iambibi commented Mar 16, 2026

Uh oh!

gtolontop commented Mar 16, 2026

Uh oh!

PuppyTransGirl left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

PuppyTransGirl Mar 16, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

iambibi commented Mar 18, 2026

Uh oh!