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

add Java tests for verifying @JvmStatic #14

Merged
merged 1 commit into from
Feb 25, 2024

Conversation

aSemy
Copy link
Collaborator

@aSemy aSemy commented Feb 25, 2024

  • Created some Java tests to verify that the library can be called from Java code
  • moved test data to a separate class so it's easier to re-use in the Java tests, and created a TestData class so it's easier to use in Java (IntelliJ didn't like the Pair class for some reason, and kept showing errors for data.getFirst())
  • Enabled JUnitPlatform (otherwise the JUnit classes weren't available in the jvmTest source set)

Related to #12

@ethauvin ethauvin merged commit 1715701 into ethauvin:master Feb 25, 2024
10 checks passed
@aSemy aSemy deleted the test-jvmstatic branch February 25, 2024 12:17
krzema12 pushed a commit to krzema12/snakeyaml-engine-kmp that referenced this pull request Mar 28, 2024
#149)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[net.thauvin.erik.urlencoder:urlencoder-lib](https://togithub.com/ethauvin/urlencoder)
| `1.4.0` -> `1.5.0` |
[![age](https://developer.mend.io/api/mc/badges/age/maven/net.thauvin.erik.urlencoder:urlencoder-lib/1.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/maven/net.thauvin.erik.urlencoder:urlencoder-lib/1.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/maven/net.thauvin.erik.urlencoder:urlencoder-lib/1.4.0/1.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/maven/net.thauvin.erik.urlencoder:urlencoder-lib/1.4.0/1.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>ethauvin/urlencoder
(net.thauvin.erik.urlencoder:urlencoder-lib)</summary>

###
[`v1.5.0`](https://togithub.com/ethauvin/urlencoder/releases/tag/1.5.0)

[Compare
Source](https://togithub.com/ethauvin/urlencoder/compare/1.4.0...1.5.0)

#### Summary

- Updated Kotlin to 1.9.24 (The Kotlin `languageVersion` is still set to
1.6, so there's no change in the library's ABI.)
-   Added additional Kotlin Multiplatform targets.
- Tidied the Kotlin Multiplatform source sets, thanks to the [new
hierarchy
structure](https://kotlinlang.org/docs/multiplatform-hierarchy.html).
- Bumped the Node version used by Kotlin Multiplatform, which is
required for Wasm support.
- Refactored urlencoder-app to remove the Java Application plugin (this
is no longer compatible with KGP), and instead use KGP's new `mainRun
{}` function.

#### What's Changed

- add Java tests for verifying `@JvmStatic` by
[@&#8203;aSemy](https://togithub.com/aSemy) in
[ethauvin/urlencoder#14
- bump Kotlin to 1.9.23, enable additional Kotlin Multiplatform targets
by [@&#8203;aSemy](https://togithub.com/aSemy) in
[ethauvin/urlencoder#15
- Bumped Gradle to version 8.7 by
[@&#8203;ethauvin](https://togithub.com/ethauvin) in
ethauvin/urlencoder@a689c56
- Bumped Dokka to version 1.9.20 by
[@&#8203;ethauvin](https://togithub.com/ethauvin) in
ethauvin/urlencoder@b59d01e
- bump version to 1.5.0-SNAPSHOT by
[@&#8203;aSemy](https://togithub.com/aSemy) in
[ethauvin/urlencoder#16
- Reworked GitHub workflows by
[@&#8203;ethauvin](https://togithub.com/ethauvin) in
ethauvin/urlencoder@1ef2045
- Tidy publishing config by [@&#8203;aSemy](https://togithub.com/aSemy)
in
[ethauvin/urlencoder#17

**Full Changelog**:
ethauvin/urlencoder@1.4.0...1.5.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/krzema12/snakeyaml-engine-kmp).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yNjkuMiIsInVwZGF0ZWRJblZlciI6IjM3LjI2OS4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants