test & lint

Author: Luca Forstner

docs/migration/draft-v9-migration-guide.md

@@ -7,6 +7,7 @@
 - Deprecated `AddRequestDataToEventOptions.transaction`. This option effectively doesn't do anything anymore, and will
   be removed in v9.
 - Deprecated `TransactionNamingScheme` type.
+- Deprecated `urlEncode`. No replacements.
 
 ## `@sentry/core`
 

packages/core/test/lib/api.test.ts

@@ -18,7 +18,7 @@ describe('API', () => {
         dsnPublicComponents,
         undefined,
         undefined,
-        'https://sentry.io:1234/subpath/api/123/envelope/?sentry_key=abc&sentry_version=7',
+        'https://sentry.io:1234/subpath/api/123/envelope/?sentry_version=7&sentry_key=abc',
       ],
       ['uses `tunnel` value when called with `tunnel` option', dsnPublicComponents, tunnel, undefined, tunnel],
       [
@@ -33,7 +33,7 @@ describe('API', () => {
         dsnPublicComponents,
         undefined,
         sdkInfo,
-        'https://sentry.io:1234/subpath/api/123/envelope/?sentry_key=abc&sentry_version=7&sentry_client=sentry.javascript.browser%2F12.31.12',
+        'https://sentry.io:1234/subpath/api/123/envelope/?sentry_version=7&sentry_key=abc&sentry_client=sentry.javascript.browser%2F12.31.12',
       ],
     ])(
       '%s',

packages/utils/src/index.ts

@@ -149,6 +149,7 @@ export {
   memoBuilder,
   arrayify,
   normalizeUrlToBase,
+  // eslint-disable-next-line deprecation/deprecation
   urlEncode,
   // eslint-disable-next-line deprecation/deprecation
   extractPathForTransaction,