Skip to content

Commit 71d7636

Browse files
committed
updates
1 parent 2ef2892 commit 71d7636

File tree

3 files changed

+69
-31
lines changed

3 files changed

+69
-31
lines changed

go.mod

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

go.sum

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

internal/exec/describe_dependents_test.go

Lines changed: 66 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -312,15 +312,17 @@ func TestDescribeDependents_WithStacksNameTemplate(t *testing.T) {
312312

313313
// Matrix-driven cases
314314
cases := []struct {
315-
name string
316-
component string
317-
stack string
318-
expected []schema.Dependent
315+
name string
316+
component string
317+
stack string
318+
dependentsStack string
319+
expected []schema.Dependent
319320
}{
320321
{
321-
name: "ue1-network-vpc",
322-
component: "vpc",
323-
stack: "ue1-network",
322+
name: "ue1-network-vpc",
323+
component: "vpc",
324+
stack: "ue1-network",
325+
dependentsStack: "",
324326
expected: []schema.Dependent{
325327
{
326328
Component: "tgw/attachment",
@@ -339,9 +341,10 @@ func TestDescribeDependents_WithStacksNameTemplate(t *testing.T) {
339341
},
340342
},
341343
{
342-
name: "uw2-network-vpc",
343-
component: "vpc",
344-
stack: "uw2-network",
344+
name: "uw2-network-vpc",
345+
component: "vpc",
346+
stack: "uw2-network",
347+
dependentsStack: "",
345348
expected: []schema.Dependent{
346349
{
347350
Component: "tgw/attachment",
@@ -353,9 +356,10 @@ func TestDescribeDependents_WithStacksNameTemplate(t *testing.T) {
353356
},
354357
},
355358
{
356-
name: "ue1-prod-vpc",
357-
component: "vpc",
358-
stack: "ue1-prod",
359+
name: "ue1-prod-vpc",
360+
component: "vpc",
361+
stack: "ue1-prod",
362+
dependentsStack: "",
359363
expected: []schema.Dependent{
360364
{
361365
Component: "tgw/attachment",
@@ -367,9 +371,10 @@ func TestDescribeDependents_WithStacksNameTemplate(t *testing.T) {
367371
},
368372
},
369373
{
370-
name: "uw2-prod-vpc",
371-
component: "vpc",
372-
stack: "uw2-prod",
374+
name: "uw2-prod-vpc",
375+
component: "vpc",
376+
stack: "uw2-prod",
377+
dependentsStack: "",
373378
expected: []schema.Dependent{
374379
{
375380
Component: "tgw/attachment",
@@ -381,9 +386,10 @@ func TestDescribeDependents_WithStacksNameTemplate(t *testing.T) {
381386
},
382387
},
383388
{
384-
name: "ue1-network-tgw-hub",
385-
component: "tgw/hub",
386-
stack: "ue1-network",
389+
name: "ue1-network-tgw-hub",
390+
component: "tgw/hub",
391+
stack: "ue1-network",
392+
dependentsStack: "",
387393
expected: []schema.Dependent{
388394
{
389395
Component: "tgw/attachment",
@@ -423,16 +429,18 @@ func TestDescribeDependents_WithStacksNameTemplate(t *testing.T) {
423429
},
424430
},
425431
{
426-
name: "uw2-network-tgw-cross-region-hub-connector",
427-
component: "tgw/cross-region-hub-connector",
428-
stack: "uw2-network",
429-
expected: []schema.Dependent{},
432+
name: "uw2-network-tgw-cross-region-hub-connector",
433+
component: "tgw/cross-region-hub-connector",
434+
stack: "uw2-network",
435+
dependentsStack: "",
436+
expected: []schema.Dependent{},
430437
},
431438
{
432-
name: "ue1-network-tgw-attachment",
433-
component: "tgw/attachment",
434-
stack: "ue1-network",
435-
expected: []schema.Dependent{},
439+
name: "ue1-network-tgw-attachment",
440+
component: "tgw/attachment",
441+
stack: "ue1-network",
442+
dependentsStack: "",
443+
expected: []schema.Dependent{},
436444
},
437445
{
438446
name: "uw2-network-tgw-attachment",
@@ -452,12 +460,42 @@ func TestDescribeDependents_WithStacksNameTemplate(t *testing.T) {
452460
stack: "uw2-prod",
453461
expected: []schema.Dependent{},
454462
},
463+
{
464+
name: "ue1-network-tgw-hub-with-same-dependents-stack",
465+
component: "tgw/hub",
466+
stack: "ue1-network",
467+
dependentsStack: "ue1-network",
468+
expected: []schema.Dependent{
469+
{
470+
Component: "tgw/attachment",
471+
ComponentType: "terraform",
472+
ComponentPath: componentPath,
473+
Stack: "ue1-network",
474+
StackSlug: "ue1-network-tgw-attachment",
475+
},
476+
},
477+
},
478+
{
479+
name: "ue1-network-tgw-hub-with-diff-dependents-stack",
480+
component: "tgw/hub",
481+
stack: "ue1-network",
482+
dependentsStack: "uw2-prod",
483+
expected: []schema.Dependent{
484+
{
485+
Component: "tgw/attachment",
486+
ComponentType: "terraform",
487+
ComponentPath: componentPath,
488+
Stack: "uw2-prod",
489+
StackSlug: "uw2-prod-tgw-attachment",
490+
},
491+
},
492+
},
455493
}
456494

457495
for _, tc := range cases {
458496
tc := tc // capture
459497
t.Run(tc.name, func(t *testing.T) {
460-
res, err := ExecuteDescribeDependents(&atmosConfig, tc.component, tc.stack, false, true, true, nil, "")
498+
res, err := ExecuteDescribeDependents(&atmosConfig, tc.component, tc.stack, false, true, true, nil, tc.dependentsStack)
461499
require.NoError(t, err)
462500

463501
// Order-agnostic equality on struct slices

0 commit comments

Comments
 (0)