igniterealtime / Smack / 1922

40%

push

travis-ci

<a href="https://github.com/igniterealtime/Smack/commit/<a class=hub.com/igniterealtime/Smack/commit/<a class="double-link" href="https://git"><a class=hub.com/igniterealtime/Smack/commit/<a class="double-link" href="https://git"><a class=hub.com/igniterealtime/Smack/commit/<a class="double-link" href="https://git"><a class=hub.com/igniterealtime/Smack/commit/fc70484cf6f3f5a8dc95a6df92a139a106bdae67">fc70484cf<a href="https://github.com/igniterealtime/Smack/commit/fc70484cf6f3f5a8dc95a6df92a139a106bdae67">&quot;&gt;&amp;lt;a href=&amp;quot;https://github.com/igniterealtime/Smack/commit/&lt;/a&gt;&lt;a class=&quot;double-link&quot; href=&quot;https://github.com/igniterealtime/Smack/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/igniterealtime/Smack/commit/fc70484cf6f3f5a8dc95a6df92a139a106bdae67&quot;&gt;fc70484cf&lt;/a&gt;&lt;a href=&quot;https://github.com/igniterealtime/Smack/commit/fc70484cf6f3f5a8dc95a6df92a139a106bdae67&quot;&gt;&amp;lt;a href=&amp;quot;https://github.com/igniterealtime/Smack/commit/fc70484cf6f3f5a8dc95a6df92a139a106bdae67&amp;quot;&amp;gt;&amp;amp;quot;&amp;amp;gt;More specific Pubsub integration tests.

XEP-0060 prohibits publishing a request that contains an item to a node that
is both &amp;amp;amp;#39;notification-only&amp;amp;amp;#39; and &amp;amp;amp;#39;transi&lt;/a>ent&amp;#39; (section 7.1.3.6)

In commit &lt;/a&gt;&lt;a class=&quot;double-link&quot; href=&quot;https://github.com/igniterealtime/Smack/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://github.com/igniterealtime/Smack/commit/<a class="double-link" href="https://github.com/igniterealtime/Smack/commit/8ed872ca639822aeff58eaaa673f953efb0ba6ee">8ed872ca6</a>&quot;&gt;8ed872ca6&lt;/a&gt;&quot;&gt;8ed872ca6&lt;/a&gt;&lt;a href=&quot;https://github.com/igniterealtime/Smack/commit/fc70484cf6f3f5a8dc95a6df92a139a106bdae67&quot;&gt; the existing pubsub publication
test was modified (to resolve a different issue) to operate on a node that&amp;#39;s both
&amp;#39;notification-only&amp;#39; and &amp;#39;transient&amp;#39;. This resulted in a test that should return
an error, even though the test implementation didn&amp;#39;t expect one.

This commit explicitly verifies that publishing an item to such a node causes
an error to be returned</a>. A new test is added that verifies that publishing an
event notification does succeed.

13830 of 36340 relevant lines covered (38.06%)

0.76 hits per line