Mock ComunicaUpdateEngine.

LDflex · Feb 8, 2019 · e807c45 · e807c45
1 parent 653d9bb
commit e807c45
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 10 deletions.
diff --git a/test/ComunicaUpdateEngine-test.js b/test/ComunicaUpdateEngine-test.js
@@ -7,8 +7,6 @@ describe('a ComunicaUpdateEngine instance', () => {
 
   it('does not yet support source objects', () => {
     engine = new ComunicaUpdateEngine('http://example.org', { dummy: 'source' });
-    // parameter doesn't get added due to mocks
-    engine._source = { dummy: 'source' };
     expect(() => engine.executeUpdate('')).toThrow();
   });
 

diff --git a/test/SubjectPathResolver-test.js b/test/SubjectPathResolver-test.js
@@ -3,6 +3,7 @@ import SubjectPathResolver from '../src/SubjectPathResolver';
 import ComunicaUpdateEngine from '../src/ComunicaUpdateEngine';
 
 jest.mock('ldflex');
+jest.mock('../src/ComunicaUpdateEngine');
 
 describe('a SubjectPathResolver', () => {
   const source = {};

diff --git a/test/__mocks__/ldflex-comunica.js b/test/__mocks__/ldflex-comunica.js
@@ -1,6 +1,9 @@
-const ComunicaEngine = jest.genMockFromModule('ldflex-comunica').default;
+export default class ComunicaEngineMock {
+  constructor(subject, source) {
+    this._source = source;
+  }
 
-async function* noResults() { /* empty */ }
-ComunicaEngine.prototype.execute.mockReturnValue(noResults());
-
-export default ComunicaEngine;
+  getDocument() {
+    return 'https://example.org/';
+  }
+}
diff --git a/test/index-test.js b/test/index-test.js
@@ -1,8 +1,12 @@
 import data from '../src';
 import auth from 'solid-auth-client';
-import ComunicaEngine from 'ldflex-comunica';
+import ComunicaUpdateEngine from '../src/ComunicaUpdateEngine';
 import { namedNode } from '@rdfjs/data-model';
 
+jest.mock('../src/ComunicaUpdateEngine');
+async function* noResults() { /* empty */ }
+ComunicaUpdateEngine.prototype.execute = jest.fn(noResults);
+
 describe('The @solid/ldflex module', () => {
   it('is an ES6 module with a default export', () => {
     expect(require('../src').default).toBe(data);
@@ -19,7 +23,7 @@ describe('The @solid/ldflex module', () => {
     });
 
     it('executes the query', () => {
-      const { constructor, execute } = ComunicaEngine.prototype;
+      const { constructor, execute } = ComunicaUpdateEngine.prototype;
       expect(constructor).toHaveBeenCalledTimes(1);
       expect(constructor.mock.calls[0][0]).toEqual(namedNode(url));
       expect(execute).toHaveBeenCalledTimes(1);
@@ -43,7 +47,7 @@ describe('The @solid/ldflex module', () => {
       });
 
       it('executes the query', async () => {
-        const { constructor, execute } = ComunicaEngine.prototype;
+        const { constructor, execute } = ComunicaUpdateEngine.prototype;
         expect(constructor).toHaveBeenCalledTimes(1);
         await expect(constructor.mock.calls[0][0]).resolves.toEqual(namedNode(webId));
         expect(execute).toHaveBeenCalledTimes(1);