Skip to content

Commit 9c869cf

Browse files
committed
Updated step regexp
1 parent 021c945 commit 9c869cf

File tree

3 files changed

+19
-18
lines changed

3 files changed

+19
-18
lines changed

driver/src/test/java/org/neo4j/driver/v1/tck/CypherComplianceSteps.java

Lines changed: 16 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -31,17 +31,16 @@
3131

3232
import org.neo4j.driver.v1.ResultCursor;
3333
import org.neo4j.driver.v1.Value;
34-
import org.neo4j.driver.v1.tck.tck.util.Types;
3534
import org.neo4j.driver.v1.tck.tck.util.runners.CypherStatementRunner;
3635
import org.neo4j.driver.v1.tck.tck.util.runners.MappedParametersRunner;
3736
import org.neo4j.driver.v1.tck.tck.util.runners.StringRunner;
3837

3938
import static org.junit.Assert.assertEquals;
4039
import static org.junit.Assert.assertTrue;
4140
import static org.neo4j.driver.v1.tck.DriverComplianceIT.session;
41+
import static org.neo4j.driver.v1.tck.Environment.mappedTypes;
4242
import static org.neo4j.driver.v1.tck.Environment.runners;
4343
import static org.neo4j.driver.v1.tck.tck.util.ResultParser.getParametersFromListOfKeysAndValues;
44-
import static org.neo4j.driver.v1.tck.tck.util.ResultParser.getTypeMapFromString;
4544
import static org.neo4j.driver.v1.tck.tck.util.ResultParser.parseExpected;
4645
import static org.neo4j.driver.v1.tck.tck.util.ResultParser.parseGiven;
4746
import static org.neo4j.driver.v1.tck.tck.util.Types.getType;
@@ -96,12 +95,11 @@ public void result_should_be_a_type_containing(String type, DataTable table) thr
9695
}
9796
}
9897

99-
@Then( "^result should be mixed: \"(.*)\"$" )
100-
public void result_should_be_mixed_a_node_b_node_l_integer( String stringTypes, DataTable table ) throws Throwable
98+
@Then( "^result should be mixed:" )
99+
public void result_should_be_mixed( DataTable table ) throws Throwable
101100
{
102101
for( CypherStatementRunner runner : runners)
103102
{
104-
Map<String,Types.Type> types = getTypeMapFromString( stringTypes );
105103
ResultCursor rc = runner.result();
106104
List<String> keys = table.topCells();
107105
Collection<Map> given = new ArrayList<>( );
@@ -117,7 +115,7 @@ public void result_should_be_mixed_a_node_b_node_l_integer( String stringTypes,
117115
tmpGiven.put( key, parseGiven( rc.record().asMap().get( key ) ) );
118116
}
119117
given.add( tmpGiven );
120-
expected.add( parseExpected( table.diffableRows().get( i + 1 ).convertedRow, keys, types ) );
118+
expected.add( parseExpected( table.diffableRows().get( i + 1 ).convertedRow, keys, mappedTypes ) );
121119
i++;
122120
}
123121
assertTrue( expected.size() > 0 );
@@ -161,4 +159,16 @@ private boolean equalRecords( Collection<Map> one, Collection<Map> other )
161159
return other.size() == 0;
162160

163161
}
162+
163+
@Then( "^result should map to types:$" )
164+
public void result_should_map_to_types(DataTable table) throws Throwable
165+
{
166+
List<String> keys = table.topCells();
167+
List<String> values = table.diffableRows().get( 1 ).convertedRow;
168+
mappedTypes = new HashMap<>( );
169+
for (int i = 0; i < keys.size(); i++)
170+
{
171+
mappedTypes.put( keys.get( i ), getType( values.get( i ) ) );
172+
}
173+
}
164174
}

driver/src/test/java/org/neo4j/driver/v1/tck/Environment.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
import java.util.Map;
2727

2828
import org.neo4j.driver.v1.Value;
29+
import org.neo4j.driver.v1.tck.tck.util.Types;
2930
import org.neo4j.driver.v1.tck.tck.util.runners.CypherStatementRunner;
3031
import org.neo4j.driver.v1.tck.tck.util.runners.MappedParametersRunner;
3132
import org.neo4j.driver.v1.tck.tck.util.runners.StatementRunner;
@@ -43,6 +44,7 @@ public class Environment
4344
public static Value expectedBoltValue;
4445
public static List<Object> listOfObjects;
4546
public static Map<String,Object> mapOfObjects;
47+
public static Map<String,Types.Type> mappedTypes;
4648

4749

4850
@Before
@@ -56,6 +58,7 @@ public void resetValues()
5658
statementRunner = null;
5759
stringRunner = null;
5860
runners = new ArrayList<>();
61+
mappedTypes = new HashMap<>( );
5962
}
6063

6164
@Before("@reset_database")

driver/src/test/java/org/neo4j/driver/v1/tck/tck/util/ResultParser.java

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,6 @@
4141
import static org.junit.Assert.assertEquals;
4242
import static org.neo4j.driver.internal.types.InternalTypeSystem.TYPE_SYSTEM;
4343
import static org.neo4j.driver.v1.tck.tck.util.Types.Type;
44-
import static org.neo4j.driver.v1.tck.tck.util.Types.getType;
4544

4645
public class ResultParser
4746
{
@@ -337,17 +336,6 @@ private static Map<String,Value> getProperties( String input )
337336
return properties;
338337
}
339338

340-
public static Map<String,Type> getTypeMapFromString( String input )
341-
{
342-
Map<String,Value> tmpMap = getProperties( input );
343-
Map<String,Type> converted = new HashMap<>();
344-
for ( String key : tmpMap.keySet() )
345-
{
346-
converted.put( key, getType( tmpMap.get( key ).asString() ) );
347-
}
348-
return converted;
349-
}
350-
351339
public static Map<String,Value> getParametersFromListOfKeysAndValues( List<String> keys, List<String> values )
352340
{
353341
assertEquals( keys.size(), values.size() );

0 commit comments

Comments
 (0)