scalatest / scalatestplus-mockito

ScalaTest + JMock provides integration support between ScalaTest and Mockito.
Apache License 2.0
16 stars 11 forks source link

Mockito 5 #33

Closed cheeseng closed 11 months ago

cheeseng commented 1 year ago

Updated to use mockito 5.8.0, and added a note in README.md about the requirement for jdk11 for mockito 5.

LuciferYang commented 1 year ago

https://github.com/scalatest/scalatestplus-mockito/blob/0e869cadf95f4188f0f652d684a8e975590c3e3c/.github/workflows/ci.yml#L15

Need to switch to installing Java 11.

cheeseng commented 1 year ago

https://github.com/scalatest/scalatestplus-mockito/blob/0e869cadf95f4188f0f652d684a8e975590c3e3c/.github/workflows/ci.yml#L15

Need to switch to installing Java 11.

Yup, you're right, CI passed now, thanks.

DieBauer commented 11 months ago

Any plans on this? The official site already advertises this, but the artifact doesn't exist: https://www.scalatest.org/plus/mockito libraryDependencies += "org.scalatestplus" %% "mockito-5-6" % "3.2.17.0" % "test"

cheeseng commented 11 months ago

@DieBauer I am sorry we overlooked this and I think the previously staged artifacts for mockito 5.6 was dropped automatically, I have re-staged with latest mockito 5.8 and hopefully @bvenners shall release it soon!

DieBauer commented 11 months ago

Thanks for the update, and effort :)

LuciferYang commented 11 months ago

It looks like need to update the PR description. The current PR description is still Updated to use mockito 5.6.0.

cheeseng commented 11 months ago

@LuciferYang Thanks for the kind reminder.

And the good news is @bvenners has just released the artifacts!

Martomate commented 10 months ago

Hello! Just so you know, the website still mentions 5.6 instead of 5.8 (https://www.scalatest.org/plus/mockito). Would be nice if it was updated.

cheeseng commented 10 months ago

@Martomate Thanks for the reminder, I have submitted the following PR for it:

https://github.com/scalatest/scalatest-website/pull/235

Cheers.