Changeset 48949

Timestamp:

09/06/2020 03:35:59 AM (5 years ago)

Author:

SergeyBiryukov

Message:

Tests: Correct assertion in Tests_Cache::test_add_get_null().

It is possible to store null in the cache without it being converted to an empty string.

Follow-up to [20089].

Props johnbillion, SergeyBiryukov.
See #38266.

File:

• trunk/tests/phpunit/tests/cache.php (modified) (2 diffs)

trunk/tests/phpunit/tests/cache.php

@@ -44 +44 @@
 
         // You can store zero in the cache.
-        $this->cache->add( $key, $val );
+        $this->assertTrue( $this->cache->add( $key, $val ) );
         $this->assertSame( $val, $this->cache->get( $key ) );
     }
@@ -52 +52 @@
         $val = null;
 
+        // You can store null in the cache.
         $this->assertTrue( $this->cache->add( $key, $val ) );
-        // Null is converted to empty string.
-        $this->assertEquals( '', $this->cache->get( $key ) );
+        $this->assertSame( $val, $this->cache->get( $key ) );
     }