Test

[jainakanksha-msft]

No description provided.

[Copilot]

Pull request overview

This PR modernizes Azurite’s dependencies (notably Azure SDK packages) and updates the codebase/tests to align with newer SDK typings and runtime behaviors (headers access, credential access, typed-array handling, and generated swagger defaults).

Changes:

• Upgrades core dependencies (e.g., @azure/ms-rest-js, @azure/storage-blob, @azure/storage-queue, axios, tedious) and adjusts tests accordingly.
• Refactors multiple tests and runtime helpers to use Uint8Array-based buffers and updated SDK option/property names (e.g., visibilityTimeout, _response.headers.get(...), client.credential).
• Updates generated swagger/artifacts to remove the default value for x-ms-blob-sequence-number and tightens Express query handling in generated request adapters.

Reviewed changes

Copilot reviewed 30 out of 39 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
tests/testutils.ts	Formats file creation loops; comments out overrideRequest helper.
tests/table/apis/table.entity.rest.test.ts	Adds non-null assertions for etag headers.
tests/table/apis/table.batch.errorhandling.test.ts	Removes RestError import and loosens error typing; minor formatting.
tests/simpleTokenCredential.ts	Switches TokenCredential-related imports to @azure/identity.
tests/queue/queueSas.test.ts	Uses (serviceClient as any).credential instead of pipeline factory introspection.
tests/queue/queueCorsRequest.test.ts	Asserts CORS headers via _response.headers.get(...) rather than response-shape properties.
tests/queue/oauth.test.ts	Updates error detail property casing and cleans up formatting.
tests/queue/apis/messages.test.ts	Renames visibilitytimeout to visibilityTimeout; adjusts timing for an expiration test.
tests/linuxbinary.test.ts	Comments out the entire linux binary test file.
tests/blob/sas.test.ts	Uses client.credential; updates hard-coded expected SAS strings.
tests/blob/oauth.test.ts	Updates error detail property casing.
tests/blob/fsStore.test.ts	Changes chunk buffering from Buffer[] to Uint8Array[].
tests/blob/blockblob.highlevel.test.ts	Updates buffer comparisons and abort logic for upload stream test.
tests/blob/blobCorsRequest.test.ts	Mirrors queue CORS assertion changes using _response.headers.get(...).
tests/blob/apis/service.test.ts	Updates expected error detail property casing; uses client.credential.
tests/blob/apis/blockblob.test.ts	Simplifies MD5 typed-array construction.
tests/blob/apis/blobbatch.test.ts	Uses client.credential for batch SAS signing.
tests/blob/apis/blob.test.ts	Removes node/browser branch for MD5 header; normalizes to Uint8Array.
swagger/blob.md	Notes removal of default for blob sequence number header parameter.
swagger/blob-storage-2021-10-04.json	Removes "default": 0 from BlobSequenceNumber parameter.
src/table/generated/utils/xml.ts	Adjusts xml2js parse callback typing; removes builder options.
src/table/generated/ExpressRequestAdapter.ts	Ensures getQuery() only returns strings.
src/queue/utils/utils.ts	Adjusts xml2js parse callback typing.
src/queue/generated/utils/xml.ts	Adjusts xml2js parse callback typing; removes builder options.
src/queue/generated/ExpressRequestAdapter.ts	Ensures getQuery() only returns strings.
src/common/utils/utils.ts	Normalizes HMAC/MD5 helpers to return/accept Uint8Array.
src/blob/persistence/SqlBlobMetadataStore.ts	Formatting tweaks; normalizes restored byte arrays to Uint8Array.
src/blob/persistence/LokiBlobMetadataStore.ts	Changes persisted-byte-array restore logic (Buffer/Uint8Array handling).
src/blob/main.ts	Minor formatting cleanup.
src/blob/handlers/SubResponseTextBodyStream.ts	Reformats class; changes end() typing to return this.
src/blob/handlers/BlobBatchSubResponse.ts	Minor formatting updates.
src/blob/handlers/BlobBatchHandler.ts	Changes stream-to-buffer copy to use Uint8Array conversion.
src/blob/handlers/AppendBlobHandler.ts	Normalizes contentMD5 to Uint8Array.
src/blob/generated/utils/xml.ts	Adjusts xml2js parse callback typing; removes builder options.
src/blob/generated/ExpressRequestAdapter.ts	Ensures getQuery() only returns strings.
src/blob/generated/artifacts/parameters.ts	Removes defaultValue: 0 for blob sequence number header.
src/blob/authentication/BlobSharedKeyAuthenticator.ts	Formatting changes plus string-to-sign line reordering.
package.json	Upgrades dependencies; removes azure-storage; adds @azure/identity.

Comments suppressed due to low confidence (1)

src/blob/persistence/LokiBlobMetadataStore.ts:3121

• restoreUint8Array no longer handles the JSON-serialized Buffer shape ({ type: "Buffer", data: [...] }). Existing Loki DB files created by earlier versions can contain this shape, so this change can break loading persisted metadata after upgrade. Consider keeping backward-compatible handling for obj.type === "Buffer" (using obj.data) in addition to the Buffer instance case.

  private restoreUint8Array(obj: any): Uint8Array | undefined {
    if (typeof obj !== "object") {
      return undefined;
    }

    if (obj instanceof Buffer) {
      return new Uint8Array(obj);
    }

    const length = Object.keys(obj).length;
    const arr = Buffer.allocUnsafe(length);

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 39 out of 48 changed files in this pull request and generated 14 comments.

[Copilot]

Pull request overview

Copilot reviewed 41 out of 50 changed files in this pull request and generated 7 comments.

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.