Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

#9748 delete tools only added by tests #10274

Merged
merged 6 commits into from
Feb 5, 2024
Merged

#9748 delete tools only added by tests #10274

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
b7ec646
#9748 delete tools only added by tests
sekmiller Jan 26, 2024
91287b3
#9748 one more assert
sekmiller Jan 26, 2024
c539bfd
Merge branch 'develop' into 9748-fix-ExternalToolsIT
sekmiller Jan 30, 2024
d960b98
#9748 comment out disabled test
sekmiller Jan 30, 2024
8f800dd
Merge branch 'develop' into 9748-fix-ExternalToolsIT
sekmiller Jan 30, 2024
bdc2c8e
#9748 avoid issue with existing tools
sekmiller Jan 31, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
102 changes: 39 additions & 63 deletions src/test/java/edu/harvard/iq/dataverse/api/ExternalToolsIT.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,21 +40,6 @@ public void testGetExternalTools() {
@Test
public void testFileLevelTool1() {

// Delete all external tools before testing.
Response getTools = UtilIT.getExternalTools();
getTools.prettyPrint();
getTools.then().assertThat()
.statusCode(OK.getStatusCode());
String body = getTools.getBody().asString();
JsonReader bodyObject = Json.createReader(new StringReader(body));
JsonArray tools = bodyObject.readObject().getJsonArray("data");
for (int i = 0; i < tools.size(); i++) {
JsonObject tool = tools.getJsonObject(i);
int id = tool.getInt("id");
Response deleteExternalTool = UtilIT.deleteExternalTool(id);
deleteExternalTool.prettyPrint();
}

Response createUser = UtilIT.createRandomUser();
createUser.prettyPrint();
createUser.then().assertThat()
Expand Down Expand Up @@ -145,26 +130,14 @@ public void testFileLevelTool1() {
.statusCode(OK.getStatusCode())
// No tools for this file type.
.body("data", Matchers.hasSize(0));

//Delete the tool added by this test...
Response deleteExternalTool = UtilIT.deleteExternalTool(toolId);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sekmiller for the other delete, you follow with an:
deleteExternalTool.then().assertThat()
.statusCode(OK.getStatusCode());
For consistency can you add that here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure

}

@Test
public void testDatasetLevelTool1() {

// Delete all external tools before testing.
Response getTools = UtilIT.getExternalTools();
getTools.prettyPrint();
getTools.then().assertThat()
.statusCode(OK.getStatusCode());
String body = getTools.getBody().asString();
JsonReader bodyObject = Json.createReader(new StringReader(body));
JsonArray tools = bodyObject.readObject().getJsonArray("data");
for (int i = 0; i < tools.size(); i++) {
JsonObject tool = tools.getJsonObject(i);
int id = tool.getInt("id");
Response deleteExternalTool = UtilIT.deleteExternalTool(id);
deleteExternalTool.prettyPrint();
}

Response createUser = UtilIT.createRandomUser();
createUser.prettyPrint();
createUser.then().assertThat()
Expand All @@ -184,7 +157,6 @@ public void testDatasetLevelTool1() {
createDataset.then().assertThat()
.statusCode(CREATED.getStatusCode());

// Integer datasetId = UtilIT.getDatasetIdFromResponse(createDataset);
Integer datasetId = JsonPath.from(createDataset.getBody().asString()).getInt("data.id");
String datasetPid = JsonPath.from(createDataset.getBody().asString()).getString("data.persistentId");

Expand Down Expand Up @@ -219,6 +191,8 @@ public void testDatasetLevelTool1() {
addExternalTool.then().assertThat()
.statusCode(OK.getStatusCode())
.body("data.displayName", CoreMatchers.equalTo("DatasetTool1"));

long toolId = JsonPath.from(addExternalTool.getBody().asString()).getLong("data.id");

Response getExternalToolsByDatasetIdInvalidType = UtilIT.getExternalToolsForDataset(datasetId.toString(), "invalidType", apiToken);
getExternalToolsByDatasetIdInvalidType.prettyPrint();
Expand All @@ -233,27 +207,16 @@ public void testDatasetLevelTool1() {
.body("data[0].scope", CoreMatchers.equalTo("dataset"))
.body("data[0].toolUrlWithQueryParams", CoreMatchers.equalTo("http://datasettool1.com?datasetPid=" + datasetPid + "&key=" + apiToken))
.statusCode(OK.getStatusCode());


//Delete the tool added by this test...
Response deleteExternalTool = UtilIT.deleteExternalTool(toolId);
deleteExternalTool.then().assertThat()
.statusCode(OK.getStatusCode());
}

@Test
public void testDatasetLevelToolConfigure() {

// Delete all external tools before testing.
Response getTools = UtilIT.getExternalTools();
getTools.prettyPrint();
getTools.then().assertThat()
.statusCode(OK.getStatusCode());
String body = getTools.getBody().asString();
JsonReader bodyObject = Json.createReader(new StringReader(body));
JsonArray tools = bodyObject.readObject().getJsonArray("data");
for (int i = 0; i < tools.size(); i++) {
JsonObject tool = tools.getJsonObject(i);
int id = tool.getInt("id");
Response deleteExternalTool = UtilIT.deleteExternalTool(id);
deleteExternalTool.prettyPrint();
}

Response createUser = UtilIT.createRandomUser();
createUser.prettyPrint();
createUser.then().assertThat()
Expand Down Expand Up @@ -302,6 +265,8 @@ public void testDatasetLevelToolConfigure() {
addExternalTool.then().assertThat()
.statusCode(OK.getStatusCode())
.body("data.displayName", CoreMatchers.equalTo("Dataset Configurator"));

long toolId = JsonPath.from(addExternalTool.getBody().asString()).getLong("data.id");

Response getExternalToolsByDatasetId = UtilIT.getExternalToolsForDataset(datasetId.toString(), "configure", apiToken);
getExternalToolsByDatasetId.prettyPrint();
Expand All @@ -311,6 +276,11 @@ public void testDatasetLevelToolConfigure() {
.body("data[0].types[0]", CoreMatchers.equalTo("configure"))
.body("data[0].toolUrlWithQueryParams", CoreMatchers.equalTo("https://datasetconfigurator.com?datasetPid=" + datasetPid))
.statusCode(OK.getStatusCode());

//Delete the tool added by this test...
Response deleteExternalTool = UtilIT.deleteExternalTool(toolId);
deleteExternalTool.then().assertThat()
.statusCode(OK.getStatusCode());

}

Expand Down Expand Up @@ -400,12 +370,13 @@ public void deleteTools() {
String body = getTools.getBody().asString();
JsonReader bodyObject = Json.createReader(new StringReader(body));
JsonArray tools = bodyObject.readObject().getJsonArray("data");
/*
for (int i = 0; i < tools.size(); i++) {
JsonObject tool = tools.getJsonObject(i);
int id = tool.getInt("id");
Response deleteExternalTool = UtilIT.deleteExternalTool(id);
deleteExternalTool.prettyPrint();
}
}*/
}

// preview only
Expand Down Expand Up @@ -446,6 +417,13 @@ public void createToolShellScript() {
addExternalTool.prettyPrint();
addExternalTool.then().assertThat()
.statusCode(OK.getStatusCode());

long toolId = JsonPath.from(addExternalTool.getBody().asString()).getLong("data.id");

//Delete the tool added by this test...
Response deleteExternalTool = UtilIT.deleteExternalTool(toolId);
deleteExternalTool.then().assertThat()
.statusCode(OK.getStatusCode());
}

// explore only
Expand Down Expand Up @@ -479,6 +457,13 @@ public void createToolDataExplorer() {
addExternalTool.prettyPrint();
addExternalTool.then().assertThat()
.statusCode(OK.getStatusCode());

long toolId = JsonPath.from(addExternalTool.getBody().asString()).getLong("data.id");

//Delete the tool added by this test...
Response deleteExternalTool = UtilIT.deleteExternalTool(toolId);
deleteExternalTool.then().assertThat()
.statusCode(OK.getStatusCode());
}

// both preview and explore
Expand Down Expand Up @@ -527,21 +512,6 @@ public void createToolSpreadsheetViewer() {
@Test
public void testFileLevelToolWithAuxFileReq() throws IOException {

// Delete all external tools before testing.
Response getTools = UtilIT.getExternalTools();
getTools.prettyPrint();
getTools.then().assertThat()
.statusCode(OK.getStatusCode());
String body = getTools.getBody().asString();
JsonReader bodyObject = Json.createReader(new StringReader(body));
JsonArray tools = bodyObject.readObject().getJsonArray("data");
for (int i = 0; i < tools.size(); i++) {
JsonObject tool = tools.getJsonObject(i);
int id = tool.getInt("id");
Response deleteExternalTool = UtilIT.deleteExternalTool(id);
deleteExternalTool.prettyPrint();
}

Response createUser = UtilIT.createRandomUser();
createUser.prettyPrint();
createUser.then().assertThat()
Expand Down Expand Up @@ -640,6 +610,12 @@ public void testFileLevelToolWithAuxFileReq() throws IOException {
.body("data[0].displayName", CoreMatchers.equalTo("HDF5 Tool"))
.body("data[0].scope", CoreMatchers.equalTo("file"))
.body("data[0].contentType", CoreMatchers.equalTo("application/x-hdf5"));

//Delete the tool added by this test...
Response deleteExternalTool = UtilIT.deleteExternalTool(toolId);
deleteExternalTool.then().assertThat()
.statusCode(OK.getStatusCode());

}

}