fix error handling in /kill and disable /kill test until we figure out whether the server should send the despawn packet or not when the entity dies

This commit is contained in:
Romain Beaumont 2015-12-11 04:32:33 +01:00
parent 9366c1b9a7
commit ce1ffe9c4f
2 changed files with 4 additions and 4 deletions

View file

@ -1,4 +1,5 @@
var Vec3 = require("vec3").Vec3;
var UserError = require('flying-squid').UserError;
module.exports.player=function(player,serv)
{
@ -45,8 +46,7 @@ module.exports.player=function(player,serv)
},
action(sel) {
var arr = serv.selectorString(sel, player.position.scaled(1/32), player.world);
if (arr instanceof Error) return arr.toString();
if (arr == null) return 'Could not find player';
if (arr.length==0) throw new UserError('Could not find player');
arr.map(entity => entity.takeDamage({damage:20}));
}

View file

@ -116,9 +116,9 @@ describe("Server with mineflayer connection", function() {
bot.on('entitySpawn',listener);
});
});
it("can use /kill",async () => {
it.skip("can use /kill",async () => { // TODO: figure out why that fails : mineflayer doesn't seem to emit entityDead
bot.chat('/kill @e[type=EnderDragon]');
const entity=await once(bot,'entityGone');
const entity=await once(bot,'entityDead');
assert.equal(entity.name,"EnderDragon");
});
describe("can use /tp",() => {