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

Improve JUnit Usage #15

Merged
merged 1 commit into from Feb 19, 2017
Merged

Improve JUnit Usage #15

merged 1 commit into from Feb 19, 2017

Conversation

marschall
Copy link
Contributor

Improve the way JUnit is used. This commit includes the following
changes:

  • update JUnit 4 version to latest
  • replace custom Assert class with JUnit provided one
  • switch from assertEquals to assertArrayEquals for arrays
  • use fail() instead of assertFalse(true)
  • used static import of asserts for better readability

Improve the way JUnit is used. This commit includes the following
changes:

- update JUnit 4 version to latest
- replace custom Assert class with JUnit provided one
- switch from assertEquals to assertArrayEquals for arrays
- use fail() instead of assertFalse(true)
- used static import of asserts for better readability
@simmac
Copy link
Owner

simmac commented Feb 19, 2017

Looks good, thank you!

@simmac simmac merged commit 00015b5 into simmac:master Feb 19, 2017
@simmac
Copy link
Owner

simmac commented Feb 19, 2017

I will notify Threema about the PRs I merged and reopen them for pulling them into the official branch

@marschall
Copy link
Contributor Author

marschall commented Feb 19, 2017

@simmac
Copy link
Owner

simmac commented Feb 19, 2017

@marschall The threema-ch/threema-msgapi-sdk-java repo is not maintained anymore, unfortunately, the official version of the Threema Gateway Java API is available here though: https://gateway.threema.ch/de/developer/sdk-java

My official branch is just a mirror of the code which can be downloaded on the gateway.threema.ch page. I try to get all the changes we make in this "community enhanced version" of the msgapi into the official version by sending all patches to the Threema team. Once they release a new version on their site, I will merge all commits to the official branch, so that the 'official' branch is always an up-to-date mirror.

tl;dr: This repo is the only maintained one on GitHub. Official version is available at https://gateway.threema.ch/de/developer/sdk-java, the branch simmac:official is a mirror. 'simmac:master' is the "community enhanced" version, I try to get Threema to merge those enhancements into the official version.

@marschall
Copy link
Contributor Author

That's for clearing that up for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants