[Question] Is there a posibility to pass Joi options that override Joi default options to .expect() method?

[terradek]

I'm using this method to pass it but I wonder if it's optimal enough?:

const joiOptions = { convert: false, presence: "required" };

frisby.timeout(2500)
    .post(baseUrl + '/auth/register' + urlOptions, user)
    .expect('jsonTypes', {
        meta: Joi.object({  status: Joi.string().valid('ok'),  error: Joi.equal(null)
        }).options(joiOptions),
        response: Joi.object({
            user: Joi.object({
                id: Joi.string().regex(/^[1-9]\d+/).min(1)
            }).options(joiOptions)
        }).options(joiOptions)

Thank you in advance

[H1Gdev]

@terradek

Yes.

Small sample code

const frisby = require('frisby');
const Joi = frisby.Joi;

const options = { convert: true };

it('Test', () => {
  return frisby.fromJSON({
    number: '1',
  })
    .expect('jsonTypes', Joi.object({
      number: Joi.number(),
    }).options(options));
});

const options = { convert: false };

Result

ValidationError: child "number" fails because ["number" must be a number]

[H1Gdev]

@terradek

Did you solve this issue?

[terradek]

@H1Gdev I thought there is a simpler method with passing defaults to .expect().
But this one works too.
Thank you!
PS It would be great to implement in future releases.

[terradek]

Voting this to be implemented in the next release as an optional parameter for expect().
Thank you.

[H1Gdev]

@vlucas

I think that proposal from @terradek is better.

	path	json	options
data type	string or array	any	object
	optional	mandatory	optional

But if impletemt this, then need to change order of parameters.

.expect('jsonTypes', path, json)

to

.expect('jsonTypes', json, path, options)

And should also change order of json and jsonStrict, too.

I think that change of interface is very affected.

Do you have any good ideas?