From f6f1095d936740bec52f65e68162804a1402d675 Mon Sep 17 00:00:00 2001
From: seotts <sotts@scratch.mit.edu>
Date: Tue, 23 Mar 2021 16:33:24 -0400
Subject: [PATCH] add justMuted variable, fix linting

---
 src/views/preview/comment/compose-comment.jsx |  6 +++-
 test/unit/components/compose-comment.test.jsx | 29 ++++++++++++-------
 2 files changed, 23 insertions(+), 12 deletions(-)

diff --git a/src/views/preview/comment/compose-comment.jsx b/src/views/preview/comment/compose-comment.jsx
index 77b1c9199..c6baa356e 100644
--- a/src/views/preview/comment/compose-comment.jsx
+++ b/src/views/preview/comment/compose-comment.jsx
@@ -292,7 +292,11 @@ class ComposeComment extends React.Component {
                         <CommentingStatus>
                             <p>
                                 <FormattedMessage
-                                    id={this.getMuteMessageInfo(this.state.status === ComposeStatus.REJECTED_MUTE).commentType}
+                                    id={
+                                        this.getMuteMessageInfo(
+                                            this.state.status === ComposeStatus.REJECTED_MUTE
+                                        ).commentType
+                                    }
                                 />
                             </p>
                             <p>
diff --git a/test/unit/components/compose-comment.test.jsx b/test/unit/components/compose-comment.test.jsx
index c8e8aa194..ddd0ff4ae 100644
--- a/test/unit/components/compose-comment.test.jsx
+++ b/test/unit/components/compose-comment.test.jsx
@@ -556,45 +556,52 @@ describe('Compose Comment test', () => {
     });
 
     test('getMuteMessageInfo: muteType set and just got muted', () => {
+        const justMuted = true;
         const commentInstance = getComposeCommentWrapper({}).instance();
         commentInstance.setState({muteType: 'unconstructive'});
-        expect(commentInstance.getMuteMessageInfo(true).commentType).toBe('comment.type.unconstructive');
-        expect(commentInstance.getMuteMessageInfo(true).muteStepContent[0]).toBe('comment.unconstructive.content1');
+        expect(commentInstance.getMuteMessageInfo(justMuted).commentType).toBe('comment.type.unconstructive');
+        expect(commentInstance.getMuteMessageInfo(justMuted)
+            .muteStepContent[0]).toBe('comment.unconstructive.content1');
     });
 
     test('getMuteMessageInfo: muteType set and already muted', () => {
+        const justMuted = false;
         const commentInstance = getComposeCommentWrapper({}).instance();
         commentInstance.setState({muteType: 'pii'});
-        expect(commentInstance.getMuteMessageInfo(false).commentType).toBe('comment.type.pii.past');
+        expect(commentInstance.getMuteMessageInfo(justMuted).commentType).toBe('comment.type.pii.past');
         // PII has the same content1 regardless of whether you were just muted
-        expect(commentInstance.getMuteMessageInfo(false).muteStepContent[0]).toBe('comment.pii.content1');
+        expect(commentInstance.getMuteMessageInfo(justMuted).muteStepContent[0]).toBe('comment.pii.content1');
 
         commentInstance.setState({muteType: 'vulgarity'});
-        expect(commentInstance.getMuteMessageInfo(false).commentType).toBe('comment.type.vulgarity.past');
-        expect(commentInstance.getMuteMessageInfo(false).muteStepContent[0]).toBe('comment.type.vulgarity.past');
+        expect(commentInstance.getMuteMessageInfo(justMuted).commentType).toBe('comment.type.vulgarity.past');
+        expect(commentInstance.getMuteMessageInfo(justMuted).muteStepContent[0]).toBe('comment.type.vulgarity.past');
     });
 
     test('getMuteMessageInfo: muteType not set and just got muted', () => {
+        const justMuted = true;
         const commentInstance = getComposeCommentWrapper({}).instance();
-        expect(commentInstance.getMuteMessageInfo(true).commentType).toBe('comment.type.general');
+        expect(commentInstance.getMuteMessageInfo(justMuted).commentType).toBe('comment.type.general');
         // general has the same content1 regardless of whether you were just muted
-        expect(commentInstance.getMuteMessageInfo(true).muteStepContent[0]).toBe('comment.general.content1');
+        expect(commentInstance.getMuteMessageInfo(justMuted).muteStepContent[0]).toBe('comment.general.content1');
     });
 
     test('getMuteMessageInfo: muteType not set and already muted', () => {
+        const justMuted = false;
         const commentInstance = getComposeCommentWrapper({}).instance();
-        expect(commentInstance.getMuteMessageInfo(false).commentType).toBe('comment.type.general.past');
+        expect(commentInstance.getMuteMessageInfo(justMuted).commentType).toBe('comment.type.general.past');
     });
 
     test('getMuteMessageInfo: muteType set to something we don\'t have messages for and just got muted', () => {
+        const justMuted = true;
         const commentInstance = getComposeCommentWrapper({}).instance();
         commentInstance.setState({muteType: 'spaghetti'});
-        expect(commentInstance.getMuteMessageInfo(true).commentType).toBe('comment.type.general');
+        expect(commentInstance.getMuteMessageInfo(justMuted).commentType).toBe('comment.type.general');
     });
 
     test('getMuteMessageInfo: muteType set to something we don\'t have messages for and already muted', () => {
+        const justMuted = false;
         const commentInstance = getComposeCommentWrapper({}).instance();
         commentInstance.setState({muteType: 'spaghetti'});
-        expect(commentInstance.getMuteMessageInfo(false).commentType).toBe('comment.type.general.past');
+        expect(commentInstance.getMuteMessageInfo(justMuted).commentType).toBe('comment.type.general.past');
     });
 });