fixed bug with history and other argument provider

Author: ashley-taylor

graphql-database-manager-test/pom.xml

@@ -92,28 +92,4 @@
             <version>2.5.4</version>
         </dependency>
     </dependencies>
-
-    <build>
-        <plugins>
-            <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-dependency-plugin</artifactId>
-                <version>3.8.1</version>
-                <executions>
-                    <execution>
-                        <id>copy</id>
-                        <phase>test-compile</phase>
-                        <goals>
-                            <goal>copy-dependencies</goal>
-                        </goals>
-                        <configuration>
-                            <includeScope>test</includeScope>
-                            <includeTypes>so,dll,dylib</includeTypes>
-                            <outputDirectory>${project.basedir}/native-libs</outputDirectory>
-                        </configuration>
-                    </execution>
-                </executions>
-            </plugin>
-        </plugins>
-    </build>
 </project>

graphql-database-manager-test/src/main/java/com/phocassoftware/graphql/database/manager/test/ArgumentProvider.java

@@ -101,14 +101,12 @@ private Tables getTables(boolean scope) {
 						tableDeleteConsumer.accept(historyTable);
 					}
 				}
-			} else if (scope) {
+			} else if (scope || withHistory) {
 				tables = new String[] { "table" + "_" + uniqueId };
 				historyTable = tables[0] + "_history";
 				createTable(client, tables[0]);
-				if (withHistory) {
-					createHistoryTable(client, historyTable);
-					tableDeleteConsumer.accept(historyTable);
-				}
+				createHistoryTable(client, historyTable);
+				tableDeleteConsumer.accept(historyTable);
 			}
 			return new Tables(tables, historyTable);
 		} catch (Exception e) {

graphql-database-manager-test/src/main/java/com/phocassoftware/graphql/database/manager/test/TestDatabaseProvider.java

@@ -61,7 +61,7 @@ public boolean supportsParameter(ParameterContext parameterContext, ExtensionCon
 		var testDatabase = getTestDatabase(testMethod);
 
 		if (testDatabase != null) {
-			Stream
+			return Stream
 				.of(testDatabase.providers())
 				.map(t -> create(t))
 				.anyMatch(provider -> parameterContext.getParameter().getType().isAssignableFrom(provider.type()));
@@ -131,7 +131,10 @@ public void beforeEach(ExtensionContext extensionContext) throws Exception {
 							.map(t -> create(t))
 							.filter(p -> type.isAssignableFrom(p.type()))
 							.findAny()
-							.orElseThrow();
+							.orElse(null);
+						if (builder == null) {
+							return null;
+						}
 						return builder.create(provider);
 
 					}
@@ -167,7 +170,6 @@ private ServerWrapper getServer() throws Exception {
 					final var streamClient = startDynamoStreamClient(port);
 
 					createTable(client, "table");
-					createHistoryTable(client, "table_history");
 					serverWrapper = new ServerWrapper(client, clientAsync, streamClient);
 				}