Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JMH benchmark setup #763

Closed
wants to merge 2 commits into from
Closed

Conversation

tmszdmsk
Copy link

So this is a performance testing setup for mockk using JMH. See README.md for more details on how to run it.

I am not entirely sure it is the right place for the module (happy to move it to a better suited place)

Copy link
Collaborator

@Raibaz Raibaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM pending a minor change.

@@ -3,4 +3,4 @@ org.gradle.configureondemand=false
org.gradle.jvmargs=-XX:MaxMetaspaceSize=768m
localrepo=build/mockk-repo
# localrepo=/Users/raibaz/.m2/repository
# kotlin.version=1.5.10
kotlin.version=1.5.31
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should stay commented (or even better removed), because the kotlin version is being set externally by the github action that runs the builds for several different versions.

It's likely the reason why builds on 1.3.* fail.

@stale
Copy link

stale bot commented Apr 16, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. If you are sure that this issue is important and should not be marked as stale just ask to put an important label.

@stale stale bot added the stale label Apr 16, 2022
@stale stale bot closed this Apr 25, 2022
aSemy added a commit to aSemy/mockk that referenced this pull request Aug 25, 2022
…tmszdmsk)

refactor test projects to be in separate directory
Raibaz added a commit that referenced this pull request Aug 25, 2022
introduce Kotlinx Benchmark tests (copied from #763)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants