Throw exception if $schema is present and it is not a string

[tizmagik]

What version of Ajv are you using? Does the issue happen if you use the latest version?

Latest

Ajv options object (see https://github.com/epoberezkin/ajv#options):

{}

JSON Schema (please make it as small as possible to reproduce the issue):

{
  "thing": {
    "type": 123,
  },
}

Data (please make it as small as posssible to reproduce the issue):

n/a

Your code (please use options, schema and data as variables):

const Ajv = require('ajv');
const ajv = new Ajv();

const isValid = ajv.validateSchema({
  thing: {
    type: 123,
  },
});

console.log(isValid); // true ??

Validation result, data AFTER validation, error messages:

none

What results did you expect?
returns false

Are you going to resolve the issue?
I'd be happy to, unless the issue is I'm doing something wrong (likely)

[epoberezkin]

thing is "an additional property" in the schema and the specification allows additional properties to be present. It's only if the value of defined keyword were invalid, schema validation would fail, e.g. minimum: false instead of a number.

Usually you don't have to explicitly validate schemas - they are validated when they are added to Ajv instance (unless you disable it).

[tizmagik]

@epoberezkin I can't re-open but here's the edge-case that I brought up on Gitter. Let me know if you'd like me to open a new issue or if you can edit the original description here.

const Ajv = require('ajv');
const ajv = new Ajv();

const isValid = ajv.validateSchema({
  $schema: {
    thing: {
      type: 'asdf',
    },
  },
});

console.log(isValid); // true ??

[epoberezkin]

in 5.0.1-beta.3