Merge pull request #420 from lightninglabs/cli-logs

Cli logs

Author: tanx

src/action/log.js

@@ -15,6 +15,9 @@ class LogAction {
     _ipcRenderer = ipcRenderer;
     _ipcRenderer.on('logs', (event, arg) => {
       store.logs.push(arg);
+      if (store.logs.length > 100) {
+        store.logs.splice(0, store.logs.length - 100);
+      }
     });
     _ipcRenderer.send('logs-ready', true);
   }

src/action/nav.js

@@ -113,7 +113,7 @@ class NavAction {
   }
 
   goCLI() {
-    // this._store.route = 'CLI';
+    this._store.route = 'CLI';
   }
 
   goCreateChannel() {

src/component/list.js

@@ -53,9 +53,15 @@ export class List extends Component {
   render() {
     return (
       <ListView
+        ref={component => (this.list = component)}
         dataSource={this.dataSource}
         renderHeader={this.props.renderHeader}
-        renderRow={this.props.renderItem}
+        renderRow={data => {
+          this.props.scrollToEnd
+            ? setTimeout(() => this.list._scrollViewRef.scrollToEnd(), 50)
+            : null;
+          return this.props.renderItem(data);
+        }}
         enableEmptySections={true}
       />
     );
@@ -66,6 +72,7 @@ List.propTypes = {
   data: PropTypes.array.isRequired,
   renderHeader: PropTypes.func,
   renderItem: PropTypes.func.isRequired,
+  scrollToEnd: PropTypes.bool,
 };
 
 //

src/component/style.js

@@ -30,6 +30,7 @@ export const color = {
   notifyLight: 'rgba(255,255,255,0.5)',
   seedBorder: 'rgba(151,151,151,0.26)',
   seedBackground: 'rgba(130,129,129,0.09)',
+  cliBackground: '#171D2C',
 };
 
 export const font = {

src/view/cli.js

@@ -0,0 +1,53 @@
+import React from 'react';
+import { StyleSheet } from 'react-native';
+import { observer } from 'mobx-react';
+import PropTypes from 'prop-types';
+import Background from '../component/background';
+import { Header, Title } from '../component/header';
+import Text from '../component/text';
+import { List, ListContent } from '../component/list';
+import { Button, BackButton } from '../component/button';
+import { color, font } from '../component/style';
+
+//
+// CLI View
+//
+
+const CLIView = ({ store, nav }) => (
+  <Background color={color.blackDark}>
+    <Header separator>
+      <BackButton onPress={() => nav.goSettings()} />
+      <Title title="Logs" />
+      <Button disabled onPress={() => {}} />
+    </Header>
+    <Background color={color.cliBackground}>
+      <ListContent>
+        <List
+          data={store.logs.slice()}
+          renderItem={text => <LogItem text={text} />}
+          scrollToEnd={true}
+        />
+      </ListContent>
+    </Background>
+  </Background>
+);
+
+CLIView.propTypes = {
+  store: PropTypes.object.isRequired,
+  nav: PropTypes.object.isRequired,
+};
+
+const iStyles = StyleSheet.create({
+  text: {
+    textAlign: 'left',
+    fontSize: font.sizeS,
+  },
+});
+
+const LogItem = ({ text }) => <Text style={iStyles.text}>{text}</Text>;
+
+LogItem.propTypes = {
+  text: PropTypes.string,
+};
+
+export default observer(CLIView);

src/view/main.js

@@ -30,6 +30,7 @@ import Setting from './setting';
 import SettingUnit from './setting-unit';
 import SettingFiat from './setting-fiat';
 import Notification from './notification';
+import CLI from './cli';
 import TransactionDetail from './transaction-detail';
 import {
   nav,
@@ -85,6 +86,7 @@ class MainView extends Component {
           <SettingFiat store={store} nav={nav} setting={setting} />
         )}
         {route === 'Notifications' && <Notification store={store} nav={nav} />}
+        {route === 'CLI' && <CLI store={store} nav={nav} />}
         {route === 'Pay' && (
           <Payment store={store} payment={payment} nav={nav} />
         )}

src/view/setting.js

@@ -56,7 +56,7 @@ const SettingView = ({ store, nav }) => {
           arrow
         />
         <SettingHeader name="ADVANCED" style={styles.advanced} />
-        <SettingItem name="CLI" onSelect={() => nav.goCLI()} arrow />
+        <SettingItem name="Logs" onSelect={() => nav.goCLI()} arrow />
       </MainContent>
     </Background>
   );

stories/screen.js

@@ -19,6 +19,7 @@ import Transaction from '../src/view/transaction';
 import Setting from '../src/view/setting';
 import SettingUnit from '../src/view/setting-unit';
 import SettingFiat from '../src/view/setting-fiat';
+import CLI from '../src/view/cli';
 import Notification from '../src/view/notification';
 import TransactionDetail from '../src/view/transaction-detail';
 import Channel from '../src/view/channel';
@@ -111,6 +112,7 @@ storiesOf('Screens', module)
     <SettingFiat store={store} nav={nav} setting={setting} />
   ))
   .add('Notifications', () => <Notification store={store} nav={nav} />)
+  .add('CLI', () => <CLI store={store} nav={nav} />)
   .add('Transactions', () => (
     <Transaction store={store} transaction={transaction} nav={nav} />
   ))
@@ -244,3 +246,27 @@ store.seedMnemonic = [
   'quit',
   'cashew',
 ];
+store.logs = [
+  '[14:00:24.995] [info] Using lnd in path lnd',
+  'Checking for update',
+  '[14:00:25.047] [info] lnd: 2018-06-28 14:00:25.039 [WRN] LTND: open /home/valentine/.config/lightning-app/lnd/lnd.conf: no such file or directory',
+  '2018-06-28 14:00:25.039 [INF] LTND: Version 0.4.2-beta commit=884c51dfdc85284ba8d063c4547d2b5665eba010',
+  '2018-06-28 14:00:25.039 [INF] LTND: Active chain: Bitcoin (network=testnet)',
+  '2018-06-28 14:00:25.039 [INF] CHDB: Checking for schema update: latest_version=1, db_version=1',
+  '[14:00:25.170] [info] lnd: 2018-06-28 14:00:25.055 [INF] RPCS: password RPC server listening on 127.0.0.1:10009',
+  '2018-06-28 14:00:25.055 [INF] RPCS: password gRPC proxy started at 127.0.0.1:8080',
+  '2018-06-28 14:00:25.055 [INF] LTND: Waiting for wallet encryption password. Use `lncli create` to create a wallet, `lncli unlock` to unlock an existing wallet, or `lncli changepassword` to change the password of an existing wallet and unlock it.',
+  '[14:00:25.541] [info] Loaded initial state',
+  '[14:00:25.557] [info] GRPC unlockerReady',
+  'Found version 0.2.0-prealpha.9 (url: Lightning-linux-x86_64v0.2.0-prealpha.9.AppImage)',
+  'Downloading update from Lightning-linux-x86_64v0.2.0-prealpha.9.AppImage',
+  'No cached update available',
+  'File has 2893 changed blocks',
+  'Full: 106,265.24 KB, To download: 59,575.39 KB (56%)',
+  'Differential download: https://github.com/lightninglabs/lightning-app/releases/download/v0.2.0-prealpha.9/Lightning-linux-x86_64v0.2.0-prealpha.9.AppImage',
+  'Redirect to https://github-production-release-asset-2e65be.s3.amazonaws.com/76898197/428914b4-7561-11e8-8826-08fde1bd29aa',
+  '[14:00:33.730] [info] lnd: 2018-06-28 14:00:33.730 [INF] LNWL: Opened wallet',
+  '[14:00:33.731] [info] lnd: 2018-06-28 14:00:33.730 [INF] LTND: Primary chain is set to: bitcoin',
+  '[14:00:33.879] [info] lnd: 2018-06-28 14:00:33.879 [INF] BTCN: Loaded 1032 addresses from file /home/valentine/.config/lightning-app/lnd/data/chain/bitcoin/testnet/peers.json',
+  '[14:00:33.893] [info] lnd: 2018-06-28 14:00:33.892 [INF] CMGR: DNS discovery failed on seed x49.seed.tbtc.petertodd.org: lookup x49.seed.tbtc.petertodd.org: No address associated with hostname',
+];

test/unit/action/log.spec.js

@@ -1,4 +1,5 @@
 import { Store } from '../../../src/store';
+import { EventEmitter } from 'events';
 import * as log from '../../../src/action/log';
 import LogAction from '../../../src/action/log';
 
@@ -38,6 +39,28 @@ describe('Action Logs Unit Tests', () => {
     });
   });
 
+  describe('constructor', () => {
+    let ipcRendererStub;
+
+    beforeEach(() => {
+      store = new Store();
+      ipcRendererStub = new EventEmitter();
+      ipcRendererStub.send = sinon.stub();
+      new LogAction(store, ipcRendererStub);
+    });
+
+    describe('constructor()', () => {
+      it('should keep logs trimmed to 100 and keep the tail of the logs', () => {
+        for (var i = 0; i < 101; i++) {
+          ipcRendererStub.emit('logs', 'some-event', i.toString());
+        }
+        expect(store.logs.length, 'to equal', 100);
+        expect(store.logs[0], 'to equal', '1');
+        expect(store.logs[99], 'to equal', '100');
+      });
+    });
+  });
+
   describe('with constructor', () => {
     beforeEach(() => {
       store = new Store();

test/unit/action/nav.spec.js

@@ -200,7 +200,7 @@ describe('Action Nav Unit Tests', () => {
     });
   });
 
-  describe.skip('goCLI()', () => {
+  describe('goCLI()', () => {
     it('should set correct route', () => {
       nav.goCLI();
       expect(store.route, 'to equal', 'CLI');