Update test files to use the `.ts` extension by DominicKramer · Pull Request #315 · googleapis/cloud-debug-nodejs

DominicKramer · 2017-07-28T20:49:45Z

No description provided.

test/nocks.ts

-'use strict';

-var nock = require('nock');
+ var nock = require('nock');


test/test-duplicate-expressions-code.js

+/*5*/    a = 0;
+/*6*/  });
+/*7*/}
+/*8*/module.exports = foo;


test/test-duplicate-nested-expressions-code.js

+/*8*/   return a;
+/*9*/ }());
+/*10*/}
+/*11*/module.exports = foo;


test/test-max-data-size-code.js

+/*3*/function foo(n) {
+/*4*/  var A = new Array(3); return n+42+A[0];
+/*5*/}
+/*6*/module.exports = foo;


test/test-this-context-code.js

+/*11*/module.exports = {
+/*12*/  foo: foo,
+/*13*/  bar: bar
+/*14*/};


test/test-try-catch-code.js

+/*8*/   return e;
+/*9*/ }
+/*10*/}
+/*11*/module.exports = foo;


test/test-v8debugapi-code.js

+/*11*/module.exports = {
+/*12*/  foo: foo,
+/*13*/  getterObject: getterObject
+/*14*/};


tsconfig.json

    "allowUnreachableCode": false,
    "allowUnusedLabels": false,
-    "alwaysStrict": true,
+//    "alwaysStrict": true,


test/misc/bench-code.js

@@ -0,0 +1,6 @@
+/*1* KEEP THIS CODE AT THE TOP TO AVOID LINE NUMBER CHANGES */


test/test-v8debugapi.ts

  id: 'fake-id-123',
-  location: { path: 'test-v8debugapi.js', line: 4 }
+  // TODO: Determine if we should be restricting to only the build directory.
+  location: { path: 'build/test/test-v8debugapi-code.js', line: 4 }


DominicKramer added 30 commits July 28, 2017 13:37

Rename nocks.js to nocks.ts

3f9117d

Fix tests associated with nocks.ts

cd16012

Disable tsconfig compiler options

7d2bff6

Change test-controller.js ext to .ts

6165914

Fix tests associated with test-controller.ts

c4022a6

Change test-debug-assert.js ext to .ts

b7d3680

Change test-debuggee.js ext to .ts

cd48dc5

Change test-debuglet.js ext to .ts

88df9a9

Change test-fat-arrow.js ext to .ts

622c675

Fix tests associated with test-fat-arrow.ts

1b180c6

Change debugger.js ext to .ts

f8cca92

Fix tests associated with debugger.ts

017c5a2

Change test-sourcemapper.js ext to .ts

bb09b61

Change test-scanner.js ext to .ts

a19074a

Change test-options-credentials.js ext to .ts

230d434

Change test-module.js ext to .ts

e898629

Fix tests associated with test-module.ts

0220f54

Change test-duplicate-expressions.js to .ts

a6e1146

Fix the tests to test-duplicate-expressions.ts

75cb893

Change ...nested-expressions.js ext to .ts

55446d5

Fix tests associated with test-duplicate-expressionst.ts

56a9ec5

Change test-max-data-size.js ext to .ts

4090685

Fix tests associated with test-max-data-size.ts

388e3f3

Change test-state.js ext to .ts

5fb31a7

Fix tests associated with test-state.ts

45546bc

Change test-this-context.js to .ts

cbc5d02

Fix tests associated with test-this-context.ts

483145b

Change test-try-catch.js ext to .ts

b4785ce

Fix tests associated with test-try-catch.ts

8af74b3

Change test-v8debugapi.js to .ts

a09cab7

DominicKramer added 9 commits July 28, 2017 13:43

Fix tests associated with test-v8debugapi.ts

ae175a7

Change test-leak.js ext to .ts

118aad9

Fix tests associated with test-leak.ts

c6bcd2d

Change bench.js ext to .ts

b30aa10

Fix tests associated with bench.ts

d329c2d

Change test-e2e.js ext to .ts

a69cf06

Change test-controller.js ext to .ts

9e3404a

Fix system tests

0095493

Rename auth-request.js to auth-request.ts

541113c

jmdobry added the in progress label Jul 28, 2017

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 28, 2017

DominicKramer requested a review from kjin July 28, 2017 20:50

ofrobots reviewed Jul 28, 2017

View reviewed changes

kjin reviewed Jul 28, 2017

View reviewed changes

kjin approved these changes Jul 31, 2017

View reviewed changes

DominicKramer merged commit a21c83d into googleapis:master Jul 31, 2017

jmdobry removed the in progress label Jul 31, 2017

DominicKramer deleted the test-prs/2-rename-to-ts branch October 9, 2017 20:50

		@@ -0,0 +1,6 @@
		/1 KEEP THIS CODE AT THE TOP TO AVOID LINE NUMBER CHANGES */

Conversation

DominicKramer commented Jul 28, 2017

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants