Diagnose tests failing only in CI

Author: smashwilson

lib/controllers/reviews-controller.js

@@ -292,15 +292,19 @@ export class BareReviewsController extends React.Component {
   addSingleComment = async (commentBody, threadID, replyToID, commentPath, position, callbacks = {}) => {
     let pendingReviewID = null;
     try {
+      console.log(`setting state: postingToThreadID=${threadID}`);
       this.setState({postingToThreadID: threadID});
 
+      console.log('calling addReviewMutation');
       const reviewResult = await addReviewMutation(this.props.relay.environment, {
         pullRequestID: this.props.pullRequest.id,
         viewerID: this.props.viewer.id,
       });
       const reviewID = reviewResult.addPullRequestReview.reviewEdge.node.id;
       pendingReviewID = reviewID;
+      console.log(`addReviewMutation returned: reviewID=${reviewID} pendingReviewID=${pendingReviewID}`);
 
+      console.log('calling addReviewCommentMutation');
       const commentPromise = addReviewCommentMutation(this.props.relay.environment, {
         body: commentBody,
         inReplyTo: replyToID,
@@ -310,28 +314,36 @@ export class BareReviewsController extends React.Component {
         path: commentPath,
         position,
       });
+      console.log('firing didSubmitComment callback');
       if (callbacks.didSubmitComment) {
         callbacks.didSubmitComment();
       }
+      console.log('awaiting comment promise');
       await commentPromise;
+      console.log('comment promise resolved successfully');
       pendingReviewID = null;
 
+      console.log('calling submitReviewMutation');
       await submitReviewMutation(this.props.relay.environment, {
         event: 'COMMENT',
         reviewID,
       });
+      console.log('submitReviewMutation resolved successfully');
       addEvent('add-single-comment', {package: 'github'});
     } catch (error) {
+      console.log(`Caught an error: ${error.stack}`);
       if (callbacks.didFailComment) {
         callbacks.didFailComment();
       }
 
       if (pendingReviewID !== null) {
         try {
+          console.log('Calling deleteReviewMutation');
           await deleteReviewMutation(this.props.relay.environment, {
             reviewID: pendingReviewID,
             pullRequestID: this.props.pullRequest.id,
           });
+          console.log('deleteReviewMutation completed');
         } catch (e) {
           /* istanbul ignore else */
           if (error.errors && e.errors) {

test/controllers/reviews-controller.test.js

@@ -291,7 +291,7 @@ describe('ReviewsController', function() {
   });
 
   describe('adding a single comment', function() {
-    it('creates a review, attaches the comment, and submits it', async function() {
+    it.only('creates a review, attaches the comment, and submits it', async function() {
       expectRelayQuery({
         name: addPrReviewMutation.operation.name,
         variables: {
@@ -347,7 +347,7 @@ describe('ReviewsController', function() {
       assert.isFalse(didFailComment.called);
     });
 
-    it('creates a notification when the review cannot be created', async function() {
+    it.only('creates a notification when the review cannot be created', async function() {
       const reportRelayError = sinon.spy();
 
       expectRelayQuery({
@@ -378,7 +378,7 @@ describe('ReviewsController', function() {
       assert.isTrue(didFailComment.called);
     });
 
-    it('creates a notification and deletes the review when the comment cannot be added', async function() {
+    it.only('creates a notification and deletes the review when the comment cannot be added', async function() {
       const reportRelayError = sinon.spy();
 
       expectRelayQuery({