Skip to content

Migrate RealmLinkTests.java to JUnit4 style.#3128

Merged
zaki50 merged 2 commits intoreleasesfrom
my/migrate_RealmLinkTest_to_Junit4
Jul 5, 2016
Merged

Migrate RealmLinkTests.java to JUnit4 style.#3128
zaki50 merged 2 commits intoreleasesfrom
my/migrate_RealmLinkTest_to_Junit4

Conversation

@zaki50
Copy link
Copy Markdown
Contributor

@zaki50 zaki50 commented Jul 5, 2016

RealmLinkTest is the last user of the deprecated TestHelper.createConfiguration() method.
I'll clean up TestHelper.createConfiguration()s in other PR.

@realm/java

@zaki50 zaki50 self-assigned this Jul 5, 2016
if (testRealm != null) {
testRealm.close();
}
Realm.deleteRealm(realmConfig);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should happen automatically when we use the TestRealmConfigurationFactory

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

@kneth
Copy link
Copy Markdown
Contributor

kneth commented Jul 5, 2016

👍

@cmelchior
Copy link
Copy Markdown
Contributor

👍

@zaki50 zaki50 merged commit 7379919 into releases Jul 5, 2016
@zaki50 zaki50 deleted the my/migrate_RealmLinkTest_to_Junit4 branch July 5, 2016 11:37
@zaki50 zaki50 removed the S:Review label Jul 5, 2016
@stk1m1
Copy link
Copy Markdown
Contributor

stk1m1 commented Jul 5, 2016

👍

@zaki50
Copy link
Copy Markdown
Contributor Author

zaki50 commented Jul 5, 2016

thanks all!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants