Merge lp:~pedronis/ubuntu-push/accept-auxcfg into lp:ubuntu-push

Proposed by Samuele Pedroni
Status: Merged
Approved by: Samuele Pedroni
Approved revision: 30
Merged at revision: 32
Proposed branch: lp:~pedronis/ubuntu-push/accept-auxcfg
Merge into: lp:ubuntu-push
Diff against target: 21 lines (+3/-1)
1 file modified
server/acceptance/acceptance_test.go (+3/-1)
To merge this branch: bzr merge lp:~pedronis/ubuntu-push/accept-auxcfg
Reviewer Review Type Date Requested Status
Samuele Pedroni Approve
Review via email: mp+203889@code.launchpad.net

Commit message

[trivial ]support supplying auxiliary configs to the server in acceptance tests

Description of the change

support supplying auxiliary configs to the server in acceptance tests

To post a comment you must log in.
30. By Samuele Pedroni

support supplying auxiliary configs to the server in acceptance tests

Revision history for this message
Samuele Pedroni (pedronis) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'server/acceptance/acceptance_test.go'
2--- server/acceptance/acceptance_test.go 2014-01-21 21:36:07 +0000
3+++ server/acceptance/acceptance_test.go 2014-01-30 08:55:34 +0000
4@@ -52,6 +52,7 @@
5 var _ = Suite(&acceptanceSuite{})
6
7 var serverCmd = flag.String("server", "", "server to test")
8+var serverAuxCfg = flag.String("auxcfg", "", "auxiliary config for the server")
9
10 func testServerConfig(addr, httpAddr string) map[string]interface{} {
11 cfg := map[string]interface{}{
12@@ -113,7 +114,8 @@
13 if err != nil {
14 c.Fatal(err)
15 }
16- server := exec.Command(*serverCmd, cfgFilename)
17+ cfgs := append(strings.Fields(*serverAuxCfg), cfgFilename)
18+ server := exec.Command(*serverCmd, cfgs...)
19 stderr, err := server.StderrPipe()
20 if err != nil {
21 c.Fatal(err)

Subscribers

People subscribed via source and target branches