add and use RTTestCase::myStringStatic()
authorDenis Ovsienko <denis@ovsienko.info>
Mon, 12 Nov 2018 10:47:13 +0000 (10:47 +0000)
committerDenis Ovsienko <denis@ovsienko.info>
Mon, 12 Nov 2018 10:47:13 +0000 (10:47 +0000)
Even after the introduction of RTTestCase::myString() some tests
continued to use fixed strings for the database props naming in their
static setup methods. When such a test failed in a way that prevented the
teardown method to run, that would fail subsequent attempts to run the
setup method until the props were manually removed or the unit test
database contents was reset.

Introduce a static counterpart of the existing solution and update the
following classes to use it:

* EntityLinkTriggerTest
* GetChildrenListTest
* ObjectCircularReferenceTest
* ObjectLogTest

tests/EntityLinkTriggerTest.php
tests/GetChildrenListTest.php
tests/ObjectCircularReferenceTest.php
tests/ObjectLogTest.php
tests/bootstrap_common.php

index 458b9c7..2ea99eb 100644 (file)
@@ -15,13 +15,13 @@ class EntityLinkTriggerTest extends RTTestCase
        public static function setUpBeforeClass ()
        {
                // add sample data
-               usePreparedInsertBlade ('Dictionary', array ('chapter_id' => 1, 'dict_value' => 'unit test object type a'));
+               usePreparedInsertBlade ('Dictionary', array ('chapter_id' => 1, 'dict_value' => self::myStringStatic ('type a', __CLASS__)));
                self::$objtypea_id = lastInsertID ();
-               usePreparedInsertBlade ('Dictionary', array ('chapter_id' => 1, 'dict_value' => 'unit test object type b'));
+               usePreparedInsertBlade ('Dictionary', array ('chapter_id' => 1, 'dict_value' => self::myStringStatic ('type b', __CLASS__)));
                self::$objtypeb_id = lastInsertID ();
-               usePreparedInsertBlade ('Dictionary', array ('chapter_id' => 1, 'dict_value' => 'unit test object type c'));
+               usePreparedInsertBlade ('Dictionary', array ('chapter_id' => 1, 'dict_value' => self::myStringStatic ('type c', __CLASS__)));
                self::$objtypec_id = lastInsertID ();
-               usePreparedInsertBlade ('Dictionary', array ('chapter_id' => 1, 'dict_value' => 'unit test object type d'));
+               usePreparedInsertBlade ('Dictionary', array ('chapter_id' => 1, 'dict_value' => self::myStringStatic ('type d', __CLASS__)));
                self::$objtyped_id = lastInsertID ();
                commitSupplementOPC (self::$objtypea_id, self::$objtypeb_id);
                commitSupplementOPC (self::$objtypea_id, self::$objtypec_id);
@@ -29,17 +29,17 @@ class EntityLinkTriggerTest extends RTTestCase
                commitSupplementOPC (self::$objtypeb_id, self::$objtypec_id);
                commitSupplementOPC (self::$objtypec_id, self::$objtypea_id);
                commitSupplementOPC (self::$objtypec_id, self::$objtypeb_id);
-               self::$objecta_id = commitAddObject ('unit test object a', NULL, self::$objtypea_id, NULL);
-               self::$objectb_id = commitAddObject ('unit test object b', NULL, self::$objtypeb_id, NULL);
-               self::$objectc_id = commitAddObject ('unit test object c', NULL, self::$objtypec_id, NULL);
-               self::$objectd_id = commitAddObject ('unit test object d', NULL, self::$objtyped_id, NULL);
-               self::$locationa_id = commitAddObject ('unit test location a', NULL, 1562, NULL);
-               self::$locationb_id = commitAddObject ('unit test location b', NULL, 1562, NULL);
-               self::$locationc_id = commitAddObject ('unit test location c', NULL, 1562, NULL);
-               self::$rowa_id = commitAddObject ('unit test row a', NULL, 1561, NULL);
-               self::$rowb_id = commitAddObject ('unit test row b', NULL, 1561, NULL);
-               self::$racka_id = commitAddObject ('unit test rack a', NULL, 1560, NULL);
-               self::$rackb_id = commitAddObject ('unit test rack b', NULL, 1560, NULL);
+               self::$objecta_id = commitAddObject (self::myStringStatic ('object a', __CLASS__), NULL, self::$objtypea_id, NULL);
+               self::$objectb_id = commitAddObject (self::myStringStatic ('object b', __CLASS__), NULL, self::$objtypeb_id, NULL);
+               self::$objectc_id = commitAddObject (self::myStringStatic ('object c', __CLASS__), NULL, self::$objtypec_id, NULL);
+               self::$objectd_id = commitAddObject (self::myStringStatic ('object d', __CLASS__), NULL, self::$objtyped_id, NULL);
+               self::$locationa_id = commitAddObject (self::myStringStatic ('location a', __CLASS__), NULL, 1562, NULL);
+               self::$locationb_id = commitAddObject (self::myStringStatic ('location b', __CLASS__), NULL, 1562, NULL);
+               self::$locationc_id = commitAddObject (self::myStringStatic ('location c', __CLASS__), NULL, 1562, NULL);
+               self::$rowa_id = commitAddObject (self::myStringStatic ('row a', __CLASS__), NULL, 1561, NULL);
+               self::$rowb_id = commitAddObject (self::myStringStatic ('row b', __CLASS__), NULL, 1561, NULL);
+               self::$racka_id = commitAddObject (self::myStringStatic ('rack a', __CLASS__), NULL, 1560, NULL);
+               self::$rackb_id = commitAddObject (self::myStringStatic ('rack b', __CLASS__), NULL, 1560, NULL);
        }
 
        public static function tearDownAfterClass ()
index 8ec0f4e..96f59f7 100644 (file)
@@ -13,10 +13,10 @@ class GetChildrenListTest extends RTTestCase
        public static function setUpBeforeClass ()
        {
                // add sample locations
-               self::$locations[] = $parent_location_id = commitAddObject ('unit test location 0', NULL, 1562, NULL);
+               self::$locations[] = $parent_location_id = commitAddObject (self::myStringStatic ('location 0', __CLASS__), NULL, 1562, NULL);
                for ($i=1; $i<=self::$num_children; $i++)
                {
-                       $child_location_id = commitAddObject ("unit test location ${i}", NULL, 1562, NULL);
+                       $child_location_id = commitAddObject (self::myStringStatic ("location ${i}", __CLASS__), NULL, 1562, NULL);
                        self::$locations[] = $child_location_id;
                        commitLinkEntities ('location', $parent_location_id, 'location', $child_location_id);
                        self::$links[] = lastInsertID();
@@ -24,13 +24,13 @@ class GetChildrenListTest extends RTTestCase
                }
 
                // add sample objects
-               usePreparedInsertBlade ('Dictionary', array ('chapter_id' => 1, 'dict_value' => 'unit test object type'));
+               usePreparedInsertBlade ('Dictionary', array ('chapter_id' => 1, 'dict_value' => self::myStringStatic ('type', __CLASS__)));
                self::$objtype_id = lastInsertID ();
                commitSupplementOPC (self::$objtype_id, self::$objtype_id);
-               self::$objects[] = $parent_object_id = commitAddObject ('unit test object 0', NULL, self::$objtype_id, NULL);
+               self::$objects[] = $parent_object_id = commitAddObject (self::myStringStatic ('object 0', __CLASS__), NULL, self::$objtype_id, NULL);
                for ($i=1; $i<=self::$num_children; $i++)
                {
-                       $child_object_id = commitAddObject ("unit test object ${i}", NULL, self::$objtype_id, NULL);
+                       $child_object_id = commitAddObject (self::myStringStatic ("object ${i}", __CLASS__), NULL, self::$objtype_id, NULL);
                        self::$objects[] = $child_object_id;
                        commitLinkEntities ('object', $parent_object_id, 'object', $child_object_id);
                        self::$links[] = lastInsertID();
@@ -38,11 +38,11 @@ class GetChildrenListTest extends RTTestCase
                }
 
                // add sample tags
-               usePreparedInsertBlade ('TagTree', array ('tag' => 'unit test tag 0'));
+               usePreparedInsertBlade ('TagTree', array ('tag' => self::myStringStatic ('tag 0', __CLASS__)));
                self::$tags[] = $parent_tag_id = lastInsertID ();
                for ($i=1; $i<=self::$num_children; $i++)
                {
-                       usePreparedInsertBlade ('TagTree', array ('parent_id' => $parent_tag_id, 'tag' => "unit test tag ${i}"));
+                       usePreparedInsertBlade ('TagTree', array ('parent_id' => $parent_tag_id, 'tag' => self::myStringStatic ("tag ${i}", __CLASS__)));
                        $parent_tag_id = lastInsertID ();
                        self::$tags[] = $parent_tag_id;
                }
index 90e8777..59a4c29 100644 (file)
@@ -12,15 +12,15 @@ class ObjectCircularReferenceTest extends RTTestCase
        public static function setUpBeforeClass ()
        {
                // add sample data
-               usePreparedInsertBlade ('Dictionary', array ('chapter_id' => 1, 'dict_value' => 'unit test object type'));
+               usePreparedInsertBlade ('Dictionary', array ('chapter_id' => 1, 'dict_value' => self::myStringStatic ('type', __CLASS__)));
                self::$objtype_id = lastInsertID ();
                commitSupplementOPC (self::$objtype_id, self::$objtype_id);
-               self::$objecta_id = commitAddObject ('unit test object a', NULL, self::$objtype_id, NULL);
-               self::$objectb_id = commitAddObject ('unit test object b', NULL, self::$objtype_id, NULL);
-               self::$objectc_id = commitAddObject ('unit test object c', NULL, self::$objtype_id, NULL);
-               self::$locationa_id = commitAddObject ('unit test location a', NULL, 1562, NULL);
-               self::$locationb_id = commitAddObject ('unit test location b', NULL, 1562, NULL);
-               self::$locationc_id = commitAddObject ('unit test location c', NULL, 1562, NULL);
+               self::$objecta_id = commitAddObject (self::myStringStatic ('object a', __CLASS__), NULL, self::$objtype_id, NULL);
+               self::$objectb_id = commitAddObject (self::myStringStatic ('object b', __CLASS__), NULL, self::$objtype_id, NULL);
+               self::$objectc_id = commitAddObject (self::myStringStatic ('object c', __CLASS__), NULL, self::$objtype_id, NULL);
+               self::$locationa_id = commitAddObject (self::myStringStatic ('location a', __CLASS__), NULL, 1562, NULL);
+               self::$locationb_id = commitAddObject (self::myStringStatic ('location b', __CLASS__), NULL, 1562, NULL);
+               self::$locationc_id = commitAddObject (self::myStringStatic ('location c', __CLASS__), NULL, 1562, NULL);
        }
 
        public static function tearDownAfterClass ()
index 7c76ba6..e72ed1d 100644 (file)
@@ -16,10 +16,10 @@ class ObjectLogTest extends RTTestCase
                $sic['logentry'] = $_REQUEST['logentry'] = 'unit test log entry';
 
                // add sample data
-               self::$rack_id = commitAddObject ('unit test rack', NULL, 1560, NULL);
-               self::$row_id = commitAddObject ('unit test row', NULL, 1561, NULL);
-               self::$location_id = commitAddObject ('unit test location', NULL, 1562, NULL);
-               self::$object_id = commitAddObject ('unit test object', NULL, 1, NULL);
+               self::$rack_id = commitAddObject (self::myStringStatic ('rack', __CLASS__), NULL, 1560, NULL);
+               self::$row_id = commitAddObject (self::myStringStatic ('row', __CLASS__), NULL, 1561, NULL);
+               self::$location_id = commitAddObject (self::myStringStatic ('location', __CLASS__), NULL, 1562, NULL);
+               self::$object_id = commitAddObject (self::myStringStatic ('object', __CLASS__), NULL, 1, NULL);
        }
 
        public static function tearDownAfterClass ()
index 5a01464..fdb2aff 100644 (file)
@@ -14,6 +14,10 @@ class RTTestCase extends RTTestCaseShim
        {
                return sprintf ('%s-%s-%u', $s, get_class ($this), getmypid());
        }
+       protected static function myStringStatic ($s, $c)
+       {
+               return sprintf ('%s-%s-%u', $s, $c, getmypid());
+       }
 }
 
 // Sanity check DB connection