Skip to content

Commit 34a0686

Browse files
Tests: extract builder with default test model creator.
1 parent 0e77bad commit 34a0686

File tree

5 files changed

+19
-10
lines changed

5 files changed

+19
-10
lines changed

tests/objectbox-java-test/src/test/java/io/objectbox/AbstractObjectBoxTest.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -236,6 +236,13 @@ byte[] createTestModelWithTwoEntities(boolean withIndex) {
236236
return modelBuilder.build();
237237
}
238238

239+
/**
240+
* When not using the {@link #store} of this to create a builder with the default test model.
241+
*/
242+
protected BoxStoreBuilder createBuilderWithTestModel() {
243+
return new BoxStoreBuilder(createTestModel(null));
244+
}
245+
239246
private void addTestEntity(ModelBuilder modelBuilder, @Nullable IndexType simpleStringIndexType) {
240247
lastEntityUid = ++lastUid;
241248
EntityBuilder entityBuilder = modelBuilder.entity("TestEntity").id(++lastEntityId, lastEntityUid);

tests/objectbox-java-test/src/test/java/io/objectbox/BoxStoreBuilderTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ protected BoxStore createBoxStore() {
5555
@Before
5656
public void setUpBuilder() {
5757
BoxStore.clearDefaultStore();
58-
builder = new BoxStoreBuilder(createTestModel(null)).directory(boxStoreDir);
58+
builder = createBuilderWithTestModel().directory(boxStoreDir);
5959
}
6060

6161
@Test

tests/objectbox-java-test/src/test/java/io/objectbox/BoxStoreTest.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,7 @@ public void openSamePath_afterClose_works() {
176176
@Test
177177
public void testOpenTwoBoxStoreTwoFiles() {
178178
File boxStoreDir2 = new File(boxStoreDir.getAbsolutePath() + "-2");
179-
BoxStoreBuilder builder = new BoxStoreBuilder(createTestModel(null)).directory(boxStoreDir2);
179+
BoxStoreBuilder builder = createBuilderWithTestModel().directory(boxStoreDir2);
180180
builder.entity(new TestEntity_());
181181
}
182182

@@ -192,7 +192,7 @@ public void testDeleteAllFiles_staticDir() {
192192
closeStoreForTest();
193193

194194
File boxStoreDir2 = new File(boxStoreDir.getAbsolutePath() + "-2");
195-
BoxStoreBuilder builder = new BoxStoreBuilder(createTestModel(null)).directory(boxStoreDir2);
195+
BoxStoreBuilder builder = createBuilderWithTestModel().directory(boxStoreDir2);
196196
BoxStore store2 = builder.build();
197197
store2.close();
198198

@@ -217,7 +217,7 @@ public void testDeleteAllFiles_baseDirName() {
217217
File dbDir = new File(basedir, name);
218218
assertFalse(dbDir.exists());
219219

220-
BoxStoreBuilder builder = new BoxStoreBuilder(createTestModel(null)).baseDirectory(basedir).name(name);
220+
BoxStoreBuilder builder = createBuilderWithTestModel().baseDirectory(basedir).name(name);
221221
BoxStore store2 = builder.build();
222222
store2.close();
223223

@@ -285,7 +285,7 @@ public void testCallInReadTxWithRetry_callback() {
285285
final int[] countHolder = {0};
286286
final int[] countHolderCallback = {0};
287287

288-
BoxStoreBuilder builder = new BoxStoreBuilder(createTestModel(null)).directory(boxStoreDir)
288+
BoxStoreBuilder builder = createBuilderWithTestModel().directory(boxStoreDir)
289289
.failedReadTxAttemptCallback((result, error) -> {
290290
assertNotNull(error);
291291
countHolderCallback[0]++;

tests/objectbox-java-test/src/test/java/io/objectbox/BoxStoreValidationTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ protected BoxStore createBoxStore() {
5353
@Before
5454
public void setUpBuilder() {
5555
BoxStore.clearDefaultStore();
56-
builder = new BoxStoreBuilder(createTestModel(null)).directory(boxStoreDir);
56+
builder = createBuilderWithTestModel().directory(boxStoreDir);
5757
}
5858

5959
@Test

tests/objectbox-java-test/src/test/java/io/objectbox/query/QueryTest.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2017-2023 ObjectBox Ltd. All rights reserved.
2+
* Copyright 2017-2024 ObjectBox Ltd. All rights reserved.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -16,6 +16,9 @@
1616

1717
package io.objectbox.query;
1818

19+
import org.junit.Test;
20+
import org.junit.function.ThrowingRunnable;
21+
1922
import java.util.ArrayList;
2023
import java.util.Arrays;
2124
import java.util.Date;
@@ -34,8 +37,7 @@
3437
import io.objectbox.relation.MyObjectBox;
3538
import io.objectbox.relation.Order;
3639
import io.objectbox.relation.Order_;
37-
import org.junit.Test;
38-
import org.junit.function.ThrowingRunnable;
40+
3941

4042
import static io.objectbox.TestEntity_.simpleBoolean;
4143
import static io.objectbox.TestEntity_.simpleByteArray;
@@ -1234,7 +1236,7 @@ public void testForEachBreak() {
12341236
// TODO can we improve? More than just "still works"?
12351237
public void testQueryAttempts() {
12361238
store.close();
1237-
BoxStoreBuilder builder = new BoxStoreBuilder(createTestModel(null)).directory(boxStoreDir)
1239+
BoxStoreBuilder builder = createBuilderWithTestModel().directory(boxStoreDir)
12381240
.queryAttempts(5)
12391241
.failedReadTxAttemptCallback((result, error) -> {
12401242
if (error != null) {

0 commit comments

Comments
 (0)