After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 647200 - libsocialweb Aggregation test fails
libsocialweb Aggregation test fails
Status: RESOLVED FIXED
Product: folks
Classification: Platform
Component: libsocialweb backend
git master
Other Linux
: Normal normal
: Unset
Assigned To: folks-maint
folks-maint
Depends on:
Blocks:
 
 
Reported: 2011-04-08 18:30 UTC by Travis Reitter
Modified: 2011-04-11 15:15 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
bug647200.patch (3.25 KB, patch)
2011-04-11 12:29 UTC, Alban Crequy
accepted-commit_now Details | Review

Description Travis Reitter 2011-04-08 18:30:51 UTC
I consistently hit this failed assertion in the lsw aggregation test because the test never reaches the final IndividualAggregator:individuals-changed signal handler for the linking:

      timer_id = Timeout.add_seconds (5, () =>
        {
          assert_not_reached ();
        });
      main_loop.run ();
      Source.remove (timer_id);

      this._lsw_backend.tear_down ();

(ie, the final timeout check)

It's not clear why the individuals-changed signal doesn't fire the second time.
Comment 1 Travis Reitter 2011-04-08 18:32:39 UTC
This failure happens with folks git master and libsocialweb 0.25.15.
Comment 2 Alban Crequy 2011-04-11 12:29:26 UTC
Created attachment 185695 [details] [review]
bug647200.patch
Comment 3 Travis Reitter 2011-04-11 14:28:22 UTC
Review of attachment 185695 [details] [review]:

Looks good. Please merge.
Comment 4 Alban Crequy 2011-04-11 15:15:46 UTC
Merged.