Skip to content

Commit 99618fe

Browse files
BarsaTMBarsaTM
committed
More review stuff
- Moved `wrap exception in meaning message` logic closer to OpenCypherGremlinDatabase impl. - Cleaned places where wrapping exception is not necessary - Misc.
1 parent 8b2769d commit 99618fe

File tree

12 files changed

+43
-72
lines changed

12 files changed

+43
-72
lines changed

‎database/opencypher/src/main/java/com/neueda/jetbrains/plugin/graphdb/database/opencypher/gremlin/OpenCypherGremlinDatabase.java

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
import com.neueda.jetbrains.plugin.graphdb.database.api.query.GraphQueryResultRow;
1010
import com.neueda.jetbrains.plugin.graphdb.database.neo4j.bolt.data.Neo4jBoltQueryResultColumn;
1111
import com.neueda.jetbrains.plugin.graphdb.database.neo4j.bolt.data.Neo4jBoltQueryResultRow;
12+
import com.neueda.jetbrains.plugin.graphdb.database.opencypher.gremlin.exceptions.OpenCypherGremlinException;
1213
import com.neueda.jetbrains.plugin.graphdb.database.opencypher.gremlin.query.OpenCypherGremlinQueryResult;
1314
import org.apache.tinkerpop.gremlin.driver.Client;
1415
import org.apache.tinkerpop.gremlin.driver.Cluster;
@@ -23,10 +24,9 @@
2324
import java.util.concurrent.ExecutionException;
2425
import java.util.stream.Collectors;
2526

26-
import static java.util.Collections.emptyList;
27-
import static java.util.Collections.singletonList;
28-
import static java.util.Collections.singletonMap;
29-
import static java.util.stream.Collectors.toList;
27+
import static com.neueda.jetbrains.plugin.graphdb.database.opencypher.gremlin.exceptions.ExceptionWrapper.*;
28+
import static java.util.Collections.*;
29+
import static java.util.stream.Collectors.*;
3030

3131
/**
3232
* Communicates with TinkerPop database by translating Cypher to Gremlin
@@ -99,7 +99,8 @@ public GraphQueryResult execute(String query, Map<String, Object> statementParam
9999
if (query.toUpperCase().startsWith("EXPLAIN")) {
100100
return new OpenCypherGremlinQueryResult(0, emptyList(), emptyList(), emptyList(), emptyList());
101101
} else {
102-
throw e;
102+
String exceptionMessage = wrapExceptionInMeaningMessage(e);
103+
throw new OpenCypherGremlinException(exceptionMessage, e);
103104
}
104105
}
105106
}

‎ui/jetbrains/src/main/java/com/neueda/jetbrains/plugin/graphdb/jetbrains/util/ExceptionErrorMessages.java renamed to ‎database/opencypher/src/main/java/com/neueda/jetbrains/plugin/graphdb/database/opencypher/gremlin/exceptions/ExceptionErrorMessages.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package com.neueda.jetbrains.plugin.graphdb.jetbrains.util;
1+
package com.neueda.jetbrains.plugin.graphdb.database.opencypher.gremlin.exceptions;
22

33
public enum ExceptionErrorMessages {
44
ERROR_OCCURRED("Error occurred."),

‎ui/jetbrains/src/main/java/com/neueda/jetbrains/plugin/graphdb/jetbrains/util/ExceptionWrapper.java renamed to ‎database/opencypher/src/main/java/com/neueda/jetbrains/plugin/graphdb/database/opencypher/gremlin/exceptions/ExceptionWrapper.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package com.neueda.jetbrains.plugin.graphdb.jetbrains.util;
1+
package com.neueda.jetbrains.plugin.graphdb.database.opencypher.gremlin.exceptions;
22

33
import java.io.PrintWriter;
44
import java.io.StringWriter;

‎database/opencypher/src/main/java/com/neueda/jetbrains/plugin/graphdb/database/opencypher/gremlin/exceptions/OpenCypherGremlinException.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
package com.neueda.jetbrains.plugin.graphdb.database.opencypher.gremlin.exceptions;
2+
3+
public class OpenCypherGremlinException extends RuntimeException {
4+
public OpenCypherGremlinException(String message, Throwable cause) {
5+
super(message, cause);
6+
}
7+
}

‎testing/integration-tinkerpop/src/test/java/com/neueda/jetbrains/plugin/graphdb/test/integration/opencypher/gremlin/ExceptionWrapperTest.java

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
package com.neueda.jetbrains.plugin.graphdb.test.integration.opencypher.gremlin;
2+
3+
import com.neueda.jetbrains.plugin.graphdb.database.opencypher.gremlin.exceptions.ExceptionWrapper;
4+
import org.junit.Test;
5+
6+
import static org.junit.Assert.*;
7+
8+
public class ExceptionWrapperTest {
9+
@Test
10+
public void shouldEllipseString() throws Exception {
11+
String shortString = "DataSource[Cosmos] - metadata refresh failed. Reason: java.util.concurrent.ExecutionException:org.apache.tinkerpop.gremlin.driver.exception.ResponseException:...";
12+
String longString = "DataSource[Cosmos] - metadata refresh failed. Reason: java.util.concurrent.ExecutionException:" +
13+
"org.apache.tinkerpop.gremlin.driver.exception.ResponseException: ActivityId : d0015df6-09d2-48a6-b3ab-32d026213a42";
14+
String truncatedString = ExceptionWrapper.ellipseString(longString, ExceptionWrapper.SHORT_STRING_LENGTH);
15+
assertEquals(shortString, truncatedString);
16+
}
17+
}

‎ui/jetbrains/src/main/java/com/neueda/jetbrains/plugin/graphdb/jetbrains/ui/console/graph/GraphPanelInteractions.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
import com.neueda.jetbrains.plugin.graphdb.visualization.VisualizationApi;
1414
import com.neueda.jetbrains.plugin.graphdb.visualization.events.EventType;
1515

16-
import static com.neueda.jetbrains.plugin.graphdb.jetbrains.util.ExceptionWrapper.wrapExceptionInMeaningMessage;
16+
import static com.neueda.jetbrains.plugin.graphdb.database.opencypher.gremlin.exceptions.ExceptionWrapper.wrapExceptionInMeaningMessage;
1717

1818
public class GraphPanelInteractions {
1919

‎ui/jetbrains/src/main/java/com/neueda/jetbrains/plugin/graphdb/jetbrains/ui/console/log/LogPanel.java

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,9 @@
2727
import java.util.HashMap;
2828
import java.util.Map;
2929

30-
import static com.neueda.jetbrains.plugin.graphdb.jetbrains.ui.console.event.QueryParametersRetrievalErrorEvent.PARAMS_ERROR_COMMON_MSG;
31-
import static com.neueda.jetbrains.plugin.graphdb.jetbrains.ui.datasource.interactions.DataSourceDialog.HEIGHT;
32-
import static com.neueda.jetbrains.plugin.graphdb.jetbrains.ui.datasource.interactions.DataSourceDialog.THICKNESS;
33-
import static com.neueda.jetbrains.plugin.graphdb.jetbrains.util.ExceptionWrapper.*;
30+
import static com.neueda.jetbrains.plugin.graphdb.database.opencypher.gremlin.exceptions.ExceptionWrapper.*;
31+
import static com.neueda.jetbrains.plugin.graphdb.jetbrains.ui.console.event.QueryParametersRetrievalErrorEvent.*;
32+
import static com.neueda.jetbrains.plugin.graphdb.jetbrains.ui.datasource.interactions.DataSourceDialog.*;
3433

3534
public class LogPanel implements Disposable {
3635
private static final String SHOW_DETAILS = "Details...";
@@ -159,7 +158,7 @@ public void error(@Nullable String message) {
159158
private String printException(Exception exception) {
160159
String errorMessage;
161160
if (exception.getMessage() != null) {
162-
errorMessage = wrapExceptionInMeaningMessage(exception);
161+
errorMessage = exception.getMessage();
163162
} else {
164163
errorMessage = exception.toString();
165164
}

‎ui/jetbrains/src/main/java/com/neueda/jetbrains/plugin/graphdb/jetbrains/ui/console/params/ParametersPanel.java

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,7 @@
99
import com.intellij.openapi.editor.Document;
1010
import com.intellij.openapi.editor.Editor;
1111
import com.intellij.openapi.editor.EditorFactory;
12-
import com.intellij.openapi.fileEditor.FileDocumentManager;
13-
import com.intellij.openapi.fileEditor.FileEditor;
14-
import com.intellij.openapi.fileEditor.FileEditorManager;
15-
import com.intellij.openapi.fileEditor.FileEditorManagerEvent;
16-
import com.intellij.openapi.fileEditor.FileEditorManagerListener;
12+
import com.intellij.openapi.fileEditor.*;
1713
import com.intellij.openapi.fileEditor.ex.FileEditorManagerEx;
1814
import com.intellij.openapi.fileEditor.impl.EditorTabPresentationUtil;
1915
import com.intellij.openapi.fileEditor.impl.EditorWindow;
@@ -36,8 +32,7 @@
3632
import javax.swing.*;
3733
import java.awt.*;
3834

39-
import static com.neueda.jetbrains.plugin.graphdb.jetbrains.ui.console.event.QueryParametersRetrievalErrorEvent.PARAMS_ERROR_COMMON_MSG;
40-
import static com.neueda.jetbrains.plugin.graphdb.jetbrains.util.ExceptionWrapper.wrapExceptionInMeaningMessage;
35+
import static com.neueda.jetbrains.plugin.graphdb.jetbrains.ui.console.event.QueryParametersRetrievalErrorEvent.*;
4136

4237
public class ParametersPanel implements ParametersProvider {
4338

@@ -83,7 +78,7 @@ private void initializeUi() {
8378
}
8479
String errorMessage;
8580
if (exception.getMessage() != null) {
86-
errorMessage = String.format("%s: %s", PARAMS_ERROR_COMMON_MSG, wrapExceptionInMeaningMessage(exception));
81+
errorMessage = String.format("%s: %s", PARAMS_ERROR_COMMON_MSG, exception.getMessage());
8782
} else {
8883
errorMessage = PARAMS_ERROR_COMMON_MSG;
8984
}

‎ui/jetbrains/src/main/java/com/neueda/jetbrains/plugin/graphdb/jetbrains/ui/datasource/actions/DataSourceAction.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,6 @@
1111
import javax.swing.*;
1212
import java.io.IOException;
1313

14-
import static com.neueda.jetbrains.plugin.graphdb.jetbrains.util.ExceptionWrapper.wrapExceptionInMeaningMessage;
15-
1614
public class DataSourceAction extends AnAction {
1715

1816
private DataSourceApi dataSource;
@@ -29,7 +27,7 @@ public void actionPerformed(AnActionEvent e) {
2927
try {
3028
FileUtil.openFile(project, FileUtil.getDataSourceFile(project, dataSource));
3129
} catch (IOException exception) {
32-
Notifier.error("Open editor error", wrapExceptionInMeaningMessage(exception));
30+
Notifier.error("Open editor error", exception.getMessage());
3331
}
3432
}
3533
}

‎ui/jetbrains/src/main/java/com/neueda/jetbrains/plugin/graphdb/jetbrains/ui/datasource/interactions/DataSourceDialog.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
import javax.swing.*;
2323
import java.awt.*;
2424

25-
import static com.neueda.jetbrains.plugin.graphdb.jetbrains.util.ExceptionWrapper.*;
25+
import static com.neueda.jetbrains.plugin.graphdb.database.opencypher.gremlin.exceptions.ExceptionWrapper.*;
2626

2727
public abstract class DataSourceDialog extends DialogWrapper {
2828
public static final int THICKNESS = 10;

‎ui/jetbrains/src/main/java/com/neueda/jetbrains/plugin/graphdb/jetbrains/ui/datasource/interactions/DataSourceInteractions.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,6 @@
2525
import java.util.List;
2626
import java.util.stream.Collectors;
2727

28-
import static com.neueda.jetbrains.plugin.graphdb.jetbrains.util.ExceptionWrapper.wrapExceptionInMeaningMessage;
29-
3028
public class DataSourceInteractions {
3129

3230
private final DataSourcesView dataSourcesView;
@@ -131,7 +129,7 @@ public void mouseClicked(MouseEvent e) {
131129
try {
132130
FileUtil.openFile(project, FileUtil.getDataSourceFile(project, dataSource));
133131
} catch (IOException exception) {
134-
Notifier.error("Open editor error", wrapExceptionInMeaningMessage(exception));
132+
Notifier.error("Open editor error", exception.getMessage());
135133
}
136134
}
137135
}

‎ui/jetbrains/src/test/java/com/neueda/jetbrains/plugin/graphdb/jetbrains/ui/helpers/ExceptionWrapperTest.java

Lines changed: 0 additions & 44 deletions
This file was deleted.

0 commit comments

Comments
 (0)