Opened on 04/18/2017 at 09:14:32 AM

Closed on 08/29/2019 at 05:43:52 PM

Last modified on 10/08/2019 at 05:52:22 PM

#5149 closed change (rejected)

[emscripten] Verify that unit tests don't leak any objects

Reported by: trev Assignee: tlucas
Priority: P2 Milestone:
Module: Core Keywords: closed-in-favor-of-gitlab
Cc: oleksandr, tlucas, sergz Blocked By:
Blocking: #4122 Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):



Now that we do "manual" garbage collection, it's important that we never forget to release object references.

What to change

Verify that by the end of each unit test all objects have been garbage collected. The trivial implementation would be adding a filter/subscription counter - global variables incremented in Filter/Subscription constructor and decremented in the corresponding destructor. At the end of each unit test these counters should have the value zero. A more generic solution would be desirable, maybe by using sandboxed-module to instrument some Emscripten code and track all objects exposed by bindings - question is whether this approach is feasible (normally this kind of instrumentation is done with a special build but we wouldn't want to require a special builds for tests).

Attachments (0)

Change History (4)

comment:1 Changed on 11/19/2017 at 12:34:54 PM by oleksandr

  • Cc oleksandr tlucas added
  • Owner set to tlucas

comment:2 Changed on 11/22/2017 at 03:05:21 PM by sergz

  • Cc sergz added

comment:3 Changed on 08/16/2019 at 10:08:43 AM by yoyo9996


Last edited on 10/08/2019 at 05:52:22 PM by kzar

comment:4 Changed on 08/29/2019 at 05:43:52 PM by sebastian

  • Keywords closed-in-favor-of-gitlab added
  • Resolution set to rejected
  • Status changed from new to closed

Sorry, but we switched to GitLab. If this issue is still relevant, please file it again in the new issue tracker.

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from tlucas.
Note: See TracTickets for help on using tickets.