billing invoices and history linter fixes

Author: na3d

src/api/configs/invoices.js

@@ -11,8 +11,8 @@ export const config = genConfig({
       singular: 'items',
       endpoint: id => `/account/invoices/${id}/items`,
       supports: [ONE],
-      },
     },
+  },
 });
 
 export const actions = genActions(config);

src/billing/layouts/DashboardPage.js

@@ -85,7 +85,7 @@ export class DashboardPage extends Component {
             </FormGroup>
             <FormGroup className="row">
               <div className="col-sm-12 text-right">
-                <Link to={`/billing/history`}>Billing History</Link>
+                <Link to="/billing/history">Billing History</Link>
               </div>
             </FormGroup>
             <h3 className="sub-header">Account Balance</h3>

src/billing/layouts/HistoryPage.js

@@ -5,9 +5,7 @@ import moment from 'moment-timezone';
 import { getStorage } from '~/storage';
 import { Card, CardHeader } from 'linode-components/cards';
 import { Table } from 'linode-components/tables';
-import { ListBody } from 'linode-components/lists/bodies';
-import { LinkCell, ButtonCell } from 'linode-components/tables/cells';
-import { account, invoices } from '~/api';
+import { LinkCell } from 'linode-components/tables/cells';
 
 import { setSource } from '~/actions/source';
 
@@ -50,7 +48,7 @@ export class HistoryPage extends Component {
             />
             <div className="row">
               <div className="col-sm-12 text-right">
-              <strong>Current Balance: ${account.balance}</strong>
+                <strong>Current Balance: ${account.balance}</strong>
               </div>
             </div>
           </Card>

src/billing/layouts/InvoicePage.js

@@ -5,10 +5,6 @@ import moment from 'moment-timezone';
 import { getStorage } from '~/storage';
 import { Card, CardHeader } from 'linode-components/cards';
 import { Table } from 'linode-components/tables';
-import { List } from 'linode-components/lists';
-import { ListBody } from 'linode-components/lists/bodies';
-import { LinkCell, ButtonCell } from 'linode-components/tables/cells';
-import { objectFromMapByLabel } from '~/api/util';
 import { invoices } from '~/api';
 
 import { setSource } from '~/actions/source';
@@ -40,7 +36,7 @@ export class InvoicePage extends Component {
                   label: 'From',
                   headerClassName: 'DateColumn',
                   formatFn: (from) => {
-                    if(!from) {
+                    if (!from) {
                       return;
                     }
                     const time = moment.utc(from, moment.iso_8601).tz(timezone);
@@ -52,7 +48,7 @@ export class InvoicePage extends Component {
                   label: 'To',
                   headerClassName: 'DateColumn',
                   formatFn: (to) => {
-                    if(!to) {
+                    if (!to) {
                       return;
                     }
                     const time = moment.utc(to, moment.iso_8601).tz(timezone);
@@ -71,7 +67,7 @@ export class InvoicePage extends Component {
                   headerClassName: 'IntegerColumn text-right',
                   className: 'text-right',
                   formatFn: (unitPrice) => {
-                    if(!unitPrice) {
+                    if (!unitPrice) {
                       return;
                     }
                     return `$${parseFloat(unitPrice).toFixed(4)}`;
@@ -92,7 +88,7 @@ export class InvoicePage extends Component {
             />
             <div className="row">
               <div className="col-sm-12 text-right">
-              <strong>Invoice Total: ${invoice.total.toFixed(2)}</strong>
+                <strong>Invoice Total: ${invoice.total.toFixed(2)}</strong>
               </div>
             </div>
           </Card>
@@ -105,6 +101,7 @@ export class InvoicePage extends Component {
 InvoicePage.propTypes = {
   dispatch: PropTypes.func,
   invoice: PropTypes.object.isRequired,
+  items: PropTypes.object.isRequired,
 };
 
 function select(state, ownProps) {

test/billing/layouts/HistoryPage.spec.js

@@ -31,7 +31,7 @@ describe('billing/layouts/HistoryPage', () => {
     const rowCount = Object.keys(invoices.invoices).length;
     expect(page.find('.TableRow').length).to.equal(rowCount);
   });
-  
+
   it('renders account balance', () => {
     const page = mount(
       <HistoryPage

test/billing/layouts/InvoicePage.spec.js

@@ -2,7 +2,6 @@ import React from 'react';
 import sinon from 'sinon';
 import { mount } from 'enzyme';
 import { expect } from 'chai';
-import moment from 'moment-timezone';
 
 import { InvoicePage } from '~/billing/layouts/InvoicePage';
 import { api } from '@/data';

test/data/invoices.js

@@ -17,28 +17,28 @@ export const apiTestInvoice = {
     data: [
       testItem,
       {
-      ...testItem,
-      label: 'Linode 1024 - ln (1234)',
+        ...testItem,
+        label: 'Linode 1024 - ln (1234)',
       },
       {
-      ...testItem,
-      label: 'Storage Volume - sv (123)',
+        ...testItem,
+        label: 'Storage Volume - sv (123)',
       },
       {
-      ...testItem,
-      label: 'Backup Service - Linode 1024 - ln (1234)',
+        ...testItem,
+        label: 'Backup Service - Linode 1024 - ln (1234)',
       },
       {
-      ...testItem,
-      label: 'Nodebalancer - nb (432)',
+        ...testItem,
+        label: 'Nodebalancer - nb (432)',
       },
       {
-      ...testItem,
-      label: 'Linode 1024 - ln (1235)',
+        ...testItem,
+        label: 'Linode 1024 - ln (1235)',
       },
       {
-      ...testItem,
-      label: 'Linode 1024 - ln (1236)',
+        ...testItem,
+        label: 'Linode 1024 - ln (1236)',
       },
     ],
   },