github/hugo
1
0
Fork 0
mirror of https://github.com/gohugoio/hugo.git synced 2024-12-27 21:02:11 +00:00
Code Issues Projects Releases Packages Wiki Activity

commands: Fix failing Travis server test

Browse source
This commit is contained in:
Bjørn Erik Pedersen 2018-04-12 09:31:53 +02:00
parent 14c35c8a56
commit 9c782d5147
No known key found for this signature in database
GPG key ID: 330E6E2BD4859D8F
2 changed files with 9 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
commands/server.go
View file

@ -41,8 +41,6 @@ import (
type serverCmd struct {
// Can be used to stop the server. Useful in tests
stop <-chan bool
// Can be used to receive notification about when the server is started. Useful in tests.
started chan<- bool
disableLiveReload bool
navigateToChanged bool
@ -60,11 +58,11 @@ type serverCmd struct {
}
func newServerCmd() *serverCmd {
return newServerCmdSignaled(nil, nil)
return newServerCmdSignaled(nil)
}
func newServerCmdSignaled(stop <-chan bool, started chan<- bool) *serverCmd {
cc := &serverCmd{stop: stop, started: started}
func newServerCmdSignaled(stop <-chan bool) *serverCmd {
cc := &serverCmd{stop: stop}
cc.baseBuilderCmd = newBuilderCmd(&cobra.Command{
Use: "server",
@ -400,10 +398,6 @@ func (c *commandeer) serve(s *serverCmd) error {
}()
}
if s.started != nil {
s.started <- true
}
jww.FEEDBACK.Println("Press Ctrl+C to stop")
if s.stop != nil {

13
commands/server_test.go
View file

@ -38,9 +38,9 @@ func TestServer(t *testing.T) {
os.RemoveAll(dir)
}()
stop, started := make(chan bool), make(chan bool)
stop := make(chan bool)
scmd := newServerCmdSignaled(stop, started)
scmd := newServerCmdSignaled(stop)
cmd := scmd.getCommand()
cmd.SetArgs([]string{"-s=" + dir, fmt.Sprintf("-p=%d", port)})
@ -50,11 +50,10 @@ func TestServer(t *testing.T) {
assert.NoError(err)
}()
select {
case <-started:
case <-time.After(2 * time.Second):
t.Fatal("server start took too long")
}
// There is no way to know exactly when the server is ready for connections.
// We could improve by something like https://golang.org/pkg/net/http/httptest/#Server
// But for now, let us sleep and pray!
time.Sleep(2 * time.Second)
resp, err := http.Get("http://localhost:1331/")
assert.NoError(err)