fix: improve test robustness and docs for per-request timeout

- Add success handlers to all per-request timeout integration tests so
  they fail fast instead of hanging if timeout is not applied
- Remove accidental .releaserc change from this branch
- Clarify README docs for using options with the global pool

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: dhensby

.releaserc

@@ -4,4 +4,3 @@ plugins:
   - '@semantic-release/release-notes-generator'
   - '@semantic-release/npm'
   - '@semantic-release/github'
-  - '@semantic-release/git'

README.md

@@ -870,6 +870,8 @@ The optional `options` argument allows per-request configuration overrides:
 const request = new sql.Request(pool, { requestTimeout: 60000 })
 ```
 
+**Note:** When using the global pool, you must still pass `undefined` as the first argument to use options: `new sql.Request(undefined, { requestTimeout: 60000 })`.
+
 ### Events
 
 - **recordset(columns)** - Dispatched when metadata for new recordset are parsed.

test/common/tests.js

@@ -1205,7 +1205,10 @@ module.exports = (sql, driver) => {
 
       new sql.ConnectionPool(config).connect().then(conn => {
         const req = new sql.Request(conn, { requestTimeout: 1000 })
-        req.query('waitfor delay \'00:00:05\';select 1').catch(err => {
+        req.query('waitfor delay \'00:00:05\';select 1').then(() => {
+          conn.close()
+          done(new Error('Expected request to timeout'))
+        }).catch(err => {
           assert.ok((message ? (message.exec(err.message) != null) : (err instanceof sql.RequestError)))
 
           conn.close()
@@ -1246,7 +1249,12 @@ module.exports = (sql, driver) => {
         const tx = new sql.Transaction(conn, { requestTimeout: 1000 })
         tx.begin().then(() => {
           const req = tx.request()
-          req.query('waitfor delay \'00:00:05\';select 1').catch(err => {
+          req.query('waitfor delay \'00:00:05\';select 1').then(() => {
+            tx.rollback().catch(() => {}).then(() => {
+              conn.close()
+              done(new Error('Expected request to timeout'))
+            })
+          }).catch(err => {
             assert.ok((message ? (message.exec(err.message) != null) : (err instanceof sql.RequestError)))
 
             tx.rollback().then(() => {
@@ -1271,7 +1279,10 @@ module.exports = (sql, driver) => {
 
       new sql.ConnectionPool(config).connect().then(conn => {
         const req = new sql.Request(conn, { requestTimeout: 1000 })
-        req.execute('__testDelay').catch(err => {
+        req.execute('__testDelay').then(() => {
+          conn.close()
+          done(new Error('Expected request to timeout'))
+        }).catch(err => {
           assert.ok((message ? (message.exec(err.message) != null) : (err instanceof sql.RequestError)))
 
           conn.close()
@@ -1288,7 +1299,12 @@ module.exports = (sql, driver) => {
       new sql.ConnectionPool(config).connect().then(conn => {
         const ps = new sql.PreparedStatement(conn, { requestTimeout: 1000 })
         ps.prepare('waitfor delay \'00:00:05\';select 1').then(() => {
-          ps.execute().catch(err => {
+          ps.execute().then(() => {
+            ps.unprepare().catch(() => {}).then(() => {
+              conn.close()
+              done(new Error('Expected request to timeout'))
+            })
+          }).catch(err => {
             assert.ok((message ? (message.exec(err.message) != null) : (err instanceof sql.RequestError)))
 
             ps.unprepare().then(() => {