update tests

Comments:
- ignore readonly ModelStatic props
- casts models to `any` since we can't augment for each test

Author: hamiltoes

test/service-module/model-base.test.ts

@@ -130,6 +130,7 @@ describe('makeModel / BaseModel', function () {
     const { BaseModel, makeServicePlugin } = feathersVuex(feathers, {
       serverAlias: 'myApi'
     })
+    //@ts-ignore
     BaseModel.modelName = 'TestModel'
     const plugin = makeServicePlugin({
       servicePath: 'todos',
@@ -220,19 +221,20 @@ describe('makeModel / BaseModel', function () {
       plugins: [todosPlugin, tasksPlugin]
     })
     const { models } = myApi
+    const anyModels = models as any
 
-    assert(models.myApi.Todo === Todo)
-    assert(!models.theirApi.Todo, `Todo stayed out of the 'theirApi' namespace`)
-    assert(models.theirApi.Task === Task)
-    assert(!models.myApi.Task, `Task stayed out of the 'myApi' namespace`)
+    assert(anyModels.myApi.Todo === Todo)
+    assert(!anyModels.theirApi.Todo, `Todo stayed out of the 'theirApi' namespace`)
+    assert(anyModels.theirApi.Task === Task)
+    assert(!anyModels.myApi.Task, `Task stayed out of the 'myApi' namespace`)
 
     assert.equal(
-      models.myApi.byServicePath[Todo.servicePath],
+      anyModels.myApi.byServicePath[Todo.servicePath],
       Todo,
       'also registered in models.byServicePath'
     )
     assert.equal(
-      models.theirApi.byServicePath[Task.servicePath],
+      anyModels.theirApi.byServicePath[Task.servicePath],
       Task,
       'also registered in models.byServicePath'
     )

test/service-module/model-instance-defaults.test.ts

@@ -322,11 +322,11 @@ describe('Models - Default Values', function() {
       public constructor(data?, options?) {
         // Pass { merge: false } in the third arg to prevent BaseModel from
         // doing its own merge
-        super(data, options, { merge: false })
+        super(data, { merge: false })
 
         // Calling merge here overwrites the Class's default location.
         // You could also write `this.location = data.location`
-        return Person.merge(this, data)
+        Person.merge(this, data)
       }
     }
 

test/service-module/model-methods.test.ts

@@ -235,6 +235,7 @@ describe('Models - Methods', function() {
 
   it('instance.save passes params to update', function() {
     const { Task } = makeContext()
+    //@ts-ignore
     Task.preferUpdate = true
 
     const task = new Task({ id: 1, test: true })

test/service-module/model-relationships.test.ts

@@ -188,7 +188,7 @@ function makeContext() {
   class Item extends BaseModel {
     public static modelName = 'Item'
     public get todos() {
-      return BaseModel.models.Todo.findInStore({ query: {} }).data
+      return (BaseModel.models as any).Todo.findInStore({ query: {} }).data
     }
     public static instanceDefaults() {
       return {

test/utils.test.ts

@@ -84,6 +84,7 @@ describe('Utils', function() {
       feathersClient,
       { serverAlias: 'hydrate' }
     )
+    const anyModels = models as any
 
     class User extends BaseModel {
       public static modelName = 'User'
@@ -107,7 +108,7 @@ describe('Utils', function() {
     store.commit('users/addServerItem')
     assert(store.state.users.keyedById['abcdefg'], 'server document added')
     assert(store.state.users.keyedById['abcdefg'] instanceof Object, 'server document is pure javascript object')
-    hydrateApi({ api: models.hydrate })
+    hydrateApi({ api: anyModels.hydrate })
     assert(store.state.users.keyedById['abcdefg'] instanceof User, 'document hydrated')
   })