Skip to content

Bugfix #124 (double click on tree node) and #118 (special characters in connection name) #129

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

Merged
merged 2 commits into from
Jan 4, 2021
Merged

Bugfix #124 (double click on tree node) and #118 (special characters in connection name) #129

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
77f9f0b
fixes #124 handles double-click event on tree-table view
PhilippSalvisberg Jan 4, 2021
78f0ae8
fixes #118 (again) spaces are encoded as plus signs in connection names
PhilippSalvisberg Jan 4, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
fixes #124 handles double-click event on tree-table view 
open package body at the position of the first failed assert when a test is selected.
open the package spec at the position of test declaration when a test is selected.
open the package spec at line 1 if a the node contains only one package.
  • Loading branch information
@PhilippSalvisberg
PhilippSalvisberg committed Jan 4, 2021
commit 77f9f0bc526bafec2c79097e010585c17f2984e5
74 changes: 61 additions & 13 deletions sqldev/src/main/java/org/utplsql/sqldev/ui/runner/RunnerPanel.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -405,6 +405,19 @@ public boolean include(final RowFilter.Entry<? extends TestOverviewTableModel, ?
}
}

private void openItemNode(final ItemNode node) {
if (!node.getPackageName().equals("***")) {
final UtplsqlDao dao = new UtplsqlDao(DatabaseTools.getConnection(currentRun.getConnectionName()));
final String source = dao.getSource(node.getOwnerName(), "PACKAGE", node.getPackageName().toUpperCase()).trim();
final UtplsqlParser parser = new UtplsqlParser(source);
int line = 1;
if (node.getUserObject() instanceof Test) {
line = parser.getLineOf(node.getProcedureName());
}
openEditor(node.getOwnerName(), "PACKAGE", node.getPackageName().toUpperCase(), line, 1);
}
}

private void openTest(final Test test) {
final UtplsqlDao dao = new UtplsqlDao(DatabaseTools.getConnection(currentRun.getConnectionName()));
final String source = dao.getSource(test.getOwnerName(), "PACKAGE", test.getObjectName().toUpperCase()).trim();
Expand All @@ -414,11 +427,21 @@ private void openTest(final Test test) {
}

private void openSelectedTest() {
final int rowIndex = testOverviewTable.getSelectedRow();
if (rowIndex != -1) {
final int row = testOverviewTable.convertRowIndexToModel(rowIndex);
final Test test = testOverviewTableModel.getTest(row);
openTest(test);
if (!showSuitesCheckBoxMenuItem.isSelected()) {
// table
final int rowIndex = testOverviewTable.getSelectedRow();
if (rowIndex != -1) {
final int row = testOverviewTable.convertRowIndexToModel(rowIndex);
final Test test = testOverviewTableModel.getTest(row);
openTest(test);
}
} else {
// tree-table
TreePath path = testOverviewTreeTable.getTree().getSelectionPath();
if (path != null) {
ItemNode itemNode = (ItemNode) path.getLastPathComponent();
openItemNode(itemNode);
}
}
}

Expand All @@ -427,14 +450,27 @@ private void openSelectedFailure() {
if (rowIndex != -1) {
final int row = failuresTable.convertRowIndexToModel(rowIndex);
final Expectation expectation = failuresTableModel.getExpectation(row);
final Test test = testOverviewTableModel
.getTest(testOverviewTable.convertRowIndexToModel(testOverviewTable.getSelectedRow()));
final Integer callerLine = expectation.getCallerLine();
if (callerLine != null) {
openEditor(test.getOwnerName(), "PACKAGE BODY", test.getObjectName().toUpperCase(),
expectation.getCallerLine(), 1);
Test test = null;
if (!showSuitesCheckBoxMenuItem.isSelected()) {
// table
test = testOverviewTableModel
.getTest(testOverviewTable.convertRowIndexToModel(testOverviewTable.getSelectedRow()));
} else {
openTest(test);
// tree-table
TreePath path = testOverviewTreeTable.getTree().getSelectionPath();
if (path != null) {
ItemNode itemNode = (ItemNode) path.getLastPathComponent();
test = ((Test)itemNode.getUserObject());
}
}
if (test != null) {
final Integer callerLine = expectation.getCallerLine();
if (callerLine != null) {
openEditor(test.getOwnerName(), "PACKAGE BODY", test.getObjectName().toUpperCase(),
expectation.getCallerLine(), 1);
} else {
openTest(test);
}
}
}
}
Expand Down Expand Up @@ -1412,7 +1448,19 @@ public Component getTableCellRendererComponent(final JTable table, final Object
testOverviewCodeCoverageMenuItem.setEnabled(true);
}
});

testOverviewTreeTable.addMouseListener(new MouseAdapter() {
@Override
public void mouseClicked(final MouseEvent e) {
if (e.getClickCount() == 2) {
if (failuresTable.getSelectedRowCount() == 1) {
// open failure only if a Test node is selected
openSelectedFailure();
} else {
openSelectedTest();
}
}
}
});
final JTree overviewTreeTableName = testOverviewTreeTable.getTree();
overviewTreeTableName.setCellRenderer(new DefaultTreeCellRenderer() {
private static final long serialVersionUID = 580783625740405285L;
Expand Down