Fix for nested props after required is used #31

Author: aboutlo

.travis.yml

@@ -1,6 +1,6 @@
 language: node_js
 
 node_js:
-- "11"
+- "11.10.1" # tmp workaround https://stackoverflow.com/questions/55059748/travis-jest-typeerror-cannot-assign-to-read-only-property-symbolsymbol-tostr
 - "10"
 - "8"

src/BaseSchema.js

@@ -2,12 +2,9 @@
 const {
   flat,
   omit,
-  hasCombiningKeywords,
   isFluentSchema,
   last,
   patchIdsWithParentId,
-  valueOf,
-  FORMATS,
   REQUIRED,
   setAttribute,
   setComposeType,
@@ -162,7 +159,7 @@ const BaseSchema = (
       : [REQUIRED]
     return options.factory({
       schema: { ...schema, required },
-      options,
+      ...options,
     })
   },
 

src/BaseSchema.test.js

@@ -126,7 +126,12 @@ describe('BaseSchema', () => {
         const prop = 'foo'
         expect(
           S.object()
-            .prop(prop, S.string().required())
+            .prop(
+              prop,
+              S.string()
+                .required()
+                .minLength(3)
+            )
             .valueOf().required
         ).toEqual([prop])
       })