Skip to content

Commit

Permalink
build: fix tests with compilation errors (#797)
Browse files Browse the repository at this point in the history
For some odd reason, we had some tests with broken builds.
Seems like fixing #761 surfaced them up.

* Generated test cases used "exports" instead of "export"
* Fix neptune test
  • Loading branch information
Elad Ben-Israel authored Sep 27, 2018
1 parent 016f3a8 commit 9d00e36
Show file tree
Hide file tree
Showing 48 changed files with 49 additions and 49 deletions.
2 changes: 1 addition & 1 deletion create-missing-libraries.sh
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ EOM
import { Test, testCase } from 'nodeunit';
import {} from '../lib';
exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-amazonmq/test/test.amazonmq.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { Test, testCase } from 'nodeunit';
import {} from '../lib';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-appsync/test/test.appsync.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-athena/test/test.athena.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-batch/test/test.batch.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-budgets/test/test.budgets.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-cloud9/test/test.cloud9.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-cognito/test/test.cognito.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-config/test/test.config.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-dax/test/test.dax.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-dms/test/test.dms.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-ecs/test/test.ecs.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-efs/test/test.efs.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-eks/test/test.eks.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-elasticache/test/test.elasticache.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-emr/test/test.emr.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-gamelift/test/test.gamelift.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-glue/test/test.glue.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-guardduty/test/test.guardduty.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-inspector/test/test.inspector.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-iot/test/test.iot.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-iot1click/test/test.iot1click.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { Test, testCase } from 'nodeunit';
import {} from '../lib';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-logs/test/test.logs.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
4 changes: 2 additions & 2 deletions packages/@aws-cdk/aws-neptune/test/test.neptune.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@ import cdk = require('@aws-cdk/cdk');
import { Test } from 'nodeunit';
import { NeptuneDatabase } from '../lib';

exports = {
export = {
'check that instantiation works'(test: Test) {
const stack = new cdk.Stack();

const vpc = new ec2.VpcNetwork(this, 'VPC');
const vpc = new ec2.VpcNetwork(stack, 'VPC');

new NeptuneDatabase(stack, 'Database', {
masterUser: {
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-opsworks/test/test.opsworks.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-redshift/test/test.redshift.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-sagemaker/test/test.sagemaker.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { Test, testCase } from 'nodeunit';
import {} from '../lib';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-sam/test/test.sam.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-sdb/test/test.sdb.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-serverless/test/test.serverless.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { Test, testCase } from 'nodeunit';
import {} from '../lib';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-ses/test/test.ses.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-ssm/test/test.ssm.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-waf/test/test.waf.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-wafregional/test/test.wafregional.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-workspaces/test/test.workspaces.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Test, testCase } from 'nodeunit';

exports = testCase({
export = testCase({
notTested(test: Test) {
test.ok(true, 'No tests are specified for this package.');
test.done();
Expand Down

0 comments on commit 9d00e36

Please sign in to comment.