feat(lib): throw error when a token is used in a construct name

Closes #1068

packages/cdktf/lib/terraform-resource.ts

@@ -1,5 +1,5 @@
 import { Construct } from "constructs";
-import { Token } from "./tokens";
+import { Token, Tokenization } from "./tokens";
 import { TerraformElement } from "./terraform-element";
 import { TerraformProvider } from "./terraform-provider";
 import { keysToSnakeCase, deepMerge } from "./util";
@@ -60,6 +60,12 @@ export class TerraformResource
   constructor(scope: Construct, id: string, config: TerraformResourceConfig) {
     super(scope, id);
 
+    if (Tokenization.containsToken(id)) {
+      throw new Error(
+        "You can not use a Token (e.g. a reference to an attribute) as name of a construct"
+      );
+    }
+
     this.terraformResourceType = config.terraformResourceType;
     this.terraformGeneratorMetadata = config.terraformGeneratorMetadata;
     if (Array.isArray(config.dependsOn)) {

packages/cdktf/lib/tokens/token.ts

@@ -115,6 +115,10 @@ export class Tokenization {
     return [];
   }
 
+  public static containsToken(x: any): boolean {
+    return Tokenization.reverse(x).length > 0;
+  }
+
   /**
    * Un-encode a string potentially containing encoded tokens
    */

packages/cdktf/test/resource.test.ts

@@ -160,3 +160,21 @@ test("numeric attributes", () => {
 
   expect(Testing.synth(stack)).toMatchSnapshot();
 });
+
+test("tokens as ids", () => {
+  const app = Testing.app();
+  const stack = new TerraformStack(app, "test");
+  new TestProvider(stack, "provider", {});
+
+  const foo = new TestResource(stack, "resource", {
+    name: "foo",
+  });
+
+  expect(() => {
+    new TestResource(stack, `resource-${foo.stringValue}`, {
+      name: "foo",
+    });
+  }).toThrowErrorMatchingInlineSnapshot(
+    `"You can not use a Token (e.g. a reference to an attribute) as name of a construct"`
+  );
+});