chore: upgrade typescript, eslint and fix lint

Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>

Author: skjnldsv

.eslintrc.js

@@ -3,21 +3,18 @@ module.exports = {
 		__webpack_nonce__: true,
 		_: true,
 		$: true,
-		moment: true,
-		escapeHTML: true,
-		oc_userconfig: true,
 		dayNames: true,
+		escapeHTML: true,
 		firstDay: true,
-		'cypress/globals': true,
-	},
-	parserOptions: {
-		parser: '@typescript-eslint/parser',
+		moment: true,
+		oc_userconfig: true,
+		sinon: true,
 	},
 	plugins: [
 		'cypress',
 	],
 	extends: [
-		'@nextcloud',
+		'@nextcloud/eslint-config/typescript',
 		'plugin:cypress/recommended',
 	],
 	rules: {

__mocks__/@nextcloud/dialogs.ts

@@ -0,0 +1,30 @@
+/**
+ * @copyright Copyright (c) 2023 John Molakvoæ <skjnldsv@protonmail.com>
+ *
+ * @author John Molakvoæ <skjnldsv@protonmail.com>
+ *
+ * @license AGPL-3.0-or-later
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ *
+ */
+
+import { jest } from '@jest/globals'
+
+export const showMessage = jest.fn()
+export const showSuccess = jest.fn()
+export const showWarning = jest.fn()
+export const showInfo = jest.fn()
+export const showError = jest.fn()
+export const showUndo = jest.fn()

__tests__/tsconfig.json

@@ -0,0 +1,6 @@
+{
+	"extends": "../tsconfig.json",
+	"compilerOptions": {
+		"verbatimModuleSyntax": false
+	}
+}

apps/files/src/actions/editLocallyAction.spec.ts

@@ -21,11 +21,11 @@
  */
 import { action } from './editLocallyAction'
 import { expect } from '@jest/globals'
-import { File, Folder, Permission } from '@nextcloud/files'
+import { File, Permission } from '@nextcloud/files'
 import { FileAction } from '../services/FileAction'
+import * as ncDialogs from '@nextcloud/dialogs'
 import axios from '@nextcloud/axios'
 import type { Navigation } from '../services/Navigation'
-import ncDialogs from '@nextcloud/dialogs'
 
 const view = {
 	id: 'files',
@@ -140,7 +140,7 @@ describe('Edit locally action execute tests', () => {
 
 	test('Edit locally fails and show error', async () => {
 		jest.spyOn(axios, 'post').mockImplementation(async () => ({}))
-		jest.spyOn(ncDialogs, 'showError')
+		jest.spyOn(ncDialogs, 'showError').mockImplementation(async () => ({}))
 
 		const file = new File({
 			id: 1,

apps/files/src/actions/openFolderAction.spec.ts

@@ -21,7 +21,7 @@
  */
 import { action } from './openFolderAction'
 import { expect } from '@jest/globals'
-import { File, Folder, Permission } from '@nextcloud/files'
+import { File, Folder, Node, Permission } from '@nextcloud/files'
 import { FileAction } from '../services/FileAction'
 import type { Navigation } from '../services/Navigation'
 
@@ -139,8 +139,7 @@ describe('Open folder action execute tests', () => {
 		const goToRouteMock = jest.fn()
 		window.OCP = { Files: { Router: { goToRoute: goToRouteMock } } }
 
-		// @ts-ignore null as Node
-		const exec = await action.exec(null, view, '/')
+		const exec = await action.exec(null as unknown as Node, view, '/')
 		expect(exec).toBe(false)
 		expect(goToRouteMock).toBeCalledTimes(0)
 	})

apps/files/src/actions/sidebarAction.ts

@@ -65,4 +65,3 @@ export const action = new FileAction({
 })
 
 registerFileAction(action)
-

apps/files/src/components/SidebarTab.vue

@@ -1,4 +1,3 @@
-
 <!--
   - @copyright Copyright (c) 2019 John Molakvoæ <skjnldsv@protonmail.com>
   -

apps/files/src/services/RouterService.ts

@@ -19,27 +19,26 @@
  * along with this program. If not, see <http://www.gnu.org/licenses/>.
  *
  */
-import type { Route } from 'vue-router';
-import type VueRouter from 'vue-router';
-import type { Dictionary } from 'vue-router/types/router';
-import type { Location } from 'vue-router/types/router';
+import type { Route } from 'vue-router'
+import type VueRouter from 'vue-router'
+import type { Dictionary, Location } from 'vue-router/types/router'
 
 export default class RouterService {
 
-	private _router: VueRouter;
+	private _router: VueRouter
 
 	constructor(router: VueRouter) {
 		this._router = router
 	}
 
 	/**
 	 * Trigger a route change on the files app
-	 * 
+	 *
 	 * @param path the url path, eg: '/trashbin?dir=/Deleted'
 	 * @param replace replace the current history
 	 * @see https://router.vuejs.org/guide/essentials/navigation.html#navigate-to-a-different-location
 	 */
-	goTo(path: string, replace: boolean = false): Promise<Route> {
+	goTo(path: string, replace = false): Promise<Route> {
 		return this._router.push({
 			path,
 			replace,
@@ -68,4 +67,5 @@ export default class RouterService {
 			replace,
 		} as Location)
 	}
+
 }

apps/files/src/views/Navigation.cy.ts

@@ -161,7 +161,7 @@ describe('Quota rendering', () => {
 		// TODO: remove when @nextcloud/l10n 2.0 is released
 		// https://github.com/nextcloud/nextcloud-l10n/pull/542
 		cy.stub(L10n, 'translate', (app, text, vars = {}, number) => {
-			cy.log({app, text, vars, number})
+			cy.log({ app, text, vars, number })
 			return text.replace(/%n/g, '' + number).replace(/{([^{}]*)}/g, (match, key) => {
 				return vars[key]
 			})

apps/files_sharing/src/components/SharingEntryInternal.vue

@@ -1,4 +1,3 @@
-
 <template>
 	<ul>
 		<SharingEntrySimple ref="shareEntrySimple"