From e9ee50babb3bbfe7a6e774460a46782abefd5107 Mon Sep 17 00:00:00 2001
From: Kiran Kamineni <kiran.k.kamineni@intel.com>
Date: Thu, 17 Jan 2019 11:49:39 -0800
Subject: Minor refactoring of definition unit tests

The definition unit tests needed some small changes to
ensure consistency between the source and the tests.
P3: Sort both the slices before running DeepEqual

Issue-ID: MULTICLOUD-440
Change-Id: If797322e44321a580cb1441bd5c60b812799f844
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>
---
 src/k8splugin/rb/definition_test.go | 24 ++++++++++++++++++------
 1 file changed, 18 insertions(+), 6 deletions(-)

(limited to 'src/k8splugin/rb')

diff --git a/src/k8splugin/rb/definition_test.go b/src/k8splugin/rb/definition_test.go
index 1e488678..0e4ffd91 100644
--- a/src/k8splugin/rb/definition_test.go
+++ b/src/k8splugin/rb/definition_test.go
@@ -21,13 +21,14 @@ package rb
 import (
 	"k8splugin/db"
 	"reflect"
+	"sort"
 	"strings"
 	"testing"
 
 	pkgerrors "github.com/pkg/errors"
 )
 
-func TestCreate(t *testing.T) {
+func TestCreateDefinition(t *testing.T) {
 	testCases := []struct {
 		label         string
 		inp           Definition
@@ -83,7 +84,7 @@ func TestCreate(t *testing.T) {
 	}
 }
 
-func TestList(t *testing.T) {
+func TestListDefinition(t *testing.T) {
 
 	testCases := []struct {
 		label         string
@@ -145,6 +146,17 @@ func TestList(t *testing.T) {
 					t.Fatalf("List returned an unexpected error %s", err)
 				}
 			} else {
+				// Since the order of returned slice is not guaranteed
+				// Check both and return error if both don't match
+				sort.Slice(got, func(i, j int) bool {
+					return got[i].UUID < got[i].UUID
+				})
+				// Sort both as it is not expected that testCase.expected
+				// is sorted
+				sort.Slice(testCase.expected, func(i, j int) bool {
+					return testCase.expected[i].UUID < testCase.expected[i].UUID
+				})
+
 				if reflect.DeepEqual(testCase.expected, got) == false {
 					t.Errorf("List Resource Bundle returned unexpected body: got %v;"+
 						" expected %v", got, testCase.expected)
@@ -154,7 +166,7 @@ func TestList(t *testing.T) {
 	}
 }
 
-func TestGet(t *testing.T) {
+func TestGetDefinition(t *testing.T) {
 
 	testCases := []struct {
 		label         string
@@ -214,7 +226,7 @@ func TestGet(t *testing.T) {
 	}
 }
 
-func TestDelete(t *testing.T) {
+func TestDeleteDefinition(t *testing.T) {
 
 	testCases := []struct {
 		label         string
@@ -253,7 +265,7 @@ func TestDelete(t *testing.T) {
 	}
 }
 
-func TestUpload(t *testing.T) {
+func TestUploadDefinition(t *testing.T) {
 	testCases := []struct {
 		label         string
 		inp           string
@@ -325,7 +337,7 @@ func TestUpload(t *testing.T) {
 					"123e4567-e89b-12d3-a456-426655441111": []byte(
 						"{\"name\":\"testresourcebundle\"," +
 							"\"description\":\"testresourcebundle\"," +
-							"\"uuid\":\"123e4567-e89b-12d3-a456-426655440000\"," +
+							"\"uuid\":\"123e4567-e89b-12d3-a456-426655441111\"," +
 							"\"service-type\":\"firewall\"}"),
 				},
 			},
-- 
cgit 1.2.3-korg