Merge pull request #93 from tlivings/master

accept a config where exclude is not defined
ExpediaGroup · Mar 2, 2022 · 0120f81 · 0120f81
2 parents dcab31d + cd73f9e
commit 0120f81
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,13 @@
+
+### v3.0.1
+
+- [FIXED] - if `exclude` is undefined in a component config
+
+### v3.0.0
+
+- Modernized to utilize new graphql-tools
+- [BREAKING] - directives are not imported and maintained throughout the tree : top level component is responsible for directive implementations
+
 ### v2.2.0
 
 - [FEATURE] - Add "hook" for custom `makeExecutableSchema` function

diff --git a/lib/transforms/__tests__.js b/lib/transforms/__tests__.js
@@ -4,6 +4,14 @@ const { exclusions } = require('./index.js');
 const Test = require('tape');
 const { FilterTypes, FilterObjectFields } = require('graphql-tools');
 
+Test('exclusions() accepts null exclude arg', (t) => {
+  t.plan(1);
+
+  t.doesNotThrow(() => {
+    exclusions();
+  }, 'does not explode');
+});
+
 Test('exclusions() throws an error if passed a malformed exclusion', (t) => {
   try {
     exclusions(['mal.form.ed']);

diff --git a/lib/transforms/index.js b/lib/transforms/index.js
@@ -2,7 +2,7 @@
 
 const { FilterTypes, FilterObjectFields } = require('graphql-tools');
 
-const exclusions = function(exclusions) {
+const exclusions = function(exclusions = []) {
   return exclusions.map((exclusion) => {
     if (typeof exclusion === 'string') {
       const parts = exclusion.split('.');

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "graphql-component",
-  "version": "3.0.0",
+  "version": "3.0.1",
   "description": "Build, customize and compose GraphQL schemas in a componentized fashion",
   "keywords": [
     "graphql",