make OnlyNeeded the default upsertGraph fetchStrategy

studiozeffa · Aug 25, 2019 · b8e7f70 · b8e7f70
1 parent c31b3ea
commit b8e7f70
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/lib/queryBuilder/graph/GraphOptions.js b/lib/queryBuilder/graph/GraphOptions.js
@@ -33,7 +33,7 @@ class GraphOptions {
     }
 
     if (!this.options[FETCH_STRATEGY]) {
-      return strategy === FetchStrategy.OnlyIdentifiers;
+      return strategy === FetchStrategy.OnlyNeeded;
     } else {
       return this.options[FETCH_STRATEGY] === strategy;
     }

diff --git a/tests/unit/relations/ManyToManyRelation.js b/tests/unit/relations/ManyToManyRelation.js
@@ -253,9 +253,11 @@ describe('ManyToManyRelation', () => {
         }
       });
     }).to.throwException(err => {
-      expect(err.message).to.equal(
-        "OwnerModel.relationMappings.testRelation: Cannot find module '/not/a/path/to/a/model'"
-      );
+      expect(
+        err.message.startsWith(
+          "OwnerModel.relationMappings.testRelation: Cannot find module '/not/a/path/to/a/model'"
+        )
+      ).to.equal(true);
     });
   });