Merge pull request #441 from codacy/feature/upload-batch-size-param-CY-5727

feature: support issues-only analysis and log upload parsing failures CY-5727

Author: pedrocodacy

README.md

@@ -196,7 +196,7 @@ Found [Clone] 7 duplicated lines with 10 tokens:
 * `analyze` - Run a Codacy analysis over a directory/files
     * `--help` - Displays all the configuration options, their meaning and possible values.
     * `--verbose` - Run the tool with verbose output
-    * `--tool` - [Choose the tool](https://docs.codacy.com/repositories-configure/codacy-configuration-file/#which-tools-can-be-configured-and-which-name-should-i-use) to analyze the code (e.g. brakeman)
+    * `--tool` - [Choose the tool](https://docs.codacy.com/repositories-configure/codacy-configuration-file/#which-tools-can-be-configured-and-which-name-should-i-use) to analyze the code (e.g. brakeman), or "metrics", "duplication", "issues" to run only a specific tool category
     * `--directory` - Choose the directory to be analysed
     * `--codacy-api-base-url` or env.`CODACY_API_BASE_URL` - Change the Codacy installation API URL to retrieve the configuration (e.g. Enterprise installation)
     * `--output` - Send the output results to a file

cli/src/main/scala/com/codacy/analysis/cli/analysis/ToolSelector.scala

@@ -34,6 +34,10 @@ class ToolSelector(toolRepository: ToolRepository) {
       case Some("duplication") =>
         duplicationToolsEither.map(_.map(_.to[ITool]))
 
+      case Some("issues") =>
+        val toolsEither = tools(None, configuration, languages)
+        toolsEither.map(_.map(_.to[ITool]))
+
       case Some(_) =>
         val toolsEither = tools(toolInput, configuration, languages)
         toolsEither.map(_.map(_.to[ITool]))

core/src/main/scala/com/codacy/analysis/core/utils/HttpHelper.scala

@@ -2,7 +2,8 @@ package com.codacy.analysis.core.utils
 
 import io.circe.parser.parse
 import io.circe.{Json, ParsingFailure}
-import scalaj.http.{Http, HttpRequest, HttpResponse, HttpOptions}
+import org.log4s.{Logger, getLogger}
+import scalaj.http.{Http, HttpOptions, HttpRequest, HttpResponse}
 
 class HttpHelper(apiUrl: String, extraHeaders: Map[String, String], allowUnsafeSSL: Boolean) {
 
@@ -11,6 +12,8 @@ class HttpHelper(apiUrl: String, extraHeaders: Map[String, String], allowUnsafeS
 
   private val remoteUrl = apiUrl + "/2.0"
 
+  private val logger: Logger = getLogger
+
   private def httpOptions = if (allowUnsafeSSL) Seq(HttpOptions.allowUnsafeSSL) else Seq.empty
 
   def get(endpoint: String): Either[ParsingFailure, Json] = {
@@ -40,7 +43,14 @@ class HttpHelper(apiUrl: String, extraHeaders: Map[String, String], allowUnsafeS
       .headers(headers)
       .timeout(connectionTimeoutMs, readTimeoutMs)
 
-    parse(request.asString.body)
+    val response = request.asString
+    val bodyAsString = response.body
+    parse(bodyAsString) match {
+      case failure @ Left(_) =>
+        logger.warn(s"Post to $endpoint failed. Response was a ${response.code} and returned $bodyAsString")
+        failure
+      case success => success
+    }
   }
 
 }