Add tests for #16713 and #16698

Signed-off-by: William Desportes <williamdes@wdes.fr>
phpmyadmin · Mar 5, 2021 · d08ddac · d08ddac
1 parent 8da837a
commit d08ddac
Showing 1 changed file with 34 additions and 0 deletions.
diff --git a/test/classes/UtilTest.php b/test/classes/UtilTest.php
@@ -2600,4 +2600,38 @@ public function testCurrentUserHasNotUserPrivilegeAndNotDbPrivilegeAndNotTablePr
         $this->assertFalse(Util::currentUserHasPrivilege('EVENT', 'my_data_base', 'my_data_table'));
         $GLOBALS['dbi'] = $oldDbi;
     }
+
+    public function testGetScriptNameForOption(): void
+    {
+        $this->assertEquals(
+            './',
+            Util::getScriptNameForOption('db_structures.php', 'database'), // Notice the typo on db_structure.php
+            'Fallback to a relative path, impossible to build a valid route link'
+        );
+        $this->assertEquals(
+            'index.php?route=/database/structure&amp;lang=en',
+            Util::getScriptNameForOption('db_structure.php', 'database'),
+            'Support the legacy value for database'
+        );
+        $this->assertEquals(
+            'index.php?route=/table/structure&amp;lang=en',
+            Util::getScriptNameForOption('tbl_structure.php', 'table'),
+            'Support the legacy value for table'
+        );
+        $this->assertEquals(
+            'index.php?route=/server/status&amp;lang=en',
+            Util::getScriptNameForOption('server_status.php', 'server'),
+            'Support the legacy value for server'
+        );
+        $this->assertEquals(
+            'index.php?route=/database/structure&amp;lang=en',
+            Util::getScriptNameForOption('structure', 'database'),
+            'The new value to go to database structure'
+        );
+        $this->assertEquals(
+            './',
+            Util::getScriptNameForOption('structure', 'databasesss'), // Notice the typo on databases
+            'Fallback to a relative path, impossible to build a valid route link'
+        );
+    }
 }