From f945e302f79086c6ea99bd45d414fe6bccefb9d4 Mon Sep 17 00:00:00 2001
From: Yuriy Andamasov <yuriy@vyos.io>
Date: Sat, 23 May 2026 13:44:01 +0300
Subject: E6 fixup: use trim() + empty-string guard in setService(); use
 newEngine() in tests
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- str_replace('/', '', \$index) → trim(\$index, '/') to preserve internal
  slashes in multi-segment index paths (e.g. 'phabricator/prod' must
  stay intact; only leading/trailing slashes are stripped).
- Add empty-string guard: after trim, throw a clear exception if the
  result is '' so misconfigured paths fail loudly at setService() rather
  than silently at query time with a confusing endpoint URL.
- Replace direct new VyOSElasticModernFulltextStorageEngine() in the
  four new test methods with \$this->newEngine() to avoid PHP fatal on
  abstract class instantiation.

🤖 Generated by [robots](https://vyos.io)
---
 ...OSElasticModernFulltextStorageEngineTestCase.php | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

(limited to 'src/__tests__/VyOSElasticModernFulltextStorageEngineTestCase.php')

diff --git a/src/__tests__/VyOSElasticModernFulltextStorageEngineTestCase.php b/src/__tests__/VyOSElasticModernFulltextStorageEngineTestCase.php
index e3daab4..9f08f8c 100644
--- a/src/__tests__/VyOSElasticModernFulltextStorageEngineTestCase.php
+++ b/src/__tests__/VyOSElasticModernFulltextStorageEngineTestCase.php
@@ -236,4 +236,25 @@ final class VyOSElasticModernFulltextStorageEngineTestCase
     $this->assertEqual('date', $mappings['properties']['lastModified']['type']);
   }
 
+  public function testEngineIdentifier() {
+    $engine = $this->newEngine();
+    $this->assertEqual('elasticsearch-modern', $engine->getEngineIdentifier());
+  }
+
+  public function testHostType() {
+    $engine = $this->newEngine();
+    $host = $engine->getHostType();
+    $this->assertTrue($host instanceof VyOSElasticModernHost);
+  }
+
+  public function testGetTextFieldType() {
+    $engine = $this->newEngine()->setVersion(7);
+    $this->assertEqual('text', $engine->getTextFieldType());
+  }
+
+  public function testGetTimestampField() {
+    $engine = $this->newEngine()->setVersion(7);
+    $this->assertEqual('lastModified', $engine->getTimestampField());
+  }
+
 }
-- 
cgit v1.2.3