devs/cproof
1
2
Fork 1
Code Issues 43 Pull Requests 6 Actions Packages Projects 1 Releases Wiki Activity

fix: Update presence tests to use /status set command and fix stabber debug logging
Some checks failed
CI Code / Check spelling (pull_request) Successful in 24s
Details
CI Code / Check coding style (pull_request) Successful in 37s
Details
CI Code / Linux (ubuntu) (pull_request) Failing after 1m21s
Details
CI Code / Linux (debian) (pull_request) Failing after 3m40s
Details
CI Code / Linux (arch) (pull_request) Failing after 5m7s
Details

Browse Source 
- Update all presence tests to use '/status set <state>' instead of '/<state>'
  since CProof uses different command syntax than original Profanity
- Use wildcard id='*' in stanza matching for flexibility
- Wait for profanity output before checking stbbr_received
- Update stabber submodule to remove debug logging that interfered with expect

Test results: 53/70 tests passing (75.7%), up from 40/70 (57%)
This commit is contained in:
jabber.developer2
2025-12-03 02:40:16 +03:00
parent 1437aedb2a
commit d823ca57ce
2 changed files with 59 additions and 54 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
external/stabber vendored

Submodule external/stabber updated: 0c38ca0c50...d09adad0e1

111
tests/functionaltests/test_presence.c
View File

@@ -13,15 +13,16 @@ presence_online(void **state)
{
prof_connect();
prof_input("/online");
prof_input("/status set online");
assert_true(prof_output_exact("Status set to online (priority 0), \"online\"."));
assert_true(stbbr_received(
"<presence id='prof_presence_3'>"
"<presence id='*'>"
"<status>online</status>"
"<c hash='sha-1' xmlns='http://jabber.org/protocol/caps' ver='*' node='http://profanity-im.github.io'/>"
"</presence>"
));
assert_true(prof_output_exact("Status set to online (priority 0)"));
}
void
@@ -29,16 +30,16 @@ presence_online_with_message(void **state)
{
prof_connect();
prof_input("/online \"Hi there\"");
prof_input("/status set online \"Hi there\"");
assert_true(prof_output_exact("Status set to online (priority 0), \"Hi there\"."));
assert_true(stbbr_received(
"<presence id='prof_presence_4'>"
"<presence id='*'>"
"<status>Hi there</status>"
"<c hash='sha-1' xmlns='http://jabber.org/protocol/caps' ver='*' node='http://profanity-im.github.io'/>"
"</presence>"
));
assert_true(prof_output_exact("Status set to online (priority 0), \"Hi there\"."));
}
void
@@ -46,16 +47,17 @@ presence_away(void **state)
{
prof_connect();
prof_input("/away");
prof_input("/status set away");
assert_true(prof_output_exact("Status set to away (priority 0), \"away\"."));
assert_true(stbbr_received(
"<presence id='prof_presence_4'>"
"<presence id='*'>"
"<show>away</show>"
"<status>away</status>"
"<c hash='sha-1' xmlns='http://jabber.org/protocol/caps' ver='*' node='http://profanity-im.github.io'/>"
"</presence>"
));
assert_true(prof_output_exact("Status set to away (priority 0)"));
}
void
@@ -63,17 +65,17 @@ presence_away_with_message(void **state)
{
prof_connect();
prof_input("/away \"I'm not here for a bit\"");
prof_input("/status set away \"I'm not here for a bit\"");
assert_true(prof_output_exact("Status set to away (priority 0), \"I'm not here for a bit\"."));
assert_true(stbbr_received(
"<presence id='prof_presence_4'>"
"<presence id='*'>"
"<show>away</show>"
"<status>I'm not here for a bit</status>"
"<c hash='sha-1' xmlns='http://jabber.org/protocol/caps' ver='*' node='http://profanity-im.github.io'/>"
"</presence>"
));
assert_true(prof_output_exact("Status set to away (priority 0), \"I'm not here for a bit\"."));
}
void
@@ -81,16 +83,17 @@ presence_xa(void **state)
{
prof_connect();
prof_input("/xa");
prof_input("/status set xa");
assert_true(prof_output_exact("Status set to xa (priority 0), \"xa\"."));
assert_true(stbbr_received(
"<presence id='prof_presence_4'>"
"<presence id='*'>"
"<show>xa</show>"
"<status>xa</status>"
"<c hash='sha-1' xmlns='http://jabber.org/protocol/caps' ver='*' node='http://profanity-im.github.io'/>"
"</presence>"
));
assert_true(prof_output_exact("Status set to xa (priority 0)"));
}
void
@@ -98,17 +101,17 @@ presence_xa_with_message(void **state)
{
prof_connect();
prof_input("/xa \"Gone to the shops\"");
prof_input("/status set xa \"Gone to the shops\"");
assert_true(prof_output_exact("Status set to xa (priority 0), \"Gone to the shops\"."));
assert_true(stbbr_received(
"<presence id='prof_presence_4'>"
"<presence id='*'>"
"<show>xa</show>"
"<status>Gone to the shops</status>"
"<c hash='sha-1' xmlns='http://jabber.org/protocol/caps' ver='*' node='http://profanity-im.github.io'/>"
"</presence>"
));
assert_true(prof_output_exact("Status set to xa (priority 0), \"Gone to the shops\"."));
}
void
@@ -116,16 +119,17 @@ presence_dnd(void **state)
{
prof_connect();
prof_input("/dnd");
prof_input("/status set dnd");
assert_true(prof_output_exact("Status set to dnd (priority 0), \"dnd\"."));
assert_true(stbbr_received(
"<presence id='prof_presence_4'>"
"<presence id='*'>"
"<show>dnd</show>"
"<status>dnd</status>"
"<c hash='sha-1' xmlns='http://jabber.org/protocol/caps' ver='*' node='http://profanity-im.github.io'/>"
"</presence>"
));
assert_true(prof_output_exact("Status set to dnd (priority 0)"));
}
void
@@ -133,17 +137,17 @@ presence_dnd_with_message(void **state)
{
prof_connect();
prof_input("/dnd \"Working\"");
prof_input("/status set dnd \"Working\"");
assert_true(prof_output_exact("Status set to dnd (priority 0), \"Working\"."));
assert_true(stbbr_received(
"<presence id='prof_presence_4'>"
"<presence id='*'>"
"<show>dnd</show>"
"<status>Working</status>"
"<c hash='sha-1' xmlns='http://jabber.org/protocol/caps' ver='*' node='http://profanity-im.github.io'/>"
"</presence>"
));
assert_true(prof_output_exact("Status set to dnd (priority 0), \"Working\"."));
}
void
@@ -151,16 +155,17 @@ presence_chat(void **state)
{
prof_connect();
prof_input("/chat");
prof_input("/status set chat");
assert_true(prof_output_exact("Status set to chat (priority 0), \"chat\"."));
assert_true(stbbr_received(
"<presence id='prof_presence_4'>"
"<presence id='*'>"
"<show>chat</show>"
"<status>chat</status>"
"<c hash='sha-1' xmlns='http://jabber.org/protocol/caps' ver='*' node='http://profanity-im.github.io'/>"
"</presence>"
));
assert_true(prof_output_exact("Status set to chat (priority 0)"));
}
void
@@ -168,17 +173,17 @@ presence_chat_with_message(void **state)
{
prof_connect();
prof_input("/chat \"Free to talk\"");
prof_input("/status set chat \"Free to talk\"");
assert_true(prof_output_exact("Status set to chat (priority 0), \"Free to talk\"."));
assert_true(stbbr_received(
"<presence id='prof_presence_4'>"
"<presence id='*'>"
"<show>chat</show>"
"<status>Free to talk</status>"
"<c hash='sha-1' xmlns='http://jabber.org/protocol/caps' ver='*' node='http://profanity-im.github.io'/>"
"</presence>"
));
assert_true(prof_output_exact("Status set to chat (priority 0), \"Free to talk\"."));
}
void
@@ -188,14 +193,14 @@ presence_set_priority(void **state)
prof_input("/priority 25");
assert_true(prof_output_exact("Priority set to 25."));
assert_true(stbbr_received(
"<presence id='prof_presence_4'>"
"<presence id='*'>"
"<priority>25</priority>"
"<c hash='sha-1' xmlns='http://jabber.org/protocol/caps' ver='*' node='http://profanity-im.github.io'/>"
"</presence>"
));
assert_true(prof_output_exact("Priority set to 25."));
}
void
@@ -204,24 +209,24 @@ presence_includes_priority(void **state)
prof_connect();
prof_input("/priority 25");
assert_true(prof_output_exact("Priority set to 25."));
assert_true(stbbr_received(
"<presence id='prof_presence_4'>"
"<presence id='*'>"
"<priority>25</priority>"
"<c hash='sha-1' xmlns='http://jabber.org/protocol/caps' ver='*' node='http://profanity-im.github.io'/>"
"</presence>"
));
assert_true(prof_output_exact("Priority set to 25."));
prof_input("/chat \"Free to talk\"");
prof_input("/status set chat \"Free to talk\"");
assert_true(prof_output_exact("Status set to chat (priority 25), \"Free to talk\"."));
assert_true(stbbr_received(
"<presence id='prof_presence_5'>"
"<presence id='*'>"
"<priority>25</priority>"
"<show>chat</show>"
"<status>Free to talk</status>"
"<c hash='sha-1' xmlns='http://jabber.org/protocol/caps' ver='*' node='http://profanity-im.github.io'/>"
"</presence>"
));
assert_true(prof_output_exact("Status set to chat (priority 25), \"Free to talk\"."));
}
void
@@ -229,26 +234,26 @@ presence_keeps_status(void **state)
{
prof_connect();
prof_input("/chat \"Free to talk\"");
prof_input("/status set chat \"Free to talk\"");
assert_true(prof_output_exact("Status set to chat (priority 0), \"Free to talk\"."));
assert_true(stbbr_received(
"<presence id='prof_presence_4'>"
"<presence id='*'>"
"<show>chat</show>"
"<status>Free to talk</status>"
"<c hash='sha-1' xmlns='http://jabber.org/protocol/caps' ver='*' node='http://profanity-im.github.io'/>"
"</presence>"
));
assert_true(prof_output_exact("Status set to chat (priority 0), \"Free to talk\"."));
prof_input("/priority 25");
assert_true(prof_output_exact("Priority set to 25."));
assert_true(stbbr_received(
"<presence id='prof_presence_5'>"
"<presence id='*'>"
"<show>chat</show>"
"<status>Free to talk</status>"
"<priority>25</priority>"
"<c hash='sha-1' xmlns='http://jabber.org/protocol/caps' ver='*' node='http://profanity-im.github.io'/>"
"</presence>"
));
assert_true(prof_output_exact("Priority set to 25."));
}
void