fix:comments

Author: wenytang-ms

package.json

@@ -169,7 +169,8 @@
       },
       {
         "command": "java.project.build.project",
-        "title": "%contributes.commands.java.project.build.project%"
+        "title": "%contributes.commands.java.project.build.project%",
+        "category": "Java"
       },
       {
         "command": "java.project.clean.workspace",
@@ -187,7 +188,8 @@
       },
       {
         "command": "java.project.rebuild",
-        "title": "%contributes.commands.java.project.rebuild%"
+        "title": "%contributes.commands.java.project.rebuild%",
+        "category": "Java"
       },
       {
         "command": "java.view.package.revealInProjectExplorer",

src/views/dependencyDataProvider.ts

@@ -83,15 +83,15 @@ export class DependencyDataProvider implements TreeDataProvider<ExplorerNode> {
                 window.showErrorMessage("The URI of the project is not available, you can try to trigger the command 'Java: Build Project' from Command Palette.");
                 return;
             }
-            commands.executeCommand(Commands.BUILD_PROJECT, Uri.parse(node.uri), false /*isFullBuild*/);
+            return commands.executeCommand(Commands.BUILD_PROJECT, Uri.parse(node.uri), false /*isFullBuild*/);
         }));
         context.subscriptions.push(instrumentOperationAsVsCodeCommand(Commands.JAVA_PROJECT_REBUILD, async (node: INodeData) => {
             if (!node.uri) {
                 sendError(new Error("Uri not available when rebuilding project"));
                 window.showErrorMessage("The URI of the project is not available, you can try to trigger the command 'Java: Rebuild Project' from Command Palette.");
                 return;
             }
-            commands.executeCommand(Commands.BUILD_PROJECT, Uri.parse(node.uri), true /*isFullBuild*/);
+            return commands.executeCommand(Commands.BUILD_PROJECT, Uri.parse(node.uri), true /*isFullBuild*/);
         }));
 
         this.setRefreshDebounceFunc();

test/suite/buildTask.test.ts

@@ -25,6 +25,18 @@ suite("Build Task Tests", () => {
         assert.strictEqual(exportJarTask.definition.isFullBuild, false);
     });
 
+    test("test resolving build task defaults to incremental build", async function() {
+        const task: Task = new Task({
+            type: BuildTaskProvider.type,
+            paths: [ BuildTaskProvider.workspace ],
+        }, TaskScope.Workspace, BuildTaskProvider.defaultTaskName, BuildTaskProvider.type);
+
+        const resolvedTask: Task | undefined = await new BuildTaskProvider().resolveTask(task);
+
+        assert.ok(resolvedTask !== undefined);
+        assert.strictEqual(resolvedTask.definition.isFullBuild, false);
+    });
+
     test("test categorizePaths()", async function() {
         const [includes, excludes, invalid] = categorizePaths([
             BuildTaskProvider.workspace,