PR updates

src/js_user.hpp

@@ -130,12 +130,13 @@ class User : public SharedUser {
         , m_app(nullptr)
     {
     }
-    User(User&&) = default;
+
+    // Remove copy constructors to avoid destroying the listener Token
     User(const User&) = delete;
     User& operator=(const User&) = delete;
 
+    User(User&&) = default;
     User& operator=(User&&) = default;
-    // User& operator=(User const&) = default;
 
     using CallbackTokenPair = std::pair<Protected<typename T::Function>, Token>;
     std::forward_list<CallbackTokenPair> m_notification_tokens;

tests/js/app-tests.js

@@ -226,18 +226,22 @@ module.exports = {
 
     const listenerEvent = () => {
       eventListenerCalls += 1;
-      console.log("calls thus far: ", eventListenerCalls);
     };
+
     app.addListener(listenerEvent);
 
     let credentials = Realm.Credentials.anonymous();
     let user = await app.logIn(credentials);
     TestCase.assertEqual(eventListenerCalls, 1);
+
     await user.logOut();
     TestCase.assertEqual(eventListenerCalls, 2);
     credentials = Realm.Credentials.anonymous();
+
     user = await app.logIn(credentials);
     TestCase.assertEqual(eventListenerCalls, 3);
+
+    // Remove the listener and verify that the event listener was not fired
     app.removeListener(listenerEvent);
     await user.logOut();
     TestCase.assertEqual(eventListenerCalls, 3);

tests/spec/unit_tests.js

@@ -31,7 +31,7 @@ RealmLogging.patch(Realm);
 
 const RealmTests = require("../js");
 
-jasmine.DEFAULT_TIMEOUT_INTERVAL = 300000000;
+jasmine.DEFAULT_TIMEOUT_INTERVAL = 30000;
 let isDebuggerAttached = typeof v8debug === "object";
 if (!isDebuggerAttached && isNodeProcess) {
   isDebuggerAttached = /--debug|--inspect/.test(process.execArgv.join(" "));