[service bus] Update tests to use secrets safely (Azure#33431)

Updates the SB tests as follows:
- Currently the connection string is expected to be loaded from a .env
saved on disk. This PR changes this by accessing the connection strings
safely using ARM requests authenticated using Entra ID
- Updates connection-string-based tests to be skipped if the connection
string is not available

Additionally, it deletes a sample that was using a SAS key which is a
risky auth method that we should no longer advertise. It also refreshes
the samples.

Live tests run:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=4666221&view=results

Author: deyaaeldeen

common/config/rush/pnpm-lock.yaml

@@ -36,7 +36,7 @@
     "build:samples": "echo Obsolete.",
     "build:test": "echo skipped. actual commands inlined in browser test scripts",
     "check-format": "dev-tool run vendored prettier --list-different --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.ts\" \"test/**/*.ts\" \"samples-dev/**/*.ts\" \"*.{js,json}\"",
-    "clean": "dev-tool run vendored rimraf --glob dist dist-* types *.tgz *.log coverage coverage-browser .nyc_output",
+    "clean": "dev-tool run vendored rimraf --glob dist dist-* types *.tgz *.log coverage coverage-browser .nyc_output *.tsbuildinfo",
     "execute:samples": "dev-tool samples run samples-dev",
     "extract-api": "dev-tool run build-package && dev-tool run extract-api",
     "format": "dev-tool run vendored prettier --write --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"samples/**/*.{ts,js}\" \"src/**/*.ts\" \"test/**/*.ts\" \"samples-dev/**/*.ts\" \"*.{js,json}\"",
@@ -109,6 +109,7 @@
     "@azure-tools/test-recorder": "^4.1.0",
     "@azure-tools/test-utils-vitest": "^1.0.0",
     "@azure-tools/vite-plugin-browser-test-map": "^1.0.0",
+    "@azure/arm-servicebus": "^6.1.0",
     "@azure/dev-tool": "^1.0.0",
     "@azure/eslint-plugin-azure-sdk": "^3.0.0",
     "@azure/identity": "^4.7.0",

sdk/search/search-documents/samples-dev/vectors.ts renamed to sdk/search/search-documents/assets/vectors.ts

@@ -18,7 +18,6 @@ These sample programs show how to use the JavaScript client libraries for Azure
 | [topicSubscriptionWithRuleOperationsSample.js][topicsubscriptionwithruleoperationssample] | Demonstrates how to filter messages in Service Bus                                                                                                             |
 | [receiveMessagesLoop.js][receivemessagesloop]                                             | Demonstrates how to receive Service Bus messages in a loop                                                                                                     |
 | [receiveMessagesStreaming.js][receivemessagesstreaming]                                   | Demonstrates how to receive Service Bus messages in a stream                                                                                                   |
-| [usingNamedKeyCredential.js][usingnamedkeycredential]                                     | This sample demonstrates how to authenticate using AzureNamedKeyCredential                                                                                     |
 | [browseMessages.js][browsemessages]                                                       | Demonstrates how to browse a Service Bus message                                                                                                               |
 | [deleteMessages.js][deletemessages]                                                       | Demonstrates deleting messages from a queue.                                                                                                                   |
 | [session.js][session]                                                                     | Demonstrates how to send/receive messages to/from session enabled queues/subscriptions in Service Bus                                                          |
@@ -78,7 +77,6 @@ Take a look at our [API Documentation][apiref] for more information about the AP
 [topicsubscriptionwithruleoperationssample]: https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/servicebus/service-bus/samples/v7/javascript/topicSubscriptionWithRuleOperationsSample.js
 [receivemessagesloop]: https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/servicebus/service-bus/samples/v7/javascript/receiveMessagesLoop.js
 [receivemessagesstreaming]: https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/servicebus/service-bus/samples/v7/javascript/receiveMessagesStreaming.js
-[usingnamedkeycredential]: https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/servicebus/service-bus/samples/v7/javascript/usingNamedKeyCredential.js
 [browsemessages]: https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/servicebus/service-bus/samples/v7/javascript/browseMessages.js
 [deletemessages]: https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/servicebus/service-bus/samples/v7/javascript/deleteMessages.js
 [session]: https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/servicebus/service-bus/samples/v7/javascript/session.js

sdk/servicebus/service-bus/package.json

@@ -13,8 +13,7 @@ const { ServiceBusAdministrationClient } = require("@azure/service-bus");
 const { DefaultAzureCredential } = require("@azure/identity");
 
 // Load the .env file if it exists
-require("dotenv").config();
-
+require("dotenv/config");
 // Define connection string and related Service Bus entity names here
 const fqdn = process.env.SERVICEBUS_FQDN || "<your-servicebus-namespace>.servicebus.windows.net";
 const queueName = process.env.QUEUE_NAME || "<queue name>";

sdk/servicebus/service-bus/samples-dev/usingNamedKeyCredential.ts

@@ -17,8 +17,7 @@ const { ServiceBusClient, delay } = require("@azure/service-bus");
 const { DefaultAzureCredential } = require("@azure/identity");
 
 // Load the .env file if it exists
-require("dotenv").config();
-
+require("dotenv/config");
 // Define connection string and related Service Bus entity names here
 const fqdn = process.env.SERVICEBUS_FQDN || "<your-servicebus-namespace>.servicebus.windows.net";
 const queueName = process.env.QUEUE_NAME || "<queue name>";

sdk/servicebus/service-bus/samples/v7/javascript/README.md

@@ -12,8 +12,7 @@ const { ServiceBusAdministrationClient } = require("@azure/service-bus");
 const { DefaultAzureCredential } = require("@azure/identity");
 
 // Load the .env file if it exists
-require("dotenv").config();
-
+require("dotenv/config");
 // Define connection string and related Service Bus entity names here
 const fqdn = process.env.SERVICEBUS_FQDN || "<your-servicebus-namespace>.servicebus.windows.net";
 

sdk/servicebus/service-bus/samples/v7/javascript/advanced/administrationClient.js

@@ -15,8 +15,7 @@ const { ServiceBusClient } = require("@azure/service-bus");
 const { DefaultAzureCredential } = require("@azure/identity");
 
 // Load the .env file if it exists
-require("dotenv").config();
-
+require("dotenv/config");
 // Define connection string and related Service Bus entity names here
 const fqdn = process.env.SERVICEBUS_FQDN || "<your-servicebus-namespace>.servicebus.windows.net";
 const queueName = process.env.QUEUE_NAME || "<queue name>";

sdk/servicebus/service-bus/samples/v7/javascript/advanced/deferral.js

@@ -16,8 +16,7 @@ const { ServiceBusClient } = require("@azure/service-bus");
 const { DefaultAzureCredential } = require("@azure/identity");
 
 // Load the .env file if it exists
-require("dotenv").config();
-
+require("dotenv/config");
 // Define connection string and related Service Bus entity names here
 const fqdn = process.env.SERVICEBUS_FQDN || "<your-servicebus-namespace>.servicebus.windows.net";
 const queueName = process.env.QUEUE_NAME || "<queue name>";

sdk/servicebus/service-bus/samples/v7/javascript/advanced/listingEntities.js

@@ -12,7 +12,7 @@
 const { ServiceBusAdministrationClient, ServiceBusClient } = require("@azure/service-bus");
 const { DefaultAzureCredential } = require("@azure/identity");
 
-require("dotenv").config();
+require("dotenv/config");
 
 async function main() {
   // Define connection string and related Service Bus entity names here
@@ -89,7 +89,7 @@ async function main() {
 
   console.log("Deleting topic...");
   await serviceBusAdminClient.deleteTopic(topicName);
-  client.close();
+  await client.close();
 }
 
 main().catch((err) => {

sdk/servicebus/service-bus/samples/v7/javascript/advanced/movingMessagesToDLQ.js

@@ -13,8 +13,7 @@
 const { ServiceBusClient, delay, isServiceBusError } = require("@azure/service-bus");
 const { DefaultAzureCredential } = require("@azure/identity");
 
-require("dotenv").config();
-
+require("dotenv/config");
 const fqdn = process.env.SERVICEBUS_FQDN || "<your-servicebus-namespace>.servicebus.windows.net";
 
 // NOTE: this sample uses a session enabled queue but would also work a session enabled subscription.

sdk/servicebus/service-bus/samples/v7/javascript/advanced/processMessageFromDLQ.js

@@ -25,8 +25,7 @@ const { ServiceBusClient } = require("@azure/service-bus");
 const { DefaultAzureCredential } = require("@azure/identity");
 
 // Load the .env file if it exists
-require("dotenv").config();
-
+require("dotenv/config");
 // Define connection string and related Service Bus entity names here
 const fqdn = process.env.SERVICEBUS_FQDN || "<your-servicebus-namespace>.servicebus.windows.net";
 const userEventsQueueName = process.env.QUEUE_NAME_WITH_SESSIONS || "<queue name>";