adding tests for domains (#91)

access_test.go

@@ -165,3 +165,56 @@ func TestACL(t *testing.T) {
 	assert.Nil(t, err)
 	assert.Len(t, acls, 1)
 }
+
+func TestNewDomain(t *testing.T) {
+	mocks.On(mockConfig)
+	defer mocks.Off()
+	client := mockClient()
+
+	assert.Nil(t, client.NewDomain("test", "t"))
+}
+
+func TestDomain_Update(t *testing.T) {
+	mocks.On(mockConfig)
+	defer mocks.Off()
+	client := mockClient()
+
+	// no realm name
+	domain := Domain{
+		client: client,
+	}
+
+	assert.Error(t, domain.Update())
+	domain.Realm = "test"
+	assert.Nil(t, domain.Update())
+}
+
+func TestDomain_Delete(t *testing.T) {
+	mocks.On(mockConfig)
+	defer mocks.Off()
+	client := mockClient()
+
+	// no realm name
+	domain := Domain{
+		client: client,
+	}
+
+	assert.Error(t, domain.Delete())
+	domain.Realm = "test"
+	assert.Nil(t, domain.Delete())
+}
+
+func TestDomain_Sync(t *testing.T) {
+	mocks.On(mockConfig)
+	defer mocks.Off()
+	client := mockClient()
+
+	// no realm name
+	domain := Domain{
+		client: client,
+	}
+
+	assert.Error(t, domain.Sync(DomainSyncOptions{}))
+	domain.Realm = "test"
+	assert.Nil(t, domain.Sync(DomainSyncOptions{}))
+}

proxmox.go

@@ -275,6 +275,10 @@ func (c *Client) handleResponse(res *http.Response, v interface{}) error {
 		return fmt.Errorf("bad request: %s - %s", res.Status, string(body))
 	}
 
+	// if nil passed dont bother to do any unmarshalling
+	if nil == v {
+		return nil
+	}
 	// account for everything being in a data key
 	var datakey map[string]json.RawMessage
 	if err := json.Unmarshal(body, &datakey); err != nil {

tests/mocks/pve7x/access.go

@@ -830,4 +830,23 @@ func access() {
     ]
 }`)
 
+	gock.New(config.C.URI).
+		Post("^/access/domains").
+		Reply(200).
+		JSON(``)
+
+	gock.New(config.C.URI).
+		Put("^/access/domains/test$").
+		Reply(200).
+		JSON(``)
+
+	gock.New(config.C.URI).
+		Delete("^/access/domains/test$").
+		Reply(200).
+		JSON(``)
+
+	gock.New(config.C.URI).
+		Post("^/access/domains/test$").
+		Reply(200).
+		JSON(``)
 }