refactor: remove unused test entity command from built-in commands

- Deleted the newTestEntityCmd function and its registration in the built-in commands.
- Streamlined the command registration process by eliminating unnecessary code.

This change helps maintain cleaner code and reduces potential confusion regarding unused commands.

Author: robinbraemer

pkg/edition/java/proxy/builtin_commands.go

@@ -2,17 +2,14 @@ package proxy
 
 import (
 	"go.minekube.com/brigodier"
-	. "go.minekube.com/common/minecraft/component"
 	"go.minekube.com/gate/pkg/command"
-	"go.minekube.com/gate/pkg/edition/java/proto/packet/brigadier"
 )
 
 func (p *Proxy) registerBuiltinCommands() []string {
 	return []string{
 		p.command.Register(newServerCmd(p)).Name(),
 		p.command.Register(newGlistCmd(p)).Name(),
 		p.command.Register(newSendCmd(p)).Name(),
-		p.command.Register(newTestEntityCmd(p)).Name(),
 	}
 }
 
@@ -21,14 +18,3 @@ func hasCmdPerm(proxy *Proxy, perm string) brigodier.RequireFn {
 		return !proxy.cfg.RequireBuiltinCommandPermissions || c.Source.HasPermission(perm)
 	})
 }
-
-func newTestEntityCmd(proxy *Proxy) brigodier.LiteralNodeBuilder {
-	return brigodier.Literal("testentity").
-		Then(brigodier.Argument("player", brigadier.PlayerArgument).
-			Executes(command.Command(func(c *command.Context) error {
-				playerName := c.String("player")
-				return c.SendMessage(&Text{
-					Content: "EntityArgumentType: You specified player '" + playerName + "' - this uses proper minecraft:entity argument type!",
-				})
-			})))
-}