fix: Refactor logger and fix badges in README

README.md

@@ -1,3 +1,8 @@
-# @nextcloud/upload [![npm last version](https://img.shields.io/npm/v/@nextcloud/upload.svg?style=flat-square)](https://www.npmjs.com/package/@nextcloud/upload) ![Codecov](https://img.shields.io/codecov/c/github/nextcloud/nextcloud-upload?style=flat-square)
+# @nextcloud/upload
+[![npm last version](https://img.shields.io/npm/v/@nextcloud/upload.svg?style=flat-square)](https://www.npmjs.com/package/@nextcloud/upload)
+![Codecov](https://img.shields.io/codecov/c/github/nextcloud-libraries/nextcloud-upload?style=flat-square)
 
 Nextcloud upload helpers for Nextcloud apps 
+
+## Usage
+📘 [API documentation](https://nextcloud-libraries.github.io/nextcloud-upload)

__tests__/tsconfig.json

@@ -1,9 +1,7 @@
 {
   "extends": "../tsconfig.json",
+  "include": ["./**/*.ts", "../lib"],
   "compilerOptions": {
-	// Allow us to pass js files to ts-jest without babel
-    "allowJs": true,
-  },
-  "include": ["./**/*.ts"],
+    "rootDir": ".."
+  }
 }
-

__tests__/utils/logger.spec.ts

@@ -0,0 +1,11 @@
+import { expect, test, vi } from 'vitest'
+import logger from '../../lib/utils/logger'
+
+// Just ensure correct app is set, rest is up to that library to test
+test('logger', () => {
+	const spy = vi.spyOn(window.console, 'warn')
+
+	logger.warn('test')
+	expect(spy).toHaveBeenCalled()
+	expect(spy.mock.calls[0][1]).toHaveProperty('app', '@nextcloud/upload')
+})

lib/utils/logger.ts

@@ -20,19 +20,9 @@
  *
  */
 
-import { getCurrentUser, type NextcloudUser } from '@nextcloud/auth'
 import { getLoggerBuilder } from '@nextcloud/logger'
 
-const getLogger = (user: NextcloudUser|null) => {
-	if (user === null) {
-		return getLoggerBuilder()
-			.setApp('uploader')
-			.build()
-	}
-	return getLoggerBuilder()
-		.setApp('uploader')
-		.setUid(user.uid)
-		.build()
-}
-
-export default getLogger(getCurrentUser())
+export default getLoggerBuilder()
+	.setApp('@nextcloud/upload')
+	.detectUser()
+	.build()