test: move createConflict() from fixtures to util (#1375)

The files in test/integration/fixtures are repeatedly `require()`d and run
in the `initialize()` function of `test/integration/setup.js`.  The
`scenarios.js` does not need this treatment.

test/integration/api/datasets.js

@@ -8,7 +8,7 @@ const { getOrNotFound } = require('../../../lib/util/promise');
 const should = require('should');
 const { sql } = require('slonik');
 const { QueryOptions } = require('../../../lib/util/db');
-const { createConflict } = require('../fixtures/scenarios');
+const { createConflict } = require('../../util/scenarios');
 const { omit } = require('ramda');
 
 const { exhaust } = require(appRoot + '/lib/worker/worker');

test/integration/api/projects.js

@@ -5,7 +5,7 @@ const { testService } = require('../setup');
 const testData = require('../../data/xml');
 const { QueryOptions } = require('../../../lib/util/db');
 const { Actor } = require('../../../lib/model/frames');
-const { createConflict } = require('../fixtures/scenarios');
+const { createConflict } = require('../../util/scenarios');
 // eslint-disable-next-line import/no-dynamic-require
 const { exhaust } = require(appRoot + '/lib/worker/worker');
 

test/integration/fixtures/scenarios.js → test/util/scenarios.js

@@ -1,6 +1,6 @@
 const appRoot = require('app-root-path');
 const { exhaust } = require(appRoot + '/lib/worker/worker');
-const testData = require('../../data/xml');
+const testData = require(appRoot + '/test/data/xml');
 
 const createConflict = async (user, container) => {
   await user.post('/v1/projects/1/forms/simpleEntity/submissions')