Remove extraneous fmt.Printf calls.
Within test cases, t.Logf is better. Elsewhere, we shouldn't output at allrc.
diff --git a/acceptance/openstack/blockstorage/v1/volumes_test.go b/acceptance/openstack/blockstorage/v1/volumes_test.go
index f84f5cb..6739a99 100644
--- a/acceptance/openstack/blockstorage/v1/volumes_test.go
+++ b/acceptance/openstack/blockstorage/v1/volumes_test.go
@@ -3,7 +3,6 @@
package v1
import (
- "fmt"
"os"
"testing"
@@ -68,7 +67,7 @@
t.Error(err)
return
}
- fmt.Printf("Got volume: %+v\n", v)
+ t.Logf("Got volume: %+v\n", v)
if v.Name != "gophercloud-updated-volume" {
t.Errorf("Unable to update volume: Expected name: gophercloud-updated-volume\nActual name: %s", v.Name)
diff --git a/acceptance/openstack/compute/v2/servers_test.go b/acceptance/openstack/compute/v2/servers_test.go
index 6bdc9e9..05cde73 100644
--- a/acceptance/openstack/compute/v2/servers_test.go
+++ b/acceptance/openstack/compute/v2/servers_test.go
@@ -3,7 +3,6 @@
package v2
import (
- "fmt"
"os"
"testing"
@@ -42,7 +41,7 @@
return true, nil
})
- fmt.Printf("--------\n%d servers listed on %d pages.\n", count, pages)
+ t.Logf("--------\n%d servers listed on %d pages.\n", count, pages)
}
func networkingClient() (*gophercloud.ServiceClient, error) {
diff --git a/openstack/objectstorage/v1/objects/requests.go b/openstack/objectstorage/v1/objects/requests.go
index 1b6cb5c..0ad0315 100644
--- a/openstack/objectstorage/v1/objects/requests.go
+++ b/openstack/objectstorage/v1/objects/requests.go
@@ -49,7 +49,6 @@
if opts != nil {
full, query, err := opts.ToObjectListParams()
if err != nil {
- fmt.Printf("Error building query string: %v", err)
return pagination.Pager{Err: err}
}
url += query
diff --git a/pagination/linked.go b/pagination/linked.go
index 447d4b1..461fa49 100644
--- a/pagination/linked.go
+++ b/pagination/linked.go
@@ -39,8 +39,6 @@
return "", nil
}
- fmt.Printf("key = %#v, path = %#v, value = %#v\n", key, path, value)
-
if len(path) > 0 {
submap, ok = value.(map[string]interface{})
if !ok {