1
39%
master: 40%

Ran 03 Mar 2022 04:28PM UTC

Files 1256

Run time 5min

Badge

Embed ▾

Committed 03 Mar 2022 04:06PM UTC coverage: 38.706% (+0.03%) from 38.674%

Job # #2680.1

Build Type

push

github-actions

Committed by

guusdk

Commit Message

SINT: Removing invalid test

The implementation of the test that is being removed depends on a server
characteristic that will cause a loop of presence stanzas (which obviously is
bad). A RFC3921-compliant client can send an 'acknowledgement' after receiving
a presence 'subscribed' stanza, in the form of a presence 'subscribe' stanza.
See section 8.2 of RFC3921.

When a server implementation does not ignore this acknowledgement, the domain
of the recipient MUST (RFC6121 section 3.1.3) respond with a 'subscribed' on
behalf of the recipient (which is what the now removed test was verifying).
This can trigger the RFC3921-compliant sender to again receive 'subscribed',
that it again can acknowledge, which causes a loop.

To test RFC6121, the subscription state of the recipient must somehow be
modified to reflect a different state than that of the initiator. I'm not sure
if that is feasible with the SINT framework.

Run Details

16213 of 41888 relevant lines covered (38.71%)

0.39 hits per line

igniterealtime / Smack / #2680 / 1
39%
master: 40%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job #2680.1

igniterealtime / Smack / #2680 / 1 39% master: 40%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job #2680.1

igniterealtime / Smack / #2680 / 1
39%
master: 40%

README BADGES
x