Fix all hints discovered by the new linter `npm run eslintfix` Change-Id: Ic07aede1b95e7dd53bbd05c062e2afcef5508c10
diff --git a/polygerrit-ui/app/elements/change/gr-change-view/gr-change-view.ts b/polygerrit-ui/app/elements/change/gr-change-view/gr-change-view.ts index 6cda197..d2f4485 100644 --- a/polygerrit-ui/app/elements/change/gr-change-view/gr-change-view.ts +++ b/polygerrit-ui/app/elements/change/gr-change-view/gr-change-view.ts
@@ -654,12 +654,10 @@ getPluginLoader() .awaitPluginsLoaded() .then(() => { - this._dynamicTabHeaderEndpoints = getPluginEndpoints().getDynamicEndpoints( - 'change-view-tab-header' - ); - this._dynamicTabContentEndpoints = getPluginEndpoints().getDynamicEndpoints( - 'change-view-tab-content' - ); + this._dynamicTabHeaderEndpoints = + getPluginEndpoints().getDynamicEndpoints('change-view-tab-header'); + this._dynamicTabContentEndpoints = + getPluginEndpoints().getDynamicEndpoints('change-view-tab-content'); if ( this._dynamicTabContentEndpoints.length !== this._dynamicTabHeaderEndpoints.length @@ -811,9 +809,8 @@ * Changes active primary tab. */ _setActivePrimaryTab(e: SwitchTabEvent) { - const primaryTabs = this.shadowRoot!.querySelector<PaperTabsElement>( - '#primaryTabs' - ); + const primaryTabs = + this.shadowRoot!.querySelector<PaperTabsElement>('#primaryTabs'); const activeTabName = this._setActiveTab( primaryTabs, { @@ -831,12 +828,10 @@ activeTabName ); if (pluginIndex !== -1) { - this._selectedTabPluginEndpoint = this._dynamicTabContentEndpoints[ - pluginIndex - ]; - this._selectedTabPluginHeader = this._dynamicTabHeaderEndpoints[ - pluginIndex - ]; + this._selectedTabPluginEndpoint = + this._dynamicTabContentEndpoints[pluginIndex]; + this._selectedTabPluginHeader = + this._dynamicTabHeaderEndpoints[pluginIndex]; } else { this._selectedTabPluginEndpoint = ''; this._selectedTabPluginHeader = ''; @@ -1859,10 +1854,10 @@ // TODO(TS): code needs second thought, // it might be that nulls were assigned to trigger some bindings if (!change.topic) { - change.topic = (null as unknown) as undefined; + change.topic = null as unknown as undefined; } if (!change.reviewer_updates) { - change.reviewer_updates = (null as unknown) as undefined; + change.reviewer_updates = null as unknown as undefined; } const latestRevisionSha = this._getLatestRevisionSHA(change); if (!latestRevisionSha) @@ -2411,9 +2406,10 @@ _handleFileActionTap(e: CustomEvent<{path: string; action: string}>) { e.preventDefault(); - const controls = this.$.fileListHeader.shadowRoot!.querySelector<GrEditControls>( - '#editControls' - ); + const controls = + this.$.fileListHeader.shadowRoot!.querySelector<GrEditControls>( + '#editControls' + ); if (!controls) throw new Error('Missing edit controls'); assertIsDefined(this._change, '_change'); if (!this._patchRange)
diff --git a/polygerrit-ui/app/elements/change/gr-change-view/gr-change-view_test.ts b/polygerrit-ui/app/elements/change/gr-change-view/gr-change-view_test.ts index ba4a91a..5508ee0 100644 --- a/polygerrit-ui/app/elements/change/gr-change-view/gr-change-view_test.ts +++ b/polygerrit-ui/app/elements/change/gr-change-view/gr-change-view_test.ts
@@ -1041,9 +1041,8 @@ const expectedStatuses = [ChangeStates.MERGED, ChangeStates.WIP]; assert.deepEqual(element._changeStatuses, expectedStatuses); flush(); - const statusChips = element.shadowRoot!.querySelectorAll( - 'gr-change-status' - ); + const statusChips = + element.shadowRoot!.querySelectorAll('gr-change-status'); assert.equal(statusChips.length, 2); }); @@ -1459,7 +1458,8 @@ .stub(element, 'loadData') .callsFake(() => Promise.resolve()); const collapseStub = sinon.stub(element.$.fileList, 'collapseAllDiffs'); - const value: AppElementChangeViewParams = createAppElementChangeViewParams(); + const value: AppElementChangeViewParams = + createAppElementChangeViewParams(); element.params = value; await flush(); assert.isTrue(reloadStub.calledOnce); @@ -1474,7 +1474,8 @@ const recreateSpy = sinon.spy(); element.addEventListener('recreate-change-view', recreateSpy); - const value: AppElementChangeViewParams = createAppElementChangeViewParams(); + const value: AppElementChangeViewParams = + createAppElementChangeViewParams(); element.params = value; await flush(); assert.isFalse(recreateSpy.calledOnce); @@ -1873,10 +1874,10 @@ const div = document.createElement('div'); element.$.replyDialog.draft = '> quote text\n\n some draft text'; element.$.replyDialog.quote = '> quote text\n\n'; - const e = ({ + const e = { target: div, preventDefault: sinon.spy(), - } as unknown) as MouseEvent; + } as unknown as MouseEvent; element._handleReplyTap(e); assert.equal( element.$.replyDialog.draft,