fix tests

Author: dplewis

integration/test/ParseObjectTest.js

@@ -552,12 +552,7 @@ describe('Parse Object', () => {
     expect(() => {
       item.set({ 'foo^bar': 'baz' });
     }).toThrow(error);
-    try {
-      await item.save({ 'foo^bar': 'baz' });
-      expect(true).toBe(false);
-    } catch (e) {
-      expect(e).toEqual(error);
-    }
+    await expectAsync(item.save({ 'foo^bar': 'baz' })).toBeRejectedWith(error);
   });
 
   it('cannot use invalid key names in multiple sets', () => {

src/ParseObject.ts

@@ -1357,7 +1357,7 @@ class ParseObject<T extends Attributes = Attributes> {
    * @returns {Promise} A promise that is fulfilled when the save
    * completes.
    */
-  save(
+  async save(
     arg1?: undefined | string | { [attr: string]: any } | null,
     arg2?: SaveOptions | any,
     arg3?: SaveOptions
@@ -1403,9 +1403,8 @@ class ParseObject<T extends Attributes = Attributes> {
         .then(() => controller.save(unsavedObjects, saveOptions))
         .then((savedOjbects: this[]) => savedOjbects.pop());
     }
-    return controller.save(unsaved, saveOptions).then(() => {
-      return controller.save(this, saveOptions);
-    }) as Promise<ParseObject> as Promise<this>;
+    await controller.save(unsaved, saveOptions);
+    return controller.save(this, saveOptions) as Promise<ParseObject> as Promise<this>;
   }
 
   /**

src/__tests__/ParseObject-test.js

@@ -1951,12 +1951,9 @@ describe('ParseObject', () => {
     const child = new ParseObject('Item');
     parent.set('child', child);
     child.set('parent', parent);
-    try {
-      await parent.save();
-      expect(true).toBe(false);
-    } catch (e) {
-      expect(e.message).toBe('Cannot create a pointer to an unsaved Object.');
-    }
+    await expect(parent.save()).rejects.toThrowError(
+      'Cannot create a pointer to an unsaved Object.'
+    );
   });
 
   it('will fail for deeper unsaved objects', async () => {
@@ -1965,12 +1962,9 @@ describe('ParseObject', () => {
     const grandchild = new ParseObject('Item');
     parent.set('child', child);
     child.set('child', grandchild);
-    try {
-      await parent.save();
-      expect(true).toBe(false);
-    } catch (e) {
-      expect(e.message).toBe('Cannot create a pointer to an unsaved Object.');
-    }
+    await expect(parent.save()).rejects.toThrowError(
+      'Cannot create a pointer to an unsaved Object.'
+    );
   });
 
   it('does not mark shallow objects as dirty', () => {