mirror of
https://github.com/hasura/graphql-engine.git
synced 2024-12-15 17:31:56 +03:00
b75ab7233e
#### Issue: https://github.com/hasura/graphql-engine-mono/issues/2179 ### Problem: 1. There are few warnings when tests are executed with cli if it is built with `race` flag It is because of a race condition between stdin in migrate_delete for the prompt and stdout for other commands. 2. The integration test of the console used to behave as follows: ``` - Initially there won't be any wg.adds in wait-group so the second go-routine created in console-test (integration-test) - It will send the interrupt signal to the channel before the server has been started - So practically the server won't start ``` 3. The read and write for consoleopts.WG has been happening in different go-routines without control or lock system(reading and writing into same memory location without lock system). So there is a chance of data-race(warning for data-race while integration tests are executed) 4. Data-race errors from `promptui` package ### Solution: 1. Use `--force` flag to avoid getting the input from the prompt in `migrate_delete_test.go` 2. Introduced two fields in server and console opts to let know other go-routines whether the server has been started or not 3. To avoid data-race above which are shared b/w go-routines to know the status of servers has been operated atomically. 4. using new package https://github.com/AlecAivazis/survey https://github.com/hasura/graphql-engine-mono/pull/2231 GitOrigin-RevId: 387eb1be74f24dda34bb3588314b5a909adac227
55 lines
1.2 KiB
Go
55 lines
1.2 KiB
Go
package util
|
|
|
|
import (
|
|
"github.com/AlecAivazis/survey/v2"
|
|
)
|
|
|
|
func GetYesNoPrompt(message string) (promptResp bool, err error) {
|
|
prompt := &survey.Confirm{
|
|
Message: message,
|
|
Default: true,
|
|
}
|
|
err = survey.AskOne(prompt, &promptResp)
|
|
return promptResp, err
|
|
}
|
|
|
|
func GetSelectPrompt(message string, options []string) (selection string, err error) {
|
|
prompt := &survey.Select{
|
|
Message: message,
|
|
Options: options,
|
|
}
|
|
err = survey.AskOne(prompt, &selection)
|
|
return selection, err
|
|
}
|
|
|
|
func GetInputPrompt(message string) (input string, err error) {
|
|
prompt := &survey.Input{
|
|
Message: message,
|
|
}
|
|
err = survey.AskOne(prompt, &input)
|
|
return input, err
|
|
}
|
|
|
|
func GetInputPromptWithDefault(message string, def string) (input string, err error) {
|
|
prompt := &survey.Input{
|
|
Message: message,
|
|
Default: def,
|
|
}
|
|
err = survey.AskOne(prompt, &input)
|
|
return input, err
|
|
}
|
|
|
|
func validateDirPath(a interface{}) error {
|
|
err := FSCheckIfDirPathExists(a.(string))
|
|
return err
|
|
}
|
|
|
|
func GetFSPathPrompt(message string, def string) (input string, err error) {
|
|
prompt := &survey.Input{
|
|
Message: message,
|
|
Default: def,
|
|
}
|
|
err = survey.AskOne(prompt, &input, survey.WithValidator(validateDirPath))
|
|
return input, err
|
|
}
|