Make sure we pass a string to a storagebuilder (#2645)

* Make sure we pass a string to a storagebuilder

* Add test

* Fix lint

---------

Co-authored-by: Romain Beaumont <romain.rom1@gmail.com>

Author: u9g

lib/plugins/blocks.js

@@ -470,9 +470,11 @@ function inject (bot, { version, storageBuilder, hideErrors }) {
     if (bot.supportFeature('dimensionIsAnInt')) {
       return dimensionNames[dimension]
     } else if (bot.supportFeature('dimensionIsAString') || bot.supportFeature('dimensionIsAWorld')) {
-      return dimension
+      return worldName
     }
   }
+  // only exposed for testing
+  bot._getDimensionName = () => worldName
 
   async function switchWorld () {
     if (bot.world) {
@@ -503,9 +505,10 @@ function inject (bot, { version, storageBuilder, hideErrors }) {
   bot._client.on('login', (packet) => {
     if (bot.supportFeature('dimensionIsAnInt')) {
       dimension = packet.dimension
+      worldName = dimensionToFolderName(dimension)
     } else {
       dimension = packet.dimension
-      worldName = packet.worldName
+      worldName = /^minecraft:.+/.test(packet.worldName) ? packet.worldName : `minecraft:${packet.worldName}`
     }
     switchWorld()
   })

test/externalTests/dimensionName.js

@@ -0,0 +1,5 @@
+const assert = require('assert')
+
+module.exports = () => async (bot) => {
+  assert.strictEqual(bot._getDimensionName(), 'minecraft:overworld')
+}