-
Notifications
You must be signed in to change notification settings - Fork 59
Develop #103
Conversation
fix calls without validate
log validation error use mimeType for POST
move postman tests
bump version to 2.0.0 update docs add regex to schema
@shrikster could you help review this? I think it's ready for release, but would like you opinion. |
Reviewing tomorrow 👍 @jgroom33 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You made a real progress seems like a huge step for us !!
I am still in middle of the review ..
src/handler.ts
Outdated
console.log(); | ||
if(validateSchema.errors.length > 0) { | ||
for (let erroredSchema of validateSchema.errors){ | ||
console.log(JSON.stringify(erroredSchema.details, null, 2)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe we should use console.error ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I updated this to use writeErrorMessage()
src/test.ts
Outdated
}) | ||
|
||
nunjucksEnv.addGlobal('Env', function (envi: string) { | ||
let environ = process.env[envi] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can be const
rename fail tests add schema failure
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wondered what this required raw
field in the Schema is about
The raw object allows the file to be parsed by nunjucks as each new response object is saved.
This isn't really the way nunjucks was designed to be used, but it seems to work out. |
@eykrehbein any other topics to discuss? |
This is ready for review. It implements HAR as the schema.
Resolve #94
Resolve #97
Resolve #22