Merge pull request #6 from aBMania/master

fix empty config bug

src/Reflector.ts

@@ -99,7 +99,7 @@ class Reflector {
      */
     public setPropertyForClass(target: any, propertyName: string) {
         let props = this.getClassProperties(target);
-        if (props === null) {
+        if (props.length === 0) {
             this._reflect.defineMetadata("tsconvict:properties", [], target.constructor);
             props = this.getClassProperties(target);
         }
@@ -112,7 +112,7 @@ class Reflector {
      * @param target A constructor of a config class.
      */
     public getClassProperties(target: any): string[] {
-        return this._reflect.getMetadata("tsconvict:properties", target.constructor) || null;
+        return this._reflect.getMetadata("tsconvict:properties", target.constructor) || [];
     }
 }
 

src/test/scenarios/empty_config/EmptyConfig.model.ts

@@ -0,0 +1,2 @@
+export default class EmptyConfig {
+}

src/test/scenarios/empty_config/EmptyConfig.test.ts

@@ -0,0 +1,38 @@
+/**
+ * @module test/models
+ */
+import {suite, test} from "mocha-typescript";
+import * as assert from "assert";
+import "mocha";
+import EmptyConfig from './EmptyConfig.model';
+import {TSConvict} from '../../../index';
+
+let tsConvict: TSConvict<EmptyConfig>;
+
+@suite('Test a config with only simple values')
+export class EmptyConfigTest {
+
+    /**
+     * Make a new one each and every test.
+     */
+    public before() {
+        tsConvict = new TSConvict(EmptyConfig);
+    }
+
+    @test('Test getting an empty config')
+    public validConfig() {
+        const myValidConfig: EmptyConfig = tsConvict.load({});
+        // Make sure we got a proper serialized type back
+        assert.strictEqual(
+            (myValidConfig instanceof EmptyConfig),
+            true,
+            'Expected the config to be an instance of EmptyConfig'
+        );
+
+        assert.deepEqual(
+            myValidConfig,
+            {},
+            'Expected the config to be empty'
+        );
+    }
+}