Updated comment in root directory's build.gradle file

Signed-off-by: Aashir Siddiqui <[email protected]>

galasa-managers-parent/build.gradle

@@ -22,6 +22,26 @@ subprojects {
    plugins.withId('jacoco')  {
         rootProject.tasks.named('jacocoMerge').get().executionData(tasks.named('test').get())
    }
+
+    // Uncomment the block below to get details about deprecations.
+
+//    tasks.withType(JavaCompile) {
+//         options.compilerArgs << '-Xlint:unchecked'
+//         options.deprecation = true
+//     }
+
+    // This task will suppress warnings (only in javadoc) about missing description for (@return, @throws etc..)
+    /*
+        Example warning:
+        
+        warning: no description for @throws
+         * @throws SeleniumManagerException
+    */
+    // However, it will not suppress warnings / errors in the code itself
+    tasks.withType(Javadoc) {
+        options.addStringOption('Xdoclint:none', '-quiet')
+    }
+
 }
 
 // Define the artifact

galasa-managers-parent/galasa-managers-cicsts-parent/dev.galasa.cicsts.manager/src/main/java/dev/galasa/cicsts/ICeci.java

@@ -75,8 +75,8 @@ public interface ICeci {
      * @param parseOutput parse the command output and store in {@link ICeciResponse}. Setting to false can improve performance on commands
      * that contain a lot of output fields, e.g. <code>ASSIGN</code>.<br><br>
      * The following examples shows how to retrieve a specific returned value:<br><code>
-     * issueCommand(ICicsTerminal, "ASSIGN USERID(&VAR)", false)<br>
-     * retrieveVariableText(ICicsTerminal, "ASSIGN USERID(&VAR)", false)
+     * issueCommand(ICicsTerminal, "ASSIGN USERID(&amp;VAR)", false)<br>
+     * retrieveVariableText(ICicsTerminal, "ASSIGN USERID(&amp;VAR)", false)
      * </code>
      * @return an {@link ICeciResponse} object containing the command's response.
      * @throws CeciException 
@@ -239,7 +239,7 @@ public interface ICeci {
 
     /**
      * Retrieve the content of the current EXEC Interface Block (EIB)
-     * @param terminal an {@link ICicsTerminal} object logged on to the CICS region and in an active CECI session.
+     * @param ceciTerminal an {@link ICicsTerminal} object logged on to the CICS region and in an active CECI session.
      * @return the {@link IExecInterfaceBlock} 
      * @throws CeciException
      */
@@ -252,7 +252,7 @@ public interface ICeci {
      * For example, the test could first issue <code>CEOT TRANIDONLY</code>
      * @param programName the name of the PROGRAM
      * @param commarea a string representing the COMMAREA. If null, COMMAREA will be omitted from the command. Can be CECI variable name populated with
-     *  (<b>&</b>name set via {@link #defineVariableText(ICicsTerminal, String, String)}) or the actual data. The value of DATALENGTH in the command will be 
+     *  (<b>&amp;</b>)name set via {@link #defineVariableText(ICicsTerminal, String, String)}) or the actual data. The value of DATALENGTH in the command will be 
      * be allowed to default.
      * @param sysid the system name where the CICS region where the link request is to be routed. If null, SYSID will be omitted from the command.
      * @param transid the name of the mirror transaction on the remote region. If null, TRANSID will be omitted from the command.
@@ -263,8 +263,8 @@ public interface ICeci {
     public ICeciResponse linkProgram(@NotNull ICicsTerminal ceciTerminal, @NotNull String programName, String commarea, String sysid, String transid, boolean synconreturn) throws CeciException;
 
     /**
-     * EXEC CICS LINK to a PROGRAM with a CHANNEL. Use {@link #putContainer(ICicsTerminal, String, String, String)} to create the container(s) on the CHANNEL 
-     * and {@link #getContainer(ICicsTerminal, String, String, String)} to retrieve the content after the LINK.
+     * EXEC CICS LINK to a PROGRAM with a CHANNEL. Use {@link #putContainer(ICicsTerminal, String, String, String, String, String, String)} to create the container(s) on the CHANNEL 
+     * and {@link #getContainer(ICicsTerminal, String, String, String, String, String)} to retrieve the content after the LINK.
      * @param ceciTerminal an {@link ICicsTerminal} object logged on to the CICS region and in an active CECI session.
      * If mixed case is required, the terminal should be presented with no upper case translate status. 
      * For example, the test could first issue <code>CEOT TRANIDONLY</code>  
@@ -279,7 +279,7 @@ public interface ICeci {
      * @param ceciTerminal an {@link ICicsTerminal} object logged on to the CICS region and in an active CECI session.
      * @param channelName the CHANNELNAME
      * @param containerName the COTAINER name
-     * @param content a string representing the container contents. Can be CECI variable name populated with (<b>&</b>name set via {@link #defineVariableText(ICicsTerminal, String, String)}) 
+     * @param content a string representing the container contents. Can be CECI variable name populated with (<b>&amp;</b>)name set via {@link #defineVariableText(ICicsTerminal, String, String)}) 
      * or the actual data. The value of FLENGTH in the command will be set to the data length.
      * @param dataType BIT or CHAR. If null, DATATYPE will be omitted from the command.
      * @param fromCcsid provides a value for FROMCCSID. If null, will be omitted from the command.
@@ -296,8 +296,7 @@ public interface ICeci {
      * For example, the test could first issue <code>CEOT TRANIDONLY</code>
      * @param channelName the CHANNELNAME
      * @param containerName the CONTAINER name
-     * @param variableName the CECI variable name. Data can be retrieved using {@link #retrieveVariableText(ICicsTerminal, String)} or {@link #retrieveVariableHex(ICicsTerminal, String)}
-     * @param dataType BIT or CHAR. If null, DATATYPE will be omitted from the command. 
+     * @param variableName the CECI variable name. Data can be retrieved using {@link #retrieveVariableText(ICicsTerminal, String)}
      * @param intoCcsid provides a value for INTOCCSID. If null, will be omitted from the command.
      * @param intoCodepage provides a value for INTOCODEPAGE. If null, will be omitted from the command.
      * @return an {@link ICeciResponse} object containing the command's response.

galasa-managers-parent/galasa-managers-cicsts-parent/dev.galasa.cicsts.manager/src/main/java/dev/galasa/cicsts/ICemt.java

@@ -16,7 +16,7 @@ public interface ICemt {
     /** 
      * Inquire a CEMT resource using the resource type and name.
      * This does not support inquiries of multiple resources at once. 
-     * Will return {@link null} if the resource is not found.
+     * Will return null if the resource is not found.
      * @param cemtTerminal an {@link ITerminal} object logged on to the CICS region and in an active CEMT session.
      * If mixed case is required, the terminal should be presented with no upper case translate status. 
      * For example, the test could first issue <code>CEOT TRANIDONLY</code>

galasa-managers-parent/galasa-managers-cicsts-parent/dev.galasa.cicsts.manager/src/main/java/dev/galasa/cicsts/IExecInterfaceBlock.java

@@ -1,8 +1,8 @@
-/*
- * Copyright contributors to the Galasa project
- *
- * SPDX-License-Identifier: EPL-2.0
- */
+/*
+ * Copyright contributors to the Galasa project
+ *
+ * SPDX-License-Identifier: EPL-2.0
+ */
 package dev.galasa.cicsts;
 
 /**
@@ -51,7 +51,6 @@ public interface IExecInterfaceBlock {
 
     /**
      * Returns the value of the EIBCPOSN field in the EIB.
-     * @param hex return a char array containing the hex values of the field
      * @return
      */
     public int getEIBCPOSN();

galasa-managers-parent/galasa-managers-cicsts-parent/dev.galasa.cicsts.manager/src/main/java/dev/galasa/cicsts/cicsresource/IJvmserver.java

@@ -1,8 +1,8 @@
-/*
- * Copyright contributors to the Galasa project
- *
- * SPDX-License-Identifier: EPL-2.0
- */
+/*
+ * Copyright contributors to the Galasa project
+ *
+ * SPDX-License-Identifier: EPL-2.0
+ */
 package dev.galasa.cicsts.cicsresource;
 
 import java.util.List;
@@ -394,7 +394,6 @@ public enum PurgeType {
 
 	/**
 	 * Delete the JVM server logs
-     * @param rasPath path in Results Archive Store
      * @throws CicsJvmserverResourceException
 	 */
 	public void clearJvmLogs() throws CicsJvmserverResourceException;

galasa-managers-parent/galasa-managers-cloud-parent/dev.galasa.cloud.manager/src/main/java/dev/galasa/cloud/internal/CloudManagerImpl.java

@@ -168,7 +168,7 @@ public void provisionGenerate() throws ManagerException, ResourceUnavailableExce
 	 * 
 	 * @param field The test field
 	 * @param annotations any annotations with the ecosystem
-	 * @return a {@link IKubernetesEcosystem} ecosystem
+	 * @return a {@link dev.galasa.galasaecosystem.IKubernetesEcosystem} ecosystem
 	 * @throws InsufficientResourcesAvailableException 
 	 * @throws KubernetesManagerException if there is a problem generating a ecosystem
 	 */

galasa-managers-parent/galasa-managers-cloud-parent/dev.galasa.docker.manager/src/main/java/dev/galasa/docker/internal/DockerContainerImpl.java

@@ -417,7 +417,7 @@ public Map<String, List<InetSocketAddress>> getExposedPorts() throws DockerManag
     /**
      * Retrieves the first open socket `
      * 
-     * @param exposed port
+     * @param exposedPort
      * @return InetSocketAddress
      */
     @Override
@@ -432,7 +432,7 @@ public InetSocketAddress getFirstSocketForExposedPort(String exposedPort) {
     /**
      * Retrieves a random open socket
      * 
-     * @param exposed port
+     * @param exposedPort
      * @return InetSocketAddress
      */
     @Override
@@ -447,7 +447,7 @@ public InetSocketAddress getRandomSocketForExposedPort(String exposedPort) {
     /**
      * Submit exec commands to be executed on the docker container.
      * 
-     * @param String... commands
+     * @param command
      * @return IDockerExec
      */
     @Override
@@ -460,7 +460,7 @@ public IDockerExec exec(String... command) throws DockerManagerException {
      * timeout.
      * 
      * @param timeout
-     * @param String... commands
+     * @param command
      * @return IDockerExec
      */
     @Override
@@ -678,8 +678,8 @@ public void discard() throws DockerManagerException {
     /**
      * Allows a file to be stored on a running docker container
      * 
-     * @param path
-     * @param InputStream
+     * @param absolutePath
+     * @param file
      */
     @Override
     public void storeFile(String absolutePath, InputStream file) throws DockerManagerException {

galasa-managers-parent/galasa-managers-cloud-parent/dev.galasa.docker.manager/src/main/java/dev/galasa/docker/internal/DockerImageImpl.java

@@ -43,7 +43,7 @@ public class DockerImageImpl implements IDockerImage {
      * @param framework
      * @param dockerManager
      * @param dockerEngine
-     * @param imageName
+     * @param fullImageName
      */
     public DockerImageImpl(IFramework framework, DockerManagerImpl dockerManager, 
             DockerEngineImpl dockerEngine, String fullImageName) {

galasa-managers-parent/galasa-managers-cloud-parent/dev.galasa.docker.manager/src/main/java/dev/galasa/docker/internal/DockerManagerImpl.java

@@ -71,10 +71,10 @@ public class DockerManagerImpl extends AbstractManager implements IDockerManager
      * 
      * Docker Environment is generated at this stage
      * 
-     * @param IFramework - the galasa framework
-     * @param List<IManager> - list of all the managers
-     * @param List<Imanager> - list of all the active managers
-     * @param Class<?> - the test class
+     * @param framework - the galasa framework
+     * @param allManagers - list of all the managers
+     * @param activeManagers - list of all the active managers
+
      * @throws ManagerException
      */
     @Override
@@ -104,8 +104,8 @@ public void initialise(@NotNull IFramework framework, @NotNull List<IManager> al
     /**
      * Makes sure that the docker manager is added to the list of active managers, and adds the dependency on http manager.
      * 
-     * @param List<IManager> - list of all the managers
-     * @param List<IManager> - list of the active managers
+     * @param allManagers - list of all the managers
+     * @param activeManagers - list of the active managers
      * @throws ManagerException
      */
     @Override

galasa-managers-parent/galasa-managers-cloud-parent/dev.galasa.docker.manager/src/main/java/dev/galasa/docker/internal/DockerRegistryImpl.java

@@ -65,7 +65,7 @@ public class DockerRegistryImpl {
 	 * 
 	 * @param framework
 	 * @param dockerManager
-	 * @param registryUrl
+	 * @param registryId
 	 * @throws DockerManagerException
 	 */
 	public DockerRegistryImpl(IFramework framework, DockerManagerImpl dockerManager, String registryId)
@@ -95,9 +95,7 @@ public DockerRegistryImpl(IFramework framework, DockerManagerImpl dockerManager,
 	/**
 	 * Checks the registry for and image.
 	 * 
-	 * @param namespace
-	 * @param repository
-	 * @param tag
+	 * @param image
 	 * @return true/false
 	 */
 	public boolean doYouHave(DockerImageImpl image) {
@@ -130,8 +128,7 @@ public boolean doYouHave(DockerImageImpl image) {
 	/**
 	 * Registry authentication
 	 * 
-	 * @param namespace
-	 * @param repository
+	 * @param image
 	 * @throws DockerManagerException
 	 */
 	public void registryAuthenticate(DockerImageImpl image) throws DockerManagerException {
@@ -155,7 +152,6 @@ public void registryAuthenticate(DockerImageImpl image) throws DockerManagerExce
 	 * Attempts to gain a bearer token from realm, if unauthorized tries basic credentials login 
 	 * retreive token
 	 * 
-	 * @param client
 	 * @return String token
 	 * @throws DockerManagerException
 	 */
@@ -191,7 +187,6 @@ public String retrieveBearerToken() throws DockerManagerException {
 	/**
 	 * Uses basic crednetials to gain a basic auth token.
 	 * 
-	 * @param client
 	 * @return String token
 	 * @throws DockerManagerException
 	 */

galasa-managers-parent/galasa-managers-comms-parent/dev.galasa.http.manager/src/main/java/dev/galasa/http/HttpClientResponse.java

@@ -240,11 +240,11 @@ public static HttpClientResponse<String> textResponse(CloseableHttpResponse http
     }
 
     /**
-     * Create an {@link HttpClientResponse} with a {@link JSONObject} content type
+     * Create an {@link HttpClientResponse} with a &lt;com.google.gson.JsonObject&gt; content type
      * from an {@link HttpResponse}.
      * 
      * @param httpResponse
-     * @return - {@link HttpClientResponse} with a {@link JSONObject} content type
+     * @return - {@link HttpClientResponse} with a &lt;com.google.gson.JsonObject&gt; content type
      * @throws HttpClientException
      */
     public static HttpClientResponse<JsonObject> jsonResponse(CloseableHttpResponse httpResponse)
@@ -253,14 +253,14 @@ public static HttpClientResponse<JsonObject> jsonResponse(CloseableHttpResponse
     }
 
     /**
-     * Create an {@link HttpClientResponse} with a {@link JSONObject} content type
+     * Create an {@link HttpClientResponse} with a &lt;com.google.gson.JsonObject&gt; content type
      * from an {@link HttpResponse}. If contentOnBadResponse is true, an attempt
      * will be made to retrieve the content even on a non 200 status code, otherwise
      * the content will be null in such an instance.
      * 
      * @param httpResponse
      * @param contentOnBadResponse
-     * @return - {@link HttpClientResponse} with a {@link JSONObject} content type
+     * @return - {@link HttpClientResponse} with a &lt;com.google.gson.JsonObject&gt; content type
      * @throws HttpClientException
      */
     public static HttpClientResponse<JsonObject> jsonResponse(CloseableHttpResponse httpResponse,