Merge branch 'main' into fix/331-cobigen-urls

.github/workflows/nightly-build.yml

@@ -24,7 +24,7 @@ jobs:
     if: ${{ needs.verify_commit.outputs.RUN_BUILD == 'true' }}
     strategy:
       matrix:
-        os: [windows-latest, ubuntu-latest, macos-latest]
+        os: [windows-latest, ubuntu-latest, macos-latest, macos-13]
 
     steps:
       - uses: actions/checkout@v3

.mvn/maven.config

@@ -1 +1 @@
--Drevision=2024.05.001-alpha-SNAPSHOT
+-Drevision=2024.06.001-alpha-SNAPSHOT

cli/pom.xml

@@ -90,6 +90,13 @@
       <artifactId>jansi</artifactId>
       <version>${jansi.version}</version>
     </dependency>
+    <dependency>
+      <!-- required for advanced mocking -->
+      <groupId>net.bytebuddy</groupId>
+      <artifactId>byte-buddy</artifactId>
+      <version>1.14.15</version>
+      <scope>test</scope>
+    </dependency>
   </dependencies>
 
   <build>

cli/src/main/java/com/devonfw/tools/ide/cli/CliArgument.java

@@ -5,17 +5,16 @@
 import java.util.Objects;
 
 /**
- * Represents an argument for a command-line-interface (CLI) and a chain to all its {@link #getNext(boolean)
- * successors}.
+ * Represents an argument for a command-line-interface (CLI) and a chain to all its {@link #getNext(boolean) successors}.
  *
- * @since 1.0.0
  * @see #getNext(boolean)
+ * @since 1.0.0
  */
 public class CliArgument {
 
   /**
-   * The {@link #get() argument} to indicate the end of the options. If this string is given as argument, any further
-   * arguments are treated as values. This allows to provide values (e.g. a filename) starting with a hyphen ('-').
+   * The {@link #get() argument} to indicate the end of the options. If this string is given as argument, any further arguments are treated as values. This
+   * allows to provide values (e.g. a filename) starting with a hyphen ('-').
    */
   public static final String END_OPTIONS = "--";
 
@@ -144,8 +143,7 @@ public boolean isStart() {
 
   /**
    * @param successors the number of {@link #getNext() successors} expected.
-   * @return {@code true} if at least the given number of {@link #getNext() successors} are available, {@code false}
-   *         otherwise.
+   * @return {@code true} if at least the given number of {@link #getNext() successors} are available, {@code false} otherwise.
    */
   public boolean hasMoreSuccessorsThan(int successors) {
 
@@ -173,8 +171,7 @@ public CliArgument getNext() {
   }
 
   /**
-   * @param splitShortOpts - if {@code true} then combined short options will be split (so instead of "-fbd" you will
-   *        get "-f", "-b", "-d").
+   * @param splitShortOpts - if {@code true} then combined short options will be split (so instead of "-fbd" you will get "-f", "-b", "-d").
    * @return the next {@link CliArgument} or {@code null} if this is the {@link #isEnd() end}.
    */
   public CliArgument getNext(boolean splitShortOpts) {
@@ -194,8 +191,7 @@ public CliArgument getNext(boolean splitShortOpts) {
   }
 
   /**
-   * @return the {@code «key»} part if the {@link #get() argument} has the has the form {@code «key»=«value»}. Otherwise
-   *         the {@link #get() argument} itself.
+   * @return the {@code «key»} part if the {@link #get() argument} has the has the form {@code «key»=«value»}. Otherwise the {@link #get() argument} itself.
    */
   public String getKey() {
 
@@ -204,8 +200,7 @@ public String getKey() {
   }
 
   /**
-   * @return the {@code «value»} part if the {@link #get() argument} has the has the form {@code «key»=«value»}.
-   *         Otherwise {@code null}.
+   * @return the {@code «value»} part if the {@link #get() argument} has the has the form {@code «key»=«value»}. Otherwise {@code null}.
    */
   public String getValue() {
 
@@ -228,8 +223,8 @@ private void initKeyValue() {
   }
 
   /**
-   * @return a {@link String} representing all arguments from this {@link CliArgument} recursively along is
-   *         {@link #getNext(boolean) next} arguments to the {@link #isEnd() end}.
+   * @return a {@link String} representing all arguments from this {@link CliArgument} recursively along is {@link #getNext(boolean) next} arguments to the
+   * {@link #isEnd() end}.
    */
   public String getArgs() {
 

cli/src/main/java/com/devonfw/tools/ide/commandlet/AbstractUpdateCommandlet.java

@@ -1,11 +1,5 @@
 package com.devonfw.tools.ide.commandlet;
 
-import java.nio.file.Files;
-import java.nio.file.Path;
-import java.util.HashSet;
-import java.util.List;
-import java.util.Set;
-
 import com.devonfw.tools.ide.context.GitContext;
 import com.devonfw.tools.ide.context.IdeContext;
 import com.devonfw.tools.ide.property.StringProperty;
@@ -15,6 +9,12 @@
 import com.devonfw.tools.ide.tool.ToolCommandlet;
 import com.devonfw.tools.ide.variable.IdeVariables;
 
+import java.nio.file.Files;
+import java.nio.file.Path;
+import java.util.HashSet;
+import java.util.List;
+import java.util.Set;
+
 /**
  * Abstract {@link Commandlet} base-class for both {@link UpdateCommandlet} and {@link CreateCommandlet}.
  */
@@ -78,8 +78,8 @@ private void setupConf(Path template, Path conf) {
           this.context.debug("Configuration {} already exists - skipping to copy from {}", confPath, child);
         } else {
           if (!basename.equals("settings.xml")) {
-            this.context.info("Copying template {} to {}.", child, confPath);
-            this.context.getFileAccess().copy(child, confPath);
+            this.context.info("Copying template {} to {}.", child, conf);
+            this.context.getFileAccess().copy(child, conf);
           }
         }
       }
@@ -104,8 +104,8 @@ private void updateSettings() {
           String message = "Missing your settings at " + settingsPath + " and no SETTINGS_URL is defined.\n"
               + "Further details can be found here: https://github.com/devonfw/IDEasy/blob/main/documentation/settings.asciidoc\n"
               + "Please contact the technical lead of your project to get the SETTINGS_URL for your project.\n"
-              + "In case you just want to test IDEasy you may simply hit return to install the default settings.\n"
-              + "Settings URL [" + IdeContext.DEFAULT_SETTINGS_REPO_URL + "]:";
+              + "In case you just want to test IDEasy you may simply hit return to install the default settings.\n" + "Settings URL ["
+              + IdeContext.DEFAULT_SETTINGS_REPO_URL + "]:";
           repository = this.context.askForInput(message, IdeContext.DEFAULT_SETTINGS_REPO_URL);
         } else if ("-".equals(repository)) {
           repository = IdeContext.DEFAULT_SETTINGS_REPO_URL;
@@ -127,8 +127,7 @@ private void updateSoftware() {
       Set<ToolCommandlet> toolCommandlets = new HashSet<>();
 
       // installed tools in IDE_HOME/software
-      List<Path> softwares = this.context.getFileAccess().listChildren(this.context.getSoftwarePath(),
-          Files::isDirectory);
+      List<Path> softwares = this.context.getFileAccess().listChildren(this.context.getSoftwarePath(), Files::isDirectory);
       for (Path software : softwares) {
         String toolName = software.getFileName().toString();
         ToolCommandlet toolCommandlet = this.context.getCommandletManager().getToolCommandletOrNull(toolName);

cli/src/main/java/com/devonfw/tools/ide/commandlet/Commandlet.java

@@ -1,18 +1,18 @@
 package com.devonfw.tools.ide.commandlet;
 
+import com.devonfw.tools.ide.context.IdeContext;
+import com.devonfw.tools.ide.property.KeywordProperty;
+import com.devonfw.tools.ide.property.Property;
+import com.devonfw.tools.ide.tool.ToolCommandlet;
+import com.devonfw.tools.ide.version.VersionIdentifier;
+
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
 
-import com.devonfw.tools.ide.context.IdeContext;
-import com.devonfw.tools.ide.property.KeywordProperty;
-import com.devonfw.tools.ide.property.Property;
-import com.devonfw.tools.ide.tool.ToolCommandlet;
-import com.devonfw.tools.ide.version.VersionIdentifier;
-
 /**
  * A {@link Commandlet} is a sub-command of the IDE CLI.
  */
@@ -68,8 +68,7 @@ public List<Property<?>> getValues() {
   }
 
   /**
-   * @param nameOrAlias the potential {@link Property#getName() name} or {@link Property#getAlias() alias} of the
-   *        requested {@link Property}.
+   * @param nameOrAlias the potential {@link Property#getName() name} or {@link Property#getAlias() alias} of the requested {@link Property}.
    * @return the requested {@link Property property} or {@code null} if not found.
    */
   public Property<?> getOption(String nameOrAlias) {
@@ -108,8 +107,7 @@ protected void addKeyword(Property<?> property) {
   protected <P extends Property<?>> P add(P property) {
 
     if (this.multiValued != null) {
-      throw new IllegalStateException(
-          "The multi-valued property " + this.multiValued + " can not be followed by " + property);
+      throw new IllegalStateException("The multi-valued property " + this.multiValued + " can not be followed by " + property);
     }
     this.propertiesList.add(property);
     if (property.isOption()) {
@@ -147,8 +145,7 @@ private void add(String name, Property<?> property, boolean alias) {
   public abstract String getName();
 
   /**
-   * @return the first keyword of this {@link Commandlet}. Typically the same as {@link #getName() name} but may also
-   *         differ (e.g. "set" vs. "set-version").
+   * @return the first keyword of this {@link Commandlet}. Typically the same as {@link #getName() name} but may also differ (e.g. "set" vs. "set-version").
    */
   public String getKeyword() {
 
@@ -167,17 +164,15 @@ protected <C extends Commandlet> C getCommandlet(Class<C> commandletType) {
   }
 
   /**
-   * @return {@code true} if {@link IdeContext#getIdeHome() IDE_HOME} is required for this commandlet, {@code false}
-   *         otherwise.
+   * @return {@code true} if {@link IdeContext#getIdeHome() IDE_HOME} is required for this commandlet, {@code false} otherwise.
    */
   public boolean isIdeHomeRequired() {
 
     return true;
   }
 
   /**
-   * @return {@code true} to suppress the {@link com.devonfw.tools.ide.step.StepImpl#logSummary(boolean) step summary
-   *         success message}.
+   * @return {@code true} to suppress the {@link com.devonfw.tools.ide.step.StepImpl#logSummary(boolean) step summary success message}.
    */
   public boolean isSuppressStepSuccess() {
 
@@ -190,8 +185,7 @@ public boolean isSuppressStepSuccess() {
   public abstract void run();
 
   /**
-   * @return {@code true} if this {@link Commandlet} is the valid candidate to be {@link #run()}, {@code false}
-   *         otherwise.
+   * @return {@code true} if this {@link Commandlet} is the valid candidate to be {@link #run()}, {@code false} otherwise.
    * @see Property#validate()
    */
   public boolean validate() {
@@ -217,8 +211,8 @@ public String toString() {
   }
 
   /**
-   * @return the {@link ToolCommandlet} set in a {@link Property} of this commandlet used for auto-completion of a
-   *         {@link VersionIdentifier} or {@code null} if not exists or not configured.
+   * @return the {@link ToolCommandlet} set in a {@link Property} of this commandlet used for auto-completion of a {@link VersionIdentifier} or {@code null} if
+   * not exists or not configured.
    */
   public ToolCommandlet getToolForVersionCompletion() {
 

cli/src/main/java/com/devonfw/tools/ide/commandlet/CompleteCommandlet.java

@@ -1,20 +1,20 @@
 package com.devonfw.tools.ide.commandlet;
 
-import java.util.List;
-
 import com.devonfw.tools.ide.cli.CliArguments;
 import com.devonfw.tools.ide.completion.CompletionCandidate;
 import com.devonfw.tools.ide.context.AbstractIdeContext;
 import com.devonfw.tools.ide.context.IdeContext;
-import com.devonfw.tools.ide.property.StringListProperty;
+import com.devonfw.tools.ide.property.StringProperty;
+
+import java.util.List;
 
 /**
  * {@link Commandlet} for auto-completion.
  */
 public final class CompleteCommandlet extends Commandlet {
 
-  /** {@link StringListProperty} with the current CLI arguments to complete. */
-  public final StringListProperty args;
+  /** {@link StringProperty} with the current CLI arguments to complete. */
+  public final StringProperty args;
 
   /**
    * The constructor.
@@ -25,7 +25,7 @@ public CompleteCommandlet(IdeContext context) {
 
     super(context);
     addKeyword(getName());
-    this.args = add(new StringListProperty("", false, "args"));
+    this.args = add(new StringProperty("", false, true, "args"));
   }
 
   @Override

cli/src/main/java/com/devonfw/tools/ide/commandlet/RepositoryCommandlet.java

@@ -1,18 +1,12 @@
 package com.devonfw.tools.ide.commandlet;
 
 import com.devonfw.tools.ide.context.IdeContext;
-import com.devonfw.tools.ide.property.PathProperty;
 import com.devonfw.tools.ide.property.RepositoryProperty;
 import com.devonfw.tools.ide.tool.ToolCommandlet;
 
-import java.io.FileInputStream;
-import java.io.IOException;
-import java.io.InputStream;
 import java.nio.file.Files;
 import java.nio.file.Path;
-import java.util.ArrayList;
 import java.util.List;
-import java.util.Properties;
 
 import static com.devonfw.tools.ide.commandlet.RepositoryConfig.loadProperties;
 
@@ -65,8 +59,7 @@ public void run() {
         return;
       }
 
-      List <Path> propertiesFiles = this.context.getFileAccess().listChildren(repositories,
-          path -> path.getFileName().toString().endsWith(".properties"));
+      List<Path> propertiesFiles = this.context.getFileAccess().listChildren(repositories, path -> path.getFileName().toString().endsWith(".properties"));
 
       boolean forceMode = this.context.isForceMode();
       for (Path propertiesFile : propertiesFiles) {
@@ -93,8 +86,7 @@ private void doImportRepository(Path repositoryFile, boolean forceMode) {
     String repository = repositoryConfig.path();
     String gitUrl = repositoryConfig.gitUrl();
     if (repository == null || "".equals(repository) || gitUrl == null || "".equals(gitUrl)) {
-      this.context.warning("Invalid repository configuration {} - both 'path' and 'git-url' have to be defined."
-          , repositoryFile.getFileName().toString());
+      this.context.warning("Invalid repository configuration {} - both 'path' and 'git-url' have to be defined.", repositoryFile.getFileName().toString());
       return;
     }
 
@@ -112,11 +104,10 @@ private void doImportRepository(Path repositoryFile, boolean forceMode) {
     if (buildCmd != null && !buildCmd.isEmpty()) {
       String[] command = buildCmd.split("\\s+");
       ToolCommandlet commandlet = this.context.getCommandletManager().getToolCommandlet(command[0]);
-      List<String> args = new ArrayList<>(command.length - 1);
+
       for (int i = 1; i < command.length; i++) {
-        args.add(command[i]);
+        commandlet.arguments.addValue(command[i]);
       }
-      commandlet.arguments.setValue(args);
       commandlet.run();
     } else {
       this.context.info("Build command not set. Skipping build for repository.");