Skip to content

Commit

Permalink
fix brew tests
Browse files Browse the repository at this point in the history
  • Loading branch information
jan-xyz committed Feb 6, 2025
1 parent 80ff389 commit a060b7b
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 13 deletions.
18 changes: 9 additions & 9 deletions internal/plugins/brewbottles/plugin_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,23 +19,23 @@ func TestGetMissingPackage(t *testing.T) {
{
name: "successfully get packages",
prepareCommander: func(c *dotfiles.MockCommander) {
c.OnOutput("brew", []string{"list", "--formula"}).Return([]byte("foo"), nil)
c.OnOutput("brew", []string{"list", "--casks"}).Return([]byte("bar"), nil)
c.OnOutput("/opt/homebrew/bin/brew", []string{"list", "--formula"}).Return([]byte("foo"), nil)
c.OnOutput("/opt/homebrew/bin/brew", []string{"list", "--casks"}).Return([]byte("bar"), nil)
},
expectedPackages: []string{"baz"},
},
{
name: "fails if list casks fails",
prepareCommander: func(c *dotfiles.MockCommander) {
c.OnOutput("brew", []string{"list", "--formula"}).Return([]byte("foo"), nil)
c.OnOutput("brew", []string{"list", "--casks"}).Return([]byte{}, errors.New("boom"))
c.OnOutput("/opt/homebrew/bin/brew", []string{"list", "--formula"}).Return([]byte("foo"), nil)
c.OnOutput("/opt/homebrew/bin/brew", []string{"list", "--casks"}).Return([]byte{}, errors.New("boom"))
},
wantErr: true,
},
{
name: "fails if list forumla fails",
prepareCommander: func(c *dotfiles.MockCommander) {
c.OnOutput("brew", []string{"list", "--formula"}).Return([]byte{}, errors.New("boom"))
c.OnOutput("/opt/homebrew/bin/brew", []string{"list", "--formula"}).Return([]byte{}, errors.New("boom"))
},
wantErr: true,
},
Expand Down Expand Up @@ -74,7 +74,7 @@ func TestAdd(t *testing.T) {
name: "successfully install packages",
input: []string{"bar", "foo"},
prepareCommander: func(c *dotfiles.MockCommander) {
c.OnOutput("brew", []string{"install", "bar", "foo"}).Return(nil, nil)
c.OnOutput("/opt/homebrew/bin/brew", []string{"install", "bar", "foo"}).Return(nil, nil)
},
wantErr: false,
},
Expand All @@ -88,7 +88,7 @@ func TestAdd(t *testing.T) {
name: "fails if installing packages fails",
input: []string{"bar", "foo"},
prepareCommander: func(c *dotfiles.MockCommander) {
c.OnOutput("brew", []string{"install", "bar", "foo"}).Return(nil, errors.New("boom"))
c.OnOutput("/opt/homebrew/bin/brew", []string{"install", "bar", "foo"}).Return(nil, errors.New("boom"))
},
wantErr: true,
},
Expand Down Expand Up @@ -123,14 +123,14 @@ func TestUpdate(t *testing.T) {
{
name: "successfully update packages",
prepareCommander: func(c *dotfiles.MockCommander) {
c.OnOutput("brew", []string{"upgrade"}).Return(nil, nil)
c.OnOutput("/opt/homebrew/bin/brew", []string{"upgrade"}).Return(nil, nil)
},
wantErr: false,
},
{
name: "fails if update packages fails",
prepareCommander: func(c *dotfiles.MockCommander) {
c.OnOutput("brew", []string{"upgrade"}).Return(nil, errors.New("boom"))
c.OnOutput("/opt/homebrew/bin/brew", []string{"upgrade"}).Return(nil, errors.New("boom"))
},
wantErr: true,
},
Expand Down
8 changes: 4 additions & 4 deletions internal/plugins/brewtaps/plugin_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ func TestGetMissingPackage(t *testing.T) {
name: "succesfully get taps",
configuredApps: []string{"bar", "foo"},
prepareCommander: func(c *dotfiles.MockCommander) {
c.OnOutput("brew", []string{"tap"}).Return([]byte("bar"), nil)
c.OnOutput("/opt/homebrew/bin/brew", []string{"tap"}).Return([]byte("bar"), nil)
},
expectedApps: []string{"foo"},
wantErr: false,
Expand All @@ -30,7 +30,7 @@ func TestGetMissingPackage(t *testing.T) {
name: "fail if calling brew fails",
configuredApps: []string{"bar", "foo"},
prepareCommander: func(c *dotfiles.MockCommander) {
c.OnOutput("brew", []string{"tap"}).Return([]byte{}, errors.New("boom"))
c.OnOutput("/opt/homebrew/bin/brew", []string{"tap"}).Return([]byte{}, errors.New("boom"))
},
wantErr: true,
},
Expand Down Expand Up @@ -68,15 +68,15 @@ func TestAdd(t *testing.T) {
name: "succefully install taps",
input: []string{"bar", "foo"},
prepareCommander: func(c *dotfiles.MockCommander) {
c.OnOutput("brew", []string{"tap", "bar", "foo"}).Return(nil, nil)
c.OnOutput("/opt/homebrew/bin/brew", []string{"tap", "bar", "foo"}).Return(nil, nil)
},
wantErr: false,
},
{
name: "fail if calling brew fails",
input: []string{"bar", "foo"},
prepareCommander: func(c *dotfiles.MockCommander) {
c.OnOutput("brew", []string{"tap", "bar", "foo"}).Return(nil, errors.New("boom"))
c.OnOutput("/opt/homebrew/bin/brew", []string{"tap", "bar", "foo"}).Return(nil, errors.New("boom"))
},
wantErr: true,
},
Expand Down

0 comments on commit a060b7b

Please sign in to comment.