Handle POJO models

[jrowlingson]

We went off the path a bit and have some model tests that are testing models which are simple EmberObjects.

The following input:

const worker = this.subject({ user });

Produces the following output:

const worker = run(() => this.owner.lookup('service:store').createRecord('worker', { user }));

This obviously will cause the test to fail because this is not an EmberData model object. I can submit a failing test if you feel this usage should be handled.

[rwjblue]

Well, I think this type of test is fine if using a type other than model with moduleFor, but the issue is mostly that we have a specific thing to replace this.subject() in a moduleForModel (or moduleFor('model:foo'...)) test. I don't know how we could know if a given underlying thing was a "real" DS.Model or not.

[jrowlingson]

@rwjblue 👍 makes sense - i'll defer to you as to whether the issue should be closed or not given that there may not be an (easy) path forward.

For reference there are a few options for manual migration. Either import the model directly or via const foo = this.owner.factoryFor('model:foo').create();