opendev/gerrit
Code Issues Proposed changes

Fix implicitly declared variables

Browse Source 
Some variables were used without declaration. Doesn't work with es-6
modules.

Change-Id: I4c99137eaba05cf2ae19891a9d0c7127db95b9ed
This commit is contained in:
Dmitrii Filippov
2020-03-04 17:24:57 +01:00
parent 8d152e1e35
commit eefacb0757
18 changed files with 26 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
polygerrit-ui/app/elements/change/gr-messages-list/gr-messages-list_test.html
View File

@@ -149,7 +149,6 @@ limitations under the License.
// comment API.
commentApiWrapper = fixture('basic');
element = commentApiWrapper.$.messagesList;
loadCommentSpy = sandbox.spy(commentApiWrapper.$.commentAPI, 'loadAll');
element.messages = messages;
// Stub methods on the changeComments object after changeComments has
@@ -476,7 +475,7 @@ limitations under the License.
// comment API.
commentApiWrapper = fixture('basic');
element = commentApiWrapper.$.messagesList;
loadCommentSpy = sandbox.spy(commentApiWrapper.$.commentAPI, 'loadAll');
sandbox.spy(commentApiWrapper.$.commentAPI, 'loadAll');
element.messages = messages;
// Stub methods on the changeComments object after changeComments has
@@ -506,7 +505,7 @@ limitations under the License.
element._hideAutomated = false;
MockInteractions.tap(element.$.automatedMessageToggle);
flushAsynchronousOperations();
allMessageEls = getMessages();
const allMessageEls = getMessages();
allHiddenMessageEls = getHiddenMessages();
// Autogenerated messages are now hidden.