#3106
39%
master: 40%

Ran 26 Apr 2024 09:02AM UTC

Jobs 1

Files 1281

Run time 2min

Badge

Embed ▾

Committed 26 Apr 2024 08:56AM UTC coverage: 39.415%. Remained the same

Build # #3106

Build Type

push

github

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

16922 of 42933 relevant lines covered (39.41%)

0.39 hits per line

Uncovered Existing Lines

Lines	Coverage	∆	File
84	0.0	0.0%	smack-integration-test/src/main/java/org/jivesoftware/smack/roster/RosterIntegrationTest.java

Jobs

ID	Job ID	Ran	Files	Coverage
1	#3106.1	26 Apr 2024 09:02AM UTC	1281	39.41

igniterealtime / Smack / #3106
39%
master: 40%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build #3106

igniterealtime / Smack / #3106 39% master: 40%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build #3106

igniterealtime / Smack / #3106
39%
master: 40%

README BADGES
x