mirror of
				https://gitee.com/gitea/gitea
				synced 2025-11-04 16:40:24 +08:00 
			
		
		
		
	After some discussion, introduce a new slice `brokenArgs` to make `gitCmd.Run()` return errors if any dynamic argument is invalid. Co-authored-by: delvh <dev.lh@web.de>
		
			
				
	
	
		
			45 lines
		
	
	
		
			1.4 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
			
		
		
	
	
			45 lines
		
	
	
		
			1.4 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
// Copyright 2022 The Gitea Authors. All rights reserved.
 | 
						|
// Use of this source code is governed by a MIT-style
 | 
						|
// license that can be found in the LICENSE file.
 | 
						|
 | 
						|
package git
 | 
						|
 | 
						|
import (
 | 
						|
	"context"
 | 
						|
	"testing"
 | 
						|
 | 
						|
	"github.com/stretchr/testify/assert"
 | 
						|
)
 | 
						|
 | 
						|
func TestRunWithContextStd(t *testing.T) {
 | 
						|
	cmd := NewCommand(context.Background(), "--version")
 | 
						|
	stdout, stderr, err := cmd.RunStdString(&RunOpts{})
 | 
						|
	assert.NoError(t, err)
 | 
						|
	assert.Empty(t, stderr)
 | 
						|
	assert.Contains(t, stdout, "git version")
 | 
						|
 | 
						|
	cmd = NewCommand(context.Background(), "--no-such-arg")
 | 
						|
	stdout, stderr, err = cmd.RunStdString(&RunOpts{})
 | 
						|
	if assert.Error(t, err) {
 | 
						|
		assert.Equal(t, stderr, err.Stderr())
 | 
						|
		assert.Contains(t, err.Stderr(), "unknown option:")
 | 
						|
		assert.Contains(t, err.Error(), "exit status 129 - unknown option:")
 | 
						|
		assert.Empty(t, stdout)
 | 
						|
	}
 | 
						|
 | 
						|
	cmd = NewCommand(context.Background())
 | 
						|
	cmd.AddDynamicArguments("-test")
 | 
						|
	assert.ErrorIs(t, cmd.Run(&RunOpts{}), ErrBrokenCommand)
 | 
						|
 | 
						|
	cmd = NewCommand(context.Background())
 | 
						|
	cmd.AddDynamicArguments("--test")
 | 
						|
	assert.ErrorIs(t, cmd.Run(&RunOpts{}), ErrBrokenCommand)
 | 
						|
 | 
						|
	subCmd := "version"
 | 
						|
	cmd = NewCommand(context.Background()).AddDynamicArguments(subCmd) // for test purpose only, the sub-command should never be dynamic for production
 | 
						|
	stdout, stderr, err = cmd.RunStdString(&RunOpts{})
 | 
						|
	assert.NoError(t, err)
 | 
						|
	assert.Empty(t, stderr)
 | 
						|
	assert.Contains(t, stdout, "git version")
 | 
						|
}
 |