Add test for handling exception in callback functions

[karczex]

This change is 

[pmem-bot]

@szyrom and @ldorau please review this pull request

[lukaszstolarczuk]

Reviewed 1 of 1 files at r1.
Reviewable status: complete! all files reviewed, all discussions resolved

[ldorau]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @karczex)

---

test/database_spec.js, line 332 at r1 (raw file):

             db.get_keys_below('B', ex);
        } catch (e) {
            expect(e.message).to.equal('TestException');

This test fails ... ?

[karczex]

---

test/database_spec.js, line 332 at r1 (raw file):

Previously, ldorau (Lukasz Dorau) wrote…

This test fails ... ?

Yes, it fails. It's test for this issue #51

[ldorau]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @karczex)

---

test/database_spec.js, line 332 at r1 (raw file):

Previously, karczex wrote…

Yes, it fails. It's test for this issue #51

So when this test will not fail?

[karczex]

---

test/database_spec.js, line 332 at r1 (raw file):

Previously, ldorau (Lukasz Dorau) wrote…

So when this test will not fail?

I don't know.

[ldorau]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @karczex)

---

test/database_spec.js, line 332 at r1 (raw file):

Previously, karczex wrote…

I don't know.

Ah, so this PR just waits for a fix?

[karczex]

---

test/database_spec.js, line 332 at r1 (raw file):

Previously, ldorau (Lukasz Dorau) wrote…

Ah, so this PR just waits for a fix?

Yes, it's bug reproducer. Unfortunately build system doesn't support tests which are failing. I think this PR may stay as draft until fixed.

[ldorau]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @karczex)

---

test/database_spec.js, line 332 at r1 (raw file):

Previously, karczex wrote…

Yes, it's bug reproducer. Unfortunately build system doesn't support tests which are failing. I think this PR may stay as draft until fixed.

I see, OK