allow non-Symbol object properties on RSAA

README.md

@@ -813,9 +813,9 @@ A *Redux Standard API-calling Action* MUST
 - be a plain JavaScript object,
 - have an `[RSAA]` property.
 
-A *Redux Standard API-calling Action* MUST NOT
+A *Redux Standard API-calling Action* MAY
 
-- include properties other than `[RSAA]`.
+- include properties other than `[RSAA]` (but will be ignored by redux-api-middleware).
 
 #### `[RSAA]`
 

src/validation.js

@@ -82,12 +82,6 @@ function validateRSAA(action) {
     return validationErrors;
   }
 
-  for (let key in action) {
-    if (key !== RSAA) {
-      validationErrors.push(`Invalid root key: ${key}`);
-    }
-  }
-
   const callAPI = action[RSAA];
   if (!isPlainObject(callAPI)) {
     validationErrors.push('[RSAA] property must be a plain JavaScript object');

test/index.js

@@ -94,16 +94,17 @@ test('validateRSAA/isValidRSAA must identify conformant RSAAs', t => {
   );
 
   const action2 = {
-    [RSAA]: {},
-    invalidKey: ''
+    [RSAA]: {
+      endpoint: '',
+      method: 'GET',
+      types: ['REQUEST', 'SUCCESS', 'FAILURE']
+    },
+    anotherKey: 'foo'
   };
-  t.ok(
-    validateRSAA(action2).includes('Invalid root key: invalidKey'),
-    'RSAAs must not have properties other than [RSAA] (validateRSAA)'
-  );
-  t.notOk(
-    isValidRSAA(action2),
-    'RSAAs must not have properties other than [RSAA] (isValidRSAA)'
+  t.equal(
+    validateRSAA(action2).length,
+    0,
+    'RSAAs may have properties other than [RSAA] (validateRSAA)'
   );
 
   const action3 = {
@@ -565,6 +566,20 @@ test('validateRSAA/isValidRSAA must identify conformant RSAAs', t => {
     '[RSAA].ok property must be a function (isRSAA)'
   );
 
+  const action30 = {
+    [RSAA]: {
+      endpoint: '',
+      method: 'GET',
+      types: ['REQUEST', 'SUCCESS', 'FAILURE']
+    },
+    [Symbol('action30 Symbol')]: 'foo'
+  };
+  t.equal(
+    validateRSAA(action30).length,
+    0,
+    'RSAAs may have Symbol properties other than [RSAA] (validateRSAA)'
+  );
+
   t.end();
 });