Skip to content

Commit

Permalink
TRELLO-2793: pro connect fix (#1867)
Browse files Browse the repository at this point in the history
* TRELLO-2793: pro connect fix
  • Loading branch information
ssedoudbgouv authored Feb 1, 2025
1 parent 51c452a commit b3adaf5
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 7 deletions.
6 changes: 0 additions & 6 deletions app/models/proconnect/ProConnectClaim.scala
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ import play.api.libs.json.Reads

case class ProConnectClaim(
sub: String,
custom: CustomField,
email: String,
givenName: String,
usualName: String,
Expand All @@ -18,15 +17,10 @@ case class ProConnectClaim(
idp_id: String
)

case class CustomField(
profession: String
)

object ProConnectClaim {

implicit val config: JsonConfiguration.Aux[Json.MacroOptions] = JsonConfiguration(JsonNaming.SnakeCase)

implicit val customFieldReads: Reads[CustomField] = Json.reads[CustomField]
implicit val proConnectClaimReads: Reads[ProConnectClaim] = Json.reads[ProConnectClaim]

}
2 changes: 1 addition & 1 deletion test/tasks/report/SampleDataGenerationTaskTest.scala
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ class SampleDataGenerationTaskTest(implicit ee: ExecutionEnv)
def check(): Future[Unit] =
for {
// Old reports should still exists
x <- readReport(reportExpired)
_ <- readReport(reportExpired)
_ = println(s"------------------ x = ${reportExpired.id} ------------------")
_ <- readReport(reportExpired).map(_.isDefined must beTrue)
_ <- readReport(reportExpiredSeenByPro).map(_.isDefined must beTrue)
Expand Down

0 comments on commit b3adaf5

Please sign in to comment.