From 133a64a73dd5607294d6b76d9176437f19721f74 Mon Sep 17 00:00:00 2001
From: md_5 <git@md-5.net>
Date: Fri, 27 Sep 2024 08:21:26 +1000
Subject: [PATCH] Improve Registry#getOrThrow messages

---
 src/main/java/org/bukkit/Registry.java           | 8 ++++----
 src/test/java/org/bukkit/support/TestServer.java | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/main/java/org/bukkit/Registry.java b/src/main/java/org/bukkit/Registry.java
index feb22f7e..30ba3dab 100644
--- a/src/main/java/org/bukkit/Registry.java
+++ b/src/main/java/org/bukkit/Registry.java
@@ -65,7 +65,7 @@ public interface Registry<T extends Keyed> extends Iterable<T> {
         public Advancement getOrThrow(@NotNull NamespacedKey key) {
             Advancement advancement = get(key);
 
-            Preconditions.checkArgument(advancement != null, "No registry entry found for key " + key);
+            Preconditions.checkArgument(advancement != null, "No Advancement registry entry found for key %s.", key);
 
             return advancement;
         }
@@ -133,7 +133,7 @@ public interface Registry<T extends Keyed> extends Iterable<T> {
         public KeyedBossBar getOrThrow(@NotNull NamespacedKey key) {
             KeyedBossBar keyedBossBar = get(key);
 
-            Preconditions.checkArgument(keyedBossBar != null, "No registry entry found for key " + key);
+            Preconditions.checkArgument(keyedBossBar != null, "No KeyedBossBar registry entry found for key %s.", key);
 
             return keyedBossBar;
         }
@@ -307,7 +307,7 @@ public interface Registry<T extends Keyed> extends Iterable<T> {
         public MemoryKey getOrThrow(@NotNull NamespacedKey key) {
             MemoryKey memoryKey = get(key);
 
-            Preconditions.checkArgument(memoryKey != null, "No registry entry found for key " + key);
+            Preconditions.checkArgument(memoryKey != null, "No MemoryKey registry entry found for key %s.", key);
 
             return memoryKey;
         }
@@ -428,7 +428,7 @@ public interface Registry<T extends Keyed> extends Iterable<T> {
         public T getOrThrow(@NotNull NamespacedKey key) {
             T object = get(key);
 
-            Preconditions.checkArgument(object != null, "No registry entry found for key " + key);
+            Preconditions.checkArgument(object != null, "No %s registry entry found for key %s.", type, key);
 
             return object;
         }
diff --git a/src/test/java/org/bukkit/support/TestServer.java b/src/test/java/org/bukkit/support/TestServer.java
index 4040ecb2..c67e0784 100644
--- a/src/test/java/org/bukkit/support/TestServer.java
+++ b/src/test/java/org/bukkit/support/TestServer.java
@@ -63,7 +63,7 @@ public final class TestServer {
             public Keyed getOrThrow(@NotNull NamespacedKey key) {
                 Keyed keyed = get(key);
 
-                Preconditions.checkArgument(keyed != null, "No registry entry found for key " + key);
+                Preconditions.checkArgument(keyed != null, "No %s registry entry found for key %s.", aClass, key);
 
                 return keyed;
             }