Skip to content

Commit

Permalink
[easy][cicd] Fix testscripts (#2545)
Browse files Browse the repository at this point in the history
## Summary

## How was it tested?

---------

Co-authored-by: John Lago <[email protected]>
  • Loading branch information
mikeland73 and Lagoja authored Mar 3, 2025
1 parent 41daddd commit 5192ba6
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 6 deletions.
7 changes: 6 additions & 1 deletion .github/workflows/cli-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,12 @@ jobs:
steps:
- uses: actions/checkout@v4
- uses: DeterminateSystems/nix-installer-action@main
- run: nix build .
- name: Build flake
run: |
if ! nix build .; then
echo "::warning::If this fails, you probably have to run 'devbox run update-hash'"
exit 1
fi
- run: ./result/bin/devbox version

golangci-lint:
Expand Down
6 changes: 3 additions & 3 deletions flake.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion testscripts/testrunner/testrunner.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ func Main(m *testing.M) {
commands := map[string]func(){
"devbox": func() {
// Call the devbox CLI directly:
boxcli.Execute(context.Background(), os.Args[1:])
os.Exit(boxcli.Execute(context.Background(), os.Args[1:]))
},
"print": func() { // Not 'echo' because we don't expand variables
fmt.Println(strings.Join(os.Args[1:], " "))
Expand Down
2 changes: 1 addition & 1 deletion vendor-hash
Original file line number Diff line number Diff line change
@@ -1 +1 @@
sha256-nU+3vFjOAYHRbVJiZuI/78jxDbWB5Ghdt05eWQoeWJA=
sha256-9ZaouEHCuavaRnEVa8cgGrAwyZWXUzyPhcUH02qkfWo=

0 comments on commit 5192ba6

Please sign in to comment.