For Building

cbuildci.yml → .cbuildci.yml

@@ -4,3 +4,4 @@ builds:
   build:
     timeoutInMinutes: 5
     image: 'aws/codebuild/nodejs:8.11.0'
+    useCache: true

app/app.js

@@ -16,8 +16,14 @@ import { ConnectedRouter } from 'connected-react-router/immutable';
 import createHistory from 'history/createBrowserHistory';
 // import 'sanitize.css/sanitize.css';
 
-// Import root app
+// Import root app and constants
 import App from 'containers/App';
+import {
+    WINDOW_BLUR,
+    WINDOW_FOCUS,
+    WINDOW_VISIBLE,
+    WINDOW_HIDDEN,
+} from 'containers/App/constants';
 
 // Import providers
 import LanguageProvider from 'containers/LanguageProvider';
@@ -49,6 +55,23 @@ history.listen((location, action) => {
 });
 
 const store = configureStore(initialState, history);
+
+// Dispatch events on window focus/visibility events.
+window.addEventListener('blur', () => {
+    store.dispatch({ type: WINDOW_BLUR });
+});
+window.addEventListener('focus', () => {
+    store.dispatch({ type: WINDOW_FOCUS });
+});
+window.addEventListener('visibilitychange', () => {
+    if (document.visibilityState === 'visible') {
+        store.dispatch({ type: WINDOW_VISIBLE });
+    }
+    if (document.visibilityState === 'hidden') {
+        store.dispatch({ type: WINDOW_HIDDEN });
+    }
+});
+
 const MOUNT_NODE = document.getElementById('app');
 
 const render = (messages) => {

app/components/CommitMeta/index.js

@@ -15,7 +15,7 @@ function CommitMeta({
             {author.login && (
                 <span className="mr-1" style={{ lineHeight: '22px' }}>
                     <img
-                        src={`https://github.com/${author.login}.png`}
+                        src={`https://github.com/${author.login}.png?size=60`}
                         width="20"
                         height="20"
                         style={{ marginTop: '-2px', borderRadius: '4px' }}

app/components/ExecutionStartMessage/tests/__snapshots__/index.test.js.snap

@@ -7,7 +7,7 @@ exports[`<ExecutionStartMessage/> should render expected JSX for "opened" pull r
             synchronize {{actionHtml} to}
             other {{actionHtml}}
         }
-        {pullRequestLinkHtml}"
+        {pullRequestLinkHtml} by {usernameHtml}"
   id="app.components.ExecutionStartMessage.startedForPullRequest"
   values={
     Object {
@@ -27,7 +27,7 @@ exports[`<ExecutionStartMessage/> should render expected JSX for "opened" pull r
           }
         />
       </strong>,
-      "event": "pull_request",
+      "eventType": "pull_request",
       "pullRequestLinkHtml": <a
         href="https://git.com/foo/bar/pull/5"
       >
@@ -51,6 +51,12 @@ exports[`<ExecutionStartMessage/> should render expected JSX for "opened" pull r
           />
         </InjectIntl(Time)>
       </strong>,
+      "username": "bsmith",
+      "usernameHtml": <a
+        href="https://git.com/bsmith"
+      >
+        bsmith
+      </a>,
     }
   }
 />
@@ -86,6 +92,12 @@ exports[`<ExecutionStartMessage/> should render expected JSX for "opened" pull r
       pull request #5
     </span>
   </a>
+   by 
+  <a
+    href="https://git.com/bsmith"
+  >
+    bsmith
+  </a>
 </span>
 `;
 
@@ -98,7 +110,7 @@ exports[`<ExecutionStartMessage/> should render expected JSX for "rerun" user ev
   id="app.components.ExecutionStartMessage.startedByUserAction"
   values={
     Object {
-      "event": "check_run",
+      "eventType": "check_run",
       "identifier": "rerun",
       "identifierHtml": <strong>
         <FormattedMessage
@@ -124,12 +136,11 @@ exports[`<ExecutionStartMessage/> should render expected JSX for "rerun" user ev
           />
         </InjectIntl(Time)>
       </strong>,
-      "username": "amekkawi-office",
+      "username": "bsmith",
       "usernameHtml": <a
-        href="https://git.com/amekkawi-office"
+        href="https://git.com/bsmith"
       >
-        @
-        amekkawi-office
+        bsmith
       </a>,
     }
   }
@@ -159,9 +170,9 @@ exports[`<ExecutionStartMessage/> should render expected JSX for "rerun" user ev
   </strong>
    initiated by 
   <a
-    href="https://git.com/amekkawi-office"
+    href="https://git.com/bsmith"
   >
-    @amekkawi-office
+    bsmith
   </a>
 </span>
 `;
@@ -173,7 +184,7 @@ exports[`<ExecutionStartMessage/> should render expected JSX for "synchronize" p
             synchronize {{actionHtml} to}
             other {{actionHtml}}
         }
-        {pullRequestLinkHtml}"
+        {pullRequestLinkHtml} by {usernameHtml}"
   id="app.components.ExecutionStartMessage.startedForPullRequest"
   values={
     Object {
@@ -193,7 +204,7 @@ exports[`<ExecutionStartMessage/> should render expected JSX for "synchronize" p
           }
         />
       </strong>,
-      "event": "pull_request",
+      "eventType": "pull_request",
       "pullRequestLinkHtml": <a
         href="https://git.com/foo/bar/pull/5"
       >
@@ -217,6 +228,12 @@ exports[`<ExecutionStartMessage/> should render expected JSX for "synchronize" p
           />
         </InjectIntl(Time)>
       </strong>,
+      "username": "bsmith",
+      "usernameHtml": <a
+        href="https://git.com/bsmith"
+      >
+        bsmith
+      </a>,
     }
   }
 />
@@ -252,16 +269,22 @@ exports[`<ExecutionStartMessage/> should render expected JSX for "synchronize" p
       pull request #5
     </span>
   </a>
+   by 
+  <a
+    href="https://git.com/bsmith"
+  >
+    bsmith
+  </a>
 </span>
 `;
 
 exports[`<ExecutionStartMessage/> should render expected JSX for unknown event 1`] = `
 <FormattedMessage
-  defaultMessage="Started {relativeCreateTimeHtml} after event \\"{event}\\""
+  defaultMessage="Started {relativeCreateTimeHtml} after event \\"{eventType}\\""
   id="app.components.ExecutionStartMessage.startedWithUnknownEvent"
   values={
     Object {
-      "event": "foobar",
+      "eventType": "foobar",
       "relativeCreateTimeHtml": <strong>
         <InjectIntl(Time)
           date={1500000000000}
@@ -305,7 +328,7 @@ exports[`<ExecutionStartMessage/> should render expected JSX for unknown user ev
   id="app.components.ExecutionStartMessage.startedByUserAction"
   values={
     Object {
-      "event": "check_run",
+      "eventType": "check_run",
       "identifier": "foobar",
       "identifierHtml": <strong>
         <FormattedMessage
@@ -331,12 +354,11 @@ exports[`<ExecutionStartMessage/> should render expected JSX for unknown user ev
           />
         </InjectIntl(Time)>
       </strong>,
-      "username": "amekkawi-office",
+      "username": "bsmith",
       "usernameHtml": <a
-        href="https://git.com/amekkawi-office"
+        href="https://git.com/bsmith"
       >
-        @
-        amekkawi-office
+        bsmith
       </a>,
     }
   }
@@ -366,9 +388,9 @@ exports[`<ExecutionStartMessage/> should render expected JSX for unknown user ev
   </strong>
    initiated by 
   <a
-    href="https://git.com/amekkawi-office"
+    href="https://git.com/bsmith"
   >
-    @amekkawi-office
+    bsmith
   </a>
 </span>
 `;

app/components/ExecutionStartMessage/tests/index.test.js

@@ -22,7 +22,7 @@ describe('<ExecutionStartMessage/>', () => {
                 repo="bar"
                 createTime={1500000000000}
                 event={{
-                    event: 'foobar',
+                    type: 'foobar',
                 }}
             />,
         );
@@ -36,11 +36,16 @@ describe('<ExecutionStartMessage/>', () => {
                 repo="bar"
                 createTime={1500000000000}
                 event={{
-                    event: 'pull_request',
+                    type: 'pull_request',
                     action: 'opened',
                     pull_request: {
                         number: 5,
                     },
+                    sender: {
+                        login: 'bsmith',
+                        type: 'User',
+                        id: 13106037,
+                    },
                 }}
             />,
         );
@@ -54,11 +59,16 @@ describe('<ExecutionStartMessage/>', () => {
                 repo="bar"
                 createTime={1500000000000}
                 event={{
-                    event: 'pull_request',
+                    type: 'pull_request',
                     action: 'synchronize',
                     pull_request: {
                         number: 5,
                     },
+                    sender: {
+                        login: 'bsmith',
+                        type: 'User',
+                        id: 13106037,
+                    },
                 }}
             />,
         );
@@ -72,13 +82,13 @@ describe('<ExecutionStartMessage/>', () => {
                 repo="bar"
                 createTime={1500000000000}
                 event={{
-                    event: 'check_run',
+                    type: 'check_run',
                     action: 'requested_action',
                     requested_action: {
                         identifier: 'rerun',
                     },
                     sender: {
-                        login: 'amekkawi-office',
+                        login: 'bsmith',
                         type: 'User',
                         id: 13106037,
                     },
@@ -95,13 +105,13 @@ describe('<ExecutionStartMessage/>', () => {
                 repo="bar"
                 createTime={1500000000000}
                 event={{
-                    event: 'check_run',
+                    type: 'check_run',
                     action: 'requested_action',
                     requested_action: {
                         identifier: 'foobar',
                     },
                     sender: {
-                        login: 'amekkawi-office',
+                        login: 'bsmith',
                         type: 'User',
                         id: 13106037,
                     },

app/components/ExecutionStopMessage/tests/__snapshots__/index.test.js.snap

@@ -72,7 +72,6 @@ exports[`<ExecutionStopMessage/> should render expected JSX when user requests s
       "stopUserHtml": <a
         href="https://git.com/foo"
       >
-        @
         foo
       </a>,
     }
@@ -99,7 +98,7 @@ exports[`<ExecutionStopMessage/> should render expected JSX when user requests s
   <a
     href="https://git.com/foo"
   >
-    @foo
+    foo
   </a>
 
 </span>
@@ -140,7 +139,6 @@ exports[`<ExecutionStopMessage/> should render expected JSX when user requests s
       "stopUserHtml": <a
         href="https://git.com/foo"
       >
-        @
         foo
       </a>,
     }
@@ -167,7 +165,7 @@ exports[`<ExecutionStopMessage/> should render expected JSX when user requests s
   <a
     href="https://git.com/foo"
   >
-    @foo
+    foo
   </a>
    and took 
   <strong>

app/components/ExecutionsBreadcrumb/index.js

@@ -23,7 +23,7 @@ function ExecutionsBreadcrumb({
     if (repo != null) {
         if (commit == null) {
             repoHtml = (
-                <li className="breadcrumb-item">
+                <li className="breadcrumb-item active">
                     <i className="fab fa-github fa-fw mr-1"/>
                     {owner}/{repo}
                 </li>
@@ -46,7 +46,7 @@ function ExecutionsBreadcrumb({
             commitHtml = (
                 <li className="breadcrumb-item active">
                     <i className="fas fa-code-branch fa-fw mr-1"/>
-                    {commit.substr(0, 10)}
+                    <code>{commit.substr(0, 10)}</code>
                 </li>
             );
         }
@@ -55,7 +55,7 @@ function ExecutionsBreadcrumb({
                 <li className="breadcrumb-item">
                     <Link to={`/repo/${owner}/${repo}/commit/${commit}`}>
                         <i className="fas fa-code-branch fa-fw mr-1"/>
-                        {commit.substr(0, 10)}
+                        <code>{commit.substr(0, 10)}</code>
                     </Link>
                 </li>
             );

app/components/PageHeader/tests/__snapshots__/index.test.js.snap

@@ -1,23 +1,14 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`<PageHeader /> should render expected JSX 1`] = `
-<h1
-  className="mt-3 pb-2"
-  style={
-    Object {
-      "borderBottom": "1px solid #CCC",
-      "paddingBottom": "8px",
-    }
-  }
->
+<PageHeader__H1>
   Foobar
-</h1>
+</PageHeader__H1>
 `;
 
 exports[`<PageHeader /> should render expected JSX 2`] = `
 <h1
-  class="mt-3 pb-2"
-  style="padding-bottom:8px;border-bottom:1px solid #CCC"
+  class="PageHeader__H1-cGQWxJ bjGECH"
 >
   Foobar
 </h1>