Failing test

Author: SOHELAHMED7

tests/specs/issue_fix/88_in_case_of_updating_a_model_generator_creates_redundant_inverse_relations/index.php

@@ -0,0 +1,13 @@
+<?php
+
+return [
+    'openApiPath' => '@specs/issue_fix/88_in_case_of_updating_a_model_generator_creates_redundant_inverse_relations/index.yaml',
+    'generateUrls' => false,
+    'generateModels' => true,
+    'excludeModels' => [
+        'Error',
+    ],
+    'generateControllers' => false,
+    'generateMigrations' => false,
+    'generateModelFaker' => false, // `generateModels` must be `true` in order to use `generateModelFaker` as `true`
+];

tests/specs/issue_fix/88_in_case_of_updating_a_model_generator_creates_redundant_inverse_relations/index.yaml

@@ -0,0 +1,33 @@
+openapi: "3.0.0"
+
+info:
+  version: 1.0.0
+  title: '#88'
+
+paths:
+  /:
+    get:
+      responses:
+        '200':
+          description: The response
+
+components:
+  schemas:
+    Address:
+      type: object
+      properties:
+        id:
+          type: integer
+        name:
+          type: string
+    Human:
+      type: object
+      properties:
+        id:
+          type: integer
+        name:
+          type: string
+        address:
+          $ref: '#/components/schemas/Address'
+
+

tests/specs/issue_fix/88_in_case_of_updating_a_model_generator_creates_redundant_inverse_relations/mysql/models/Address.php

@@ -0,0 +1,10 @@
+<?php
+
+namespace app\models;
+
+class Address extends \app\models\base\Address
+{
+
+
+}
+

tests/specs/issue_fix/88_in_case_of_updating_a_model_generator_creates_redundant_inverse_relations/mysql/models/Human.php

@@ -0,0 +1,10 @@
+<?php
+
+namespace app\models;
+
+class Human extends \app\models\base\Human
+{
+
+
+}
+

tests/specs/issue_fix/88_in_case_of_updating_a_model_generator_creates_redundant_inverse_relations/mysql/models/base/Address.php

@@ -0,0 +1,30 @@
+<?php
+
+/**
+ * This file is generated by Gii, do not change manually!
+ */
+
+namespace app\models\base;
+
+/**
+ * This is the model class for table "addresses".
+ *
+ * @property int $id
+ * @property string $name
+ *
+ */
+abstract class Address extends \yii\db\ActiveRecord
+{
+    public static function tableName()
+    {
+        return '{{%addresses}}';
+    }
+
+    public function rules()
+    {
+        return [
+            'trim' => [['name'], 'trim'],
+            'name_string' => [['name'], 'string'],
+        ];
+    }
+}

tests/specs/issue_fix/88_in_case_of_updating_a_model_generator_creates_redundant_inverse_relations/mysql/models/base/Human.php

@@ -0,0 +1,39 @@
+<?php
+
+/**
+ * This file is generated by Gii, do not change manually!
+ */
+
+namespace app\models\base;
+
+/**
+ * This is the model class for table "humans".
+ *
+ * @property int $id
+ * @property string $name
+ * @property int $address_id
+ *
+ * @property \app\models\Address $address
+ */
+abstract class Human extends \yii\db\ActiveRecord
+{
+    public static function tableName()
+    {
+        return '{{%humans}}';
+    }
+
+    public function rules()
+    {
+        return [
+            'trim' => [['name'], 'trim'],
+            'name_string' => [['name'], 'string'],
+            'address_id_integer' => [['address_id'], 'integer'],
+            'address_id_exist' => [['address_id'], 'exist', 'targetRelation' => 'address'],
+        ];
+    }
+
+    public function getAddress()
+    {
+        return $this->hasOne(\app\models\Address::class, ['id' => 'address_id']);
+    }
+}

tests/unit/IssueFixTest.php

@@ -1014,4 +1014,18 @@ public function test22BugRulesRequiredIsGeneratedBeforeDefault()
         ]);
         $this->checkFiles($actualFiles, $expectedFiles);
     }
+
+    // https://github.com/php-openapi/yii2-openapi/issues/88
+    public function test88InCaseOfUpdatingAModelGeneratorCreatesRedundantInverseRelations()
+    {
+        $testFile = Yii::getAlias("@specs/issue_fix/88_in_case_of_updating_a_model_generator_creates_redundant_inverse_relations/index.php");
+        $this->runGenerator($testFile);
+        $actualFiles = FileHelper::findFiles(Yii::getAlias('@app'), [
+            'recursive' => true,
+        ]);
+        $expectedFiles = FileHelper::findFiles(Yii::getAlias("@specs/issue_fix/88_in_case_of_updating_a_model_generator_creates_redundant_inverse_relations/mysql"), [
+            'recursive' => true,
+        ]);
+        $this->checkFiles($actualFiles, $expectedFiles);
+    }
 }