Scalafix fixes (#563)

Co-authored-by: Jan Kolena <[email protected]>
avast · May 17, 2021 · 9f6a029 · 9f6a029
1 parent 75ad62a
commit 9f6a029
Show file tree

Hide file tree

Showing 10 changed files with 12 additions and 9 deletions.
diff --git a/cats-effect/src/test/scala/com/avast/sst/catseffect/syntax/FOpsTest.scala b/cats-effect/src/test/scala/com/avast/sst/catseffect/syntax/FOpsTest.scala
@@ -8,6 +8,7 @@ import org.scalatest.funsuite.AsyncFunSuite
 import scala.concurrent.ExecutionContext
 import scala.concurrent.duration.{Duration, TimeUnit}
 
+@SuppressWarnings(Array("scalafix:Disable.get", "scalafix:DisableSyntax.var"))
 class FOpsTest extends AsyncFunSuite {
 
   implicit private val timer: Timer[IO] = IO.timer(ExecutionContext.global)

diff --git a/fs2-kafka-pureconfig/src/main/scala/com/avast/sst/fs2kafka/pureconfig/ConfigReaders.scala b/fs2-kafka-pureconfig/src/main/scala/com/avast/sst/fs2kafka/pureconfig/ConfigReaders.scala
@@ -8,6 +8,7 @@ import pureconfig.error.CannotConvert
 import pureconfig.generic.ProductHint
 import pureconfig.generic.semiauto.deriveReader
 
+@SuppressWarnings(Array("scalafix:DisableSyntax.=="))
 trait ConfigReaders {
 
   implicit protected def hint[T]: ProductHint[T] = ProductHint.default

diff --git a/fs2-kafka/src/main/scala/com/avast/sst/fs2kafka/ConsumerConfig.scala b/fs2-kafka/src/main/scala/com/avast/sst/fs2kafka/ConsumerConfig.scala
@@ -40,6 +40,7 @@ object ConsumerConfig {
 
   private val officialDefaults = ApacheConsumerConfig.configDef().defaultValues().asScala
 
+  @SuppressWarnings(Array("scalafix:DisableSyntax.asInstanceOf"))
   private def default[A](key: String): A = officialDefaults(key).asInstanceOf[A]
 
   private def defaultMillis(key: String): FiniteDuration = FiniteDuration(default[Int](key).toLong, MILLISECONDS)

diff --git a/fs2-kafka/src/main/scala/com/avast/sst/fs2kafka/ProducerConfig.scala b/fs2-kafka/src/main/scala/com/avast/sst/fs2kafka/ProducerConfig.scala
@@ -28,6 +28,7 @@ object ProducerConfig {
 
   private val officialDefaults = ApacheProducerConfig.configDef().defaultValues().asScala
 
+  @SuppressWarnings(Array("scalafix:DisableSyntax.asInstanceOf"))
   private def default[A](key: String): A = officialDefaults(key).asInstanceOf[A]
 
   private def defaultMillis(key: String): FiniteDuration = FiniteDuration(default[Int](key).toLong, MILLISECONDS)

diff --git a/http4s-client-blaze/src/test/scala/com/avast/sst/http4s/client/Http4SBlazeClientTest.scala b/http4s-client-blaze/src/test/scala/com/avast/sst/http4s/client/Http4SBlazeClientTest.scala
@@ -1,7 +1,7 @@
 package com.avast.sst.http4s.client
 
 import cats.effect._
-import org.http4s.headers.{`User-Agent`, AgentComment, AgentProduct}
+import org.http4s.headers._
 import org.scalatest.funsuite.AsyncFunSuite
 
 import scala.concurrent.ExecutionContext

diff --git a/...t/scala/com/avast/sst/http4s/server/micrometer/MicrometerHttp4sMetricsOpsModuleTest.scala b/...t/scala/com/avast/sst/http4s/server/micrometer/MicrometerHttp4sMetricsOpsModuleTest.scala
@@ -1,12 +1,12 @@
 package com.avast.sst.http4s.server.micrometer
 
-import java.util.concurrent.TimeUnit
-
 import cats.effect.IO
 import io.micrometer.core.instrument.simple.SimpleMeterRegistry
 import org.http4s.{Method, Status}
 import org.scalatest.funsuite.AnyFunSuite
 
+import java.util.concurrent.TimeUnit
+
 class MicrometerHttp4sMetricsOpsModuleTest extends AnyFunSuite {
 
   test("http4s MetricsOps for Micrometer") {

diff --git a/...r-micrometer/src/test/scala/com/avast/sst/http4s/server/micrometer/RouteMetricsTest.scala b/...r-micrometer/src/test/scala/com/avast/sst/http4s/server/micrometer/RouteMetricsTest.scala
@@ -1,12 +1,12 @@
 package com.avast.sst.http4s.server.micrometer
 
-import java.util.concurrent.TimeUnit
-
 import cats.effect.SyncIO
 import io.micrometer.core.instrument.simple.SimpleMeterRegistry
 import org.http4s.Response
 import org.scalatest.funsuite.AnyFunSuite
 
+import java.util.concurrent.TimeUnit
+
 class RouteMetricsTest extends AnyFunSuite {
 
   test("Single route metrics") {

diff --git a/...r/src/test/scala/com/avast/sst/http4s/server/middleware/CorrelationIdMiddlewareTest.scala b/...r/src/test/scala/com/avast/sst/http4s/server/middleware/CorrelationIdMiddlewareTest.scala
@@ -1,7 +1,5 @@
 package com.avast.sst.http4s.server.middleware
 
-import java.net.InetSocketAddress
-
 import cats.effect.{ContextShift, IO, Resource, Timer}
 import com.avast.sst.http4s.server.Http4sRouting
 import org.http4s.client.blaze.BlazeClientBuilder
@@ -11,6 +9,7 @@ import org.http4s.util.CaseInsensitiveString
 import org.http4s.{Header, HttpRoutes, Request, Uri}
 import org.scalatest.funsuite.AsyncFunSuite
 
+import java.net.InetSocketAddress
 import scala.concurrent.ExecutionContext
 
 @SuppressWarnings(Array("scalafix:Disable.get", "scalafix:Disable.toString", "scalafix:Disable.createUnresolved"))

diff --git a/jvm/src/test/scala/com/avast/sst/jvm/system/console/ConsoleModuleTest.scala b/jvm/src/test/scala/com/avast/sst/jvm/system/console/ConsoleModuleTest.scala
@@ -1,10 +1,9 @@
 package com.avast.sst.jvm.system.console
 
-import java.io.{ByteArrayInputStream, ByteArrayOutputStream}
-
 import cats.effect.SyncIO
 import org.scalatest.funsuite.AnyFunSuite
 
+import java.io.{ByteArrayInputStream, ByteArrayOutputStream}
 import scala.{Console => SConsole}
 
 class ConsoleModuleTest extends AnyFunSuite {

diff --git a/lettuce/src/main/scala/com/avast/sst/lettuce/LettuceModule.scala b/lettuce/src/main/scala/com/avast/sst/lettuce/LettuceModule.scala
@@ -29,6 +29,7 @@ object LettuceModule {
   }
 
   /** Makes [[io.lettuce.core.api.StatefulRedisConnection]] initialized with the given config and optionally [[io.lettuce.core.resource.ClientResources]]. */
+  @SuppressWarnings(Array("scalafix:DisableSyntax.==", "scalafix:DisableSyntax.null"))
   def makeConnection[F[_]: Async, K, V](
       config: LettuceConfig,
       clientResources: Option[ClientResources] = None