Add tests for nil *Resource (#2227)

* Add tests for nil *Resource * Update sdk/resource/resource_test.go Co-authored-by: Tyler Yahn <[email protected]> Co-authored-by: Tyler Yahn <[email protected]>
open-telemetry · Sep 10, 2021 · 360d130 · 360d130
1 parent 9e7812d
commit 360d130
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/sdk/resource/resource.go b/sdk/resource/resource.go
@@ -132,6 +132,9 @@ func (r *Resource) Attributes() []attribute.KeyValue {
 }
 
 func (r *Resource) SchemaURL() string {
+	if r == nil {
+		return ""
+	}
 	return r.schemaURL
 }
 

diff --git a/sdk/resource/resource_test.go b/sdk/resource/resource_test.go
@@ -205,6 +205,17 @@ func TestMerge(t *testing.T) {
 	}
 }
 
+func TestEmpty(t *testing.T) {
+	var res *resource.Resource
+	assert.Equal(t, "", res.SchemaURL())
+	assert.Equal(t, "", res.String())
+	assert.Equal(t, []attribute.KeyValue(nil), res.Attributes())
+
+	it := res.Iter()
+	assert.Equal(t, 0, it.Len())
+	assert.True(t, res.Equal(res))
+}
+
 func TestDefault(t *testing.T) {
 	res := resource.Default()
 	require.False(t, res.Equal(resource.Empty()))