Replace empty value by random value (#522)

* Update doc

* Declare option

* Permit test to handle multiple requires

* Add tests

* Add replaceEmptyByRandomValue test option

* Handle replaceEmptyByRandomValue option

docs/README.md

@@ -123,6 +123,7 @@ jsf.locate('faker');
 - `reuseProperties` — If enabled, it will try to generate missing properties from existing ones. Only when `fillProperties` is enabled too  (default: `false`)
 - `fillProperties` — If enabled, it will try to generate missing properties to fulfill the schema definition (default: `true`)
 - `random` — Setup a custom _randonmess_ generator, useful for getting deterministic results (default: `Math.random`)
+- `replaceEmptyByRandomValue` — Remplace default empty value by a random value (default: `false`)
 
 ## Building
 

src/class/OptionRegistry.js

@@ -25,6 +25,7 @@ defaults.maxLength = null;
 defaults.resolveJsonPath = false;
 defaults.reuseProperties = false;
 defaults.fillProperties = true;
+defaults.replaceEmptyByRandomValue = false;
 
 defaults.random = Math.random;
 

src/core/traverse.js

@@ -25,7 +25,9 @@ function traverse(schema, path, resolve, rootSchema) {
     }
 
     if (optionAPI('useDefaultValue') && 'default' in schema) {
-      return schema.default;
+      if (schema.default !== '' || !optionAPI('replaceEmptyByRandomValue')) {
+        return schema.default;
+      }
     }
 
     if ('template' in schema) {

tests/schema/core/option/replaceEmptyByRandomValue.js

+module.exports = {
+  register(jsf) {
+    return jsf.option({
+      replaceEmptyByRandomValue: true,
+    });
+  },
+};

tests/schema/core/option/useDefaultValue.json

@@ -10,6 +10,30 @@
         },
         "equal": "Hello",
         "require": "core/option/useDefaultValue"
+      },
+      {
+        "description": "should handle useDefaultValue option with an empty default value",
+        "schema": {
+          "type": "string",
+          "default": ""
+        },
+        "equal": "",
+        "require": "core/option/useDefaultValue"
+      },
+      {
+        "description": "should handle useDefaultValue & replaceEmptyByRandomValue option",
+        "schema": {
+          "test-response": {
+            "type": "string",
+            "default": ""
+          }
+        },
+        "valid": true,
+        "notEmpty": ["test-response"],
+        "require": [
+          "core/option/useDefaultValue",
+          "core/option/replaceEmptyByRandomValue"
+        ]
       }
     ]
   }

tests/schema/main.spec.js

@@ -29,7 +29,11 @@ function seed() {
         });
 
         if (test.require) {
-          require(`./${test.require}`).register(jsf);
+          test.require = Array.isArray(test.require) ? test.require : [test.require];
+
+          test.require.map(toRequire => {
+            return require(`./${toRequire}`).register(jsf);
+          });
         }
 
         const schema = typeof test.schema === 'string'