Initial commit

Author: jjjimenez100

.gitignore

@@ -0,0 +1,64 @@
+# Intellij based ides
+.idea/
+
+# Logs
+logs
+*.log
+npm-debug.log*
+yarn-debug.log*
+yarn-error.log*
+
+# Runtime data
+pids
+*.pid
+*.seed
+*.pid.lock
+
+# Directory for instrumented libs generated by jscoverage/JSCover
+lib-cov
+
+# Coverage directory used by tools like istanbul
+coverage
+
+# nyc test coverage
+.nyc_output
+
+# Grunt intermediate storage (http://gruntjs.com/creating-plugins#storing-task-files)
+.grunt
+
+# Bower dependency directory (https://bower.io/)
+bower_components
+
+# node-waf configuration
+.lock-wscript
+
+# Compiled binary addons (https://nodejs.org/api/addons.html)
+build/Release
+
+# Dependency directories
+node_modules/
+jspm_packages/
+
+# TypeScript v1 declaration files
+typings/
+
+# Optional npm cache directory
+.npm
+
+# Optional eslint cache
+.eslintcache
+
+# Optional REPL history
+.node_repl_history
+
+# Output of 'npm pack'
+*.tgz
+
+# Yarn Integrity file
+.yarn-integrity
+
+# dotenv environment variables file
+.env
+
+# next.js build output
+.next

Readme.md

@@ -0,0 +1,102 @@
+# Xendit Coding Exercise
+
+The goal of these exercises are to assess your proficiency in software engineering that is related to the daily work that we do at Xendit. Please follow the instructions below to complete the assessment.
+
+## Setup
+
+1. Create a new repository in your own github profile named `backend-coding-test` and commit the contents of this folder
+2. Ensure `node (>8.6 and <= 10)` and `npm` are installed
+3. Run `npm install`
+4. Run `npm test`
+5. Run `npm start`
+6. Hit the server to test health `curl localhost:8010/health` and expect a `200` response 
+
+## Tasks
+
+Below will be your set of tasks to accomplish. Please work on each of these tasks in order. Success criteria will be defined clearly for each task
+
+1. [Documentation](#documentation)
+2. [Implement Tooling](#implement-tooling)
+3. [Implement Pagination](#implement-pagination)
+4. [Refactoring](#refactoring)
+5. [Security](#security)
+6. [Load Testing](#load-testing)
+
+### Documentation
+
+Please deliver documentation of the server that clearly explains the goals of this project and clarifies the API response that is expected.
+
+#### Success Criteria
+
+1. A pull request against `master` of your fork with a clear description of the change and purpose and merge it
+3. **[BONUS]** Create an easy way to deploy and view the documentation in a web format and include instructions to do so
+
+### Implement Tooling
+
+Please implement the following tooling:
+
+1. `eslint` - for linting
+2. `nyc` - for code coverage
+3. `pre-push` - for git pre push hook running tests
+4. `winston` - for logging
+
+#### Success Criteria
+
+1. Create a pull request against `master` of your fork with the new tooling and merge it
+    1. `eslint` should have an opinionated format
+    2. `nyc` should aim for test coverage of `80%` across lines, statements, and branches
+    3. `pre-push` should run the tests before allowing pushing using `git`
+    4. `winston` should be used to replace console logs and all errors should be logged as well. Logs should go to disk.
+2. Ensure that tooling is connected to `npm test`
+3. Create a separate pull request against `master` of your fork with the linter fixes and merge it
+4. Create a separate pull request against `master` of your fork to increase code coverage to acceptable thresholds and merge it
+5. **[BONUS]** Add integration to CI such as Travis or Circle
+6. **[BONUS]** Add Typescript support
+
+### Implement Pagination
+
+Please implement pagination to retrieve pages of the resource `rides`.
+
+1. Create a pull request against `master` with your changes to the `GET /rides` endpoint to support pagination including:
+    1. Code changes
+    2. Tests
+    3. Documentation
+2. Merge the pull request
+
+### Refactoring
+
+Please implement the following refactors of the code:
+
+1. Convert callback style code to use `async/await`
+2. Reduce complexity at top level control flow logic and move logic down and test independently
+3. **[BONUS]** Split between functional and imperative function and test independently
+
+#### Success Criteria
+
+1. A pull request against `master` of your fork for each of the refactors above with:
+    1. Code changes
+    2. Tests
+
+### Security
+
+Please implement the following security controls for your system:
+
+1. Ensure the system is not vulnerable to [SQL injection](https://www.owasp.org/index.php/SQL_Injection)
+2. **[BONUS]** Implement an additional security improvement of your choice
+
+#### Success Criteria
+
+1. A pull request against `master` of your fork with:
+    1. Changes to the code
+    2. Tests ensuring the vulnerability is addressed
+
+### Load Testing
+
+Please implement load testing to ensure your service can handle a high amount of traffic
+
+#### Success Criteria
+
+1. Implement load testing using `artillery`
+    1. Create a PR against `master` of your fork including artillery
+    2. Ensure that load testing is able to be run using `npm test:load`. You can consider using a tool like `forever` to spin up a daemon and kill it after the load test has completed.
+    3. Test all endpoints under at least `100 rps` for `30s` and ensure that `p99` is under `50ms`

index.js

@@ -0,0 +1,21 @@
+'use strict';
+
+const express = require('express');
+const app = express();
+const port = 8010;
+
+const bodyParser = require('body-parser');
+const jsonParser = bodyParser.json();
+
+const sqlite3 = require('sqlite3').verbose();
+const db = new sqlite3.Database(':memory:');
+
+const buildSchemas = require('./src/schemas');
+
+db.serialize(() => {
+    buildSchemas(db);
+
+    const app = require('./src/app')(db);
+
+    app.listen(port, () => console.log(`App started and listening on port ${port}`));
+});