From 3f361972aca5f08ba7d921e58832ab6a4e7a98ae Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?William=20Tis=C3=A4ter?= Date: Mon, 26 May 2014 23:44:46 +0200 Subject: [PATCH] Remove TODO comments --- command/agent/catalog_endpoint_test.go | 1 - command/agent/health_endpoint_test.go | 3 --- command/agent/rpc_client_test.go | 1 - command/agent/ui_endpoint_test.go | 1 - consul/catalog_endpoint_test.go | 1 - 5 files changed, 7 deletions(-) diff --git a/command/agent/catalog_endpoint_test.go b/command/agent/catalog_endpoint_test.go index bbbeaea8d0..af0a71f5cc 100644 --- a/command/agent/catalog_endpoint_test.go +++ b/command/agent/catalog_endpoint_test.go @@ -175,7 +175,6 @@ func TestCatalogNodes_Blocking(t *testing.T) { // Should block for a while if time.Now().Sub(start) < 50*time.Millisecond { - // TODO: Failing t.Fatalf("too fast") } diff --git a/command/agent/health_endpoint_test.go b/command/agent/health_endpoint_test.go index 03706de870..c1f75a2698 100644 --- a/command/agent/health_endpoint_test.go +++ b/command/agent/health_endpoint_test.go @@ -105,7 +105,6 @@ func TestHealthServiceChecks(t *testing.T) { // Should be 1 health check for consul nodes := obj.(structs.HealthChecks) if len(nodes) != 1 { - // TODO: Failing t.Fatalf("bad: %v", obj) } } @@ -129,7 +128,6 @@ func TestHealthServiceNodes(t *testing.T) { t.Fatalf("err: %v", err) } - // TODO: Failing assertIndex(t, resp) // Should be 1 health check for consul @@ -176,7 +174,6 @@ func TestHealthServiceNodes_PassingFilter(t *testing.T) { t.Fatalf("err: %v", err) } - // TODO: Failing assertIndex(t, resp) // Should be 0 health check for consul diff --git a/command/agent/rpc_client_test.go b/command/agent/rpc_client_test.go index 66d48065ce..1ced706afe 100644 --- a/command/agent/rpc_client_test.go +++ b/command/agent/rpc_client_test.go @@ -243,7 +243,6 @@ OUTER1: } } if !found { - // TODO: Failing t.Fatalf("should log client accept") } diff --git a/command/agent/ui_endpoint_test.go b/command/agent/ui_endpoint_test.go index da4f7e590f..709a6d4f9e 100644 --- a/command/agent/ui_endpoint_test.go +++ b/command/agent/ui_endpoint_test.go @@ -101,7 +101,6 @@ func TestUiNodeInfo(t *testing.T) { t.Fatalf("err: %v", err) } - // TODO: Failing assertIndex(t, resp) // Should be 1 node for the server diff --git a/consul/catalog_endpoint_test.go b/consul/catalog_endpoint_test.go index 233c39fd28..f9721ca098 100644 --- a/consul/catalog_endpoint_test.go +++ b/consul/catalog_endpoint_test.go @@ -545,7 +545,6 @@ func TestCatalogListServices_Timeout(t *testing.T) { // Should block at least 100ms if time.Now().Sub(start) < 100*time.Millisecond { - // TODO: Failing t.Fatalf("too fast") }