style: format

packages/cloudapi-gql/src/schema/queries/users.js

@@ -16,7 +16,9 @@ module.exports = {
     const { list, get } = api.users;
 
     return args.id
-      ? get(args).then(user => [user]).then(user =>
+      ? get(args)
+          .then(user => [user])
+          .then(user =>
             Object.assign(user, {
               isUser: true
             })

packages/cp-frontend/src/app.js

@@ -17,9 +17,7 @@ class App extends Component {
   render() {
     return (
       <ApolloProvider client={client} store={store}>
-        <ThemeProvider theme={theme}>
+        <ThemeProvider theme={theme}>{Router}</ThemeProvider>
-          {Router}
-        </ThemeProvider>
       </ApolloProvider>
     );
   }

packages/cp-frontend/src/components/deployment-group/delete.js

@@ -6,7 +6,7 @@ const DeploymentGroupDelete = ({
   deploymentGroup,
   onCancelClick = () => {},
   onConfirmClick = () => {}
-}) =>
+}) => (
   <div>
     <ModalHeading>
       Deleting a deployment group: <br /> {deploymentGroup.name}
@@ -20,7 +20,8 @@ const DeploymentGroupDelete = ({
       Cancel
     </Button>
     <Button onClick={onConfirmClick}>Delete deployment group</Button>
-  </div>;
+  </div>
+);
 
 DeploymentGroupDelete.propTypes = {
   deploymentGroup: PropTypes.object.isRequired,

packages/cp-frontend/src/components/instances/empty.js

@@ -3,9 +3,10 @@ import React from 'react';
 import { Col, Row } from 'react-styled-flexboxgrid';
 import { P } from 'joyent-ui-toolkit';
 
-export default () =>
+export default () => (
   <Row>
     <Col xs={12}>
       <P>You don't have any instances</P>
     </Col>
-  </Row>;
+  </Row>
+);

packages/cp-frontend/src/components/instances/list-item.js

@@ -65,7 +65,16 @@ const StyledCard = Card.extend`
 
   background-color: ${props => props.theme.white};
 
-  ${isOr('stopping', 'stopped', 'offline', 'destroyed', 'failed', 'deleted', 'incomplete', 'unknown')`
+  ${isOr(
+    'stopping',
+    'stopped',
+    'offline',
+    'destroyed',
+    'failed',
+    'deleted',
+    'incomplete',
+    'unknown'
+  )`
     background-color: ${props => props.theme.background};
 
     & [name="card-options"] > button {
@@ -80,7 +89,6 @@ const InstanceCard = ({
   onStatusMouseOver = () => {},
   onMouseOut = () => {}
 }) => {
-
   const statusProps = STATUSES.reduce(
     (acc, name) =>
       Object.assign(acc, {
@@ -92,42 +100,34 @@ const InstanceCard = ({
   const label = (instance.healthy || 'UNKNOWN').toLowerCase();
   const icon = <HealthyIcon healthy={instance.healthy} />;
 
-  const handleHealthMouseOver = (evt) => {
+  const handleHealthMouseOver = evt => {
     onHealthMouseOver(evt, instance);
-  }
+  };
 
-  const handleStatusMouseOver = (evt) => {
+  const handleStatusMouseOver = evt => {
     onStatusMouseOver(evt, instance);
-  }
+  };
 
-  const handleMouseOut = (evt) => {
+  const handleMouseOut = evt => {
     onMouseOut(evt);
-  }
+  };
 
   return (
     <StyledCard collapsed={true} key={instance.uuid} {...statusProps}>
       <CardView>
-        <CardTitle>
+        <CardTitle>{instance.name}</CardTitle>
-          {instance.name}
-        </CardTitle>
         <CardDescription>
-          <div
+          <div onMouseOver={handleHealthMouseOver} onMouseOut={handleMouseOut}>
-            onMouseOver={handleHealthMouseOver}
-            onMouseOut={handleMouseOut}
-          >
             <CardInfo
               icon={icon}
-              iconPosition='left'
+              iconPosition="left"
               label={label}
-              color='dark'
+              color="dark"
             />
           </div>
         </CardDescription>
         <CardDescription>
-          <div
+          <div onMouseOver={handleStatusMouseOver} onMouseOut={handleMouseOut}>
-            onMouseOver={handleStatusMouseOver}
-            onMouseOut={handleMouseOut}
-          >
             <Label>
               <Dot {...statusProps} />
               {titleCase(instance.status)}
@@ -136,7 +136,7 @@ const InstanceCard = ({
         </CardDescription>
       </CardView>
     </StyledCard>
-  )
+  );
 };
 
 InstanceCard.propTypes = {

packages/cp-frontend/src/components/manifest/edit-or-create.js

@@ -143,31 +143,29 @@ class ManifestEditorBundle extends Component {
     const { ManifestEditor } = this.state;
     const { children, ...rest } = this.props;
 
-    return (
+    return <ManifestEditor {...rest}>{children}</ManifestEditor>;
-      <ManifestEditor {...rest}>
-        {children}
-      </ManifestEditor>
-    );
   }
 }
 
-const MEditor = ({ input, defaultValue, readOnly }) =>
+const MEditor = ({ input, defaultValue, readOnly }) => (
   <ManifestEditorBundle
     mode="yaml"
     {...input}
     value={input.value || defaultValue}
     readOnly={readOnly}
-  />;
+  />
+);
 
-const EEditor = ({ input, defaultValue, readOnly }) =>
+const EEditor = ({ input, defaultValue, readOnly }) => (
   <ManifestEditorBundle
     mode="ini"
     {...input}
     value={input.value || defaultValue}
     readOnly={readOnly}
-  />;
+  />
+);
 
-export const Name = ({ handleSubmit, onCancel, dirty }) =>
+export const Name = ({ handleSubmit, onCancel, dirty }) => (
   <form onSubmit={handleSubmit}>
     <Row>
       <Col xs={12} md={3} lg={3}>
@@ -185,7 +183,8 @@ export const Name = ({ handleSubmit, onCancel, dirty }) =>
         Next
       </Button>
     </ButtonsRow>
-  </form>;
+  </form>
+);
 
 export const Manifest = ({
   handleSubmit,
@@ -193,7 +192,7 @@ export const Manifest = ({
   dirty,
   defaultValue = '',
   loading
-}) =>
+}) => (
   <form onSubmit={handleSubmit}>
     <Field name="manifest" defaultValue={defaultValue} component={MEditor} />
     <ButtonsRow>
@@ -208,30 +207,31 @@ export const Manifest = ({
         Environment
       </Button>
     </ButtonsRow>
-  </form>;
+  </form>
+);
 
 const File = ({ id, name, value, onRemoveFile, readOnly }) => {
-  const removeButton = !readOnly
+  const removeButton = !readOnly ? (
-    ? <FilenameRemove type="button" onClick={onRemoveFile} secondary>
+    <FilenameRemove type="button" onClick={onRemoveFile} secondary>
       Remove
     </FilenameRemove>
-    : null;
+  ) : null;
 
-  const fileEditor = !readOnly
+  const fileEditor = !readOnly ? (
-    ? <Field
+    <Field name={`file-value-${id}`} defaultValue={value} component={EEditor} />
-        name={`file-value-${id}`}
+  ) : (
-        defaultValue={value}
+    <EEditor input={{ value }} readOnly />
-        component={EEditor}
+  );
-      />
-    : <EEditor input={{ value }} readOnly />;
 
-  const input = !readOnly
+  const input = !readOnly ? (
-    ? <FilenameInput type="text" placeholder="Filename including extension…" />
+    <FilenameInput type="text" placeholder="Filename including extension…" />
-    : <FilenameInput
+  ) : (
+    <FilenameInput
       type="text"
       placeholder="Filename including extension…"
       value={name}
-      />;
+    />
+  );
 
   return (
     <FileCard>
@@ -247,15 +247,15 @@ const File = ({ id, name, value, onRemoveFile, readOnly }) => {
 };
 
 const Files = ({ files, onAddFile, onRemoveFile, readOnly }) => {
-  const footer = !readOnly
+  const footer = !readOnly ? (
-    ? <Button type="button" onClick={onAddFile} secondary>
+    <Button type="button" onClick={onAddFile} secondary>
       Create new .env file
     </Button>
-    : null;
+  ) : null;
 
   return (
     <div>
-      {files.map(({ id, ...rest }) =>
+      {files.map(({ id, ...rest }) => (
         <File
           key={id}
           id={id}
@@ -263,7 +263,7 @@ const Files = ({ files, onAddFile, onRemoveFile, readOnly }) => {
           readOnly={readOnly}
           {...rest}
         />
-      )}
+      ))}
       {footer}
     </div>
   );
@@ -280,18 +280,16 @@ export const Environment = ({
   readOnly = false,
   loading
 }) => {
-  const envEditor = !readOnly
+  const envEditor = !readOnly ? (
-    ? <Field
+    <Field name="environment" defaultValue={defaultValue} component={EEditor} />
-        name="environment"
+  ) : (
-        defaultValue={defaultValue}
+    <EEditor input={{ value: defaultValue }} readOnly />
-        component={EEditor}
+  );
-      />
-    : <EEditor input={{ value: defaultValue }} readOnly />;
 
   const footerDivider = !readOnly ? <EnvironmentDivider /> : null;
 
-  const footer = !readOnly
+  const footer = !readOnly ? (
-    ? <ButtonsRow>
+    <ButtonsRow>
       <Button type="button" onClick={onCancel} secondary>
         Cancel
       </Button>
@@ -303,7 +301,7 @@ export const Environment = ({
         Continue
       </Button>
     </ButtonsRow>
-    : null;
+  ) : null;
 
   return (
     <form onSubmit={handleSubmit}>
@@ -348,18 +346,18 @@ export const Review = ({
   environmentToggles,
   ...state
 }) => {
-  const serviceList = forceArray(state.services).map(({ name, config }) =>
+  const serviceList = forceArray(state.services).map(({ name, config }) => (
     <ServiceCard key={name}>
-      <ServiceName>
+      <ServiceName>{name}</ServiceName>
-        {name}
-      </ServiceName>
       <Dl>
         <dt>
           <ImageTitle>Image:</ImageTitle> <Image>{config.image}</Image>
         </dt>
       </Dl>
-      {config.environment && config.environment.length ? <ServiceDivider /> : null}
+      {config.environment && config.environment.length ? (
-      {config.environment && config.environment.length ?
+        <ServiceDivider />
+      ) : null}
+      {config.environment && config.environment.length ? (
         <ServiceEnvironmentTitle
           expanded={environmentToggles[name]}
           onClick={() => onEnvironmentToggle(name)}
@@ -370,12 +368,14 @@ export const Review = ({
             up={environmentToggles[name]}
           />
         </ServiceEnvironmentTitle>
-      : null}
+      ) : null}
-      {config.environment && config.environment.length && environmentToggles[name]
+      {config.environment &&
-        ? <EnvironmentReview environment={config.environment} />
+      config.environment.length &&
-        : null}
+      environmentToggles[name] ? (
+        <EnvironmentReview environment={config.environment} />
+      ) : null}
     </ServiceCard>
-  );
+  ));
 
   return (
     <form onSubmit={handleSubmit}>
@@ -396,9 +396,8 @@ export const Progress = ({ stage, create, edit }) => {
   const _nameCompleted = stage !== 'name';
   const _nameActive = stage === 'name';
 
-  const _name = !create
+  const _name = !create ? null : (
-    ? null
+    <ProgressbarItem>
-    : <ProgressbarItem>
       <ProgressbarButton
         zIndex="10"
         completed={_nameCompleted}
@@ -407,7 +406,8 @@ export const Progress = ({ stage, create, edit }) => {
       >
         Name the group
       </ProgressbarButton>
-      </ProgressbarItem>;
+    </ProgressbarItem>
+  );
 
   const _manifestCompleted = ['environment', 'review'].indexOf(stage) >= 0;
   const _manifestActive = create ? stage === 'manifest' : stage === 'edit';

packages/cp-frontend/src/components/messaging/error.js

@@ -2,8 +2,9 @@ import React from 'react';
 import PropTypes from 'prop-types';
 import { Message } from 'joyent-ui-toolkit';
 
-const ErrorMessage = ({ title, message = "Ooops, there's been an error" }) =>
+const ErrorMessage = ({ title, message = "Ooops, there's been an error" }) => (
-  <Message title={title} message={message} type="ERROR" />;
+  <Message title={title} message={message} type="ERROR" />
+);
 
 ErrorMessage.propTypes = {
   title: PropTypes.string,

packages/cp-frontend/src/components/messaging/loader.js

@@ -27,10 +27,9 @@ const Msg = P.extend`
   margin-bottom: 0;
 `;
 
-export default ({ msg }) =>
+export default ({ msg }) => (
   <Container>
     <Loader />
-    <Msg>
+    <Msg>{msg || 'Loading...'}</Msg>
-      {msg || 'Loading...'}
+  </Container>
-    </Msg>
+);
-  </Container>;

packages/cp-frontend/src/components/messaging/modal-error.js

@@ -7,13 +7,15 @@ const StyledHeading = styled(ModalHeading)`
   color: ${props => props.theme.red};
 `;
 
-const ModalErrorMessage = ({ title, message, onCloseClick }) =>
+const ModalErrorMessage = ({ title, message, onCloseClick }) => (
   <div>
     <StyledHeading>{title}</StyledHeading>
-    <ModalText marginBottom="3">{message}
+    <ModalText marginBottom="3">{message}</ModalText>
-    </ModalText>
+    <Button onClick={onCloseClick} secondary>
-    <Button onClick={onCloseClick} secondary>Close </Button>
+      Close{' '}
-  </div>;
+    </Button>
+  </div>
+);
 
 ModalErrorMessage.propTypes = {
   title: PropTypes.string,

packages/cp-frontend/src/components/messaging/warning.js

@@ -2,8 +2,9 @@ import React from 'react';
 import PropTypes from 'prop-types';
 import { Message } from 'joyent-ui-toolkit';
 
-const WarningMessage = ({ title, message }) =>
+const WarningMessage = ({ title, message }) => (
-  <Message title={title} message={message} type="WARNING" />;
+  <Message title={title} message={message} type="WARNING" />
+);
 
 WarningMessage.propTypes = {
   title: PropTypes.string,

packages/cp-frontend/src/components/navigation/breadcrumb.js

@@ -24,27 +24,22 @@ const BreadcrumbContainer = styled.div`
 const getBreadcrumbItems = (...links) =>
   forceArray(links).map(({ pathname, name }, i) => {
     const item =
-      i + 1 >= links.length
+      i + 1 >= links.length ? (
-        ? name
+        name
-        : <BreadcrumbLink to={pathname}>
+      ) : (
-            {name}
+        <BreadcrumbLink to={pathname}>{name}</BreadcrumbLink>
-          </BreadcrumbLink>;
-
-    return (
-      <BreadcrumbItem key={name}>
-        {item}
-      </BreadcrumbItem>
       );
+
+    return <BreadcrumbItem key={name}>{item}</BreadcrumbItem>;
   });
 
-const NavBreadcrumb = ({ links = [] }) =>
+const NavBreadcrumb = ({ links = [] }) => (
   <BreadcrumbContainer>
     <Grid>
-      <Breadcrumb>
+      <Breadcrumb>{getBreadcrumbItems(...links)}</Breadcrumb>
-        {getBreadcrumbItems(...links)}
-      </Breadcrumb>
     </Grid>
-  </BreadcrumbContainer>;
+  </BreadcrumbContainer>
+);
 
 NavBreadcrumb.propTypes = {
   links: PropTypes.arrayOf(

packages/cp-frontend/src/components/navigation/header.js

@@ -12,20 +12,17 @@ const StyledLogo = Img.extend`
   height: ${remcalc(25)};
 `;
 
-const NavHeader = ({ datacenter, username }) =>
+const NavHeader = ({ datacenter, username }) => (
   <Header>
     <HeaderBrand>
       <Link to="/">
         <StyledLogo src={Logo} />
       </Link>
     </HeaderBrand>
-    <HeaderItem>
+    <HeaderItem>{datacenter}</HeaderItem>
-      {datacenter}
+    <HeaderItem>{username}</HeaderItem>
-    </HeaderItem>
+  </Header>
-    <HeaderItem>
+);
-      {username}
-    </HeaderItem>
-  </Header>;
 
 NavHeader.propTypes = {
   datacenter: PropTypes.string,

packages/cp-frontend/src/components/navigation/menu.js

@@ -11,20 +11,19 @@ import {
 } from 'joyent-ui-toolkit';
 
 const getMenuItems = (...links) =>
-  forceArray(links).map(({ pathname, name }) =>
+  forceArray(links).map(({ pathname, name }) => (
     <SectionListItem key={pathname}>
       <SectionListNavLink activeClassName="active" to={pathname}>
         {name}
       </SectionListNavLink>
     </SectionListItem>
-  );
+  ));
 
-const Menu = ({ links = [] }) =>
+const Menu = ({ links = [] }) => (
   <LayoutContainer plain>
-    <SectionList>
+    <SectionList>{getMenuItems(...links)}</SectionList>
-      {getMenuItems(...links)}
+  </LayoutContainer>
-    </SectionList>
+);
-  </LayoutContainer>;
 
 Menu.propTypes = {
   links: PropTypes.arrayOf(

packages/cp-frontend/src/components/navigation/not-found.js

@@ -39,6 +39,6 @@ NotFound.propTypes = {
   message: PropTypes.string,
   link: PropTypes.string,
   to: PropTypes.string
-}
+};
 
 export default NotFound;

packages/cp-frontend/src/components/service/delete.js

@@ -6,7 +6,7 @@ const ServiceDelete = ({
   service,
   onCancelClick = () => {},
   onConfirmClick = () => {}
-}) =>
+}) => (
   <div>
     <ModalHeading>
       Deleting a service: <br /> {service.name}
@@ -19,7 +19,8 @@ const ServiceDelete = ({
       Cancel
     </Button>
     <Button onClick={onConfirmClick}>Delete service</Button>
-  </div>;
+  </div>
+);
 
 ServiceDelete.propTypes = {
   service: PropTypes.object.isRequired,

packages/cp-frontend/src/components/service/metrics.js

@@ -2,13 +2,9 @@ import React, { Component } from 'react';
 import PropTypes from 'prop-types';
 import { MetricGraph } from 'joyent-ui-toolkit';
 
-const ServiceMetrics = ({
+const ServiceMetrics = ({ metricsData, graphDurationSeconds }) => {
-  metricsData,
-  graphDurationSeconds
-}) => {
-
   // metricsData should prob be an array rather than an object
-  const metricGraphs = Object.keys(metricsData).map((key) => (
+  const metricGraphs = Object.keys(metricsData).map(key => (
     // should also have a header, w metric name and number of instances (omit everything else from design for copilot)
     <MetricGraph
       key={key}
@@ -17,19 +13,15 @@ const ServiceMetrics = ({
       height={292}
       graphDurationSeconds={graphDurationSeconds}
     />
-  ))
+  ));
   // This needs layout!!!
-  return (
+  return <div>{metricGraphs}</div>;
-    <div>
+};
-      {metricGraphs}
-    </div>
-  )
-}
 
 ServiceMetrics.propTypes = {
   // metricsData should prob be an array rather than an object
   metricsData: PropTypes.object.isRequired,
   graphDurationSeconds: PropTypes.number.isRequired
-}
+};
 
 export default ServiceMetrics;

packages/cp-frontend/src/components/service/scale.js

@@ -15,7 +15,7 @@ const ServiceScale = ({
   onCancelClick = () => {},
   invalid,
   pristine
-}) =>
+}) => (
   <form onSubmit={handleSubmit}>
     <ModalHeading>
       Scaling a service: <br />
@@ -38,7 +38,8 @@ const ServiceScale = ({
     <Button type="submit" disabled={pristine || invalid} secondary>
       Scale
     </Button>
-  </form>;
+  </form>
+);
 
 ServiceScale.propTypes = {
   service: PropTypes.object.isRequired,

packages/cp-frontend/src/components/services/empty.js

@@ -16,7 +16,7 @@ const StyledBox = styled.div`
   }
 `;
 
-export default () =>
+export default () => (
   <LayoutContainer>
     <Row>
       <Col>
@@ -56,4 +56,5 @@ export default () =>
         </Row>
       </Col>
     </Row>
-  </LayoutContainer>;
+  </LayoutContainer>
+);

packages/cp-frontend/src/components/services/list-item.js

@@ -57,27 +57,27 @@ const ServiceListItem = ({
     : service.instances.length;
 
   const childrenItems = children.length
-    ? children.map(service =>
+    ? children.map(service => (
         <ServiceListItem
           key={service.id}
           deploymentGroup={deploymentGroup}
           service={service}
           isChild
         />
-    ) : null;
+      ))
+    : null;
 
-
+  const title = isChild ? (
-  const title = isChild
+    <CardTitle>{service.name}</CardTitle>
-    ? <CardTitle>
+  ) : (
-        {service.name}
+    <CardTitle>
-      </CardTitle>
-    : <CardTitle>
       <TitleInnerContainer>
         <StyledAnchor to={to} secondary active={service.instancesActive}>
           {service.name}
         </StyledAnchor>
       </TitleInnerContainer>
-      </CardTitle>;
+    </CardTitle>
+  );
 
   const subtitle = (
     <CardSubTitle>
@@ -86,8 +86,8 @@ const ServiceListItem = ({
     </CardSubTitle>
   );
 
-  const header = !isChild
+  const header = !isChild ? (
-    ? <StyledCardHeader>
+    <StyledCardHeader>
       {title}
       <CardDescription>
         <CardInfo
@@ -101,7 +101,7 @@ const ServiceListItem = ({
       </CardDescription>
       <CardOptions onClick={handleCardOptionsClick} />
     </StyledCardHeader>
-    : null;
+  ) : null;
 
   let healthyInfo = null;
   if (service.instancesActive) {
@@ -111,27 +111,22 @@ const ServiceListItem = ({
     const label = `${healthy} of ${total} healthy`;
 
     healthyInfo = (
-      <CardInfo
+      <CardInfo icon={icon} iconPosition="left" label={label} color="dark" />
-        icon={icon}
+    );
-        iconPosition='left'
-        label={label}
-        color='dark'
-      />
-    )
   }
 
-  const view = children.length
+  const view = children.length ? (
-    ? <CardGroupView>
+    <CardGroupView>{childrenItems}</CardGroupView>
-        {childrenItems}
+  ) : (
-      </CardGroupView>
+    <CardView>
-    : <CardView>
       {isChild && title}
       {isChild && subtitle}
       <CardDescription>
         <Status service={service} />
         {healthyInfo}
       </CardDescription>
-      </CardView>;
+    </CardView>
+  );
 
   return (
     <Card

packages/cp-frontend/src/components/services/quick-actions.js

@@ -1,6 +1,11 @@
 import React from 'react';
 import PropTypes from 'prop-types';
-import { Tooltip, TooltipButton, TooltipDivider, TooltipList } from 'joyent-ui-toolkit';
+import {
+  Tooltip,
+  TooltipButton,
+  TooltipDivider,
+  TooltipList
+} from 'joyent-ui-toolkit';
 
 const ServicesQuickActions = ({
   show,
@@ -11,7 +16,7 @@ const ServicesQuickActions = ({
   onStopClick = () => {},
   onStartClick = () => {},
   onScaleClick = () => {},
-  onDeleteClick = () => {},
+  onDeleteClick = () => {}
 }) => {
   if (!show) {
     return null;
@@ -46,22 +51,22 @@ const ServicesQuickActions = ({
   }, null);
 
   const startService =
-    status === 'RUNNING'
+    status === 'RUNNING' ? null : (
-      ? null
+      <li>
-      : <li>
         <TooltipButton onClick={handleStartClick} disabled={disabled}>
           Start
         </TooltipButton>
-        </li>;
+      </li>
+    );
 
   const stopService =
-    status === 'STOPPED'
+    status === 'STOPPED' ? null : (
-      ? null
+      <li>
-      : <li>
         <TooltipButton onClick={handleStopClick} disabled={disabled}>
           Stop
         </TooltipButton>
-        </li>;
+      </li>
+    );
 
   return (
     <Tooltip {...position} onBlur={onBlur}>

packages/cp-frontend/src/components/services/status.js

@@ -37,16 +37,18 @@ const ServiceStatus = ({ service }) => {
       );
     });
 
-  return service.transitionalStatus
+  return service.transitionalStatus ? (
-    ? <StyledStatusContainer>
+    <StyledStatusContainer>
       <StatusLoader />
       <StyledTransitionalStatus>
         {service.status ? service.status.toLowerCase() : ''}
       </StyledTransitionalStatus>
     </StyledStatusContainer>
-    : <StyledStatusContainer>
+  ) : (
+    <StyledStatusContainer>
       {getInstanceStatuses(service.instanceStatuses)}
-      </StyledStatusContainer>;
+    </StyledStatusContainer>
+  );
 };
 
 ServiceStatus.propTypes = {

packages/cp-frontend/src/containers/deployment-group/delete.js

@@ -5,7 +5,7 @@ import DeploymentGroupDeleteMutation from '@graphql/DeploymentGroupDeleteMutatio
 import DeploymentGroupQuery from '@graphql/DeploymentGroup.gql';
 import { Loader, ModalErrorMessage } from '@components/messaging';
 import { DeploymentGroupDelete as DeploymentGroupDeleteComponent } from '@components/deployment-group';
-import { Modal } from 'joyent-ui-toolkit'
+import { Modal } from 'joyent-ui-toolkit';
 import { withNotFound, GqlPaths } from '@containers/navigation';
 
 export class DeploymentGroupDelete extends Component {
@@ -21,7 +21,10 @@ export class DeploymentGroupDelete extends Component {
     const { history, match, loading, error } = this.props;
 
     const handleCloseClick = evt => {
-      const closeUrl = match.url.split('/').slice(0, -2).join('/');
+      const closeUrl = match.url
+        .split('/')
+        .slice(0, -2)
+        .join('/');
       history.replace(closeUrl);
     };
 
@@ -37,24 +40,21 @@ export class DeploymentGroupDelete extends Component {
       return (
         <Modal width={460} onCloseClick={handleCloseClick}>
           <ModalErrorMessage
-            title='Ooops!'
+            title="Ooops!"
-            message='An error occurred while loading your deployment group.'
+            message="An error occurred while loading your deployment group."
             onCloseClick={handleCloseClick}
           />
         </Modal>
       );
     }
 
-    const {
+    const { deploymentGroup, deleteDeploymentGroup } = this.props;
-      deploymentGroup,
-      deleteDeploymentGroup
-    } = this.props;
 
     if (this.state.error) {
       return (
         <Modal width={460} onCloseClick={handleCloseClick}>
           <ModalErrorMessage
-            title='Ooops!'
+            title="Ooops!"
             message={`An error occurred while attempting to delete the ${deploymentGroup.name} deployment group.`}
             onCloseClick={handleCloseClick}
           />

packages/cp-frontend/src/containers/deployment-groups/create.js

@@ -5,9 +5,10 @@ import { Progress } from '@components/manifest/edit-or-create';
 import { LayoutContainer } from '@components/layout';
 import { Title } from '@components/navigation';
 
-export default ({ match }) =>
+export default ({ match }) => (
   <LayoutContainer>
     <Title>Creating deployment group</Title>
     <Progress stage={match.params.stage} create />
     <ManifestEditOrCreate create />
-  </LayoutContainer>;
+  </LayoutContainer>
+);

packages/cp-frontend/src/containers/deployment-groups/list.js

@@ -140,20 +140,18 @@ export const DeploymentGroupList = ({
     );
   }
 
-  const groups = forceArray(deploymentGroups).map(({ slug, name }) =>
+  const groups = forceArray(deploymentGroups).map(({ slug, name }) => (
     <Col xs={12} sm={4} md={3} lg={3} key={slug}>
       <Box>
         <StyledLink to={`${match.path}/${slug}`}>
-          <ServiceTitle>
+          <ServiceTitle>{name}</ServiceTitle>
-            {name}
-          </ServiceTitle>
         </StyledLink>
         <StyledIconButton to={`${match.url}/${slug}/delete`}>
           <BinIcon />
         </StyledIconButton>
       </Box>
     </Col>
-  );
+  ));
 
   const create = [
     <Col xs={12} sm={4} md={3} lg={3} key="~create">
@@ -165,18 +163,16 @@ export const DeploymentGroupList = ({
       </BoxCreate>
     </Col>
   ].concat(
-    forceArray(importable).map(({ slug, name }) =>
+    forceArray(importable).map(({ slug, name }) => (
       <Col xs={12} sm={4} md={3} lg={3} key={slug}>
         <BoxCreate>
           <StyledCreateLink to={`${match.path}/~import/${slug}`}>
             <Oval>&#10549;</Oval>
-            <CreateTitle>
+            <CreateTitle>{name}</CreateTitle>
-              {name}
-            </CreateTitle>
           </StyledCreateLink>
         </BoxCreate>
       </Col>
-    )
+    ))
   );
 
   return (

packages/cp-frontend/src/containers/instances/list.js

@@ -20,7 +20,6 @@ export const InstanceList = ({
   instancesTooltip,
   toggleInstancesTooltip
 }) => {
-
   const _title = <Title>Instances</Title>;
 
   if (loading && !forceArray(instances).length) {
@@ -62,15 +61,12 @@ export const InstanceList = ({
   };
 
   const handleMouseOver = (evt, instance, type) => {
-
     const label = evt.currentTarget;
     const labelRect = label.getBoundingClientRect();
-    const offset = type === 'healthy'
+    const offset = type === 'healthy' ? 48 : type === 'status' ? 36 : 0;
-      ? 48 : type === 'status' ? 36 : 0;
 
     const position = {
-      left:
+      left: `${window.scrollX + labelRect.left + offset}px`,
-        `${window.scrollX + labelRect.left + offset}px`,
       top: `${window.scrollY + labelRect.bottom}px`
     };
 
@@ -78,16 +74,16 @@ export const InstanceList = ({
       instance,
       position,
       type
-    }
+    };
 
     toggleInstancesTooltip(tooltipData);
   };
 
-  const handleMouseOut = (evt) => {
+  const handleMouseOut = evt => {
     toggleInstancesTooltip({ show: false });
   };
 
-  const instanceList = instances.map((instance, index) =>
+  const instanceList = instances.map((instance, index) => (
     <InstanceListItem
       instance={instance}
       key={instance.id}
@@ -95,7 +91,7 @@ export const InstanceList = ({
       onStatusMouseOver={handleStatusMouseOver}
       onMouseOut={handleMouseOut}
     />
-  );
+  ));
 
   const _instances = !instanceList.length ? <EmptyInstances /> : instanceList;
 
@@ -105,7 +101,7 @@ export const InstanceList = ({
       {_instances}
     </LayoutContainer>
   );
-}
+};
 
 const mapStateToProps = (state, ownProps) => ({
   instancesTooltip: state.ui.instances.tooltip
@@ -151,8 +147,5 @@ const InstanceListGql = graphql(InstancesQuery, {
 export default compose(
   UiConnect,
   InstanceListGql,
-  withNotFound([
+  withNotFound([GqlPaths.DEPLOYMENT_GROUP, GqlPaths.SERVICES])
-    GqlPaths.DEPLOYMENT_GROUP,
-    GqlPaths.SERVICES
-  ])
 )(InstanceList);

packages/cp-frontend/src/containers/instances/tooltip.js

@@ -12,33 +12,30 @@ const StyledContainer = styled.div`
 const healthMessages = {
   healthy: 'Your instance is operating as expected',
   unhealthy: 'Your instance is not operating as expected',
-  maintenance: 'You\'ve set your instance to this manually, use the Container Pilot CLI to change',
+  maintenance:
-  unknown: 'We\'ve connected to your instance but we have no health information',
+    "You've set your instance to this manually, use the Container Pilot CLI to change",
-  unavailable: 'We cannot connect to your instance',
+  unknown: "We've connected to your instance but we have no health information",
+  unavailable: 'We cannot connect to your instance'
 };
 
 const statusMessages = {
   running: 'Your instance is operating',
   provisioning: 'Your instance is downloading dependencies and compiling',
-  ready: 'Your instance finished provisioning and is ready to be run, it\'ll be running soon',
+  ready:
+    "Your instance finished provisioning and is ready to be run, it'll be running soon",
   stopping: 'Your instance is going to be stopped soon',
-  stopped: 'Your instance isn\'t doing anything, you can start it',
+  stopped: "Your instance isn't doing anything, you can start it",
   offline: 'We have no idea what this means, do you??????',
   failed: 'Your instance has crashed',
-  unknown: 'We cannot work out what status your instance is in',
+  unknown: 'We cannot work out what status your instance is in'
 };
 
-export const InstancesTooltip = ({
+export const InstancesTooltip = ({ instancesTooltip }) => {
-  instancesTooltip
-}) => {
-
   if (instancesTooltip.show) {
-    const {
+    const { type, instance } = instancesTooltip;
-      type,
-      instance
-    } = instancesTooltip;
 
-    const message = type === 'healthy'
+    const message =
+      type === 'healthy'
         ? healthMessages[instance.healthy.toLowerCase()]
         : type === 'status'
           ? statusMessages[instance.status.toLowerCase()]
@@ -50,7 +47,7 @@ export const InstancesTooltip = ({
           <TooltipLabel>{message}</TooltipLabel>
         </Tooltip>
       </StyledContainer>
-    )
+    );
   }
 
   return null;

packages/cp-frontend/src/containers/manifest/edit-or-create.js

@@ -175,7 +175,9 @@ class DeploymentGroupEditOrCreate extends Component {
       name.replace(/^\$/, '')
     );
 
-    const vars = uniq(names).map(name => `\n${name}=`).join('');
+    const vars = uniq(names)
+      .map(name => `\n${name}=`)
+      .join('');
 
     return `# define your interpolatable variables here\n${vars}`;
   }

packages/cp-frontend/src/containers/manifest/index.js

@@ -47,12 +47,12 @@ const Manifest = ({
   }
 
   const _notice =
-    deploymentGroup && deploymentGroup.imported && !manifest
+    deploymentGroup && deploymentGroup.imported && !manifest ? (
-      ? <WarningMessage
+      <WarningMessage
         title="Be aware"
         message="Since this Deployment Group was imported, it doesn&#x27;t have the initial manifest."
       />
-      : null;
+    ) : null;
 
   return (
     <LayoutContainer>

packages/cp-frontend/src/containers/metrics/metrics-data-hoc.js

@@ -11,25 +11,16 @@ export const MetricNames = [
 export const withServiceMetricsPolling = ({
   pollingInterval = 1000 // in milliseconds
 }) => {
-  return (WrappedComponent) => {
+  return WrappedComponent => {
-
     return class extends Component {
-
       componentDidMount() {
-
         this._poll = setInterval(() => {
-          const {
+          const { loading, error, service, fetchMoreMetrics } = this.props;
-            loading,
-            error,
-            service,
-            fetchMoreMetrics
-          } = this.props;
 
           if (!loading && !error && service) {
             const previousEnd = service.instances[0].metrics[0].end;
             fetchMoreMetrics(previousEnd);
           }
-
         }, pollingInterval); // TODO this is the polling interval - think about amount is the todo I guess...
       }
 
@@ -38,24 +29,28 @@ export const withServiceMetricsPolling = ({
       }
 
       render() {
-        return <WrappedComponent {...this.props} />
+        return <WrappedComponent {...this.props} />;
-      }
-    }
-  }
       }
+    };
+  };
+};
 
 export const withServiceMetricsGql = ({
   gqlQuery,
   graphDurationSeconds,
   updateIntervalSeconds
 }) => {
-
+  const getPreviousMetrics = (
-  const getPreviousMetrics = (previousResult, serviceId, instanceId, metricName) => {
+    previousResult,
+    serviceId,
+    instanceId,
+    metricName
+  ) => {
     return previousResult.deploymentGroup.services
-      .find(s => s.id === serviceId).instances
+      .find(s => s.id === serviceId)
-      .find(i => i.id === instanceId).metrics
+      .instances.find(i => i.id === instanceId)
-      .find(m => m.name === metricName).metrics;
+      .metrics.find(m => m.name === metricName).metrics;
-  }
+  };
 
   const getNextResult = (previousResult, fetchNextResult) => {
     const deploymentGroup = fetchNextResult.deploymentGroup;
@@ -78,9 +73,9 @@ export const withServiceMetricsGql = ({
           }))
         }))
       }
-    }
+    };
     return nextResult;
-  }
+  };
 
   return graphql(gqlQuery, {
     options(props) {
@@ -90,7 +85,10 @@ export const withServiceMetricsGql = ({
 
       // this is potentially prone to overfetching if we already have data within timeframe and we leave the page then come back to it
       const end = moment();
-      const start = moment(end).subtract(graphDurationSeconds + updateIntervalSeconds, 'seconds'); // TODO initial amount of data we wanna get - should be the same as what we display + 15 secs
+      const start = moment(end).subtract(
+        graphDurationSeconds + updateIntervalSeconds,
+        'seconds'
+      ); // TODO initial amount of data we wanna get - should be the same as what we display + 15 secs
 
       return {
         variables: {
@@ -102,27 +100,34 @@ export const withServiceMetricsGql = ({
         }
       };
     },
-    props: ({ data: { deploymentGroup, loading, error, variables, fetchMore }}) => {
+    props: ({
-
+      data: { deploymentGroup, loading, error, variables, fetchMore }
-      const fetchMoreMetrics = (previousEnd) => {
+    }) => {
+      const fetchMoreMetrics = previousEnd => {
         fetchMore({
           variables: {
             ...variables,
             start: previousEnd,
-            end: moment().utc().format()
+            end: moment()
+              .utc()
+              .format()
           },
-          updateQuery: (previousResult, { fetchMoreResult, queryVariables }) => {
+          updateQuery: (
+            previousResult,
+            { fetchMoreResult, queryVariables }
+          ) => {
             return getNextResult(previousResult, fetchMoreResult);
           }
         });
-      }
+      };
-      return ({
+      return {
         deploymentGroup,
-        service: !loading && deploymentGroup ? deploymentGroup.services[0] : null,
+        service:
+          !loading && deploymentGroup ? deploymentGroup.services[0] : null,
         loading,
         error,
         fetchMoreMetrics
-      })
+      };
     }
   });
-}
+};

packages/cp-frontend/src/containers/navigation/breadcrumb.js

@@ -9,8 +9,11 @@ import {
   serviceBySlugSelector
 } from '@root/state/selectors';
 
-export const Breadcrumb = ({ deploymentGroup, service, location: { pathname }}) => {
+export const Breadcrumb = ({
-
+  deploymentGroup,
+  service,
+  location: { pathname }
+}) => {
   const path = pathname.split('/');
 
   const links = [
@@ -41,7 +44,7 @@ Breadcrumb.propTypes = {
   deploymentGroup: PropTypes.object,
   service: PropTypes.object,
   location: PropTypes.object
-}
+};
 
 const connectBreadcrumb = connect(
   (state, ownProps) => {
@@ -59,7 +62,4 @@ const connectBreadcrumb = connect(
   dispatch => ({})
 );
 
-export default compose(
+export default compose(connectBreadcrumb, withNotFound())(Breadcrumb);
-  connectBreadcrumb,
-  withNotFound()
-)(Breadcrumb);

packages/cp-frontend/src/containers/navigation/header.js

@@ -5,8 +5,9 @@ import get from 'lodash.get';
 import PortalQuery from '@graphql/Portal.gql';
 import { Header as HeaderComponent } from '@components/navigation';
 
-export const Header = ({ datacenter, username }) =>
+export const Header = ({ datacenter, username }) => (
-  <HeaderComponent datacenter={datacenter} username={username} />;
+  <HeaderComponent datacenter={datacenter} username={username} />
+);
 
 const HeaderWithData = graphql(PortalQuery, {
   props: ({ data: { portal = {} } }) => ({

packages/cp-frontend/src/containers/navigation/menu.js

@@ -5,7 +5,6 @@ import withNotFound from './not-found-hoc';
 import { Menu as MenuComponent } from '@components/navigation';
 
 export const Menu = ({ location, match, sections }) => {
-
   if (!sections || !sections.length) {
     return null;
   }
@@ -40,7 +39,4 @@ const connectMenu = connect(
   dispatch => ({})
 );
 
-export default compose(
+export default compose(connectMenu, withNotFound())(Menu);
-  connectMenu,
-  withNotFound()
-)(Menu);

packages/cp-frontend/src/containers/navigation/not-found-hoc.js

@@ -4,26 +4,19 @@ import { NotFound } from '@components/navigation';
 export const GqlPaths = {
   DEPLOYMENT_GROUP: 'deploymentGroup',
   SERVICES: 'services'
-}
+};
-
-export default (paths) => {
-  return (WrappedComponent) => {
 
+export default paths => {
+  return WrappedComponent => {
     return class extends Component {
-
       componentWillReceiveProps(nextProps) {
-        
         if (paths) {
-          const {
+          const { error, location, history } = nextProps;
-            error,
-            location,
-            history
-          } = nextProps;
 
           if (
-            error
+            error &&
-            && (!location.state || !location.state.notFound)
+            (!location.state || !location.state.notFound) &&
-            && (error.graphQLErrors && error.graphQLErrors.length)
+            (error.graphQLErrors && error.graphQLErrors.length)
           ) {
             const graphQLError = error.graphQLErrors[0];
             if (graphQLError.message === 'Not Found') {
@@ -37,12 +30,7 @@ export default (paths) => {
       }
 
       render() {
-
+        const { error, location, match } = this.props;
-        const {
-          error,
-          location,
-          match
-        } = this.props;
 
         if (location.state && location.state.notFound) {
           const notFound = location.state.notFound;
@@ -52,10 +40,12 @@ export default (paths) => {
             let link;
             if (notFound === 'services' || notFound === 'service') {
               title = 'This service doesn’t exist';
-              to = match.url.split('/').slice(0, 3).join('/');
+              to = match.url
+                .split('/')
+                .slice(0, 3)
+                .join('/');
               link = 'Back to services';
-            }
+            } else if (notFound === 'deploymentGroup') {
-            else if(notFound === 'deploymentGroup') {
               title = 'This deployment group doesn’t exist';
               to = '/deployment-group';
               link = 'Back to dashboard';
@@ -63,17 +53,17 @@ export default (paths) => {
             return (
               <NotFound
                 title={title}
-                message='Sorry, but our princess is in another castle.'
+                message="Sorry, but our princess is in another castle."
                 to={to}
                 link={link}
               />
-            )
+            );
           }
           return null;
         }
 
-        return <WrappedComponent {...this.props} />
+        return <WrappedComponent {...this.props} />;
-      }
-    }
-  }
       }
+    };
+  };
+};

packages/cp-frontend/src/containers/service/delete.js

@@ -21,7 +21,10 @@ export class ServiceDelete extends Component {
     const { loading, error, match, history } = this.props;
 
     const handleCloseClick = evt => {
-      const closeUrl = match.url.split('/').slice(0, -2).join('/');
+      const closeUrl = match.url
+        .split('/')
+        .slice(0, -2)
+        .join('/');
       history.replace(closeUrl);
     };
 
@@ -37,8 +40,8 @@ export class ServiceDelete extends Component {
       return (
         <Modal width={460} onCloseClick={handleCloseClick}>
           <ModalErrorMessage
-            title='Ooops!'
+            title="Ooops!"
-            message='An error occurred while loading your service.'
+            message="An error occurred while loading your service."
             onCloseClick={handleCloseClick}
           />
         </Modal>
@@ -51,7 +54,7 @@ export class ServiceDelete extends Component {
       return (
         <Modal width={460} onCloseClick={handleCloseClick}>
           <ModalErrorMessage
-            title='Ooops!'
+            title="Ooops!"
             message={`An error occurred while attempting to delete your ${service.name} service.`}
             onCloseClick={handleCloseClick}
           />
@@ -60,7 +63,9 @@ export class ServiceDelete extends Component {
     }
 
     const handleConfirmClick = evt => {
-      deleteServices(service.id).then(() => handleCloseClick()).catch(err => {
+      deleteServices(service.id)
+        .then(() => handleCloseClick())
+        .catch(err => {
           this.setState({ error: err });
         });
     };

packages/cp-frontend/src/containers/service/metrics.js

@@ -10,17 +10,16 @@ import { ServiceMetrics as ServiceMetricsComponent } from '@components/service';
 import { Button } from 'joyent-ui-toolkit';
 import { Loader, ErrorMessage } from '@components/messaging';
 
-import { withServiceMetricsPolling, withServiceMetricsGql } from '@containers/metrics';
+import {
+  withServiceMetricsPolling,
+  withServiceMetricsGql
+} from '@containers/metrics';
 
 // 'width' of graph, i.e. total duration of time it'll display and truncate data to
 // amount of data we'll need to initially fetch
 const GraphDurationSeconds = 90;
 
-const ServiceMetrics = ({
+const ServiceMetrics = ({ service, loading, error }) => {
-  service,
-  loading,
-  error
-}) => {
   if (loading || !service) {
     return (
       <LayoutContainer center>
@@ -43,7 +42,7 @@ const ServiceMetrics = ({
   // metricsData should prob be an array rather than an object
   const metricsData = service.instances.reduce((metrics, instance) => {
     // gather metrics of instances according to type
-    instance.metrics.forEach((instanceMetrics) => {
+    instance.metrics.forEach(instanceMetrics => {
       if (!metrics[instanceMetrics.name]) {
         metrics[instanceMetrics.name] = [];
       }
@@ -69,10 +68,7 @@ export default compose(
     updateIntervalSeconds: 15
   }),
   withServiceMetricsPolling({ pollingInterval: 1000 }),
-  withNotFound([
+  withNotFound([GqlPaths.DEPLOYMENT_GROUP, GqlPaths.SERVICES])
-    GqlPaths.DEPLOYMENT_GROUP,
-    GqlPaths.SERVICES
-  ])
 )(ServiceMetrics);
 
 /*

packages/cp-frontend/src/containers/service/scale.js

@@ -22,7 +22,10 @@ export class ServiceScale extends Component {
     const { loading, error, match, history } = this.props;
 
     const handleCloseClick = evt => {
-      const closeUrl = match.url.split('/').slice(0, -2).join('/');
+      const closeUrl = match.url
+        .split('/')
+        .slice(0, -2)
+        .join('/');
       history.replace(closeUrl);
     };
 
@@ -38,8 +41,8 @@ export class ServiceScale extends Component {
       return (
         <Modal width={460} onCloseClick={handleCloseClick}>
           <ModalErrorMessage
-            title='Ooops!'
+            title="Ooops!"
-            message='An error occurred while loading your service.'
+            message="An error occurred while loading your service."
             onCloseClick={handleCloseClick}
           />
         </Modal>
@@ -52,7 +55,7 @@ export class ServiceScale extends Component {
       return (
         <Modal width={460} onCloseClick={handleCloseClick}>
           <ModalErrorMessage
-            title='Ooops!'
+            title="Ooops!"
             message={`An error occurred while attempting to scale your ${service.name} service.`}
             onCloseClick={handleCloseClick}
           />
@@ -70,7 +73,9 @@ export class ServiceScale extends Component {
     };
 
     const handleSubmitClick = values => {
-      scale(service.id, values.replicas).then(handleCloseClick).catch(err => {
+      scale(service.id, values.replicas)
+        .then(handleCloseClick)
+        .catch(err => {
           this.setState({ error: err });
         });
     };

packages/cp-frontend/src/containers/services/list.js

@@ -34,7 +34,7 @@ export class ServiceList extends Component {
       services,
       loading,
       error,
-      toggleServicesQuickActions,
+      toggleServicesQuickActions
     } = this.props;
 
     if (loading && !forceArray(services).length) {
@@ -73,8 +73,9 @@ export class ServiceList extends Component {
       const buttonRect = button.getBoundingClientRect();
 
       const position = {
-        left:
+        left: `${buttonRect.left +
-          `${buttonRect.left + window.scrollX + (buttonRect.right - buttonRect.left) / 2}px`,
+          window.scrollX +
+          (buttonRect.right - buttonRect.left) / 2}px`,
         top: `${buttonRect.bottom + window.scrollY}px`
       };
 
@@ -120,9 +121,7 @@ export class ServiceList extends Component {
     return (
       <LayoutContainer>
         {renderedError}
-        <StyledContainer>
+        <StyledContainer>{serviceList}</StyledContainer>
-          {serviceList}
-        </StyledContainer>
       </LayoutContainer>
     );
   }
@@ -134,7 +133,7 @@ ServiceList.propTypes = {
   loading: PropTypes.bool,
   error: PropTypes.bool,
   toggleServicesQuickActions: PropTypes.func
-}
+};
 
 const mapStateToProps = (state, ownProps) => ({});
 

packages/cp-frontend/src/containers/services/menu.js

@@ -20,11 +20,8 @@ const PaddedRow = Row.extend`
   margin-bottom: ${remcalc(18)}
 `;
 
-export const ServicesMenu = ({
-  location: { pathname },
-  history: { push }
-}) => {
 
+export const ServicesMenu = ({ location: { pathname }, history: { push } }) => {
   const toggleValue = pathname.split('-').pop();
 
   const handleToggle = evt => {

packages/cp-frontend/src/containers/services/quick-actions.js

@@ -17,13 +17,12 @@ const StyledContainer = styled.div`
 `;
 
 export class ServicesQuickActions extends Component {
-
   constructor(props) {
     super(props);
 
     this.state = {
       errors: {}
-    }
+    };
   }
 
   render() {
@@ -115,7 +114,7 @@ export class ServicesQuickActions extends Component {
             onDeleteClick={handleDeleteClick}
           />
         </StyledContainer>
-      )
+      );
     }
 
     if (quickActions || errorMessage) {
@@ -124,7 +123,7 @@ export class ServicesQuickActions extends Component {
           {errorMessage}
           {quickActions}
         </div>
-      )
+      );
     }
 
     return null;

packages/cp-frontend/src/containers/services/topology.js

@@ -47,7 +47,7 @@ export class ServicesTopology extends Component {
       services,
       loading,
       error,
-      toggleServicesQuickActions,
+      toggleServicesQuickActions
     } = this.props;
 
     if (loading && !forceArray(services).length) {
@@ -85,18 +85,27 @@ export class ServicesTopology extends Component {
       const container = this._refs.container;
       const containerRect = container.getBoundingClientRect();
       const position = {
-        top: `${containerRect.top + window.scrollY + tooltipData.position.top}px`,
+        top: `${containerRect.top +
-        left: `${containerRect.left + window.scrollX + tooltipData.position.left}px`
+          window.scrollY +
-      }
+          tooltipData.position.top}px`,
+        left: `${containerRect.left +
+          window.scrollX +
+          tooltipData.position.left}px`
+      };
       const data = {
         ...tooltipData,
         position
-      }
+      };
       toggleServicesQuickActions(data);
     };
 
     const handleNodeTitleClick = (evt, { service }) => {
-      push(`${url.split('/').slice(0, 3).join('/')}/services/${service.slug}`);
+      push(
+        `${url
+          .split('/')
+          .slice(0, 3)
+          .join('/')}/services/${service.slug}`
+      );
     };
 
     let renderedError = null;

packages/cp-frontend/src/graphql/ServiceMetrics.gql

@@ -1,8 +1,8 @@
 query Instances(
-  $deploymentGroupSlug: String!,
+  $deploymentGroupSlug: String!
-  $serviceSlug: String!,
+  $serviceSlug: String!
-  $metricNames: [MetricName]!,
+  $metricNames: [MetricName]!
-  $start: String!,
+  $start: String!
   $end: String!
 ) {
   deploymentGroup(slug: $deploymentGroupSlug) {

packages/cp-frontend/src/router.js

@@ -25,10 +25,7 @@ import {
   ServiceMetrics
 } from '@containers/service';
 
-import {
+import { InstanceList, InstancesTooltip } from '@containers/instances';
-  InstanceList,
-  InstancesTooltip
-} from '@containers/instances';
 
 import { DeploymentGroupDelete } from '@containers/deployment-group';
 
@@ -43,23 +40,27 @@ const Container = styled.div`
 
 const rootRedirect = p => <Redirect to="/deployment-groups" />;
 
-const servicesListRedirect = p =>
+const servicesListRedirect = p => (
   <Redirect
     to={`/deployment-groups/${p.match.params.deploymentGroup}/services-list`}
-  />;
+  />
+);
 
-const servicesTopologyRedirect = p =>
+const servicesTopologyRedirect = p => (
   <Redirect
-    to={`/deployment-groups/${p.match.params.deploymentGroup}/services-topology`}
+    to={`/deployment-groups/${p.match.params
-  />;
+      .deploymentGroup}/services-topology`}
+  />
+);
 
-const serviceRedirect = p =>
+const serviceRedirect = p => (
   <Redirect
     to={`/deployment-groups/${p.match.params.deploymentGroup}/services/${p.match
       .params.service}/instances`}
-  />;
+  />
+);
 
-const App = p =>
+const App = p => (
   <div>
     <Switch>
       <Route
@@ -222,7 +223,8 @@ const App = p =>
         component={servicesTopologyRedirect}
       />
     </Switch>
-  </div>;
+  </div>
+);
 
 const Router = (
   <BrowserRouter>

packages/cp-frontend/src/state/reducers/ui.js

@@ -1,5 +1,8 @@
 import { handleActions } from 'redux-actions';
-import { toggleServicesQuickActions, toggleInstancesTooltip } from '@state/actions';
+import {
+  toggleServicesQuickActions,
+  toggleInstancesTooltip
+} from '@state/actions';
 
 export const _toggleServicesQuickActions = (state, action) => {
   const { position, service, show } = action.payload;

packages/cp-frontend/src/state/selectors.js

@@ -104,8 +104,8 @@ const getInstancesHealthy = instances => {
   return instances.reduce(
     (healthy, instance) => ({
       total: healthy.total + 1,
-      healthy: instance.healthy === 'HEALTHY' ?
+      healthy:
-        healthy.healthy + 1 : healthy.healthy
+        instance.healthy === 'HEALTHY' ? healthy.healthy + 1 : healthy.healthy
     }),
     { total: 0, healthy: 0 }
   );
@@ -174,6 +174,8 @@ const processServicesForTopology = services => {
   }));
 };
 
+/* ,
+  instancesByServiceId */
 export {
   deploymentGroupBySlug as deploymentGroupBySlugSelector,
   serviceBySlug as serviceBySlugSelector,
@@ -182,6 +184,5 @@ export {
   getInstancesHealthy,
   getService,
   processServices,
-  processServicesForTopology /* ,
+  processServicesForTopology
-  instancesByServiceId */
 };

packages/cp-frontend/src/state/store.js

@@ -30,8 +30,11 @@ export const client = new ApolloClient({
             ? o.timestamp
             : o.name && o.instance
               ? `${o.name}-${o.instance}`
-                : o.name ? o.name : o.time && o.value
+              : o.name
-                  ? `${o.time}-${o.value}` : 'apollo-cache-key-not-defined';
+                ? o.name
+                : o.time && o.value
+                  ? `${o.time}-${o.value}`
+                  : 'apollo-cache-key-not-defined';
     return `${o.__typename}:${id}`;
   },
   networkInterface: createNetworkInterface({

packages/cp-frontend/test/unit/components/deployment-group/delete.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */
@@ -12,9 +11,7 @@ import { deploymentGroup } from '../../mocks';
 
 it('renders <DeploymentGroupDelete /> without throwing', () => {
   const tree = renderer
-    .create(
+    .create(<DeploymentGroupDelete deploymentGroup={deploymentGroup} />)
-      <DeploymentGroupDelete deploymentGroup={deploymentGroup} />
-    )
     .toJSON();
   expect(tree).toMatchSnapshot();
 });

packages/cp-frontend/test/unit/components/instances/empty.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */
@@ -10,10 +9,6 @@ import 'jest-styled-components';
 import EmtpyInstances from '@components/instances/empty.js';
 
 it('renders <EmtpyInstances /> without throwing', () => {
-  const tree = renderer
+  const tree = renderer.create(<EmtpyInstances />).toJSON();
-    .create(
-      <EmtpyInstances />
-    )
-    .toJSON();
   expect(tree).toMatchSnapshot();
 });

packages/cp-frontend/test/unit/components/instances/list-item.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */
@@ -11,10 +10,6 @@ import InstanceCard from '@components/instances/list-item.js';
 import { instance } from '../../mocks';
 
 it('renders <InstanceCard /> without throwing', () => {
-  const tree = renderer
+  const tree = renderer.create(<InstanceCard instance={instance} />).toJSON();
-    .create(
-      <InstanceCard instance={instance} />
-    )
-    .toJSON();
   expect(tree).toMatchSnapshot();
 });

packages/cp-frontend/test/unit/components/layout/container.js

@@ -9,10 +9,6 @@ import 'jest-styled-components';
 import Container from '@components/layout/container';
 
 it('renders <Container /> without throwing', () => {
-  const tree = renderer
+  const tree = renderer.create(<Container />).toJSON();
-    .create(
-      <Container />
-    )
-    .toJSON();
   expect(tree).toMatchSnapshot();
 });

packages/cp-frontend/test/unit/components/manifest/edit-or-create.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */

packages/cp-frontend/test/unit/components/messaging/error.js

@@ -9,10 +9,6 @@ import 'jest-styled-components';
 import Error from '@components/messaging/error';
 
 it('renders <Error /> without throwing', () => {
-  const tree = renderer
+  const tree = renderer.create(<Error />).toJSON();
-    .create(
-      <Error />
-    )
-    .toJSON();
   expect(tree).toMatchSnapshot();
 });

packages/cp-frontend/test/unit/components/messaging/loader.js

@@ -9,10 +9,6 @@ import 'jest-styled-components';
 import Loader from '@components/messaging/loader';
 
 it('renders <Loader /> without throwing', () => {
-  const tree = renderer
+  const tree = renderer.create(<Loader />).toJSON();
-    .create(
-      <Loader />
-    )
-    .toJSON();
   expect(tree).toMatchSnapshot();
 });

packages/cp-frontend/test/unit/components/messaging/modal-error.js

@@ -11,7 +11,7 @@ import ModalError from '@components/messaging/modal-error';
 it('renders <ModalError /> without throwing', () => {
   const tree = renderer
     .create(
-      <ModalError message='Modal error message' onCloseClick={() => {}}/>
+      <ModalError message="Modal error message" onCloseClick={() => {}} />
     )
     .toJSON();
   expect(tree).toMatchSnapshot();

packages/cp-frontend/test/unit/components/messaging/warning.js

@@ -9,10 +9,6 @@ import 'jest-styled-components';
 import Warning from '@components/messaging/warning';
 
 it('renders <Warning /> without throwing', () => {
-  const tree = renderer
+  const tree = renderer.create(<Warning message="Warning message" />).toJSON();
-    .create(
-      <Warning message='Warning message'/>
-    )
-    .toJSON();
   expect(tree).toMatchSnapshot();
 });

packages/cp-frontend/test/unit/components/navigation/title.js

@@ -9,10 +9,6 @@ import 'jest-styled-components';
 import Title from '@components/navigation/title';
 
 it('renders <Title /> without throwing', () => {
-  const tree = renderer
+  const tree = renderer.create(<Title />).toJSON();
-    .create(
-      <Title />
-    )
-    .toJSON();
   expect(tree).toMatchSnapshot();
 });

packages/cp-frontend/test/unit/components/service/delete.js

@@ -10,10 +10,6 @@ import { service } from '../../mocks';
 import Delete from '@components/service/delete';
 
 it('renders <Delete /> without throwing', () => {
-  const tree = renderer
+  const tree = renderer.create(<Delete service={service} />).toJSON();
-    .create(
-      <Delete service={service} />
-    )
-    .toJSON();
   expect(tree).toMatchSnapshot();
 });

packages/cp-frontend/test/unit/components/services/empty.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */

packages/cp-frontend/test/unit/components/services/list-item.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */
@@ -23,9 +22,7 @@ it('renders <ServiceListItem /> without throwing', () => {
 
 it('renders child <ServiceListItem /> without throwing', () => {
   const tree = renderer
-    .create(
+    .create(<ServiceListItem service={service} isChild />)
-      <ServiceListItem service={service} isChild />
-    )
     .toJSON();
   expect(tree).toMatchSnapshot();
 });

packages/cp-frontend/test/unit/components/services/quick-actions.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */

packages/cp-frontend/test/unit/components/services/status.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */
@@ -12,9 +11,7 @@ import { service } from '../../mocks';
 
 xit('renders <ServiceStatus /> without throwing', () => {
   const tree = renderer
-    .create(
+    .create(<ServiceStatus instanceStatuses={service.instanceStatuses} />)
-      <ServiceStatus instanceStatuses={service.instanceStatuses} />
-    )
     .toJSON();
   expect(tree).toMatchSnapshot();
 });

packages/cp-frontend/test/unit/containers/deployment-group/delete.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */

packages/cp-frontend/test/unit/containers/deployment-groups/create.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */
@@ -17,7 +16,7 @@ it('renders <DeploymentGroupCreate /> without throwing', () => {
         stage: ''
       }
     }
-  }
+  };
   const tree = renderer
     .create(
       <Store>

packages/cp-frontend/test/unit/containers/deployment-groups/import.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */

packages/cp-frontend/test/unit/containers/deployment-groups/list.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */

packages/cp-frontend/test/unit/containers/instances/list.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */

packages/cp-frontend/test/unit/containers/instances/tooltip.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */

packages/cp-frontend/test/unit/containers/manifest/edit-or-create.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */

packages/cp-frontend/test/unit/containers/navigation/breadcrumb.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */
@@ -15,7 +14,7 @@ it('renders <Breadcrumb /> without throwing', () => {
     location: {
       pathname: ''
     }
-  }
+  };
   const tree = renderer
     .create(
       <Store>

packages/cp-frontend/test/unit/containers/navigation/header.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */

packages/cp-frontend/test/unit/containers/navigation/menu.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */

packages/cp-frontend/test/unit/containers/navigation/not-found-hoc.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */

packages/cp-frontend/test/unit/containers/service/delete.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */

packages/cp-frontend/test/unit/containers/service/scale.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */

packages/cp-frontend/test/unit/containers/services/list.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */
@@ -15,10 +14,7 @@ it('renders <ServiceList /> without throwing', () => {
     .create(
       <Store>
         <Router>
-          <ServiceList
+          <ServiceList deploymentGroup={deploymentGroup} services={services} />
-            deploymentGroup={deploymentGroup}
-            services={services}
-          />
         </Router>
       </Store>
     )

packages/cp-frontend/test/unit/containers/services/menu.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */
@@ -18,7 +17,7 @@ it('renders <ServicesMenu /> without throwing', () => {
     history: {
       push: () => {}
     }
-  }
+  };
   const tree = renderer
     .create(
       <Store>

packages/cp-frontend/test/unit/containers/services/quick-actions.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */

packages/cp-frontend/test/unit/containers/services/topology.js

@@ -1,4 +1,3 @@
-
 /**
  * @jest-environment jsdom
  */
@@ -15,9 +14,7 @@ it('renders <ServicesTopology /> without throwing', () => {
     .create(
       <Store>
         <Router>
-          <ServicesTopology
+          <ServicesTopology services={services} />
-            services={services}
-          />
         </Router>
       </Store>
     )

packages/cp-frontend/test/unit/mocks/data.js

@@ -1,37 +1,31 @@
 export const instance = {
-  "id": "309ecd9f-ac03-474b-aff7-4bd2e743296c",
+  id: '309ecd9f-ac03-474b-aff7-4bd2e743296c',
-  "name": "wordpress_01",
+  name: 'wordpress_01',
-  "serviceId": "be227788-74f1-4e5b-a85f-b5c71cbae8d8",
+  serviceId: 'be227788-74f1-4e5b-a85f-b5c71cbae8d8',
-  "deploymentGroupId": "e0ea0c02-55cc-45fe-8064-3e5176a59401",
+  deploymentGroupId: 'e0ea0c02-55cc-45fe-8064-3e5176a59401',
-  "machineId": "011f7479-2d45-442d-99bf-7f6216954cc8",
+  machineId: '011f7479-2d45-442d-99bf-7f6216954cc8',
-  "status": "RUNNING",
+  status: 'RUNNING',
-  "healthy": "HEALTHY"
+  healthy: 'HEALTHY'
 };
 
 export const service = {
-  "id": "081a792c-47e0-4439-924b-2efa9788ae9e",
+  id: '081a792c-47e0-4439-924b-2efa9788ae9e',
-  "slug": "nginx",
+  slug: 'nginx',
-  "name": "Nginx",
+  name: 'Nginx',
-  "deploymentGroupId": "e0ea0c02-55cc-45fe-8064-3e5176a59401",
+  deploymentGroupId: 'e0ea0c02-55cc-45fe-8064-3e5176a59401',
-  "connections": ["be227788-74f1-4e5b-a85f-b5c71cbae8d8"],
+  connections: ['be227788-74f1-4e5b-a85f-b5c71cbae8d8'],
-  "instances": [instance],
+  instances: [instance],
-  "instanceStatuses": [{ status: "RUNNING", count: 1 }]
+  instanceStatuses: [{ status: 'RUNNING', count: 1 }]
 };
 
 export const deploymentGroup = {
-  "id": "e0ea0c02-55cc-45fe-8064-3e5176a59401",
+  id: 'e0ea0c02-55cc-45fe-8064-3e5176a59401',
-  "slug": "wordpress-blog-example",
+  slug: 'wordpress-blog-example',
-  "name": "Wordpress Blog Example"
+  name: 'Wordpress Blog Example'
 };
 
-export const services = [
+export const services = [service];
-  service
-];
 
-export const instances = [
+export const instances = [instance];
-  instance
-];
 
-export const deploymentGroups = [
+export const deploymentGroups = [deploymentGroup];
-  deploymentGroup
-]

packages/cp-frontend/test/unit/mocks/router.js

@@ -1,7 +1,4 @@
 import React from 'react';
 import { MemoryRouter } from 'react-router-dom';
 
-export default ({ children }) =>
+export default ({ children }) => <MemoryRouter>{children}</MemoryRouter>;
-  <MemoryRouter>
-    {children}
-  </MemoryRouter>;

packages/cp-frontend/test/unit/mocks/store.js

@@ -2,7 +2,8 @@ import React from 'react';
 import { client, store } from '@state/store';
 import { ApolloProvider } from 'react-apollo';
 
-export default ({ children }) =>
+export default ({ children }) => (
   <ApolloProvider client={client} store={store}>
     {children}
-  </ApolloProvider>;
+  </ApolloProvider>
+);

packages/cp-frontend/test/unit/mocks/theme.js

@@ -2,7 +2,6 @@ import React from 'react';
 import { ThemeProvider } from 'styled-components';
 import { theme } from 'joyent-ui-toolkit';
 
-export default ({ children }) =>
+export default ({ children }) => (
-  <ThemeProvider theme={theme}>
+  <ThemeProvider theme={theme}>{children}</ThemeProvider>
-    {children}
+);
-  </ThemeProvider>;

packages/cp-frontend/test/unit/state/reducers/ui.js

@@ -5,7 +5,6 @@ import {
 import state from '@state/state';
 
 describe('ui reducer', () => {
-
   it('toggleServicesQuickActions shows correctly', () => {
     const uiState = state.ui;
     const expectedUiState = {
@@ -23,8 +22,9 @@ describe('ui reducer', () => {
           }
         }
       }
-    }
+    };
-    const action = { payload: {
+    const action = {
+      payload: {
         show: true,
         position: {
           top: 10,
@@ -33,7 +33,8 @@ describe('ui reducer', () => {
         service: {
           id: 'service-id'
         }
-    }};
+      }
+    };
     const result = _toggleServicesQuickActions(uiState, action);
     expect(result).toEqual(expectedUiState);
   });
@@ -48,7 +49,7 @@ describe('ui reducer', () => {
           show: false
         }
       }
-    }
+    };
     const action = { payload: { show: false } };
     const result = _toggleServicesQuickActions(uiState, action);
     expect(result).toEqual(expectedUiState);
@@ -72,8 +73,9 @@ describe('ui reducer', () => {
           type: 'healthy'
         }
       }
-    }
+    };
-    const action = { payload: {
+    const action = {
+      payload: {
         show: true,
         position: {
           top: 10,
@@ -83,7 +85,8 @@ describe('ui reducer', () => {
           id: 'instance-id'
         },
         type: 'healthy'
-    }};
+      }
+    };
     const result = _toggleInstancesTooltip(uiState, action);
     expect(result).toEqual(expectedUiState);
   });
@@ -98,9 +101,9 @@ describe('ui reducer', () => {
           show: false
         }
       }
-    }
+    };
     const action = { payload: { show: false } };
     const result = _toggleInstancesTooltip(uiState, action);
     expect(result).toEqual(expectedUiState);
   });
-})
+});

packages/cp-frontend/test/unit/state/selectors.js

@@ -10,9 +10,7 @@ import {
 } from '@state/selectors';
 
 describe('Redux selectors and Apollo helpers', () => {
-
   describe('getInstanceStatuses', () => {
-
     it('gathers instance statuses correctly', () => {
       const service = {
         instances: [
@@ -50,8 +48,7 @@ describe('Redux selectors and Apollo helpers', () => {
   });
 
   describe('getInstancesActive', () => {
-
+    it("returns true if all instances' status is active", () => {
-    it('returns true if all instances\' status is active', () => {
       const statuses = [
         { status: 'RUNNING' },
         { status: 'READY' },
@@ -63,7 +60,7 @@ describe('Redux selectors and Apollo helpers', () => {
       expect(result).toEqual(expectedResult);
     });
 
-    it('returns false if no instances\' status is active', () => {
+    it("returns false if no instances' status is active", () => {
       const statuses = [
         { status: 'STOPPING' },
         { status: 'FAILED' },
@@ -75,7 +72,7 @@ describe('Redux selectors and Apollo helpers', () => {
       expect(result).toEqual(expectedResult);
     });
 
-    it('returns true if some instances\' status is active', () => {
+    it("returns true if some instances' status is active", () => {
       const statuses = [
         { status: 'STOPPING' },
         { status: 'FAILED' },
@@ -89,7 +86,6 @@ describe('Redux selectors and Apollo helpers', () => {
   });
 
   describe('getInstancesHealthy', () => {
-
     it('returns the number of healthy instances correctly', () => {
       const instances = [
         { healthy: 'HEALTHY' },
@@ -106,39 +102,31 @@ describe('Redux selectors and Apollo helpers', () => {
   });
 
   describe('getService', () => {
-
     it('returns the service decorated with details for display correctly', () => {
-
       const result = getService(nginxService, 0);
       expect(result).toEqual(nginxExpectedResult);
     });
 
     it('returns the consul service decorated with details for display correctly', () => {
-
       const result = getService(consulService, 1);
       expect(result).toEqual(consulExpectedResult);
     });
   });
 
   describe('processServices', () => {
-
     it('returns the services decorated with details for display correctly', () => {
-      const services = [
+      const services = [nginxService, consulService];
-        nginxService,
+      const expectedResult = [nginxExpectedResult, consulExpectedResult];
-        consulService
-      ];
-      const expectedResult = [
-        nginxExpectedResult,
-        consulExpectedResult
-      ];
       const result = processServices(services);
       expect(result).toEqual(expectedResult);
     });
 
     it('removes deleted services', () => {
-      const services = [{
+      const services = [
+        {
           status: 'DELETED'
-      }];
+        }
+      ];
       const expectedResult = [];
       const result = processServices(services);
       expect(result).toEqual(expectedResult);
@@ -146,16 +134,12 @@ describe('Redux selectors and Apollo helpers', () => {
   });
 
   describe('processServicesForTopology', () => {
-
     it('returns the services decorated with details for display correctly', () => {
       const services = [
         {
           ...nginxService,
           id: 'nginx-service-0',
-          connections: [
+          connections: ['consul-service-0', 'consul-service-1']
-            'consul-service-0',
-            'consul-service-1'
-          ]
         },
         {
           ...nginxService,
@@ -164,9 +148,7 @@ describe('Redux selectors and Apollo helpers', () => {
         {
           ...consulService,
           id: 'consul-service-0',
-          connections: [
+          connections: ['consul-service-1']
-            'consul-service-1'
-          ]
         },
         {
           ...consulService,
@@ -177,10 +159,7 @@ describe('Redux selectors and Apollo helpers', () => {
         {
           ...nginxExpectedResult,
           id: 'nginx-service-0',
-          connections: [
+          connections: ['consul-service-0', 'consul-service-1'],
-            'consul-service-0',
-            'consul-service-1'
-          ],
           connected: true,
           index: 0
         },
@@ -193,9 +172,7 @@ describe('Redux selectors and Apollo helpers', () => {
         {
           ...consulExpectedResult,
           id: 'consul-service-0',
-          connections: [
+          connections: ['consul-service-1'],
-            'consul-service-1'
-          ],
           connected: true,
           index: 2
         },

packages/cp-gql-mock-server/package.json

@@ -48,12 +48,8 @@
     "tap-xunit": "^1.7.0"
   },
   "ava": {
-    "files": [
+    "files": ["test/*.js"],
-      "test/*.js"
+    "source": ["src/*.js"],
-    ],
-    "source": [
-      "src/*.js"
-    ],
     "failFast": true
   }
 }

packages/cp-gql-mock-server/src/complex-data.json

@@ -268,8 +268,10 @@
     },
     {
       "id": "1ad6dd4e-15bd-4ea5-a0db-bf071d7958c4",
-      "slug": "extra service reported by containerpilot: redbaloonservice-https",
+      "slug":
-      "name": "Extra service reported by ContainerPilot: RedBaloonService-HTTPS",
+        "extra service reported by containerpilot: redbaloonservice-https",
+      "name":
+        "Extra service reported by ContainerPilot: RedBaloonService-HTTPS",
       "deploymentGroupId": "ba217234-9b1b-41a7-8079-08f9a4aadb0f",
       "status": "ACTIVE"
     },
@@ -578,7 +580,8 @@
       "deploymentGroupId": "ba217234-9b1b-41a7-8079-08f9a4aadb0f",
       "serviceId": "31663285-2b58-4f92-b6f5-3ef34591c3a3",
       "machineId": "c9a49cff-4438-460f-bc46-610bfecbddca",
-      "name": "instance-Extra service reported by ContainerPilot: CartService-HTTPS-0"
+      "name":
+        "instance-Extra service reported by ContainerPilot: CartService-HTTPS-0"
     },
     {
       "id": "instance-31663285-2b58-4f92-b6f5-3ef34591c3a3-1",
@@ -587,7 +590,8 @@
       "deploymentGroupId": "ba217234-9b1b-41a7-8079-08f9a4aadb0f",
       "serviceId": "31663285-2b58-4f92-b6f5-3ef34591c3a3",
       "machineId": "4252ddb5-e9b4-4d1e-aa53-6e1bdcdeab30",
-      "name": "instance-Extra service reported by ContainerPilot: CartService-HTTPS-1"
+      "name":
+        "instance-Extra service reported by ContainerPilot: CartService-HTTPS-1"
     },
     {
       "id": "instance-31663285-2b58-4f92-b6f5-3ef34591c3a3-2",
@@ -596,7 +600,8 @@
       "deploymentGroupId": "ba217234-9b1b-41a7-8079-08f9a4aadb0f",
       "serviceId": "31663285-2b58-4f92-b6f5-3ef34591c3a3",
       "machineId": "783c5eb2-2145-4fd2-a22f-274bec3b2ffc",
-      "name": "instance-Extra service reported by ContainerPilot: CartService-HTTPS-2"
+      "name":
+        "instance-Extra service reported by ContainerPilot: CartService-HTTPS-2"
     },
     {
       "id": "instance-31663285-2b58-4f92-b6f5-3ef34591c3a3-3",
@@ -605,7 +610,8 @@
       "deploymentGroupId": "ba217234-9b1b-41a7-8079-08f9a4aadb0f",
       "serviceId": "31663285-2b58-4f92-b6f5-3ef34591c3a3",
       "machineId": "f8a54a11-93f4-4b0a-9c80-be6542393448",
-      "name": "instance-Extra service reported by ContainerPilot: CartService-HTTPS-3"
+      "name":
+        "instance-Extra service reported by ContainerPilot: CartService-HTTPS-3"
     },
     {
       "id": "instance-3b954132-49fc-405c-9d91-c59b8953d7b8-0",
@@ -1747,7 +1753,8 @@
       "healthy": "UNKNOWN",
       "deploymentGroupId": "ba217234-9b1b-41a7-8079-08f9a4aadb0f",
       "serviceId": "1ad6dd4e-15bd-4ea5-a0db-bf071d7958c4",
-      "name": "instance-Extra service reported by ContainerPilot: RedBaloonService-HTTPS-0",
+      "name":
+        "instance-Extra service reported by ContainerPilot: RedBaloonService-HTTPS-0",
       "machineId": "20f5bf63-c7d7-4b80-bbe0-63dd744f1b72"
     },
     {
@@ -1756,7 +1763,8 @@
       "healthy": "UNKNOWN",
       "deploymentGroupId": "ba217234-9b1b-41a7-8079-08f9a4aadb0f",
       "serviceId": "1ad6dd4e-15bd-4ea5-a0db-bf071d7958c4",
-      "name": "instance-Extra service reported by ContainerPilot: RedBaloonService-HTTPS-1",
+      "name":
+        "instance-Extra service reported by ContainerPilot: RedBaloonService-HTTPS-1",
       "machineId": "bf10d972-17e0-4267-908e-4a8184d7c164"
     },
     {
@@ -1765,7 +1773,8 @@
       "healthy": "UNKNOWN",
       "deploymentGroupId": "ba217234-9b1b-41a7-8079-08f9a4aadb0f",
       "serviceId": "1ad6dd4e-15bd-4ea5-a0db-bf071d7958c4",
-      "name": "instance-Extra service reported by ContainerPilot: RedBaloonService-HTTPS-2",
+      "name":
+        "instance-Extra service reported by ContainerPilot: RedBaloonService-HTTPS-2",
       "machineId": "d7204a39-5005-4925-a2d5-02afbb2457db"
     },
     {
@@ -1774,7 +1783,8 @@
       "healthy": "UNKNOWN",
       "deploymentGroupId": "ba217234-9b1b-41a7-8079-08f9a4aadb0f",
       "serviceId": "1ad6dd4e-15bd-4ea5-a0db-bf071d7958c4",
-      "name": "instance-Extra service reported by ContainerPilot: RedBaloonService-HTTPS-3",
+      "name":
+        "instance-Extra service reported by ContainerPilot: RedBaloonService-HTTPS-3",
       "machineId": "80dac6f7-e459-4f42-b824-5abdd6f13d41"
     },
     {
@@ -1783,7 +1793,8 @@
       "healthy": "UNKNOWN",
       "deploymentGroupId": "ba217234-9b1b-41a7-8079-08f9a4aadb0f",
       "serviceId": "1ad6dd4e-15bd-4ea5-a0db-bf071d7958c4",
-      "name": "instance-Extra service reported by ContainerPilot: RedBaloonService-HTTPS-4",
+      "name":
+        "instance-Extra service reported by ContainerPilot: RedBaloonService-HTTPS-4",
       "machineId": "dbb819d5-9df2-48e1-8412-7cd48ca4c186"
     },
     {

packages/cp-gql-mock-server/src/resolvers.js

@@ -48,15 +48,9 @@ const cleanQuery = (q = {}) => JSON.parse(JSON.stringify(q));
 let metricDataIndex = 0;
 
 const getMetrics = query => {
-  const {
+  const { names, start, end, instanceId } = query;
-    names,
-    start,
-    end,
-    instanceId
-  } = query;
 
   const metrics = names.reduce((metrics, name) => {
-
     // pick one of the three metric data jsons, so there's variety
     const index = metricDataIndex % metricData.length;
     metricDataIndex++;
@@ -79,7 +73,10 @@ const getMetrics = query => {
     while (i < records) {
       const index = (randomIndex + i) % m.length; // loop if not enough data
       const requiredMetric = m[index];
-      requiredMetric.time = time.add(15, 'seconds').utc().format(); // we should have a new record every 15 secs
+      requiredMetric.time = time
+        .add(15, 'seconds')
+        .utc()
+        .format(); // we should have a new record every 15 secs
       requiredMetrics.push(requiredMetric);
       i++;
     }
@@ -90,13 +87,13 @@ const getMetrics = query => {
       start: s.utc().format(),
       end: time.utc().format(), // this will be used by the frontend for the next fetch
       metrics: requiredMetrics
-    }
+    };
     metrics.push(requiredMetricData);
     return metrics;
   }, []);
 
   return Promise.resolve(metrics);
-}
+};
 
 const getInstances = query => {
   const metricsResolver = ({ id }) => query =>
@@ -175,8 +172,7 @@ const getServices = query => {
       return ret;
     });
 
-  return Promise.resolve(services)
+  return Promise.resolve(services).then(services => {
-    .then((services) => {
     if (!services || !services.length) {
       throw Boom.notFound();
     }
@@ -199,7 +195,7 @@ const getDeploymentGroups = query => {
 
   return Promise.resolve(
     deploymentGroups.filter(find(cleanQuery(query))).map(addNestedResolvers)
-  ).then((deploymentGroups) => {
+  ).then(deploymentGroups => {
     if (!deploymentGroups || !deploymentGroups.length) {
       throw Boom.notFound();
     }
@@ -406,11 +402,16 @@ const updateServiceAndInstancesStatus = (
   instancesStatus
 ) => {
   return Promise.all([
-    getServices({ id: serviceId })/* ,
+    getServices({
+      id: serviceId
+    }) /* ,
     getServices({ parentId: serviceId }) */
   ])
     .then(services => {
-      return services.reduce((services, service) => services.concat(service), [])
+      return services.reduce(
+        (services, service) => services.concat(service),
+        []
+      );
     })
     .then(services => {
       updateServiceStatus(services, serviceStatus);
@@ -431,7 +432,9 @@ const updateServiceAndInstancesStatus = (
     })
     .then(() =>
       Promise.all([
-        getUnfilteredServices({ id: serviceId })/* ,
+        getUnfilteredServices({
+          id: serviceId
+        }) /* ,
         getUnfilteredServices({ parentId: serviceId }) */
       ])
     )
@@ -536,12 +539,7 @@ const parseEnvVars = (str = '') =>
 const findEnvInterpolation = (str = '') =>
   uniq(str.match(INTERPOLATE_REGEX).map(name => name.replace(/^\$/, '')));
 
-const config = ({
+const config = ({ environment = '', files = [], raw = '', _plain = false }) => {
-  environment = '',
-  files = [],
-  raw = '',
-  _plain = false
-}) => {
   const interpolatableNames = findEnvInterpolation(raw);
   const interpolatableEnv = parseEnvVars(environment);
 
@@ -591,8 +589,7 @@ const config = ({
             name,
             id: hasha(JSON.stringify(service.config.environment[name])),
             value: service.config.environment[name]
-            })
+          }))
-          )
         })
       })
     );

packages/cp-gql-schema/schema.gql

@@ -203,7 +203,6 @@ type Datacenter {
   region: String!
 }
 
-
 # we probably wont use some of these queries or arguments
 # but this way we expose the entire db through gql
 type Query {
@@ -265,7 +264,13 @@ type Query {
   ): [Service]
   importableDeploymentGroups: [DeploymentGroup]
   # start and end should be .toISOString() date strings
-  metrics(deploymentGroupId: ID!, names: [MetricName]!, instances: [ID]!, start: String!, end: String!): [InstanceMetric]
+  metrics(
+    deploymentGroupId: ID!
+    names: [MetricName]!
+    instances: [ID]!
+    start: String!
+    end: String!
+  ): [InstanceMetric]
 }
 
 type Mutation {

packages/normalized-styled-components/src/index.js

@@ -152,17 +152,17 @@ export const Button = styled.button`
   appearance: button;
 
   &::-moz-focus-inner,
-  &[type="button"]::-moz-focus-inner,
+  &[type='button']::-moz-focus-inner,
-  &[type="reset"]::-moz-focus-inner,
+  &[type='reset']::-moz-focus-inner,
-  &[type="submit"]::-moz-focus-inner {
+  &[type='submit']::-moz-focus-inner {
     border-style: none;
     padding: 0;
   }
 
   &:-moz-focusring,
-  &[type="button"]:-moz-focusring,
+  &[type='button']:-moz-focusring,
-  &[type="reset"]:-moz-focusring,
+  &[type='reset']:-moz-focusring,
-  &[type="submit"]:-moz-focusring {
+  &[type='submit']:-moz-focusring {
     outline: ${remcalc(1)} dotted ButtonText;
   }
 `;
@@ -174,24 +174,24 @@ export const Input = styled.input`
   margin: 0;
   overflow: visible;
 
-  &[type="checkbox"],
+  &[type='checkbox'],
-  &[type="radio"] {
+  &[type='radio'] {
     box-sizing: border-box;
     padding: 0;
   }
 
-  &[type="number"]::-webkit-inner-spin-button,
+  &[type='number']::-webkit-inner-spin-button,
-  &[type="number"]::-webkit-outer-spin-button {
+  &[type='number']::-webkit-outer-spin-button {
     height: auto;
   }
 
-  &[type="search"] {
+  &[type='search'] {
     appearance: textfield;
     outline-offset: ${remcalc(-2)};
   }
 
-  &[type="search"]::-webkit-search-cancel-button,
+  &[type='search']::-webkit-search-cancel-button,
-  &[type="search"]::-webkit-search-decoration {
+  &[type='search']::-webkit-search-decoration {
     appearance: none;
   }
 

packages/ui-toolkit/src/anchor/index.js

@@ -36,11 +36,7 @@ const Anchor = ({ children, ...rest }) => {
   const Views = [() => (to ? StyledLink : null), () => StyledAnchor];
   const View = Views.reduce((sel, view) => (sel ? sel : view()), null);
 
-  return (
+  return <View {...rest}>{children}</View>;
-    <View {...rest}>
-      {children}
-    </View>
-  );
 };
 
 Anchor.propTypes = {

packages/ui-toolkit/src/base/global.js

@@ -21,6 +21,6 @@ export default css`
     margin: 0;
     padding: 0;
     background: ${theme.background};
-    font-family: 'Libre Franklin'
+    font-family: 'Libre Franklin';
   }
 `;

packages/ui-toolkit/src/basealign/index.js

@@ -18,4 +18,6 @@ export default Component =>
     ? Component.extend`
         ${alignsFromProps};
       `
-    : styled(Component)`${alignsFromProps}`;
+    : styled(Component)`
+        ${alignsFromProps};
+      `;

packages/ui-toolkit/src/baseline/index.js

@@ -33,4 +33,6 @@ export default Component =>
     ? Component.extend`
         ${unitsFromProps};
       `
-    : styled(Component)`${unitsFromProps}`;
+    : styled(Component)`
+        ${unitsFromProps};
+      `;

packages/ui-toolkit/src/boxes/index.js

@@ -11,9 +11,9 @@ export const tooltipShadow = `0 ${remcalc(2)} ${remcalc(6)} ${remcalc(
 export const modalShadow = `0 0 ${remcalc(6)} ${remcalc(1)} rgba(0, 0, 0, 0.1)`;
 
 export const border = {
-  checked: css`${remcalc(1)} solid ${props => props.theme.primary}`,
+  checked: css`${remcalc(1)} solid ${props => props.theme.primary};`,
-  unchecked: css`${remcalc(1)} solid ${props => props.theme.grey}`,
+  unchecked: css`${remcalc(1)} solid ${props => props.theme.grey};`,
-  confirmed: css`${remcalc(1)} solid ${props => props.theme.grey}`,
+  confirmed: css`${remcalc(1)} solid ${props => props.theme.grey};`,
-  error: css`${remcalc(1)} solid ${props => props.theme.red}`,
+  error: css`${remcalc(1)} solid ${props => props.theme.red};`,
-  secondary: css`${remcalc(1)} solid ${props => props.theme.secondaryActive}`,
+  secondary: css`${remcalc(1)} solid ${props => props.theme.secondaryActive};`
 };

packages/ui-toolkit/src/breadcrumb/index.js

@@ -4,9 +4,10 @@ import { Row } from 'react-styled-flexboxgrid';
 /**
  * @example ./usage.md
  */
-export default ({ children, ...rest }) =>
+export default ({ children, ...rest }) => (
   <Row name="breadcrum" {...rest}>
     {children}
-  </Row>;
+  </Row>
+);
 
 export { default as Item } from './item';

packages/ui-toolkit/src/breadcrumb/item.js

@@ -21,10 +21,11 @@ const Arrow = styled.div`
   margin: ${remcalc(3)} ${remcalc(10)} ${remcalc(3)} ${remcalc(10)};
 `;
 
-export default ({ children, ...rest }) =>
+export default ({ children, ...rest }) => (
   <div>
     <Name name="breadcrum-item" {...rest}>
       {children}
     </Name>
     <Arrow />
-  </div>;
+  </div>
+);

packages/ui-toolkit/src/breakpoints/index.js

@@ -41,7 +41,7 @@ const screens = {
 
 const breakpoint = label => (...args) => css`
   @media ${screens[label]} {
-    ${css(...args)}
+    ${css(...args)};
   }
 `;
 

packages/ui-toolkit/src/button/index.js

@@ -180,15 +180,13 @@ const Button = props => {
 
   const View = Views.reduce((sel, view) => (sel ? sel : view()), null);
 
-  const children = loading
+  const children = loading ? (
-    ? <StatusLoader secondary={!secondary} tertiary={tertiary} />
+    <StatusLoader secondary={!secondary} tertiary={tertiary} />
-    : props.children;
+  ) : (
-
+    props.children
-  return (
-    <View {...props}>
-      {children}
-    </View>
   );
+
+  return <View {...props}>{children}</View>;
 };
 
 Button.propTypes = {

packages/ui-toolkit/src/card/card.js

@@ -77,11 +77,7 @@ const Card = ({
     );
   };
 
-  return (
+  return <Subscriber channel="card">{render}</Subscriber>;
-    <Subscriber channel="card">
-      {render}
-    </Subscriber>
-  );
 };
 
 Card.propTypes = {

packages/ui-toolkit/src/card/description.js

@@ -25,23 +25,18 @@ const InnerDescription = styled.div`
 `;
 
 const Description = ({ children, ...rest }) => {
-  const render = ({ collapsed = false }) =>
+  const render = ({ collapsed = false }) => (
     <StyledTitle
       collapsed={collapsed}
       name="card-description"
       xs={collapsed ? 6 : 12}
       {...rest}
     >
-      <InnerDescription collapsed={collapsed}>
+      <InnerDescription collapsed={collapsed}>{children}</InnerDescription>
-        {children}
+    </StyledTitle>
-      </InnerDescription>
-    </StyledTitle>;
-
-  return (
-    <Subscriber channel="card">
-      {render}
-    </Subscriber>
   );
+
+  return <Subscriber channel="card">{render}</Subscriber>;
 };
 
 Description.propTypes = {

packages/ui-toolkit/src/card/group-view.js

@@ -9,9 +9,8 @@ const StyledView = View.extend`
   background-color: ${props => props.grey};
 `;
 
-const GroupView = ({ children, ...rest }) =>
+const GroupView = ({ children, ...rest }) => (
-  <StyledView {...rest}>
+  <StyledView {...rest}>{children}</StyledView>
-    {children}
+);
-  </StyledView>;
 
 export default Baseline(GroupView);