Merge lp:~jonas-drange/ubuntu-push/remove-unused into lp:ubuntu-push

[Jonas G. Drange]

remove unused sess variable

Sems the gcgo compiler is right—sess is unused but declared. Let's drop it.

[system-apps-ci-bot (system-apps-ci-bot)]

FAILED: Continuous integration, rev:163
https://jenkins.canonical.com/system-apps/job/lp-ubuntu-push-ci/16/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/system-apps/job/build/993/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/993
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-1-sourcepkg/release=vivid+overlay/894/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-1-sourcepkg/release=xenial+overlay/894/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-1-sourcepkg/release=yakkety/894/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-ubuntu-push-ci/16/rebuild

review: Needs Fixing (continuous-integration)

[Ken VanDine (ken-vandine)]

looks good

review: Approve

=== modified file 'server/session/session_test.go'
--- server/session/session_test.go	2015-12-04 15:31:27 +0000
+++ server/session/session_test.go	2016-07-18 12:49:56 +0000
@@ -166,7 +166,6 @@
 }
 
 func (s *sessionSuite) TestSessionRegisterError(c *C) {
-	var sess broker.BrokerSession
 	errCh := make(chan error, 1)
 	up := make(chan interface{}, 5)
 	down := make(chan interface{}, 5)
@@ -176,7 +175,7 @@
 	brkr.err = errRegister
 	go func() {
 		var err error
-		sess, err = sessionStart(tp, brkr, cfg10msPingInterval5msExchangeTout, &tracker{sessionId: "s2"})
+		_, err = sessionStart(tp, brkr, cfg10msPingInterval5msExchangeTout, &tracker{sessionId: "s2"})
 		errCh <- err
 	}()
 	up <- protocol.ConnectMsg{Type: "connect", ClientVer: "1", DeviceId: "dev-1"}