util.stanza: Remove redundant check for attrs
authorKim Alvefur <zash@zash.se>
Mon, 25 Nov 2019 20:46:55 +0100
changeset 10447 f28718f46196
parent 10446 22db763c510c
child 10448 4eab1f5a4f3b
util.stanza: Remove redundant check for attrs A stanza can't not have attrs if created the correct way
spec/util_stanza_spec.lua
util/stanza.lua
--- a/spec/util_stanza_spec.lua	Mon Nov 25 20:44:05 2019 +0100
+++ b/spec/util_stanza_spec.lua	Mon Nov 25 20:46:55 2019 +0100
@@ -176,6 +176,13 @@
 				st.reply(not "a stanza");
 			end, "expected stanza");
 		end);
+
+		it("should reject not-stanzas", function ()
+			assert.has.error_match(function ()
+				st.reply({name="x"});
+			end, "expected stanza");
+		end);
+
 	end);
 
 	describe("#error_reply()", function()
--- a/util/stanza.lua	Mon Nov 25 20:44:05 2019 +0100
+++ b/util/stanza.lua	Mon Nov 25 20:46:55 2019 +0100
@@ -438,7 +438,7 @@
 		error("bad argument to reply: expected stanza, got "..type(orig));
 	end
 	return new_stanza(orig.name,
-		orig.attr and {
+		{
 			to = orig.attr.from,
 			from = orig.attr.to,
 			id = orig.attr.id,