update the config file logic

Author: vik-nullify

pkg/merger/merger_test.go

@@ -82,14 +82,6 @@ func TestMergeConfigFiles(t *testing.T) {
 						SeverityThreshold: models.SeverityHigh,
 						PriorityThreshold: models.PriorityImportant,
 					},
-					AWS: &models.AWS{
-						Enable:           true,
-						RoleNameToAssume: "nullify-role",
-						PrimaryAccountID: "123456789012",
-						PrimaryRegion:    "ap-southeast-2",
-						TargetRegions:    &[]string{"ap-southeast-2", "us-east-2"},
-						TargetAccounts:   &[]string{"123456789012", "123456789013"},
-					},
 				},
 			},
 			expected: &models.Configuration{
@@ -148,14 +140,6 @@ func TestMergeConfigFiles(t *testing.T) {
 						SeverityThreshold: models.SeverityHigh,
 						PriorityThreshold: models.PriorityImportant,
 					},
-					AWS: &models.AWS{
-						Enable:           true,
-						RoleNameToAssume: "nullify-role",
-						PrimaryAccountID: "123456789012",
-						PrimaryRegion:    "ap-southeast-2",
-						TargetRegions:    &[]string{"ap-southeast-2", "us-east-2"},
-						TargetAccounts:   &[]string{"123456789012", "123456789013"},
-					},
 				},
 			},
 		},
@@ -220,14 +204,6 @@ func TestMergeConfigFiles(t *testing.T) {
 							Low:      "low",
 						},
 					},
-					AWS: &models.AWS{
-						Enable:           true,
-						RoleNameToAssume: "nullify-role",
-						PrimaryAccountID: "123456789012",
-						PrimaryRegion:    "ap-southeast-2",
-						TargetRegions:    &[]string{"ap-southeast-2", "us-east-2"},
-						TargetAccounts:   &[]string{"123456789012", "123456789013"},
-					},
 				},
 			},
 			repoConfig: nil,
@@ -293,14 +269,6 @@ func TestMergeConfigFiles(t *testing.T) {
 							Low:      "low",
 						},
 					},
-					AWS: &models.AWS{
-						Enable:           true,
-						RoleNameToAssume: "nullify-role",
-						PrimaryAccountID: "123456789012",
-						PrimaryRegion:    "ap-southeast-2",
-						TargetRegions:    &[]string{"ap-southeast-2", "us-east-2"},
-						TargetAccounts:   &[]string{"123456789012", "123456789013"},
-					},
 				},
 			},
 		},
@@ -454,24 +422,6 @@ func TestMergeConfigFiles(t *testing.T) {
 				},
 			},
 		},
-		{
-			name: "attack surface with AWS integration merge",
-			globalConfig: &models.Configuration{
-				AttackSurface: &models.AttackSurface{
-					Enable: true,
-				},
-			},
-			repoConfig: nil,
-			expected: &models.Configuration{
-				EnablePullRequestReviews: models.Bool(true),
-				EnableIssueDashboards:    models.Bool(true),
-				SeverityThreshold:        parser.DefaultSeverityThreshold,
-				PriorityThreshold:        parser.DefaultPriorityThreshold,
-				AttackSurface: &models.AttackSurface{
-					Enable: true,
-				},
-			},
-		},
 	} {
 		t.Run(scenario.name, func(t *testing.T) {
 			config := MergeConfigFiles(parser.NewDefaultConfig(), scenario.globalConfig, scenario.repoConfig)