Skip to content

Commit 7c3c2b0

Browse files
committed
Enable parallel testing
Signed-off-by: Timo Sand <[email protected]>
1 parent c9fbaa9 commit 7c3c2b0

2 files changed

Lines changed: 44 additions & 44 deletions

File tree

GNUmakefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ test:
5959
testacc:
6060
@branch=$$(git rev-parse --abbrev-ref HEAD); \
6161
printf "==> Running acceptance tests on branch: \033[1m%s\033[0m...\n" "🌿 $$branch 🌿"
62-
TF_ACC=1 CGO_ENABLED=0 go test $(TEST) -v -run '^TestAcc' $(RUNARGS) $(TESTARGS) -timeout 120m -count=1
62+
TF_ACC=1 CGO_ENABLED=0 go test $(TEST) -parallel=4 -v -run '^TestAcc' $(RUNARGS) $(TESTARGS) -timeout 120m -count=1
6363

6464
sweep:
6565
@echo "WARNING: This will destroy infrastructure. Use only in development accounts."

github/resource_github_repository_test.go

Lines changed: 43 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ func TestAccGithubRepository(t *testing.T) {
4141
}
4242
`, testRepoName, testAccConf.testRepositoryVisibility)
4343

44-
resource.Test(t, resource.TestCase{
44+
resource.ParallelTest(t, resource.TestCase{
4545
PreCheck: func() { skipUnauthenticated(t) },
4646
ProviderFactories: providerFactories,
4747
Steps: []resource.TestStep{
@@ -72,7 +72,7 @@ func TestAccGithubRepository(t *testing.T) {
7272
}
7373
`, oldName, randomID, testAccConf.testRepositoryVisibility)
7474

75-
resource.Test(t, resource.TestCase{
75+
resource.ParallelTest(t, resource.TestCase{
7676
PreCheck: func() { skipUnauthenticated(t) },
7777
ProviderFactories: providerFactories,
7878
Steps: []resource.TestStep{
@@ -110,7 +110,7 @@ func TestAccGithubRepository(t *testing.T) {
110110
}
111111
`, testRepoName, testAccConf.testRepositoryVisibility)
112112

113-
resource.Test(t, resource.TestCase{
113+
resource.ParallelTest(t, resource.TestCase{
114114
PreCheck: func() { skipUnauthenticated(t) },
115115
ProviderFactories: providerFactories,
116116
Steps: []resource.TestStep{
@@ -124,7 +124,7 @@ func TestAccGithubRepository(t *testing.T) {
124124
ResourceName: "github_repository.test",
125125
ImportState: true,
126126
ImportStateVerify: true,
127-
ImportStateVerifyIgnore: []string{"auto_init", "vulnerability_alerts", "ignore_vulnerability_alerts_during_read"},
127+
ImportStateVerifyIgnore: []string{"auto_init", "vulnerability_alerts", "ignore_vulnerability_alerts_during_read", "etag"},
128128
},
129129
},
130130
})
@@ -142,7 +142,7 @@ resource "github_repository" "test" {
142142
}
143143
`
144144

145-
resource.Test(t, resource.TestCase{
145+
resource.ParallelTest(t, resource.TestCase{
146146
PreCheck: func() { skipUnauthenticated(t) },
147147
ProviderFactories: providerFactories,
148148
Steps: []resource.TestStep{
@@ -174,7 +174,7 @@ resource "github_repository" "test" {
174174
}
175175
`, testRepoName, testAccConf.testRepositoryVisibility)
176176

177-
resource.Test(t, resource.TestCase{
177+
resource.ParallelTest(t, resource.TestCase{
178178
PreCheck: func() { skipUnauthenticated(t) },
179179
ProviderFactories: providerFactories,
180180
Steps: []resource.TestStep{
@@ -214,7 +214,7 @@ resource "github_repository" "test" {
214214
}
215215
`, testRepoName, testAccConf.testRepositoryVisibility)
216216

217-
resource.Test(t, resource.TestCase{
217+
resource.ParallelTest(t, resource.TestCase{
218218
PreCheck: func() { skipUnauthenticated(t) },
219219
ProviderFactories: providerFactories,
220220
Steps: []resource.TestStep{
@@ -263,7 +263,7 @@ resource "github_repository" "test" {
263263
statecheck.ExpectKnownValue("github_repository.test", tfjsonpath.New("default_branch"), knownvalue.StringExact("main")),
264264
}
265265

266-
resource.Test(t, resource.TestCase{
266+
resource.ParallelTest(t, resource.TestCase{
267267
PreCheck: func() { skipUnauthenticated(t) },
268268
ProviderFactories: providerFactories,
269269
Steps: []resource.TestStep{
@@ -297,7 +297,7 @@ resource "github_repository" "test" {
297297
}
298298
`, testRepoName, testAccConf.testRepositoryVisibility)
299299

300-
resource.Test(t, resource.TestCase{
300+
resource.ParallelTest(t, resource.TestCase{
301301
PreCheck: func() { skipUnauthenticated(t) },
302302
ProviderFactories: providerFactories,
303303
Steps: []resource.TestStep{
@@ -326,7 +326,7 @@ resource "github_repository" "test" {
326326
}
327327
`
328328

329-
resource.Test(t, resource.TestCase{
329+
resource.ParallelTest(t, resource.TestCase{
330330
PreCheck: func() { skipUnauthenticated(t) },
331331
ProviderFactories: providerFactories,
332332
Steps: []resource.TestStep{
@@ -362,7 +362,7 @@ resource "github_repository" "test" {
362362
}
363363
`, testRepoName, testAccConf.testRepositoryVisibility, testAccConf.testPublicTemplateRepositoryOwner, testAccConf.testPublicTemplateRepository)
364364

365-
resource.Test(t, resource.TestCase{
365+
resource.ParallelTest(t, resource.TestCase{
366366
PreCheck: func() { skipUnauthenticated(t); skipIfEMUEnterprise(t) },
367367
ProviderFactories: providerFactories,
368368
Steps: []resource.TestStep{
@@ -392,7 +392,7 @@ resource "github_repository" "test" {
392392
}
393393
`, testRepoName, testAccConf.testRepositoryVisibility, testAccConf.owner, testAccConf.testOrgTemplateRepository)
394394

395-
resource.Test(t, resource.TestCase{
395+
resource.ParallelTest(t, resource.TestCase{
396396
PreCheck: func() { skipUnlessHasOrgs(t) },
397397
ProviderFactories: providerFactories,
398398
Steps: []resource.TestStep{
@@ -420,7 +420,7 @@ resource "github_repository" "test" {
420420
}
421421
`
422422

423-
resource.Test(t, resource.TestCase{
423+
resource.ParallelTest(t, resource.TestCase{
424424
PreCheck: func() { skipUnauthenticated(t) },
425425
ProviderFactories: providerFactories,
426426
Steps: []resource.TestStep{
@@ -453,7 +453,7 @@ resource "github_repository" "test" {
453453
}
454454
`, repoName)
455455

456-
resource.Test(t, resource.TestCase{
456+
resource.ParallelTest(t, resource.TestCase{
457457
PreCheck: func() { skipUnlessHasOrgs(t) },
458458
ProviderFactories: providerFactories,
459459
Steps: []resource.TestStep{
@@ -481,7 +481,7 @@ resource "github_repository" "test" {
481481
}
482482
`, repoName)
483483

484-
resource.Test(t, resource.TestCase{
484+
resource.ParallelTest(t, resource.TestCase{
485485
PreCheck: func() { skipUnlessHasOrgs(t) },
486486
ProviderFactories: providerFactories,
487487
Steps: []resource.TestStep{
@@ -507,7 +507,7 @@ resource "github_repository" "test" {
507507
}
508508
`, repoName)
509509

510-
resource.Test(t, resource.TestCase{
510+
resource.ParallelTest(t, resource.TestCase{
511511
PreCheck: func() { skipUnlessHasOrgs(t) },
512512
ProviderFactories: providerFactories,
513513
Steps: []resource.TestStep{
@@ -548,7 +548,7 @@ resource "github_repository" "test" {
548548
}
549549
`, testRepoName)
550550

551-
resource.Test(t, resource.TestCase{
551+
resource.ParallelTest(t, resource.TestCase{
552552
PreCheck: func() {
553553
skipUnauthenticated(t)
554554
skipIfEMUEnterprise(t)
@@ -593,7 +593,7 @@ resource "github_repository" "test" {
593593
}
594594
`, repoName, testAccConf.testRepositoryVisibility)
595595

596-
resource.Test(t, resource.TestCase{
596+
resource.ParallelTest(t, resource.TestCase{
597597
PreCheck: func() { skipUnauthenticated(t) },
598598
ProviderFactories: providerFactories,
599599
Steps: []resource.TestStep{
@@ -620,7 +620,7 @@ resource "github_repository" "test" {
620620
}
621621
`, repoName, testAccConf.testRepositoryVisibility)
622622

623-
resource.Test(t, resource.TestCase{
623+
resource.ParallelTest(t, resource.TestCase{
624624
PreCheck: func() { skipUnauthenticated(t) },
625625
ProviderFactories: providerFactories,
626626
Steps: []resource.TestStep{
@@ -647,7 +647,7 @@ resource "github_repository" "test" {
647647

648648
// NOTE: terraform-plugin-testing does not support asserting the nonexistence of a value
649649
// (no equivalent to the legacy TestCheckNoResourceAttr). We only verify creation succeeds.
650-
resource.Test(t, resource.TestCase{
650+
resource.ParallelTest(t, resource.TestCase{
651651
PreCheck: func() { skipUnauthenticated(t) },
652652
ProviderFactories: providerFactories,
653653
Steps: []resource.TestStep{
@@ -671,7 +671,7 @@ resource "github_repository" "test" {
671671
}
672672
`
673673

674-
resource.Test(t, resource.TestCase{
674+
resource.ParallelTest(t, resource.TestCase{
675675
PreCheck: func() { skipUnauthenticated(t) },
676676
ProviderFactories: providerFactories,
677677
Steps: []resource.TestStep{
@@ -709,7 +709,7 @@ resource "github_repository" "test" {
709709
}
710710
`
711711

712-
resource.Test(t, resource.TestCase{
712+
resource.ParallelTest(t, resource.TestCase{
713713
PreCheck: func() { skipUnauthenticated(t) },
714714
ProviderFactories: providerFactories,
715715
Steps: []resource.TestStep{
@@ -750,7 +750,7 @@ resource "github_repository" "test" {
750750
}
751751
`
752752

753-
resource.Test(t, resource.TestCase{
753+
resource.ParallelTest(t, resource.TestCase{
754754
PreCheck: func() { skipUnauthenticated(t) },
755755
ProviderFactories: providerFactories,
756756
Steps: []resource.TestStep{
@@ -791,7 +791,7 @@ resource "github_repository" "test" {
791791
// resource.TestCheckResourceAttr("github_repository.test", "primary_language", "Go"),
792792
// )
793793

794-
// resource.Test(t, resource.TestCase{
794+
// resource.ParallelTest(t, resource.TestCase{
795795
// PreCheck: func() { skipUnauthenticated(t) },
796796
// ProviderFactories: providerFactories,
797797
// Steps: []resource.TestStep{
@@ -826,7 +826,7 @@ resource "github_repository" "test" {
826826
}
827827
`, testRepoName, testAccConf.testRepositoryVisibility)
828828

829-
resource.Test(t, resource.TestCase{
829+
resource.ParallelTest(t, resource.TestCase{
830830
PreCheck: func() { skipUnauthenticated(t) },
831831
ProviderFactories: providerFactories,
832832
Steps: []resource.TestStep{
@@ -859,7 +859,7 @@ resource "github_repository" "test" {
859859
}
860860
`, testRepoName, testAccConf.testRepositoryVisibility)
861861

862-
resource.Test(t, resource.TestCase{
862+
resource.ParallelTest(t, resource.TestCase{
863863
PreCheck: func() { skipUnauthenticated(t) },
864864
ProviderFactories: providerFactories,
865865
Steps: []resource.TestStep{
@@ -913,7 +913,7 @@ resource "github_repository" "test" {
913913
`, testRepoName)
914914

915915
securityPath := tfjsonpath.New("security_and_analysis").AtSliceIndex(0)
916-
resource.Test(t, resource.TestCase{
916+
resource.ParallelTest(t, resource.TestCase{
917917
PreCheck: func() { skipUnauthenticated(t) },
918918
ProviderFactories: providerFactories,
919919
Steps: []resource.TestStep{
@@ -953,7 +953,7 @@ resource "github_repository" "test" {
953953
`, testRepoName)
954954

955955
securityPath := tfjsonpath.New("security_and_analysis").AtSliceIndex(0)
956-
resource.Test(t, resource.TestCase{
956+
resource.ParallelTest(t, resource.TestCase{
957957
PreCheck: func() {
958958
skipUnauthenticated(t)
959959
skipIfEMUEnterprise(t)
@@ -981,7 +981,7 @@ resource "github_repository" "test" {
981981
}
982982
`
983983

984-
resource.Test(t, resource.TestCase{
984+
resource.ParallelTest(t, resource.TestCase{
985985
PreCheck: func() { skipUnauthenticated(t) },
986986
ProviderFactories: providerFactories,
987987
Steps: []resource.TestStep{
@@ -1005,7 +1005,7 @@ resource "github_repository" "test" {
10051005
}
10061006
`, testRepoName)
10071007

1008-
resource.Test(t, resource.TestCase{
1008+
resource.ParallelTest(t, resource.TestCase{
10091009
PreCheck: func() { skipUnlessMode(t, enterprise) },
10101010
ProviderFactories: providerFactories,
10111011
Steps: []resource.TestStep{
@@ -1030,7 +1030,7 @@ resource "github_repository" "test" {
10301030
}
10311031
`
10321032

1033-
resource.Test(t, resource.TestCase{
1033+
resource.ParallelTest(t, resource.TestCase{
10341034
PreCheck: func() { skipUnauthenticated(t); skipIfEMUEnterprise(t) },
10351035
ProviderFactories: providerFactories,
10361036
Steps: []resource.TestStep{
@@ -1060,7 +1060,7 @@ resource "github_repository" "test" {
10601060
}
10611061
`, testRepoName)
10621062

1063-
resource.Test(t, resource.TestCase{
1063+
resource.ParallelTest(t, resource.TestCase{
10641064
PreCheck: func() { skipUnauthenticated(t) },
10651065
ProviderFactories: providerFactories,
10661066
Steps: []resource.TestStep{
@@ -1093,7 +1093,7 @@ resource "github_repository" "test" {
10931093
}
10941094
`, testRepoName)
10951095

1096-
resource.Test(t, resource.TestCase{
1096+
resource.ParallelTest(t, resource.TestCase{
10971097
PreCheck: func() { skipUnlessMode(t, enterprise) },
10981098
ProviderFactories: providerFactories,
10991099
Steps: []resource.TestStep{
@@ -1130,7 +1130,7 @@ resource "github_repository" "test" {
11301130
}
11311131
`, testRepoName, testAccConf.testPublicTemplateRepositoryOwner, testAccConf.testPublicTemplateRepository)
11321132

1133-
resource.Test(t, resource.TestCase{
1133+
resource.ParallelTest(t, resource.TestCase{
11341134
PreCheck: func() { skipUnauthenticated(t); skipIfEMUEnterprise(t) },
11351135
ProviderFactories: providerFactories,
11361136
Steps: []resource.TestStep{
@@ -1159,7 +1159,7 @@ resource "github_repository" "test" {
11591159
}
11601160
`, testRepoName, testAccConf.testPublicTemplateRepositoryOwner, testAccConf.testPublicTemplateRepository)
11611161

1162-
resource.Test(t, resource.TestCase{
1162+
resource.ParallelTest(t, resource.TestCase{
11631163
PreCheck: func() { skipUnlessMode(t, enterprise); skipIfEMUEnterprise(t) },
11641164
ProviderFactories: providerFactories,
11651165
Steps: []resource.TestStep{
@@ -1185,7 +1185,7 @@ resource "github_repository" "test" {
11851185
}
11861186
`
11871187

1188-
resource.Test(t, resource.TestCase{
1188+
resource.ParallelTest(t, resource.TestCase{
11891189
PreCheck: func() { skipUnauthenticated(t) },
11901190
ProviderFactories: providerFactories,
11911191
Steps: []resource.TestStep{
@@ -1210,7 +1210,7 @@ resource "github_repository" "test" {
12101210
}
12111211
`
12121212

1213-
resource.Test(t, resource.TestCase{
1213+
resource.ParallelTest(t, resource.TestCase{
12141214
PreCheck: func() { skipUnauthenticated(t) },
12151215
ProviderFactories: providerFactories,
12161216
Steps: []resource.TestStep{
@@ -1234,7 +1234,7 @@ resource "github_repository" "test" {
12341234
}
12351235
`
12361236

1237-
resource.Test(t, resource.TestCase{
1237+
resource.ParallelTest(t, resource.TestCase{
12381238
PreCheck: func() { skipUnauthenticated(t) },
12391239
ProviderFactories: providerFactories,
12401240
Steps: []resource.TestStep{
@@ -1262,7 +1262,7 @@ resource "github_repository" "test" {
12621262
}
12631263
`
12641264

1265-
resource.Test(t, resource.TestCase{
1265+
resource.ParallelTest(t, resource.TestCase{
12661266
PreCheck: func() { skipUnauthenticated(t) },
12671267
ProviderFactories: providerFactories,
12681268
Steps: []resource.TestStep{
@@ -1293,7 +1293,7 @@ resource "github_repository" "private" {
12931293
}
12941294
`
12951295

1296-
resource.Test(t, resource.TestCase{
1296+
resource.ParallelTest(t, resource.TestCase{
12971297
PreCheck: func() { skipUnauthenticated(t) },
12981298
ProviderFactories: providerFactories,
12991299
Steps: []resource.TestStep{
@@ -1324,7 +1324,7 @@ resource "github_repository" "private" {
13241324
}
13251325
`
13261326

1327-
resource.Test(t, resource.TestCase{
1327+
resource.ParallelTest(t, resource.TestCase{
13281328
PreCheck: func() { skipUnauthenticated(t) },
13291329
ProviderFactories: providerFactories,
13301330
Steps: []resource.TestStep{
@@ -1398,7 +1398,7 @@ func Test_expandPages(t *testing.T) {
13981398
}
13991399
`, testRepoName)
14001400

1401-
resource.Test(t, resource.TestCase{
1401+
resource.ParallelTest(t, resource.TestCase{
14021402
PreCheck: func() { skipUnauthenticated(t) },
14031403
ProviderFactories: providerFactories,
14041404
Steps: []resource.TestStep{
@@ -1443,7 +1443,7 @@ func Test_expandPages(t *testing.T) {
14431443
}
14441444
`, testRepoName)
14451445

1446-
resource.Test(t, resource.TestCase{
1446+
resource.ParallelTest(t, resource.TestCase{
14471447
PreCheck: func() { skipUnauthenticated(t) },
14481448
ProviderFactories: providerFactories,
14491449
Steps: []resource.TestStep{

0 commit comments

Comments
 (0)