add breaking test for upsert

[0ff]

There is a quirk in this mixin: if you upsert a model (i.e. you just know the id and properties, but you don't want to find it first) the createdAt-Timestamp is updated, too. This must not happen, of course.
I fear it's related to the lb-ds-juggler's implementation of default values.

Can you think of a solution besides querying for the model everytime?

[clarkbw]

Good catch! I'm looking into this in #11
Seems like the README should be updated to show this. I can get it fixed in juggler and bump the required release version; leaving a note in the README.

[clarkbw]

@0ff the fix landed in loopbackio/loopback-datasource-juggler@01af886 I'll update the package.json file to reflect this and I'll try to pull in your tests as well.

[0ff]

Awesome, thank you!

[mrshll]

For anyone coming to here because of the runtime warning, you have to make sure your model.json has the validateUpsert option set to false:

{
  "name": "Message",
  "plural": "messages",
  "options": {
    "validateUpsert": false
  }
}

[cvrabie]

Disabling validation for upsert seems like a big disadvantage. Are there any plans to address this in the future?

[ShaharHD]

I've branched off and revised a better solution IMHO.

I'm creating a pull request.