Various fixes

stanza/commands.go

@@ -119,26 +119,27 @@ func (c *Command) UnmarshalXML(d *xml.Decoder, start xml.StartElement) error {
 
 			case "affiliations":
 				a := Actions{}
-				d.DecodeElement(&a, &tt)
+				err = d.DecodeElement(&a, &tt)
 				c.CommandElement = &a
 			case "configure":
 				nt := Note{}
-				d.DecodeElement(&nt, &tt)
+				err = d.DecodeElement(&nt, &tt)
 				c.CommandElement = &nt
 			case "x":
 				f := Form{}
-				d.DecodeElement(&f, &tt)
+				err = d.DecodeElement(&f, &tt)
 				c.CommandElement = &f
 			default:
 				n := Node{}
-				e := d.DecodeElement(&n, &tt)
-				_ = e
+				err = d.DecodeElement(&n, &tt)
 				c.CommandElement = &n
 				if err != nil {
 					return err
 				}
 			}
-
+			if err != nil {
+				return err
+			}
 		case xml.EndElement:
 			if tt == start.End() {
 				return nil

stanza/pubsub_owner.go

@@ -226,7 +226,10 @@ func NewApprovePendingSubRequest(serviceId, sessionId, nodeId string) (*IQ, erro
 		return nil, err
 	}
 	var n Node
-	xml.Unmarshal(data, &n)
+	err = xml.Unmarshal(data, &n)
+	if err != nil {
+		return nil, err
+	}
 
 	iq, err := NewIQ(Attrs{Type: IQTypeSet, To: serviceId})
 	if err != nil {
@@ -406,33 +409,35 @@ func (pso *PubSubOwner) UnmarshalXML(d *xml.Decoder, start xml.StartElement) err
 
 			case "affiliations":
 				aff := AffiliationsOwner{}
-				d.DecodeElement(&aff, &tt)
+				err = d.DecodeElement(&aff, &tt)
 				pso.OwnerUseCase = &aff
 			case "configure":
 				co := ConfigureOwner{}
-				d.DecodeElement(&co, &tt)
+				err = d.DecodeElement(&co, &tt)
 				pso.OwnerUseCase = &co
 			case "default":
 				def := DefaultOwner{}
-				d.DecodeElement(&def, &tt)
+				err = d.DecodeElement(&def, &tt)
 				pso.OwnerUseCase = &def
 			case "delete":
 				del := DeleteOwner{}
-				d.DecodeElement(&del, &tt)
+				err = d.DecodeElement(&del, &tt)
 				pso.OwnerUseCase = &del
 			case "purge":
 				pu := PurgeOwner{}
-				d.DecodeElement(&pu, &tt)
+				err = d.DecodeElement(&pu, &tt)
 				pso.OwnerUseCase = &pu
 			case "subscriptions":
 				subs := SubscriptionsOwner{}
-				d.DecodeElement(&subs, &tt)
+				err = d.DecodeElement(&subs, &tt)
 				pso.OwnerUseCase = &subs
 				if err != nil {
 					return err
 				}
 			}
-
+			if err != nil {
+				return err
+			}
 		case xml.EndElement:
 			if tt == start.End() {
 				return nil

stanza/pubsub_owner_test.go

@@ -20,9 +20,6 @@ func TestNewConfigureNode(t *testing.T) {
 		t.Fatalf("failed to create a configure node request: %v", err)
 	}
 	subR.Id = "config1"
-	if err != nil {
-		t.Fatalf("Could not create request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)
@@ -136,9 +133,6 @@ func TestNewRequestDefaultConfig(t *testing.T) {
 		t.Fatalf("failed to create a default config request: %v", err)
 	}
 	subR.Id = "def1"
-	if err != nil {
-		t.Fatalf("Could not create request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)
@@ -249,9 +243,6 @@ func TestNewDelNode(t *testing.T) {
 		t.Fatalf("failed to create a node delete request: %v", err)
 	}
 	subR.Id = "delete1"
-	if err != nil {
-		t.Fatalf("Could not create request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)
@@ -324,9 +315,6 @@ func TestNewPurgeAllItems(t *testing.T) {
 		t.Fatalf("failed to create a purge all items request: %v", err)
 	}
 	subR.Id = "purge1"
-	if err != nil {
-		t.Fatalf("Could not create request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)
@@ -383,9 +371,6 @@ func TestNewApproveSubRequest(t *testing.T) {
 		t.Fatalf("failed to create a sub approval request: %v", err)
 	}
 	subR.Id = "approve1"
-	if err != nil {
-		t.Fatalf("Could not create request : %s", err)
-	}
 
 	frm, ok := subR.Extensions[0].(*stanza.Form)
 	if !ok {
@@ -423,9 +408,6 @@ func TestNewGetPendingSubRequests(t *testing.T) {
 		t.Fatalf("failed to create a get pending subs request: %v", err)
 	}
 	subR.Id = "pending1"
-	if err != nil {
-		t.Fatalf("Could not create request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)
@@ -507,9 +489,6 @@ func TestNewApprovePendingSubRequest(t *testing.T) {
 		t.Fatalf("failed to create a approve pending sub request: %v", err)
 	}
 	subR.Id = "pending2"
-	if err != nil {
-		t.Fatalf("Could not create request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)
@@ -549,9 +528,6 @@ func TestNewSubListRqPl(t *testing.T) {
 		t.Fatalf("failed to create a sub list request: %v", err)
 	}
 	subR.Id = "subman1"
-	if err != nil {
-		t.Fatalf("Could not create request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)
@@ -627,9 +603,6 @@ func TestNewAffiliationListRequest(t *testing.T) {
 		t.Fatalf("failed to create an affiliations list request: %v", err)
 	}
 	subR.Id = "ent1"
-	if err != nil {
-		t.Fatalf("Could not create request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)
@@ -721,9 +694,6 @@ func TestNewModifAffiliationRequest(t *testing.T) {
 		t.Fatalf("failed to create a modif affiliation request: %v", err)
 	}
 	subR.Id = "ent3"
-	if err != nil {
-		t.Fatalf("Could not create request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)
@@ -866,11 +836,7 @@ func TestNewFormSubmissionOwner(t *testing.T) {
 	if err != nil {
 		t.Fatalf("failed to create a form submission request: %v", err)
 	}
-
 	subR.Id = "config2"
-	if err != nil {
-		t.Fatalf("Could not create request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)

stanza/pubsub_test.go

@@ -44,9 +44,6 @@ func TestNewSubRequest(t *testing.T) {
 		t.Fatalf("failed to create a sub request: %v", err)
 	}
 	subR.Id = "sub1"
-	if err != nil {
-		t.Fatalf("Could not create a sub request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)
@@ -103,9 +100,6 @@ func TestNewUnSubRequest(t *testing.T) {
 		t.Fatalf("failed to create an unsub request: %v", err)
 	}
 	subR.Id = "unsub1"
-	if err != nil {
-		t.Fatalf("Could not create a sub request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)
@@ -167,9 +161,6 @@ func TestNewSubOptsRq(t *testing.T) {
 		t.Fatalf("failed to create a sub options request: %v", err)
 	}
 	subR.Id = "options1"
-	if err != nil {
-		t.Fatalf("Could not create a sub request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)
@@ -277,9 +268,6 @@ func TestNewFormSubmission(t *testing.T) {
 		t.Fatalf("failed to create a form submission request: %v", err)
 	}
 	subR.Id = "options2"
-	if err != nil {
-		t.Fatalf("Could not create a sub request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)
@@ -329,9 +317,6 @@ func TestNewSubAndConfig(t *testing.T) {
 		t.Fatalf("failed to create a sub and config request: %v", err)
 	}
 	subR.Id = "sub1"
-	if err != nil {
-		t.Fatalf("Could not create a sub request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)
@@ -501,9 +486,6 @@ func TestNewSpecificItemRequest(t *testing.T) {
 		t.Fatalf("failed to create a specific item request: %v", err)
 	}
 	subR.Id = "items3"
-	if err != nil {
-		t.Fatalf("Could not create an items request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated sub request : %s", e)
@@ -660,9 +642,6 @@ func TestNewDelItemFromNode(t *testing.T) {
 		t.Fatalf("failed to create a delete item from node request: %v", err)
 	}
 	subR.Id = "retract1"
-	if err != nil {
-		t.Fatalf("Could not create a del item request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated del item request : %s", e)
@@ -702,9 +681,6 @@ func TestNewCreateNode(t *testing.T) {
 		t.Fatalf("failed to create a create node request: %v", err)
 	}
 	subR.Id = "create1"
-	if err != nil {
-		t.Fatalf("Could not create a create node request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated del item request : %s", e)
@@ -777,9 +753,6 @@ func TestNewCreateAndConfigNode(t *testing.T) {
 		t.Fatalf("failed to create a create and config node request: %v", err)
 	}
 	subR.Id = "create1"
-	if err != nil {
-		t.Fatalf("Could not create a create node request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated del item request : %s", e)
@@ -828,9 +801,6 @@ func TestNewRetrieveAllSubsRequest(t *testing.T) {
 		t.Fatalf("failed to create a get all subs request: %v", err)
 	}
 	subR.Id = "subscriptions1"
-	if err != nil {
-		t.Fatalf("Could not create a create node request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated del item request : %s", e)
@@ -891,9 +861,6 @@ func TestNewRetrieveAllAffilsRequest(t *testing.T) {
 		t.Fatalf("failed to create a get all affiliations request: %v", err)
 	}
 	subR.Id = "affil1"
-	if err != nil {
-		t.Fatalf("Could not create retreive all affiliations request : %s", err)
-	}
 
 	if _, e := checkMarshalling(t, subR); e != nil {
 		t.Fatalf("Failed to check marshalling for generated retreive all affiliations request : %s", e)