Fix gamemode tests

Add test function to kill the bot
PrismarineJS · Nov 21, 2024 · c362658 · c362658
1 parent cd66986
commit c362658
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 7 deletions.
diff --git a/lib/plugins/game.js b/lib/plugins/game.js
@@ -25,10 +25,11 @@ function inject (bot, options) {
   function handleRespawnPacketData (packet) {
     bot.game.levelType = packet.levelType ?? (packet.isFlat ? 'flat' : 'default')
     bot.game.hardcore = packet.isHardcore ?? Boolean(packet.gameMode & 0b100)
-    if (bot.registry.isOlderThan('1.10')) { // gamemode is used pre 1.10 and post 1.20 but in between it's gameMode
-      bot.game.gameMode = parseGameMode(packet.gamemode)
+    // Either a respawn packet or a login packet. Depending on the packet it can be "gamemode" or "gameMode"
+    if (bot.registry.isNewerOrEqualTo('1.20.5')) {
+      bot.game.gameMode = packet.gamemode || parseGameMode(packet.gameMode)
     } else {
-      bot.game.gameMode = packet.gamemode || parseGameMode(packet.gameMode & 0b1111) // lower four bits
+      bot.game.gameMode = parseGameMode(packet.gamemode ?? packet.gameMode)
     }
     if (bot.supportFeature('segmentedRegistryCodecData')) { // 1.20.5
       if (typeof packet.dimension === 'number') {

diff --git a/test/externalTests/gamemode.js b/test/externalTests/gamemode.js
@@ -19,10 +19,8 @@ module.exports = () => {
 
   addTest('after respawn', async (bot) => {
     await bot.test.becomeCreative()
-
-    bot.chat('/kill')
-
-    await onceWithCleanup(bot, 'respawn', { timeout: 2000 })
+    bot.test.selfKill()
+    await onceWithCleanup(bot, 'respawn', { timeout: 5000 })
     // Respawn packets send the gamemode. If the bot is in creative mode, it should respawn in creative mode. Tested <1.20
     assert.strictEqual(bot.game.gameMode, 'creative', 'Wrong gamemode after respawn')
   })

diff --git a/test/externalTests/plugins/testCommon.js b/test/externalTests/plugins/testCommon.js
@@ -25,6 +25,7 @@ function inject (bot) {
   bot.test.placeBlock = placeBlock
   bot.test.runExample = runExample
   bot.test.tellAndListen = tellAndListen
+  bot.test.selfKill = selfKill
   bot.test.wait = function (ms) {
     return new Promise((resolve) => { setTimeout(resolve, ms) })
   }
@@ -232,4 +233,8 @@ function inject (bot) {
     }
     return closeExample()
   }
+
+  function selfKill () {
+    bot.chat('/kill @p')
+  }
 }