diff --git a/.github/.OwlBot.lock.yaml b/.github/.OwlBot.lock.yaml index e7c45fd3..9d507eee 100644 --- a/.github/.OwlBot.lock.yaml +++ b/.github/.OwlBot.lock.yaml @@ -1,3 +1,3 @@ docker: image: gcr.io/repo-automation-bots/owlbot-nodejs:latest - digest: sha256:c9c7828c165b1985579098978877935ee52dda2b1b538087514fd24fa2443e7a + digest: sha256:41d5457ff79c3945782ab7e23bf4d617fd7bf3f2b03b6d84808010f7d2e10ca2 diff --git a/.github/auto-approve.yml b/.github/auto-approve.yml index 90369797..a79ba66c 100644 --- a/.github/auto-approve.yml +++ b/.github/auto-approve.yml @@ -4,4 +4,9 @@ rules: changedFiles: - "package\\.json$" - "CHANGELOG\\.md$" - maxFiles: 3 \ No newline at end of file + maxFiles: 3 +- author: "renovate-bot" + title: "^(fix\\(deps\\)|chore\\(deps\\)):" + changedFiles: + - "/package\\.json$" + maxFiles: 2 diff --git a/.kokoro/test.sh b/.kokoro/test.sh index 5d6383fc..b5646aeb 100755 --- a/.kokoro/test.sh +++ b/.kokoro/test.sh @@ -32,6 +32,9 @@ if [[ $KOKORO_BUILD_ARTIFACTS_SUBDIR = *"continuous"* ]] || [[ $KOKORO_BUILD_ART } trap cleanup EXIT HUP fi +# Unit tests exercise the entire API surface, which may include +# deprecation warnings: +export MOCHA_THROW_DEPRECATION=false npm test # codecov combines coverage across integration and unit tests. Include diff --git a/CHANGELOG.md b/CHANGELOG.md index 4c268d28..015cbb1f 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,5 +1,12 @@ # Node.js Agent for Google Cloud Debug ChangeLog +### [5.2.6](https://www.github.com/googleapis/cloud-debug-nodejs/compare/v5.2.5...v5.2.6) (2021-07-06) + + +### Bug Fixes + +* Add more debugging info to the system e2e test ([#983](https://www.github.com/googleapis/cloud-debug-nodejs/issues/983)) ([5f670ee](https://www.github.com/googleapis/cloud-debug-nodejs/commit/5f670eefbb5828a9c211854f535ddce573c63d29)) + ### [5.2.5](https://www.github.com/googleapis/cloud-debug-nodejs/compare/v5.2.4...v5.2.5) (2021-06-24) diff --git a/package.json b/package.json index ca057490..787956a8 100644 --- a/package.json +++ b/package.json @@ -1,6 +1,6 @@ { "name": "@google-cloud/debug-agent", - "version": "5.2.5", + "version": "5.2.6", "author": "Google Inc.", "description": "Stackdriver Debug Agent for Node.js", "main": "./build/src/index", @@ -60,7 +60,7 @@ "@compodoc/compodoc": "1.1.11", "@types/acorn": "^4.0.2", "@types/console-log-level": "^1.4.0", - "@types/estree": "0.0.48", + "@types/estree": "0.0.49", "@types/extend": "^3.0.0", "@types/mocha": "^8.0.0", "@types/mv": "^2.1.0", diff --git a/samples/package.json b/samples/package.json index df50881d..d471277e 100644 --- a/samples/package.json +++ b/samples/package.json @@ -17,7 +17,7 @@ "test": "mocha" }, "dependencies": { - "@google-cloud/debug-agent": "^5.2.5", + "@google-cloud/debug-agent": "^5.2.6", "express": "4.17.1" }, "devDependencies": { diff --git a/system-test/test-e2e.ts b/system-test/test-e2e.ts index 90795052..d58b7984 100644 --- a/system-test/test-e2e.ts +++ b/system-test/test-e2e.ts @@ -68,7 +68,7 @@ describe('@google-cloud/debug end-to-end behavior', () => { debuggeeId: string; projectId: string; }) => { - console.log(c); + console.log('handler received:', c); if (c.error) { reject(new Error('A child reported the following error: ' + c.error)); return; @@ -91,6 +91,7 @@ describe('@google-cloud/debug end-to-end behavior', () => { } numChildrenReady++; if (numChildrenReady === CLUSTER_WORKERS) { + console.log('All children are ready'); resolve(); } }; @@ -105,7 +106,7 @@ describe('@google-cloud/debug end-to-end behavior', () => { }; for (let i = 0; i < CLUSTER_WORKERS; i++) { - // Fork child processes that sned messages to this process with IPC. + // Fork child processes that send messages to this process with IPC. // We pass UUID to the children so that they can all get the same // debuggee id. const child: Child = {transcript: ''}; @@ -126,8 +127,8 @@ describe('@google-cloud/debug end-to-end behavior', () => { afterEach(function () { this.timeout(5 * 1000); // Create a promise for each child that resolves when that child exits. - const childExitPromises = children.map(child => { - console.log(child.transcript); + const childExitPromises = children.map((child, index) => { + console.log(`child ${index} transcript: ===#`, child.transcript, '#==='); assert(child.process); const childProcess = child.process as cp.ChildProcess; childProcess.kill(); @@ -164,7 +165,10 @@ describe('@google-cloud/debug end-to-end behavior', () => { assert.ok(debuggees, 'should get a valid ListDebuggees response'); const result = debuggees.find(d => d.id === debuggeeId); - assert.ok(result, 'should find the debuggee we just registered'); + assert.ok( + result, + `should find the debuggee we just registered, expected debuggeeId: ${debuggeeId}, found: ${result}` + ); } async function verifyDeleteBreakpoints() { diff --git a/test/fixtures/fib.js b/test/fixtures/fib.js index 47ce93fc..82860cfb 100644 --- a/test/fixtures/fib.js +++ b/test/fixtures/fib.js @@ -29,6 +29,8 @@ nocks.metadataInstance(); const UUID = process.argv[2] || uuid.v4(); +console.log('UUID: ', UUID); + // eslint-disable-next-line node/no-missing-require const debuglet = require('../../..').start({ debug: {