update comments

command/agent/config.go

@@ -1352,7 +1352,6 @@ func parseMultipleIPTemplate(ipTmpl string) ([]string, error) {
 		return []string{}, fmt.Errorf("Unable to parse address template %q: %v", ipTmpl, err)
 	}
 
-	// TODO: deduplicate the parsed ips
 	ips := strings.Split(out, " ")
 	if len(ips) == 0 {
 		return []string{}, errors.New("No addresses found, please configure one.")

command/agent/config_test.go

@@ -883,7 +883,6 @@ func TestConfig_normalizeAddrs_IPv6Loopback(t *testing.T) {
 // TestConfig_normalizeAddrs_MultipleInterface asserts that normalizeAddrs will
 // handle normalizing multiple interfaces in a single protocol.
 func TestConfig_normalizeAddrs_MultipleInterfaces(t *testing.T) {
-	// TODO: add test with go-sockaddr templates
 	testCases := []struct {
 		name                    string
 		addressConfig           *Addresses

command/agent/testagent.go

@@ -263,7 +263,9 @@ func (a *TestAgent) start() (*Agent, error) {
 	if err != nil {
 		return agent, err
 	}
-	// TODO: change type or handle better
+
+	// TODO: investigate if there is a way to remove the requirement by updating test.
+	// Initial pass at implementing this is https://github.com/kevinschoonover/nomad/tree/tests.
 	a.Servers = httpServers
 	a.Server = &httpServers[0]
 	return agent, nil