Fix empty() and add more tests

Author: youchenlee

src/Setting/Setting.php

@@ -203,17 +203,14 @@ protected function hasByKey($key)
     {
         if (strpos($key, '.') !== false) {
             $setting = $this->getSubValue($key);
-
-            return (empty($setting)) ? false : true;
         } else {
             if ($this->cache->has($key.'@'.$this->lang)) {
                 $setting = $this->cache->get($key.'@'.$this->lang);
             } else {
                 $setting = $this->storage->retrieve($key, $this->lang);
             }
-
-            return (empty($setting)) ? false : true;
         }
+        return ($setting === null) ? false : true;
     }
 
     protected function forgetByKey($key)

tests/SettingsTest.php

@@ -140,6 +140,29 @@ protected function setMock()
         Schema::swap(Manager::Schema());
     }
 
+    public function testNullValue()
+    {
+        $cache = m::mock(CacheContract::class);
+        $cache->shouldReceive('has')->andReturn(false);
+        $cache->shouldReceive('add')->andReturn(true);
+
+        $setting = new Setting(new EloquentStorage(), $cache);
+        $setting->set('a', null);
+        $this->assertTrue($setting->get('a') === null);
+        $this->assertTrue($setting->get('b') === null);
+
+        $setting->set('foo.bar', null);
+        $this->assertTrue($setting->get('foo.bar') === null);
+
+        $this->assertTrue($setting->get('foo.xxx') === null);
+
+        $setting->set('foo.zzz', 0);
+        $this->assertTrue($setting->get('foo.zzz') === 0);
+
+        $setting->set('foo.yyy', []);
+        $this->assertTrue($setting->get('foo.yyy') === []);
+    }
+
     protected function migrationUp()
     {
         (new CreateSettingsTable())->up();