Filter the setting list

Author: otroboe

Diff for: README.md

@@ -1,5 +1,7 @@
 # Node JS Exercise
 
+Showing a list of settings on the `/settings` route.
+
 ## Requirements
 
 ### Global
@@ -24,6 +26,12 @@ yarn
 ```bash
 yarn start
 ```
+You can display the settings on http://localhost:8042/settings.
+
+* Check ESLint
+```bash
+yarn lint
+```
 
 * Build for production
 ```bash

Diff for: package.json

@@ -34,6 +34,7 @@
       "es6": true
     },
     "rules": {
+      "arrow-parens": ["error", "as-needed"],
       "no-console": "off"
     }
   }

Diff for: src/modules/SettingsRouter.js

@@ -19,16 +19,43 @@ class SettingsRouter {
    * @returns {Promise}
    */
   async init() {
-    const settingsData = await parseFile(this.settingsFilePath);
+    const settingsList = await parseFile(this.settingsFilePath);
     const componentList = await parseFile(this.componentsFilePath);
+    const filteredList = SettingsRouter.filterSettingsList(settingsList, componentList);
 
     this.expressRouter.get('/settings', (req, res) => {
-      res.send(settingsData);
+      res.send(filteredList);
       res.end();
     });
 
     return this.expressRouter;
   }
+
+  /**
+   * @param {Array} settingsList
+   * @param {Array} componentList
+   * @returns {Array}
+   */
+  static filterSettingsList(settingsList, componentList) {
+    // Put all the component name in an array of string (instead of objects)
+    const componentNameList = componentList.map(item => item.name);
+
+    let settingRequires;
+
+    return settingsList.filter(settingsItem => {
+      ({ requires: settingRequires } = settingsItem);
+
+      if (!Array.isArray(settingRequires)) {
+        return false;
+      }
+
+      if (settingRequires.length === 0) {
+        return true;
+      }
+
+      return settingRequires.some(requiredComponent => componentNameList.includes(requiredComponent));
+    });
+  }
 }
 
 export default SettingsRouter;