Skip to content
This repository has been archived by the owner on Sep 3, 2021. It is now read-only.

Commit

Permalink
Make #252 less painful, mark not-isolated tests
Browse files Browse the repository at this point in the history
With this commit you can run

```
npm run test-isolated
```

over and over again and the tests should pass. This does not fix the
issue mentioned above but it makes it less painful. At least I can make
my changes and run all tests exluding those that are not isolated. When I'm done
with my changes I can re-import the data and run all the tests, even those that
are not isolated.

This is what I did to get all the tests passing on my machine:

```
sudo rm /var/lib/neo4j/data/databases/* -r
sudo cp -rp ~/Downloads/temp/neo4j-graphql-js/recommendations.db/ /var/lib/neo4j/data/databases/graph.db
sudo systemctl restart neo4j.service
```

Very annoying.

@johnymontana @michaeldgraham do you know if there is a single command
that brings the database into a state where it was *before* running the
test case? Something that we could use in a `beforeEach` or `afterEach`?

See: #252 (comment)
  • Loading branch information
roschaefer committed May 22, 2019
1 parent f5af6d5 commit 467cf17
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 14 deletions.
3 changes: 2 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,8 @@
"parse-tck": "babel-node test/helpers/tck/parseTck.js",
"test-tck": "nyc ava --fail-fast test/tck/",
"report-coverage": "nyc report --reporter=text-lcov > coverage.lcov && codecov",
"test-all": "nyc ava test/*.js test/neo4j-schema/*Test.js --verbose",
"test-all": "nyc ava test/ --verbose",
"test-isolated": "nyc ava test/ --verbose --match='!*not-isolated*'",
"debug": "nodemon ./example/apollo-server/movies.js --exec babel-node --inspect-brk=9229 --nolazy",
"debug-typedefs": "nodemon ./example/apollo-server/movies-typedefs.js --exec babel-node --inspect-brk=9229 --nolazy",
"debug-interface": "nodemon ./example/apollo-server/interfaceError.js --exec babel-node --inspect-brk=9229 --nolazy"
Expand Down
26 changes: 13 additions & 13 deletions test/integration.js → test/integration/integration.js
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,7 @@ test('Handle @cypher directive on QueryType', async t => {
});
});

test('Mutation with @cypher directive', async t => {
test('Mutation with @cypher directive (not-isolated)', async t => {
t.plan(1);

let expected = {
Expand Down Expand Up @@ -202,7 +202,7 @@ test('Mutation with @cypher directive', async t => {
});
});

test('Create node mutation', async t => {
test('Create node mutation (not-isolated)', async t => {
t.plan(1);

let expected = {
Expand Down Expand Up @@ -279,7 +279,7 @@ test('Update node mutation', async t => {
});
});

test.serial('Add relationship mutation', async t => {
test.serial('Add relationship mutation (not-isolated)', async t => {
t.plan(1);

let expected = {
Expand Down Expand Up @@ -348,7 +348,7 @@ test.serial('Add relationship mutation', async t => {
});
});

test.serial('Remove relationship mutation', async t => {
test.serial('Remove relationship mutation (not-isolated)', async t => {
t.plan(1);

await client
Expand Down Expand Up @@ -613,7 +613,7 @@ test('query using inine fragment', async t => {
*/

// Temporal node property
test.serial('Temporal - Create node with temporal property', async t => {
test.serial('Temporal - Create node with temporal property (not-isolated)', async t => {
t.plan(1);

let expected = {
Expand Down Expand Up @@ -660,7 +660,7 @@ test.serial('Temporal - Create node with temporal property', async t => {
});

test.serial(
'Temporal - Create node with multiple temporal fields and input formats',
'Temporal - Create node with multiple temporal fields and input formats (not-isolated)',
async t => {
t.plan(1);

Expand Down Expand Up @@ -727,7 +727,7 @@ test.serial(
);

test.serial(
'Temporal - Create node with multiple temporal fields and input formats - with GraphQL variables',
'Temporal - Create node with multiple temporal fields and input formats - with GraphQL variables (not-isolated)',
async t => {
t.plan(1);

Expand Down Expand Up @@ -802,7 +802,7 @@ test.serial(
}
);

test.serial('Temporal - Query node with temporal field', async t => {
test.serial('Temporal - Query node with temporal field (not-isolated)', async t => {
let expected = {
data: {
Movie: [
Expand Down Expand Up @@ -881,7 +881,7 @@ test.serial('Temporal - Query node with temporal field', async t => {
});
});

test.serial('Temporal - create node with only a temporal property', async t => {
test.serial('Temporal - create node with only a temporal property (not-isolated)', async t => {
t.plan(1);

let expected = {
Expand Down Expand Up @@ -916,7 +916,7 @@ test.serial('Temporal - create node with only a temporal property', async t => {
});
});

test.serial('Temporal - temporal query argument, components', async t => {
test.serial('Temporal - temporal query argument, components (not-isolated)', async t => {
t.plan(1);

let expected = {
Expand Down Expand Up @@ -953,7 +953,7 @@ test.serial('Temporal - temporal query argument, components', async t => {
});
});

test.serial('Temporal - temporal query argument, formatted', async t => {
test.serial('Temporal - temporal query argument, formatted (not-isolated)', async t => {
t.plan(1);

let expected = {
Expand Down Expand Up @@ -990,7 +990,7 @@ test.serial('Temporal - temporal query argument, formatted', async t => {
});
});

test.serial('Add relationship with temporal property', async t => {
test.serial('Add relationship with temporal property (not-isolated)', async t => {
t.plan(1);

let expected = {
Expand Down Expand Up @@ -1031,7 +1031,7 @@ test.serial('Add relationship with temporal property', async t => {
});
});

test.serial('Query for temporal property on relationship', async t => {
test.serial('Query for temporal property on relationship (not-isolated)', async t => {
t.plan(1);

let expected = {
Expand Down
File renamed without changes.

0 comments on commit 467cf17

Please sign in to comment.