Run scalafix, now that it is working

Author: morgen-peschke

collections/test/src/peschke/UnitSpec.scala

@@ -1,6 +1,7 @@
 package peschke
 
-import org.scalatest.{EitherValues, OptionValues}
+import org.scalatest.EitherValues
+import org.scalatest.OptionValues
 import org.scalatest.matchers.must.Matchers
 import org.scalatest.wordspec.AnyWordSpec
 

core/src/peschke/Complete.scala

@@ -1,6 +1,8 @@
 package peschke
 
-import cats.{Hash, Monoid, Show}
+import cats.Hash
+import cats.Monoid
+import cats.Show
 
 /** Used as a replacement for returning Unit.
  *

shims/src/peschke/shims/resource/Bracket.scala

@@ -1,11 +1,15 @@
 package peschke.shims.resource
 
-import cats.{Id, Semigroup}
+import cats.Id
+import cats.Semigroup
 import cats.syntax.all._
 import peschke.Complete
 
-import scala.concurrent.{ExecutionContext, Future}
-import scala.util.{Failure, Success, Try}
+import scala.concurrent.ExecutionContext
+import scala.concurrent.Future
+import scala.util.Failure
+import scala.util.Success
+import scala.util.Try
 
 /** This is a poor replacement for cats-effect's MonadCancel.
  *

shims/src/peschke/shims/resource/Closer.scala

@@ -1,7 +1,7 @@
 package peschke.shims.resource
 
-import cats.syntax.all._
 import cats.ApplicativeThrow
+import cats.syntax.all._
 import peschke.Complete
 
 import java.io.Closeable

shims/src/peschke/shims/resource/ExceptionMapper.scala

@@ -1,7 +1,8 @@
 package peschke.shims.resource
 
+import cats.ApplicativeError
+import cats.Id
 import cats.syntax.all._
-import cats.{ApplicativeError, Id}
 
 import scala.concurrent.Future
 import scala.util.Try
@@ -29,7 +30,7 @@ object ExceptionMapper {
  try fa
  catch {
  case NonFatal(ex) =>
- mapErrorIfPossible(ex).map(AE.raiseError[A]).getOrElse(throw ex)
+ mapErrorIfPossible(ex).map(AE.raiseError[A]).getOrElse(throw ex) // scalafix:ok DisableSyntax.throw
  }
  }
 

shims/src/peschke/shims/resource/Managed.scala

@@ -1,7 +1,8 @@
 package peschke.shims.resource
 
+import cats.Applicative
+import cats.Monad
 import cats.syntax.all._
-import cats.{Applicative, Monad}
 
 /** This is a poor replacement for cats-effect's `Resource`
  *

shims/test/src-2.13/peschke/UnitSpec.scala

@@ -1,8 +1,9 @@
 package peschke
 
+import org.scalatest.EitherValues
+import org.scalatest.OptionValues
 import org.scalatest.matchers.must.Matchers
 import org.scalatest.wordspec.AnyWordSpec
-import org.scalatest.{EitherValues, OptionValues}
 
 trait UnitSpec
  extends AnyWordSpec

shims/test/src-2.13/peschke/shims/resource/ManagedTest.scala

@@ -4,12 +4,17 @@ import cats.Id
 import cats.data.Chain
 import cats.syntax.all._
 import org.scalatest.TryValues
-import peschke.shims.resource.TestResourceFactory.{Record, ResourceBuilder}
-import peschke.{Complete, UnitSpec}
+import peschke.Complete
+import peschke.UnitSpec
+import peschke.shims.resource.TestResourceFactory.Record
+import peschke.shims.resource.TestResourceFactory.ResourceBuilder
 
-import java.util.concurrent.atomic.{AtomicBoolean, AtomicReference}
+import java.util.concurrent.atomic.AtomicBoolean
+import java.util.concurrent.atomic.AtomicReference
 import scala.util.Try
 
+// OK to throw in this test, as the code needs to behave in the presence of exceptions
+// scalafix:off DisableSyntax.throw
 class ManagedIdTest extends UnitSpec {
  val manage: Managed.Factory[Id, Throwable] = Managed.factory[Id, Throwable]
 
@@ -1297,15 +1302,16 @@ object TestResourceFactory {
  */
  sealed abstract class Record[A] extends Product with Serializable
  object Record {
- final case class SuccessfullyOpened[A]() extends Record[A]
+ // 'ignored: Boolean = true' is a workaround for https://github.com/scalameta/scalafmt/issues/3304
+ final case class SuccessfullyOpened[A](ignored: Boolean = true) extends Record[A]
  final case class FailedToOpen[A](error: String) extends Record[A]
  final case class SuccessfullyRetrieved[A](value: A) extends Record[A]
  final case class FailedToRetrieve[A](error: String) extends Record[A]
- final case class RetrieveAttemptedAfterClose[A]() extends Record[A]
- final case class SuccessfullyClosed[A]() extends Record[A]
- final case class FailedToClose[A]() extends Record[A]
- final case class CloseAttemptedAfterClose[A]() extends Record[A]
- final case class RanOutOfResources[A]() extends Record[A]
+ final case class RetrieveAttemptedAfterClose[A](ignored: Boolean = true) extends Record[A]
+ final case class SuccessfullyClosed[A](ignored: Boolean = true) extends Record[A]
+ final case class FailedToClose[A](ignored: Boolean = true) extends Record[A]
+ final case class CloseAttemptedAfterClose[A](ignored: Boolean = true) extends Record[A]
+ final case class RanOutOfResources[A](ignored: Boolean = true) extends Record[A]
  final case class Marker[A](tag: String) extends Record[A]
  }
 }