feat(php-laravel): add tests for issue 21334 with API and model definitions

Author: Mostafa Aghajani

bin/configs/php-laravel-issue-21334.yaml

@@ -0,0 +1,8 @@
+generatorName: php-laravel
+outputDir: samples/server/petstore/php-laravel-issue-21334
+inputSpec: modules/openapi-generator/src/test/resources/3_0/issue21334.yaml
+templateDir: modules/openapi-generator/src/main/resources/php-laravel
+gitUserId: openapitools
+gitRepoId: petstore
+additionalProperties:
+  variableNamingConvention: "off"

modules/openapi-generator/src/test/resources/3_0/issue21334.yaml

@@ -0,0 +1,43 @@
+openapi: 3.0.3
+info:
+  title: Issue 21334 - Abstract PHP Variable Naming Convention
+  description: ''
+  version: 1.0.0
+paths:
+  /dummy:
+    get:
+      description: ''
+      operationId: getDummy
+      parameters:
+        - name: paramCamelCase
+          in: query
+          required: false
+          schema:
+            type: string
+        - name: ParamPascalCase
+          in: query
+          required: false
+          schema:
+            type: string
+        - name: param_withRandomCase
+          in: query
+          required: false
+          schema:
+            type: string
+      responses:
+        '200':
+          description: successful operation
+components:
+  schemas:
+    SampleModelToTestNamingConvention:
+      type: object
+      properties:
+        propCamelCase:
+          type: string
+          default: 'defaultName'
+        PropPascalCase:
+          type: string
+          default: 'DefaultName'
+        Prop_withRandomCase:
+          type: string
+          default: 'defaultName'

samples/server/petstore/php-laravel-issue-21334/.gitignore

@@ -0,0 +1 @@
+/vendor/

samples/server/petstore/php-laravel-issue-21334/.openapi-generator-ignore

@@ -0,0 +1,23 @@
+# OpenAPI Generator Ignore
+# Generated by openapi-generator https://github.com/openapitools/openapi-generator
+
+# Use this file to prevent files from being overwritten by the generator.
+# The patterns follow closely to .gitignore or .dockerignore.
+
+# As an example, the C# client generator defines ApiClient.cs.
+# You can make changes and tell OpenAPI Generator to ignore just this file by uncommenting the following line:
+#ApiClient.cs
+
+# You can match any string of characters against a directory, file or extension with a single asterisk (*):
+#foo/*/qux
+# The above matches foo/bar/qux and foo/baz/qux, but not foo/bar/baz/qux
+
+# You can recursively match patterns against a directory, file or extension with a double asterisk (**):
+#foo/**/qux
+# This matches foo/bar/qux, foo/baz/qux, and foo/bar/baz/qux
+
+# You can also negate patterns with an exclamation (!).
+# For example, you can ignore all files in a docs folder with the file extension .md:
+#docs/*.md
+# Then explicitly reverse the ignore rule for a single file:
+#!docs/README.md

samples/server/petstore/php-laravel-issue-21334/.openapi-generator/FILES

@@ -0,0 +1,9 @@
+.gitignore
+Api/DefaultApiInterface.php
+Http/Controllers/DefaultController.php
+Model/NoContent200.php
+Model/SampleModelToTestNamingConvention.php
+README.md
+composer.json
+phpunit.xml.dist
+routes.php

samples/server/petstore/php-laravel-issue-21334/.openapi-generator/VERSION

@@ -0,0 +1 @@
+7.14.0-SNAPSHOT

samples/server/petstore/php-laravel-issue-21334/Api/DefaultApiInterface.php

@@ -0,0 +1,42 @@
+<?php declare(strict_types=1);
+
+/**
+ * Issue 21334 - Abstract PHP Variable Naming Convention
+ * No description provided (generated by Openapi Generator https://github.com/openapitools/openapi-generator)
+ * PHP version 8.1
+ *
+ * The version of the OpenAPI document: 1.0.0
+ * 
+ *
+ * NOTE: This class is auto generated by OpenAPI-Generator
+ * https://openapi-generator.tech
+ * Do not edit the class manually.
+ *
+ * Source files are located at:
+ *
+ * > https://github.com/OpenAPITools/openapi-generator/blob/master/modules/openapi-generator/src/main/resources/php-laravel/
+ */
+
+
+namespace OpenAPI\Server\Api;
+
+
+interface DefaultApiInterface {
+
+
+    /**
+     * Operation getDummy
+     * @param null | string $paramCamelCase
+     * @param null | string $ParamPascalCase
+     * @param null | string $param_withRandomCase
+     * @return \OpenAPI\Server\Model\NoContent200
+     */
+    public function getDummy(
+            ?string $paramCamelCase,
+            ?string $ParamPascalCase,
+            ?string $param_withRandomCase,
+    ):
+        \OpenAPI\Server\Model\NoContent200
+    ;
+
+}

samples/server/petstore/php-laravel-issue-21334/Http/Controllers/DefaultController.php

@@ -0,0 +1,98 @@
+<?php declare(strict_types=1);
+
+/**
+ * Issue 21334 - Abstract PHP Variable Naming Convention
+ * No description provided (generated by Openapi Generator https://github.com/openapitools/openapi-generator)
+ * PHP version 8.1
+ *
+ * The version of the OpenAPI document: 1.0.0
+ * 
+ *
+ * NOTE: This class is auto generated by OpenAPI-Generator
+ * https://openapi-generator.tech
+ * Do not edit the class manually.
+ *
+ * Source files are located at:
+ *
+ * > https://github.com/OpenAPITools/openapi-generator/blob/master/modules/openapi-generator/src/main/resources/php-laravel/
+ */
+
+
+namespace OpenAPI\Server\Http\Controllers;
+
+use Crell\Serde\SerdeCommon;
+use Illuminate\Routing\Controller;
+use Illuminate\Http\JsonResponse;
+use Illuminate\Http\Request;
+use Illuminate\Support\Facades\Validator;
+
+
+use OpenAPI\Server\Api\DefaultApiInterface;
+
+class DefaultController extends Controller
+{
+    /**
+     * Constructor
+     */
+    public function __construct(
+        private readonly DefaultApiInterface $api,
+        private readonly SerdeCommon $serde = new SerdeCommon(),
+    )
+    {
+    }
+
+    /**
+     * Operation getDummy
+     *
+     * .
+     *
+     */
+    public function getDummy(Request $request): JsonResponse
+    {
+        $validator = Validator::make(
+            array_merge(
+                [
+                    
+                ],
+                $request->all(),
+            ),
+            [
+                'paramCamelCase' => [
+                    'string',
+                ],
+                'ParamPascalCase' => [
+                    'string',
+                ],
+                'param_withRandomCase' => [
+                    'string',
+                ],
+            ],
+        );
+
+        if ($validator->fails()) {
+            return response()->json(['error' => 'Invalid input'], 400);
+        }
+
+        $paramCamelCase = $request->string('paramCamelCase')->value();
+
+        $ParamPascalCase = $request->string('ParamPascalCase')->value();
+
+        $param_withRandomCase = $request->string('param_withRandomCase')->value();
+
+        try {
+            $apiResult = $this->api->getDummy($paramCamelCase, $ParamPascalCase, $param_withRandomCase);
+        } catch (\Exception $exception) {
+            // This shouldn't happen
+            report($exception);
+            return response()->json(['error' => $exception->getMessage()], 500);
+        }
+
+        if ($apiResult instanceof \OpenAPI\Server\Model\NoContent200) {
+            return response()->json($this->serde->serialize($apiResult, format: 'array'), 200);
+        }
+
+
+        // This shouldn't happen
+        return response()->abort(500);
+    }
+}

samples/server/petstore/php-laravel-issue-21334/Model/NoContent200.php

@@ -0,0 +1,46 @@
+<?php declare(strict_types=1);
+
+/**
+ * Issue 21334 - Abstract PHP Variable Naming Convention
+ * No description provided (generated by Openapi Generator https://github.com/openapitools/openapi-generator)
+ * PHP version 8.1
+ *
+ * The version of the OpenAPI document: 1.0.0
+ * 
+ *
+ * NOTE: This class is auto generated by OpenAPI-Generator
+ * https://openapi-generator.tech
+ * Do not edit the class manually.
+ *
+ * Source files are located at:
+ *
+ * > https://github.com/OpenAPITools/openapi-generator/blob/master/modules/openapi-generator/src/main/resources/php-laravel/
+ */
+
+
+/**
+ * NoContent200
+ */
+namespace OpenAPI\Server\Model;
+
+/**
+ * NoContent200
+ * @description No content for 200
+ */
+use Crell\Serde\Renaming\Cases;
+use Crell\Serde\Attributes as Serde;
+
+#[Serde\ClassSettings(renameWith: Cases::snake_case)]
+class NoContent200
+{
+    /**
+    *
+    * dummy property for no-content responses
+    * @param null | string $dummy
+    */
+
+    public function __construct(
+        public ?string $dummy = null,
+    ) {}
+}
+

samples/server/petstore/php-laravel-issue-21334/Model/SampleModelToTestNamingConvention.php

@@ -0,0 +1,53 @@
+<?php declare(strict_types=1);
+
+/**
+ * Issue 21334 - Abstract PHP Variable Naming Convention
+ * No description provided (generated by Openapi Generator https://github.com/openapitools/openapi-generator)
+ * PHP version 8.1
+ *
+ * The version of the OpenAPI document: 1.0.0
+ * 
+ *
+ * NOTE: This class is auto generated by OpenAPI-Generator
+ * https://openapi-generator.tech
+ * Do not edit the class manually.
+ *
+ * Source files are located at:
+ *
+ * > https://github.com/OpenAPITools/openapi-generator/blob/master/modules/openapi-generator/src/main/resources/php-laravel/
+ */
+
+
+/**
+ * SampleModelToTestNamingConvention
+ */
+namespace OpenAPI\Server\Model;
+
+/**
+ * SampleModelToTestNamingConvention
+ */
+use Crell\Serde\Renaming\Cases;
+use Crell\Serde\Attributes as Serde;
+
+#[Serde\ClassSettings(renameWith: Cases::snake_case)]
+class SampleModelToTestNamingConvention
+{
+    /**
+    *
+    * 
+    * @param string $propCamelCase
+    *
+    * 
+    * @param string $PropPascalCase
+    *
+    * 
+    * @param string $Prop_withRandomCase
+    */
+
+    public function __construct(
+        public string $propCamelCase = 'defaultName',
+        public string $PropPascalCase = 'DefaultName',
+        public string $Prop_withRandomCase = 'defaultName',
+    ) {}
+}
+