From c0203f13366606fb85afab739dfc77158c34b36c Mon Sep 17 00:00:00 2001
From: Alejandro Celaya <alejandro@alejandrocelaya.com>
Date: Fri, 15 Jun 2018 21:49:25 +0200
Subject: [PATCH] Implemented loading of short URLs

---
 package.json                                  |  4 +-
 src/App.js                                    |  2 +-
 src/api/ShlinkApiClient.js                    | 70 +++++++++++++++++++
 src/common/AsideMenu.js                       |  8 +++
 src/common/AsideMenu.scss                     | 15 ++++
 src/common/MainHeader.js                      |  4 +-
 src/common/MenuLayout.js                      | 21 ++++--
 src/index.js                                  |  5 +-
 src/reducers/index.js                         |  6 +-
 src/reducers/types.js                         |  8 +++
 src/servers/ServersDropdown.js                | 12 ++--
 src/servers/reducers/selectedServer.js        |  7 +-
 src/servers/reducers/server.js                |  8 +--
 src/short-urls/ShortUrlsList.js               | 33 +++++++++
 src/short-urls/reducers/shortUrlsList.js      | 21 ++++++
 .../servers/ServersDropdown.test.js           |  0
 16 files changed, 191 insertions(+), 33 deletions(-)
 create mode 100644 src/api/ShlinkApiClient.js
 create mode 100644 src/common/AsideMenu.js
 create mode 100644 src/common/AsideMenu.scss
 create mode 100644 src/reducers/types.js
 create mode 100644 src/short-urls/ShortUrlsList.js
 create mode 100644 src/short-urls/reducers/shortUrlsList.js
 rename {tests => test}/servers/ServersDropdown.test.js (100%)

diff --git a/package.json b/package.json
index 85dd3c04..91bdad98 100644
--- a/package.json
+++ b/package.json
@@ -8,6 +8,7 @@
     "@fortawesome/fontawesome-free-solid": "^5.0.13",
     "@fortawesome/react-fontawesome": "0.0.19",
     "autoprefixer": "7.1.6",
+    "axios": "^0.18.0",
     "babel-core": "6.26.0",
     "babel-eslint": "7.2.3",
     "babel-jest": "20.0.3",
@@ -46,6 +47,7 @@
     "react-router-dom": "^4.2.2",
     "reactstrap": "^6.0.1",
     "redux": "^4.0.0",
+    "redux-thunk": "^2.3.0",
     "resolve": "1.6.0",
     "style-loader": "0.19.0",
     "sw-precache-webpack-plugin": "0.11.4",
@@ -74,7 +76,7 @@
       "<rootDir>/config/setupEnzyme.js"
     ],
     "testMatch": [
-      "<rootDir>/tests/**/*.test.{js,jsx,mjs}"
+      "<rootDir>/test/**/*.test.{js,jsx,mjs}"
     ],
     "testEnvironment": "node",
     "testURL": "http://localhost",
diff --git a/src/App.js b/src/App.js
index 3e5a7cd3..58c5d3b9 100644
--- a/src/App.js
+++ b/src/App.js
@@ -9,7 +9,7 @@ import CreateServer from './servers/CreateServer';
 export default class App extends React.Component {
   render() {
     return (
-      <div>
+      <div className="container-fluid">
         <MainHeader/>
 
         <div className="app">
diff --git a/src/api/ShlinkApiClient.js b/src/api/ShlinkApiClient.js
new file mode 100644
index 00000000..b435db1a
--- /dev/null
+++ b/src/api/ShlinkApiClient.js
@@ -0,0 +1,70 @@
+import axios from 'axios';
+import { isEmpty } from 'ramda';
+
+export class ShlinkApiClient {
+  constructor(axios) {
+    this.axios = axios;
+    this._baseUrl = '';
+    this._apiKey = '';
+    this._token = '';
+  }
+
+  /**
+   * Sets the base URL to be used on any request
+   * @param {String} baseUrl
+   * @param {String} apiKey
+   */
+  setConfig = ({ url, apiKey }) => {
+    this._baseUrl = url;
+    this._apiKey = apiKey;
+  };
+
+  /**
+   * Returns the list of short URLs
+   * @param params
+   * @returns {Promise<Array>}
+   */
+  listShortUrls = (params = {}) => {
+    const { page = 1 } = params;
+    return this._performRequest(`/rest/short-codes?page=${page}`)
+      .then(resp => resp.data.shortUrls.data)
+      .catch(e => {
+        // If auth failed, reset token to force it to be regenerated, and perform a new request
+        if (e.response.status === 401) {
+          this._token = '';
+          return this.listShortUrls(params);
+        }
+
+        // Otherwise, let caller handle the rejection
+        return Promise.reject(e);
+      });
+  };
+
+  _performRequest = async (url, method = 'GET') => {
+    if (isEmpty(this._token)) {
+      this._token = await this._handleAuth();
+    }
+
+    return await this.axios({
+      method,
+      url: `${this._baseUrl}${url}`,
+      headers: { 'Authorization': `Bearer ${this._token}` }
+    }).then(resp => {
+      // Save new token
+      const { authorization = '' } = resp.headers;
+      this._token = authorization.substr('Bearer '.length);
+      return resp;
+    });
+  };
+
+  _handleAuth = async () => {
+    const resp = await this.axios({
+      method: 'POST',
+      url: `${this._baseUrl}/rest/authenticate`,
+      data: { apiKey: this._apiKey }
+    });
+    return resp.data.token;
+  };
+}
+
+export default new ShlinkApiClient(axios);
diff --git a/src/common/AsideMenu.js b/src/common/AsideMenu.js
new file mode 100644
index 00000000..3f9fb51d
--- /dev/null
+++ b/src/common/AsideMenu.js
@@ -0,0 +1,8 @@
+import React from 'react';
+import './AsideMenu.scss';
+
+export default function AsideMenu() {
+  return (
+    <aside className="aside-menu col-md-2 col-sm-2">Aside menu</aside>
+  );
+}
diff --git a/src/common/AsideMenu.scss b/src/common/AsideMenu.scss
new file mode 100644
index 00000000..77691222
--- /dev/null
+++ b/src/common/AsideMenu.scss
@@ -0,0 +1,15 @@
+@import "../utils/base";
+
+.aside-menu {
+  position: fixed !important;
+  top: $headerHeight;
+  bottom: 0;
+  left: 0;
+  z-index: 1000;
+  display: block;
+  padding: 20px;
+  overflow-x: hidden;
+  overflow-y: auto;
+  background-color: #f7f7f7;
+  border-right: 1px solid #eee;
+}
diff --git a/src/common/MainHeader.js b/src/common/MainHeader.js
index d8e8287b..8b511ac7 100644
--- a/src/common/MainHeader.js
+++ b/src/common/MainHeader.js
@@ -25,14 +25,14 @@ export default class MainHeader extends React.Component {
   render() {
     return (
       <Navbar color="primary" dark fixed="top" className="main-header" expand="md">
-        <NavbarBrand href="https://shlink.io" target="_blank">
+        <NavbarBrand tag={Link} to="/">
           <img src={shlinkLogo} alt="Shlink" className="main-header__brand-logo"/> Shlink
         </NavbarBrand>
         <NavbarToggler onClick={() => this.toggle()}/>
         <Collapse navbar isOpen={this.state.isOpen}>
           <Nav navbar className="ml-auto">
             <NavItem>
-              <NavLink tag={Link} to ="/server/create">
+              <NavLink tag={Link} to="/server/create">
                 <FontAwesomeIcon icon={plusIcon}/>&nbsp; Add server
               </NavLink>
             </NavItem>
diff --git a/src/common/MenuLayout.js b/src/common/MenuLayout.js
index e2e56f10..acadd34b 100644
--- a/src/common/MenuLayout.js
+++ b/src/common/MenuLayout.js
@@ -1,13 +1,20 @@
 import React from 'react';
-import { Switch } from 'react-router-dom';
+import { Route, Switch } from 'react-router-dom';
+import ShortUrlsList from '../short-urls/ShortUrlsList';
+import AsideMenu from './AsideMenu';
 
-export default () => {
+export default function MenuLayout() {
   return (
-    <div>
-      <nav>Left menu</nav>
-      <Switch>
-
-      </Switch>
+    <div className="row">
+      <AsideMenu />
+      <div className="col-md-10 offset-md-2 col-sm-9 offset-sm-3">
+        <Switch>
+          <Route exact
+                 path="/server/:serverId/list-short-urls/:page"
+                 component={ShortUrlsList}
+          />
+        </Switch>
+      </div>
     </div>
   );
 }
diff --git a/src/index.js b/src/index.js
index 092c2ff1..c4d951a4 100644
--- a/src/index.js
+++ b/src/index.js
@@ -4,13 +4,14 @@ import ReactDOM from 'react-dom';
 import { Provider } from 'react-redux';
 import { BrowserRouter } from 'react-router-dom';
 import { applyMiddleware, createStore } from 'redux';
+import ReduxThunk from 'redux-thunk';
+
 import App from './App';
 import './index.scss';
 import reducers from './reducers';
 import registerServiceWorker from './registerServiceWorker';
 
-// const store = createStore(reducers, {}, applyMiddleware());
-const store = createStore(reducers, applyMiddleware());
+const store = createStore(reducers, applyMiddleware(ReduxThunk));
 
 ReactDOM.render(
   <Provider store={store}>
diff --git a/src/reducers/index.js b/src/reducers/index.js
index 07a1f366..ee533601 100644
--- a/src/reducers/index.js
+++ b/src/reducers/index.js
@@ -2,10 +2,10 @@ import { combineReducers } from 'redux';
 
 import serversReducer from '../servers/reducers/server';
 import selectedServerReducer from '../servers/reducers/selectedServer';
+import shortUrlsListReducer from '../short-urls/reducers/shortUrlsList';
 
-const rootReducer = combineReducers({
+export default combineReducers({
   servers: serversReducer,
   selectedServer: selectedServerReducer,
+  shortUrlsList: shortUrlsListReducer,
 });
-
-export default rootReducer;
diff --git a/src/reducers/types.js b/src/reducers/types.js
new file mode 100644
index 00000000..7a3ab9e4
--- /dev/null
+++ b/src/reducers/types.js
@@ -0,0 +1,8 @@
+
+// Servers
+export const LOAD_SERVER = 'shlink/LOAD_SERVER';
+export const FETCH_SERVERS = 'shlink/FETCH_SERVERS';
+export const CREATE_SERVER = 'shlink/CREATE_SERVER';
+
+// Short URLs
+export const LIST_SHORT_URLS = 'shlink/LIST_SHORT_URLS';
diff --git a/src/servers/ServersDropdown.js b/src/servers/ServersDropdown.js
index dbbdd47a..108e7e0a 100644
--- a/src/servers/ServersDropdown.js
+++ b/src/servers/ServersDropdown.js
@@ -1,10 +1,10 @@
+import { isEmpty } from 'ramda';
 import React from 'react';
 import { connect } from 'react-redux';
+import { Link } from 'react-router-dom';
 import { DropdownItem, DropdownMenu, DropdownToggle, UncontrolledDropdown } from 'reactstrap';
-import { isEmpty } from 'ramda';
 
 import { listServers } from './reducers/server';
-import { loadServer } from './reducers/selectedServer';
 
 export class ServersDropdown extends React.Component {
   renderServers = () => {
@@ -16,17 +16,13 @@ export class ServersDropdown extends React.Component {
 
     return Object.values(servers).map(({ name, id }) => (
       <span key={id}>
-        <DropdownItem onClick={() => this.selectServer(id)}>
+        <DropdownItem tag={Link} to={`/server/${id}/list-short-urls/1`}>
           {name}
         </DropdownItem>
       </span>
     ));
   };
 
-  selectServer = serverId => {
-    this.props.loadServer(serverId);
-  };
-
   componentDidMount() {
     this.props.listServers();
   }
@@ -45,4 +41,4 @@ const mapStateToProps = state => ({
   servers: state.servers
 });
 
-export default connect(mapStateToProps, { listServers, loadServer })(ServersDropdown);
+export default connect(mapStateToProps, { listServers })(ServersDropdown);
diff --git a/src/servers/reducers/selectedServer.js b/src/servers/reducers/selectedServer.js
index 08c8d53e..8b637049 100644
--- a/src/servers/reducers/selectedServer.js
+++ b/src/servers/reducers/selectedServer.js
@@ -1,14 +1,13 @@
 import ServersService from '../services';
-
-const LOAD_SERVER = 'shlink/LOAD_SERVER';
+import { LOAD_SERVER } from '../../reducers/types';
 
 export default function selectedServerReducer(state = null, action) {
   switch (action.type) {
     case LOAD_SERVER:
       return action.selectedServer;
+    default:
+      return state;
   }
-
-  return state;
 }
 
 export const loadServer = serverId => {
diff --git a/src/servers/reducers/server.js b/src/servers/reducers/server.js
index 56f3c573..79a74430 100644
--- a/src/servers/reducers/server.js
+++ b/src/servers/reducers/server.js
@@ -1,7 +1,5 @@
 import ServersService from '../services';
-
-const FETCH_SERVERS = 'shlink/FETCH_SERVERS';
-const CREATE_SERVER = 'shlink/CREATE_SERVER';
+import { FETCH_SERVERS, CREATE_SERVER } from '../../reducers/types';
 
 export default function serversReducer(state = {}, action) {
   switch (action.type) {
@@ -9,9 +7,9 @@ export default function serversReducer(state = {}, action) {
       return action.servers;
     case CREATE_SERVER:
       return [ ...state, action.server ];
+    default:
+      return state;
   }
-
-  return state;
 }
 
 export const listServers = () => {
diff --git a/src/short-urls/ShortUrlsList.js b/src/short-urls/ShortUrlsList.js
new file mode 100644
index 00000000..3a4f3d19
--- /dev/null
+++ b/src/short-urls/ShortUrlsList.js
@@ -0,0 +1,33 @@
+import React from 'react';
+import { connect } from 'react-redux';
+import { listShortUrls } from './reducers/shortUrlsList';
+
+export class ShortUrlsList extends React.Component {
+  componentDidMount() {
+    const { match } = this.props;
+    this.props.listShortUrls(match.params.serverId);
+  }
+
+  render() {
+    return (
+      <ul>
+        {this.renderShortUrls()}
+      </ul>
+    );
+  }
+
+  renderShortUrls() {
+    const { shortUrlsList } = this.props;
+    if (! shortUrlsList) {
+      return '<li><i>Loading...</i></li>';
+    }
+
+    return shortUrlsList.map(shortUrl => (
+      <li key={shortUrl.shortCode}>{`${shortUrl.shortCode}`}</li>
+    ));
+  }
+}
+
+export default connect(state => ({
+  shortUrlsList: state.shortUrlsList
+}), { listShortUrls })(ShortUrlsList);
diff --git a/src/short-urls/reducers/shortUrlsList.js b/src/short-urls/reducers/shortUrlsList.js
new file mode 100644
index 00000000..71bbbc36
--- /dev/null
+++ b/src/short-urls/reducers/shortUrlsList.js
@@ -0,0 +1,21 @@
+import { LIST_SHORT_URLS } from '../../reducers/types';
+import ServersService from '../../servers/services';
+import ShlinkApiClient from '../../api/ShlinkApiClient';
+
+export default function shortUrlsListReducer(state = [], action) {
+  switch (action.type) {
+    case LIST_SHORT_URLS:
+      return action.shortUrls;
+    default:
+      return state;
+  }
+}
+
+export const listShortUrls = (serverId) => {
+  return async dispatch => {
+    const selectedServer = ServersService.findServerById(serverId);
+
+    ShlinkApiClient.setConfig(selectedServer);
+    dispatch({ type: LIST_SHORT_URLS, shortUrls: await ShlinkApiClient.listShortUrls() });
+  };
+};
diff --git a/tests/servers/ServersDropdown.test.js b/test/servers/ServersDropdown.test.js
similarity index 100%
rename from tests/servers/ServersDropdown.test.js
rename to test/servers/ServersDropdown.test.js