Backport #359

Author: etspaceman

swagger/src/main/scala/org/http4s/rho/swagger/SwaggerModelsBuilder.scala

@@ -255,13 +255,18 @@ private[swagger] class SwaggerModelsBuilder(formats: SwaggerFormats) {
     go(rr.rules::Nil)
   }
 
+  def renderMediaRange: MediaRange => String = {
+    case tpe: MediaType    => tpe.show
+    case range: MediaRange => range.show
+  }
+
   def mkOperation[F[_]](pathStr: String, rr: RhoRoute[F, _])(implicit etag: WeakTypeTag[F[_]]): Operation = {
     val parameters = collectOperationParams(rr)
 
     Operation(
       tags        = collectTags(rr),
       summary     = collectSummary(rr),
-      consumes    = rr.validMedia.toList.map(_.show),
+      consumes    = rr.validMedia.toList.map(renderMediaRange),
       produces    = rr.responseEncodings.toList.map(_.show),
       operationId = mkOperationId(pathStr, rr.method, parameters).some,
       parameters  = parameters,

swagger/src/test/scala/org/http4s/rho/swagger/SwaggerModelsBuilderSpec.scala

@@ -449,6 +449,17 @@ class SwaggerModelsBuilderSpec extends Specification {
 
       compiledRoutes should_== results
     }
+
+    "generate precise consumed media types" in {
+      val dec = new EntityDecoder[IO, List[String]] {
+        def consumes: Set[MediaRange] = Set(MediaType.application.json)
+        def decode(msg: Message[IO], strict: Boolean): DecodeResult[IO, List[String]] = ???
+      }
+      val ra = GET / "foo" ^ dec |>> { _: List[String] => "" }
+      val op = sb.mkOperation("/foo", ra)
+
+      op.consumes must_== List("application/json")
+    }
   }
 
   "SwaggerModelsBuilder.collectDefinitions" should {