From dc6fc9241abc11bea904ebab396ec4bfb034da1a Mon Sep 17 00:00:00 2001
From: Paul Mehrer <p.mehrer@metaways.de>
Date: Wed, 26 Feb 2025 15:17:34 +0100
Subject: [PATCH] tweak(tb) fix tests re property feature switch

---
 tests/tine20/Tinebase/Frontend/JsonTest.php   | 28 +++++--------------
 .../Tinebase/ModelConfigurationTest.php       | 16 -----------
 2 files changed, 7 insertions(+), 37 deletions(-)

diff --git a/tests/tine20/Tinebase/Frontend/JsonTest.php b/tests/tine20/Tinebase/Frontend/JsonTest.php
index 134c67a5297..19637f28f6a 100644
--- a/tests/tine20/Tinebase/Frontend/JsonTest.php
+++ b/tests/tine20/Tinebase/Frontend/JsonTest.php
@@ -696,28 +696,14 @@ public function testGetAllRegistryData()
         $symbols = Zend_Locale::getTranslationList('symbols', $locale);
         self::assertEquals($symbols['decimal'], $registryData['Tinebase']['decimalSeparator']);
 
-        if (Sales_Config::getInstance()->featureEnabled(Sales_Config::FEATURE_INVOICES_MODULE)) {
-            $configuredSalesModels = array_keys($registryData['Sales']['models']);
-            self::assertTrue(in_array('Invoice', $configuredSalesModels), 'Invoices is missing from configured models: '
-                . print_r($configuredSalesModels, true));
-            $copyOmitFields = array(
-                'billed_in',
-                'invoice_id',
-                'status',
-                'cleared_at',
-                'relations',
-            );
-        } else {
-            $copyOmitFields = array(
-                'billed_in',
-                'status',
-                'cleared_at',
-                'relations',
-            );
-        }
-
         self::assertTrue(isset($registryData['Timetracker']['models']['Timeaccount']['copyOmitFields']), 'Timeaccount copyOmitFields empty/missing');
-        self::assertEquals($copyOmitFields, $registryData['Timetracker']['models']['Timeaccount']['copyOmitFields']);
+        self::assertEquals([
+            'billed_in',
+            'invoice_id',
+            'status',
+            'cleared_at',
+            'relations',
+        ], $registryData['Timetracker']['models']['Timeaccount']['copyOmitFields']);
         self::assertTrue(is_array(($registryData['Timetracker']['relatableModels'][0])), 'relatableModels needs to be an numbered array');
 
         $this->_assertImportExportDefinitions($registryData);
diff --git a/tests/tine20/Tinebase/ModelConfigurationTest.php b/tests/tine20/Tinebase/ModelConfigurationTest.php
index 94455c03bcd..ff0db6c915f 100644
--- a/tests/tine20/Tinebase/ModelConfigurationTest.php
+++ b/tests/tine20/Tinebase/ModelConfigurationTest.php
@@ -37,22 +37,6 @@ public function testModelCreationTraditional()
         $this->assertNull($cObj);
     }
 
-    /**
-     * tests if the modelconfiguration is created for foreign record keys that are disabled by a feature switch
-     */
-    public function testModelConfigWithDisabledForeignRecords()
-    {
-        // TODO disable feature first
-        if (Sales_Config::getInstance()->featureEnabled(Sales_Config::FEATURE_INVOICES_MODULE)) {
-            $this->markTestSkipped('only testable when disabled');
-        }
-
-        $timesheet = new Timetracker_Model_Timesheet(array(), true);
-        $mcFields = $timesheet->getConfiguration()->getFields();
-        $this->assertEquals('string', $mcFields['invoice_id']['type']);
-        $this->assertEquals(null, $mcFields['invoice_id']['label']);
-    }
-
     /**
      * testModelConfigWithDisabledRelationApp
      */