Add more tests and fix add_notification

src/js_realm.hpp

@@ -638,6 +638,9 @@ void RealmClass<T>::add_listener(ContextType ctx, ObjectType this_object, size_t
     auto callback = Value::validated_to_function(ctx, arguments[1]);
 
     SharedRealm realm = *get_internal<T, RealmClass<T>>(this_object);
+    if (realm->is_closed()) {
+        throw ClosedRealmException();
+    }
     get_delegate<T>(realm.get())->add_notification(callback);
 }
 

tests/js/realm-tests.js

@@ -657,6 +657,8 @@ module.exports = {
 
         var person = realm.objects('PersonObject')[0];
 
+        // The tests below assert that everthing throws when
+        // operating on a closed realm
         realm.close();
 
         TestCase.assertThrows(function() {
@@ -666,6 +668,28 @@ module.exports = {
         TestCase.assertThrows(function() {
             realm.objects('PersonObject');
         });
+
+        TestCase.assertThrows(function() {
+            realm.addListener('change', () => {});
+        });
+
+        TestCase.assertThrows(function() {
+            realm.create('PersonObject', {name: 'Ari', age: 10});
+        });
+
+        TestCase.assertThrows(function() {
+            realm.delete(person);
+        });
+
+        TestCase.assertThrows(function() {
+            realm.deleteAll();
+        });
+
+        TestCase.assertThrows(function() {
+            realm.write(() => {});
+        });
+
+
     },
 
     testRealmObjectForPrimaryKey: function() {