How about we ditch that and perform the call for the two available signatures? The goal of this example is to run a test (that covers as much code as possible) against a real server in the quickest way possible.
[1]
22 - fmt.Println("This application will query the staging Ubuntu SSO Server to fetch authorisation tokens.")
23 + fmt.Println("This application will query the staging Ubuntu SSO Server" +
24 + " to fetch authorisation tokens.")
and
384 - server := newSingleServingServer("/api/v2/tokens", string(jsonServerResponseData), 200)
385 + server := newSingleServingServer("/api/v2/tokens",
386 + string(jsonServerResponseData), 200)
and
159 +func (suite *USSOTestSuite) TestNormalizeURLLeavesNonstandardPort(
160 + c *gocheck.C) {
etc.
I think Jeroen's remark about the formatting is right. It's best if we try to use the same style as the other Go projects (lp:juju-core for instance) and use long lines instead of wrapping things like that.
I'll try to review that in details later today but just a quick notes:
[0]
+ fmt.Print("Enter signature method (PLAINTEXT or HMAC-SHA1): ")
+ fmt.Scanf("%s", &signature_method)
How about we ditch that and perform the call for the two available signatures? The goal of this example is to run a test (that covers as much code as possible) against a real server in the quickest way possible.
[1]
22 - fmt.Println("This application will query the staging Ubuntu SSO Server to fetch authorisation tokens.")
23 + fmt.Println("This application will query the staging Ubuntu SSO Server" +
24 + " to fetch authorisation tokens.")
and
384 - server := newSingleServin gServer( "/api/v2/ tokens" , string( jsonServerRespo nseData) , 200) gServer( "/api/v2/ tokens" , jsonServerRespo nseData) , 200)
385 + server := newSingleServin
386 + string(
and
159 +func (suite *USSOTestSuite) TestNormalizeUR LLeavesNonstand ardPort(
160 + c *gocheck.C) {
etc.
I think Jeroen's remark about the formatting is right. It's best if we try to use the same style as the other Go projects (lp:juju-core for instance) and use long lines instead of wrapping things like that.