es-mapping-to-schema - npm Package Compare versions

Comparing version 3.3.1 to 3.3.2

index.js

@@ -83,3 +83,3 @@ const _       = require('lodash');
 
-    if (schema.type === 'string' && options[schemaType].all.maxLength) {
+    if (options[schemaType].all.maxLength && mapping.type === 'string') {
       schema.maxLength = options[schemaType].all.maxLength;
@@ -86,0 +86,0 @@ }

package.json

 {
   "name": "es-mapping-to-schema",
-  "version": "3.3.1",
+  "version": "3.3.2",
   "description": "Convert Elasticsearch mappings to Schema Inspector schemas",
@@ -5,0 +5,0 @@ "main": "index.js",

test.js

@@ -1262,2 +1262,34 @@ const chai      = require('chai');
 
+  it('should apply maxLength sanitization to all strings', () => {
+    const mapping = {
+      properties: {
+        someString: {
+          type: 'string'
+        },
+        someInt: {
+          type: 'integer'
+        }
+      }
+    };
+
+    const expectedSchema = {
+      properties: {
+        someString: {
+          maxLength: 20
+        },
+        someInt: {}
+      }
+    };
+
+    const schemas = MappingToSchema(mapping, {
+      sanitization: {
+        all: {
+          maxLength: 20
+        }
+      }
+    });
+
+    expect(schemas.sanitization).to.eql(expectedSchema);
+  });
+
   it('should apply rules to all string mappings', () => {
@@ -1264,0 +1296,0 @@ const mapping = {

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

48757

increased by1.2%

Lines of code

1636

increased by1.74%

No dependency changes