Apply auto-fixes

Author: watson

integration-tests/aiguard/index.spec.js

@@ -12,7 +12,7 @@ const { executeRequest } = require('./util')
 
 function assertHasGuardSpan (payload, predicate) {
   const spans = payload[0].filter(span => span.name === 'ai_guard')
-  assert.ok(spans.length > 0)
+  assert.ok(spans.length > 0, `Expected ${spans.length} > 0`)
   const matching = spans.find(predicate)
   assert.notStrictEqual(matching, undefined)
 }

integration-tests/appsec/iast-esbuild.spec.js

@@ -56,7 +56,7 @@ describe('esbuild support for IAST', () => {
 
         const ddStack = msgpack.decode(span.meta_struct['_dd.stack'])
         assert.ok(Object.hasOwn(ddStack.vulnerability[0], 'frames'))
-        assert.ok(ddStack.vulnerability[0].frames.length > 0)
+        assert.ok(ddStack.vulnerability[0].frames.length > 0, `Expected ${ddStack.vulnerability[0].frames.length} > 0`)
       })
     }, null, 1, true)
   }

integration-tests/cucumber/cucumber.spec.js

@@ -1229,7 +1229,7 @@ describe(`cucumber@${version} commonJS`, () => {
 
               // Only tests from the non-skipped suite ran
               const tests = events.filter(event => event.type === 'test').map(event => event.content)
-              assert.ok(tests.length > 0)
+              assert.ok(tests.length > 0, `Expected ${tests.length} > 0`)
               tests.forEach(test => {
                 assert.ok(!test.meta[TEST_SUITE].includes('farewell'))
               })
@@ -2563,7 +2563,10 @@ describe(`cucumber@${version} commonJS`, () => {
           'nyc output does not match the reported coverage (no --all flag)')
 
         eventsPromise.then(() => {
-          assert.ok(codeCoverageWithoutUntestedFiles > codeCoverageWithUntestedFiles)
+          assert.ok(
+            codeCoverageWithoutUntestedFiles > codeCoverageWithUntestedFiles,
+            `Expected ${codeCoverageWithoutUntestedFiles} > ${codeCoverageWithUntestedFiles}`
+          )
           done()
         }).catch(done)
       })
@@ -2904,7 +2907,7 @@ describe(`cucumber@${version} commonJS`, () => {
             const atfTests = tests.filter(
               t => t.meta[TEST_MANAGEMENT_IS_ATTEMPT_TO_FIX] === 'true'
             )
-            assert.ok(atfTests.length > 0)
+            assert.ok(atfTests.length > 0, `Expected ${atfTests.length} > 0`)
             for (const test of atfTests) {
               assert.ok(
                 !(TEST_IS_NEW in test.meta),
@@ -3434,7 +3437,7 @@ describe(`cucumber@${version} commonJS`, () => {
           .gatherPayloadsMaxTimeout(({ url }) => url.endsWith('/api/v2/citestcycle'), payloads => {
             const metadataDicts = payloads.flatMap(({ payload }) => payload.metadata)
 
-            assert.ok(metadataDicts.length > 0)
+            assert.ok(metadataDicts.length > 0, `Expected ${metadataDicts.length} > 0`)
             metadataDicts.forEach(metadata => {
               assert.strictEqual(metadata.test[DD_CAPABILITIES_TEST_IMPACT_ANALYSIS], '1')
               assert.strictEqual(metadata.test[DD_CAPABILITIES_EARLY_FLAKE_DETECTION], '1')

integration-tests/cypress/cypress-impacted-tests.spec.js

@@ -159,7 +159,7 @@ moduleTypes.forEach(({
                 .filter(({ payload }) => payload.metadata?.test)
                 .flatMap(({ payload }) => payload.metadata)
 
-              assert.ok(metadataDicts.length > 0)
+              assert.ok(metadataDicts.length > 0, `Expected ${metadataDicts.length} > 0`)
               metadataDicts.forEach(metadata => {
                 assert.strictEqual(metadata.test[DD_CAPABILITIES_TEST_IMPACT_ANALYSIS], '1')
                 assert.strictEqual(metadata.test[DD_CAPABILITIES_EARLY_FLAKE_DETECTION], '1')

integration-tests/cypress/cypress-itr.spec.js

@@ -538,7 +538,7 @@ moduleTypes.forEach(({
         const eventsPromise = gatherCypressPayloads(receiver, childProcess, '/api/v2/citestcycle', payloads => {
           const events = payloads.flatMap(({ payload }) => payload.events)
           const tests = events.filter(event => event.type === 'test').map(event => event.content)
-          assert.ok(tests.length > 0)
+          assert.ok(tests.length > 0, `Expected ${tests.length} > 0`)
           tests.forEach(test => {
             assert.strictEqual(test.itr_correlation_id, itrCorrelationId)
           })
@@ -621,7 +621,7 @@ moduleTypes.forEach(({
 
         const testEvents = events.filter(event => event.type === 'test')
         const testModuleEvent = events.find(event => event.type === 'test_module_end')
-        assert.ok(testEvents.length > 0)
+        assert.ok(testEvents.length > 0, `Expected ${testEvents.length} > 0`)
         assert.ok(testModuleEvent)
 
         testEvents.forEach(testEvent => {

integration-tests/cypress/cypress-reporting.spec.js

@@ -1111,7 +1111,7 @@ moduleTypes.forEach(({
             const testSessionEvent = events.find(event => event.type === 'test_session_end')
             assert.ok(testSessionEvent)
             const testEvents = events.filter(event => event.type === 'test')
-            assert.ok(testEvents.length > 0)
+            assert.ok(testEvents.length > 0, `Expected ${testEvents.length} > 0`)
           }, { hardTimeout: 30000 })
 
       await Promise.all([

integration-tests/cypress/cypress-test-management.spec.js

@@ -589,7 +589,7 @@ moduleTypes.forEach(({
               const atfTests = tests.filter(
                 t => t.meta[TEST_MANAGEMENT_IS_ATTEMPT_TO_FIX] === 'true'
               )
-              assert.ok(atfTests.length > 0)
+              assert.ok(atfTests.length > 0, `Expected ${atfTests.length} > 0`)
               for (const test of atfTests) {
                 assert.ok(
                   !(TEST_IS_NEW in test.meta),

integration-tests/debugger/snapshot-global-sample-rate.spec.js

@@ -61,14 +61,14 @@ describe('Dynamic Instrumentation', function () {
               const timeSincePrevTimestamp = timestamp - prevTimestamp
 
               // Allow for a time variance (time will tell if this is enough). Timeouts can vary.
-              assert.ok(duration >= 925)
-              assert.ok(duration < 1050)
+              assert.ok(duration >= 925, `Expected ${duration} >= 925`)
+              assert.ok(duration < 1050, `Expected ${duration} < 1050`)
 
               // A sanity check to make sure we're not saturating the event loop. We expect a lot of snapshots to be
               // sampled in the beginning of the sample window and then once the threshold is hit, we expect a "quiet"
               // period until the end of the window. If there's no "quiet" period, then we're saturating the event loop
               // and this test isn't really testing anything.
-              assert.ok(timeSincePrevTimestamp >= 250)
+              assert.ok(timeSincePrevTimestamp >= 250, `Expected ${timeSincePrevTimestamp} >= 250`)
 
               clearTimeout(state[rcConfig1.config.id].timer)
               clearTimeout(state[rcConfig2.config.id].timer)

integration-tests/debugger/snapshot-pruning.spec.js

@@ -14,7 +14,7 @@ describe('Dynamic Instrumentation', function () {
       it('should prune snapshot if payload is too large', function (done) {
         t.agent.on('debugger-input', ({ payload: [payload] }) => {
           const payloadSize = Buffer.byteLength(JSON.stringify(payload))
-          assert.ok(payloadSize < 1024 * 1024) // 1MB
+          assert.ok(payloadSize < 1024 * 1024, `Expected ${payloadSize} < ${1024 * 1024}`) // 1MB
 
           const capturesJson = JSON.stringify(payload.debugger.snapshot.captures)
           assert.match(capturesJson, /"pruned":true/)

integration-tests/debugger/snapshot.spec.js

@@ -204,7 +204,7 @@ describe('Dynamic Instrumentation', function () {
           if ('fields' in prop) {
             if (prop.notCapturedReason === 'fieldCount') {
               assert.strictEqual(Object.keys(prop.fields).length, maxFieldCount)
-              assert.ok(prop.size > maxFieldCount)
+              assert.ok(prop.size > maxFieldCount, `Expected ${prop.size} > ${maxFieldCount}`)
             } else {
               assert.ok(Object.keys(prop.fields).length < maxFieldCount)
             }
@@ -228,12 +228,12 @@ describe('Dynamic Instrumentation', function () {
           assert.strictEqual(locals.request.type, 'Request')
           assert.strictEqual(Object.keys(locals.request.fields).length, maxFieldCount)
           assert.strictEqual(locals.request.notCapturedReason, 'fieldCount')
-          assert.ok(locals.request.size > maxFieldCount)
+          assert.ok(locals.request.size > maxFieldCount, `Expected ${locals.request.size} > ${maxFieldCount}`)
 
           assert.strictEqual(locals.fastify.type, 'Object')
           assert.strictEqual(Object.keys(locals.fastify.fields).length, maxFieldCount)
           assert.strictEqual(locals.fastify.notCapturedReason, 'fieldCount')
-          assert.ok(locals.fastify.size > maxFieldCount)
+          assert.ok(locals.fastify.size > maxFieldCount, `Expected ${locals.fastify.size} > ${maxFieldCount}`)
 
           for (const value of Object.values(locals)) {
             assertMaxFieldCount(value)
@@ -300,7 +300,7 @@ describe('Dynamic Instrumentation', function () {
           const { raw } = captures.lines[t.breakpoint.line].locals.request.fields
           assert.strictEqual(raw.notCapturedReason, 'fieldCount')
           assert.strictEqual(Object.keys(raw.fields).length, 20)
-          assert.ok(raw.size > 20)
+          assert.ok(raw.size > 20, `Expected ${raw.size} > 20`)
           done()
         })