Merge pull request #29 from worksome/feature/upper-snake-case-enum-cases

fix: resolve correct name conversion for enum cases

composer.json

@@ -12,7 +12,8 @@
     "symfony/config": "^6.0",
     "symplify/autowire-array-parameter": "^11.0",
     "symplify/package-builder": "^11.0",
-    "thecodingmachine/safe": "^2.2"
+    "thecodingmachine/safe": "^2.2",
+    "jawira/case-converter": "^3.5"
   },
   "require-dev": {
     "pestphp/pest": "^1.21",

src/Fixes/UpperSnakeCaseNameFixer.php

@@ -5,7 +5,7 @@
 namespace Worksome\Graphlint\Fixes;
 
 use GraphQL\Language\AST\NameNode;
-use Illuminate\Support\Str;
+use Jawira\CaseConverter\Convert;
 use Worksome\Graphlint\ProblemDescriptor;
 use Worksome\Graphlint\Utils\NodeNameResolver;
 
@@ -30,12 +30,6 @@ public function fix(ProblemDescriptor $problemDescriptor): void
             return;
         }
 
-        $upperCase = Str::of($name)
-            ->replace('_', ' ')
-            ->title()
-            ->snake()
-            ->upper();
-
-        $node->value = $upperCase->__toString();
+        $node->value = (new Convert($name))->toMacro();
     }
 }

src/Inspections/UpperSnakeCaseEnumCaseDefinitionInspection.php

@@ -5,7 +5,7 @@
 namespace Worksome\Graphlint\Inspections;
 
 use GraphQL\Language\AST\EnumValueDefinitionNode;
-use Illuminate\Support\Str;
+use Jawira\CaseConverter\Convert;
 use Worksome\Graphlint\Fixes\UpperSnakeCaseNameFixer;
 use Worksome\Graphlint\InspectionDescription;
 use Worksome\Graphlint\ProblemsHolder;
@@ -35,7 +35,7 @@ public function visitEnumValueDefinition(
             return;
         }
 
-        $upperCase = Str::of($name)->replace('_', ' ')->title()->snake()->upper()->__toString();
+        $upperCase = (new Convert($name))->toMacro();
 
         if ($name === $upperCase) {
             return;

test-resources/Inspections/UpperSnakeCaseEnumCaseDefinitionInspectionTest/camel_case_multi_word_enum_case.graphql.inc

@@ -0,0 +1,7 @@
+enum Status {
+  inProgress
+}
+-----
+enum Status {
+  IN_PROGRESS
+}

test-resources/Inspections/UpperSnakeCaseEnumCaseDefinitionInspectionTest/pascal_case_multi_word_enum_case.graphql.inc

@@ -0,0 +1,7 @@
+enum Status {
+  InProgress
+}
+-----
+enum Status {
+  IN_PROGRESS
+}