udpated default to null, improved default logic, renamed test

This commit is contained in:
Stephen Kirby
2023-09-26 20:22:34 +00:00
parent f9346f4c23
commit 6a83fe8793
2 changed files with 3 additions and 3 deletions

View File

@@ -6,14 +6,14 @@ import {
testRequiredVariables,
} from "../test";
describe("azure-region", async () => {
describe("fly-region", async () => {
await runTerraformInit(import.meta.dir);
testRequiredVariables(import.meta.dir, {});
it("default output", async () => {
const state = await runTerraformApply(import.meta.dir, {});
expect(state.outputs.value.value).toBe("ams");
expect(state.outputs.value.value).toBe("");
});
it("customized default", async () => {

View File

@@ -270,7 +270,7 @@ data "coder_parameter" "fly_region" {
name = "flyio_region"
display_name = var.display_name
description = var.description
default = (var.default != null && var.default != "") && (contains(var.regions, var.default) || length(var.regions) == 0) ? var.default : null
default = (var.default != null && var.default != "") && ((var.default != null ? contains(var.regions, var.default) : false) || length(var.regions) == 0) ? var.default : null
mutable = var.mutable
dynamic "option" {
for_each = { for k, v in local.regions : k => v if anytrue([for d in var.regions : k == d ]) || length(var.regions) == 0 }