fix(schema): handle non-string enum values

Author: P0lip

package.json

@@ -43,7 +43,7 @@
     "inline-version": "node ./scripts/inline-version.js",
     "prelint": "tsc --noEmit",
     "lint.fix": "yarn lint.prettier --write && yarn lint.eslint --fix",
-    "lint.eslint":" eslint --cache --cache-location .cache/ --ext=.js,.ts src test-harness",
+    "lint.eslint": "eslint --cache --cache-location .cache/ --ext=.js,.ts src test-harness",
     "lint.prettier": "prettier --ignore-path .eslintignore --check {src,test-harness}/**/*.{json,yaml} docs/**/*.md README.md",
     "lint": "yarn lint.prettier && yarn lint.eslint",
     "copy.html-templates": "copyfiles -u 1 \"./src/formatters/html/*.html\" \"./dist/\"",
@@ -62,7 +62,7 @@
     "test": "jest --silent"
   },
   "dependencies": {
-    "@stoplight/better-ajv-errors": "0.0.0",
+    "@stoplight/better-ajv-errors": "0.0.2",
     "@stoplight/json": "3.9.0",
     "@stoplight/json-ref-readers": "1.2.1",
     "@stoplight/json-ref-resolver": "3.1.0",

src/functions/__tests__/schema.test.ts

@@ -269,7 +269,7 @@ describe('schema', () => {
       it('reports pretty enum errors for a string', () => {
         expect(runSchema('baz', testSchema)).toEqual([
           {
-            message: `String should be equal to one of the allowed values: foo, bar. Did you mean bar?`,
+            message: 'String should be equal to one of the allowed values: `foo`, `bar`. Did you mean `bar`?',
             path: [],
           },
         ]);
@@ -321,7 +321,7 @@ describe('schema', () => {
 
     expect(runSchema('three', testSchema)).toEqual([
       {
-        message: `String should be equal to one of the allowed values: foo, bar`,
+        message: 'String should be equal to one of the allowed values: `foo`, `bar`',
         path: [],
       },
     ]);

src/rulesets/asyncapi/__tests__/asyncapi-headers-schema-type-object.ts

@@ -63,7 +63,7 @@ describe(`Rule '${ruleName}'`, () => {
       expect.objectContaining({
         code: ruleName,
         message:
-          'Headers schema type should be `object` (`type` property should be equal to one of the allowed values: object. Did you mean object?).',
+          'Headers schema type should be `object` (`type` property should be equal to one of the allowed values: `object`. Did you mean `object`?).',
         path: ['components', 'messages', 'aMessage', 'headers', 'type'],
         severity: rule.severity,
       }),
@@ -98,7 +98,7 @@ describe(`Rule '${ruleName}'`, () => {
       expect.objectContaining({
         code: ruleName,
         message:
-          'Headers schema type should be `object` (`type` property should be equal to one of the allowed values: object. Did you mean object?).',
+          'Headers schema type should be `object` (`type` property should be equal to one of the allowed values: `object`. Did you mean `object`?).',
         path: ['components', 'messageTraits', 'aTrait', 'headers', 'type'],
         severity: rule.severity,
       }),
@@ -155,7 +155,7 @@ describe(`Rule '${ruleName}'`, () => {
         expect.objectContaining({
           code: ruleName,
           message:
-            'Headers schema type should be `object` (`type` property should be equal to one of the allowed values: object. Did you mean object?).',
+            'Headers schema type should be `object` (`type` property should be equal to one of the allowed values: `object`. Did you mean `object`?).',
           path: ['channels', 'users/{userId}/signedUp', property, 'message', 'headers', 'type'],
           severity: rule.severity,
         }),

test-harness/scenarios/examples.oas2.scenario

@@ -268,7 +268,7 @@ OpenAPI 2.0 (Swagger) detected
   95:26    error  oas2-valid-schema-example  `example` property should match format `date-time`
   99:26    error  oas2-valid-schema-example  `example` property should match format `url`
  106:11  warning  operation-tag-defined      Operation tags should be defined in global tags.
- 120:22    error  oas2-valid-schema-example  `x-example` property should be equal to one of the allowed values: foo, bar
+ 120:22    error  oas2-valid-schema-example  `x-example` property should be equal to one of the allowed values: `foo`, `bar`
  177:30    error  oas2-valid-media-example   `application/json` property should have required property `name`
  194:30    error  oas2-valid-media-example   `application/json` property should have required property `user`
 

test-harness/scenarios/examples.oas3.scenario

@@ -356,8 +356,8 @@ OpenAPI 3.x detected
  138:30  error  oas3-valid-schema-example  `example` property type should be boolean
  142:30  error  oas3-valid-schema-example  `example` property should match format `date-time`
  146:30  error  oas3-valid-schema-example  `example` property should match format `url`
- 169:20  error  oas3-valid-media-example   `example` property should be equal to one of the allowed values: foo, bar
- 185:22  error  oas3-valid-schema-example  `example` property should be equal to one of the allowed values: foo, bar
+ 169:20  error  oas3-valid-media-example   `example` property should be equal to one of the allowed values: `foo`, `bar`
+ 185:22  error  oas3-valid-schema-example  `example` property should be equal to one of the allowed values: `foo`, `bar`
  197:22  error  oas3-valid-media-example   `value` property type should be string
  199:21  error  oas3-valid-media-example   `value` property type should be string
  262:25  error  oas3-valid-media-example   `value` property should have required property `id`

yarn.lock

@@ -593,10 +593,10 @@
   dependencies:
     type-detect "4.0.8"
 
-"@stoplight/[email protected].0":
-  version "0.0.0"
-  resolved "https://registry.yarnpkg.com/@stoplight/better-ajv-errors/-/better-ajv-errors-0.0.0.tgz#18bf69c2380a00b72e4bd5fb7247c5adf615795a"
-  integrity sha512-jA1i4J5HmCXMmZdZigdeEHpisFCGVg5QfK++ouxP58pxshATJ5G40U9uGec2vYnuR2MQnzWHAXYf0f8jvJhBwg==
+"@stoplight/[email protected].2":
+  version "0.0.2"
+  resolved "https://registry.yarnpkg.com/@stoplight/better-ajv-errors/-/better-ajv-errors-0.0.2.tgz#f941e53c03838fa85cdb51c07a7fa9d21ff405be"
+  integrity sha512-XB6sISR7jRUPWXBcYGltxvn/Rq8vCfmmeGhG6SBfDJOnKfz2WMPqOOEeylCDqMRh9rtpImZLZxNyatRDQZ1ivQ==
   dependencies:
     jsonpointer "^4.0.1"
     leven "^3.1.0"