Required oneOf relation about data HOT 6 OPEN

@kettanaito finally finished the implementation, please take a look (especially check if the changes in tests make sense to you, some cases I just removed, because they were testing the behavior that doesn't exist anymore).

from data.

@kettanaito, I would like to hear your thoughts about it. If this makes sense, I'd implement this and create PR.

A simple use case is: we have a model Employee with fields id, name, email, etc., and also a company which is oneOf relation to the Company model. Employee can never exist without Company so employee.company may never be undefined, otherwise we'd use nullable(oneOf('Company')).

from data.

Hi, @timkolotov. This is a great proposal. The consistency you describe is the way to go:

• oneOf() relationships must have initial value. If not provided, the library should throw. If provided, use the current logic as-is;
• If the consumer wishes to have a nullable one-of relationship, describe it explicitly via nullable(oneOf(modelName)).
  • If marked as nullable and no initial value was provided to the relationship, set its value as null.
  • If marked as nullable and initial value was provided to the relationship, set its value to the initially provided value.
  • If marked as nullable, allow updating this relational property to null (i.e. via update etc).

Most of this is already covered by nullable(). The change would be to treat oneOf relationship as non-nullable by default.

Does this sound interesting to you to give it a try? I think the change itself should be rather scoped. I can always help with the code review.

from data.

Yep, I'm in. Currently, I just patched the types to shut the TS compiler up, which isn't really safe...

from data.

Any update on this issue? Really interested in this as I think I've run into a similar problem. Not sure of the best way around this, with strict mode due to undefined:

// My MSW handler response type
type BasketResponse = {
    basketId: string,
    currency: {
        code: string,
        symbol: string
    }
}

export const db = factory({
    basket: {
        basketId: primaryKey(String),
        currency: oneOf('currency'),
    },
    currency: {
        code: primaryKey(String),
        symbol: String,
    }
})

const currency = db.currency.create()
const basket = db.basket.create({ currency });

type currencyType = typeof currency; // PublicEntity< ... >
type basketCurrencyType = typeof basket.currency; // PublicEntity< ... > |  undefined

const value: BasketResponse = basket // 'Type 'undefined' is not assignable to type '{ code: string; symbol: string; }'.

This makes it a bit awkward to use the basket pulled directly from the db with a MSW handler typed as rest.post<CreateBasketRequest, PathParams, BasketResponse>

from data.

@harry-gocity it does look like the same issue indeed. I started working on it but didn't have much time so far. With enough luck I'll have a vacation next week and will be able to dedicate time for this.

from data.