fix(data): make non-optimistic add command entity partial

Mirror the type definition of EntityCollectionServiceBase.add on EntityServerCommands.add.

Closes EntityServerCommands.add does not allow optional id #3847

modules/data/spec/dispatchers/entity-dispatcher.spec.ts

@@ -157,6 +157,23 @@ export function commandDispatchTest(
         expect(data).toBe(hero);
       });
 
+      it('#add(hero) dispatches SAVE_ADD pessimistically with partial hero', () => {
+        const hero: Partial<Hero> = { name: 'test' };
+        dispatcher.add(hero);
+        const { entityOp, isOptimistic, data } = dispatchedAction().payload;
+        expect(entityOp).toBe(EntityOp.SAVE_ADD_ONE);
+        expect(isOptimistic).toBe(false);
+        expect(data).toBe(hero);
+
+        testStore.dispatch.calls.reset();
+
+        dispatcher.add(hero, { isOptimistic: false });
+        const specificallyPessimistic = dispatchedAction().payload;
+        expect(specificallyPessimistic.entityOp).toBe(EntityOp.SAVE_ADD_ONE);
+        expect(specificallyPessimistic.isOptimistic).toBe(false);
+        expect(specificallyPessimistic.data).toBe(hero);
+      });
+
       it('#delete(42) can dispatch SAVE_DELETE pessimistically for the id:42', () => {
         dispatcher.delete(42, { isOptimistic: false }); // optimistic by default
         const { entityOp, isOptimistic, data } = dispatchedAction().payload;

modules/data/src/dispatchers/entity-commands.ts

@@ -11,6 +11,14 @@ export interface EntityServerCommands<T> {
    * @returns A terminating Observable of the entity
    * after server reports successful save or the save error.
    */
+  add(
+    entity: Partial<T>,
+    options?: EntityActionOptions & { isOptimistic?: false }
+  ): Observable<T>;
+  add(
+    entity: T,
+    options: EntityActionOptions & { isOptimistic: true }
+  ): Observable<T>;
   add(entity: T, options?: EntityActionOptions): Observable<T>;
 
   /**
@@ -101,11 +109,11 @@ export interface EntityServerCommands<T> {
    * after server reports successful query or the query error.
    * @see getWithQuery
    */
-  loadWithQuery(queryParams: QueryParams | string,
-                options?: EntityActionOptions
+  loadWithQuery(
+    queryParams: QueryParams | string,
+    options?: EntityActionOptions
   ): Observable<T[]>;
 
-
   /**
    * Dispatch action to save the updated entity (or partial entity) in remote storage.
    * The update entity may be partial (but must have its key)