Fix RecipeValidationTest

rob-valor · Aug 15, 2024 · 26f62e2 · 26f62e2
1 parent c4baa1c
commit 26f62e2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/rewrite-core/src/main/java/org/openrewrite/config/CategoryTree.java b/rewrite-core/src/main/java/org/openrewrite/config/CategoryTree.java
@@ -327,6 +327,9 @@ CategoryTree<G> findOrAddCategory(G group, CategoryDescriptor category) {
     }
 
     void addRecipe(G group, RecipeDescriptor recipe) {
+        if (recipe.getName() == null) {
+            throw new IllegalArgumentException("Expected recipe to have a name " + recipe.getSource());
+        }
         if (!recipe.getName().contains(".")) {
             throw new IllegalArgumentException("Expected recipe with name '" + recipe.getName() + "' to have " +
                                                "a package, but it did not.");

diff --git a/rewrite-core/src/test/java/org/openrewrite/RecipeValidationTest.java b/rewrite-core/src/test/java/org/openrewrite/RecipeValidationTest.java
@@ -36,7 +36,7 @@ public static class JSpecifyAnnotatedRecipeOptions extends Recipe {
         @Option(displayName = "An optional field",
           description = "Something that can be null.")
         @Nullable
-        String name;
+        String optionalField;
 
         @Override
         public String getDisplayName() {