Skip to content

Commit 9799303

Browse files
committed
fix: ci -- run all subpackages
When moving all the tests to environment/integration, I forgot to update the CI to actually run all subpackages. It also cleans up some over-engineering on my end at the time. We now have a single test command, which, by default, runs all the tests Signed-off-by: home <[email protected]>
1 parent 2b175c5 commit 9799303

File tree

2 files changed

+4
-22
lines changed

2 files changed

+4
-22
lines changed

.dagger/main.go

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -54,13 +54,14 @@ func (m *ContainerUse) Release(ctx context.Context,
5454
// Test runs the test suite
5555
func (m *ContainerUse) Test(ctx context.Context,
5656
//+optional
57-
//+default="."
57+
//+default="./..."
5858
// Package to test
5959
pkg string,
6060
//+optional
6161
// Run tests with verbose output
6262
verboseOutput bool,
6363
//+optional
64+
//+default=true
6465
// Run tests including integration tests
6566
integration bool,
6667
) (string, error) {
@@ -82,15 +83,3 @@ func (m *ContainerUse) Test(ctx context.Context,
8283
WithExec(args).
8384
Stdout(ctx)
8485
}
85-
86-
// TestEnvironment runs the environment package tests specifically
87-
func (m *ContainerUse) TestEnvironment(ctx context.Context,
88-
//+optional
89-
// Run tests with verbose output
90-
verboseOutput bool,
91-
//+optional
92-
// Include integration tests
93-
integration bool,
94-
) (string, error) {
95-
return m.Test(ctx, "./environment", verboseOutput, integration)
96-
}

.github/workflows/test.yml

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -15,16 +15,9 @@ jobs:
1515
- name: Checkout
1616
uses: actions/checkout@v4
1717

18-
- name: Run unit tests
18+
- name: Run tests
1919
uses: dagger/[email protected]
2020
with:
2121
version: "latest"
2222
verb: call
23-
args: test-environment --verbose
24-
25-
- name: Run integration tests
26-
uses: dagger/[email protected]
27-
with:
28-
version: "latest"
29-
verb: call
30-
args: test-environment --verbose --integration
23+
args: test --verbose

0 commit comments

Comments
 (0)