Skip to content

Commit 441757a

Browse files
committed
add tests
1 parent 3d294d2 commit 441757a

File tree

9 files changed

+96
-5
lines changed

9 files changed

+96
-5
lines changed

net.lecousin.core/src/main/java/net/lecousin/framework/collections/map/IntegerMapLinkedArrayList.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ public Entry(int key, T value) {
4545
public void clear() {
4646
for (LinkedArrayList<Entry<T>> bucket : buckets)
4747
bucket.clear();
48+
size = 0;
4849
}
4950

5051
@Override
@@ -58,6 +59,7 @@ public T put(int key, T entry) {
5859
return previous;
5960
}
6061
buckets[b].add(new Entry<T>(key, entry));
62+
size++;
6163
return null;
6264
}
6365

@@ -79,6 +81,7 @@ public T remove(int key) {
7981
Entry<T> e = it.next();
8082
if (e.key == key) {
8183
it.remove();
84+
size--;
8285
return e.value;
8386
}
8487
}

net.lecousin.core/src/test/java/net/lecousin/framework/core/test/collections/sort/TestSortedAssociatedWithInteger.java

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,19 @@ protected void check(Sorted.AssociatedWithInteger<Object> sorted, TreeSet<Intege
166166
}
167167
Assert.assertEquals(sorted.size(), nb);
168168
Assert.assertFalse(it2.hasNext());
169+
170+
it1 = sorted.reverseOrderIterator();
171+
it2 = order.descendingIterator();
172+
nb = 0;
173+
while (it1.hasNext()) {
174+
Assert.assertTrue(it2.hasNext());
175+
Object o = it1.next();
176+
Integer i = it2.next();
177+
Assert.assertEquals(i.intValue(), -((Integer)o).intValue());
178+
nb++;
179+
}
180+
Assert.assertEquals(sorted.size(), nb);
181+
Assert.assertFalse(it2.hasNext());
169182
}
170183

171184
}

net.lecousin.core/src/test/java/net/lecousin/framework/core/test/collections/sort/TestSortedAssociatedWithLong.java

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,19 @@ protected void check(Sorted.AssociatedWithLong<Object> sorted, TreeSet<Long> ord
166166
}
167167
Assert.assertEquals(sorted.size(), nb);
168168
Assert.assertFalse(it2.hasNext());
169+
170+
it1 = sorted.reverseOrderIterator();
171+
it2 = order.descendingIterator();
172+
nb = 0;
173+
while (it1.hasNext()) {
174+
Assert.assertTrue(it2.hasNext());
175+
Object o = it1.next();
176+
Long i = it2.next();
177+
Assert.assertEquals(i.longValue(), -((Long)o).longValue());
178+
nb++;
179+
}
180+
Assert.assertEquals(sorted.size(), nb);
181+
Assert.assertFalse(it2.hasNext());
169182
}
170183

171184
}

net.lecousin.core/src/test/java/net/lecousin/framework/core/test/io/TestOutputToInput.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,9 @@ public void testWriteSyncReadHalfSync() throws Exception {
5151
readHalfSync(o2i, (nbRead % 2) == 0, b);
5252
nbRead++;
5353
}
54+
Assert.assertFalse(o2i.isFullDataAvailable());
5455
o2i.endOfData();
56+
Assert.assertTrue(o2i.isFullDataAvailable());
5557
while (nbRead < nbBuf * 2) {
5658
readHalfSync(o2i, (nbRead % 2) == 0, b);
5759
nbRead++;
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
package net.lecousin.framework.core.tests.collections.maps;
2+
3+
import net.lecousin.framework.collections.map.IntegerMap;
4+
import net.lecousin.framework.collections.map.IntegerMapLinkedArrayList;
5+
import net.lecousin.framework.core.test.collections.maps.TestIntegerMap;
6+
7+
public class TestIntegerMapLinkedArrayList extends TestIntegerMap {
8+
9+
@Override
10+
public IntegerMap<Object> createIntegerMap() {
11+
return new IntegerMapLinkedArrayList<Object>(20, 10);
12+
}
13+
14+
}

net.lecousin.core/src/test/java/net/lecousin/framework/core/tests/collections/sort/TestRedBlackTreeInteger.java

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -93,6 +93,24 @@ public void tests() {
9393
Assert.assertEquals(0, tree.size());
9494
Assert.assertFalse(tree.containsInstance(51, o));
9595

96+
tree.add(51, o);
97+
tree.add(52, o);
98+
Assert.assertEquals(2, tree.size());
99+
Assert.assertTrue(tree.containsInstance(51, o));
100+
Assert.assertTrue(tree.containsInstance(52, o));
101+
Assert.assertNull(tree.getPrevious(51));
102+
Assert.assertNotNull(tree.getPrevious(52));
103+
Assert.assertEquals(51, tree.getPrevious(52).getValue());
104+
Assert.assertNotNull(tree.getNext(51));
105+
Assert.assertEquals(52, tree.getNext(51).getValue());
106+
Assert.assertNull(tree.getNext(52));
107+
tree.add(53, o);
108+
tree.removeKey(51);
109+
Assert.assertNull(tree.getPrevious(52));
110+
Assert.assertEquals(53, tree.getNext(52).getValue());
111+
Assert.assertNull(tree.getNext(53));
112+
113+
tree.clear();
96114
for (int i = 0; i < 100; ++i)
97115
tree.add(i, o);
98116
Assert.assertEquals(100, tree.size());

net.lecousin.core/src/test/java/net/lecousin/framework/core/tests/collections/sort/TestRedBlackTreeLong.java

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,24 @@ public void tests() {
9292
Assert.assertEquals(0, tree.size());
9393
Assert.assertFalse(tree.containsInstance(51, o));
9494

95+
tree.add(51, o);
96+
tree.add(52, o);
97+
Assert.assertEquals(2, tree.size());
98+
Assert.assertTrue(tree.containsInstance(51, o));
99+
Assert.assertTrue(tree.containsInstance(52, o));
100+
Assert.assertNull(tree.getPrevious(51));
101+
Assert.assertNotNull(tree.getPrevious(52));
102+
Assert.assertEquals(51, tree.getPrevious(52).getValue());
103+
Assert.assertNotNull(tree.getNext(51));
104+
Assert.assertEquals(52, tree.getNext(51).getValue());
105+
Assert.assertNull(tree.getNext(52));
106+
tree.add(53, o);
107+
tree.removeKey(51);
108+
Assert.assertNull(tree.getPrevious(52));
109+
Assert.assertEquals(53, tree.getNext(52).getValue());
110+
Assert.assertNull(tree.getNext(53));
111+
112+
tree.clear();
95113
for (int i = 0; i < 100; ++i)
96114
tree.add(i, o);
97115
Assert.assertEquals(100, tree.size());

net.lecousin.core/src/test/java/net/lecousin/framework/core/tests/concurrent/TestDirectoryTasks.java

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
import net.lecousin.framework.concurrent.tasks.drives.RemoveDirectoryContentTask;
1515
import net.lecousin.framework.concurrent.tasks.drives.RemoveDirectoryTask;
1616
import net.lecousin.framework.core.test.LCCoreAbstractTest;
17+
import net.lecousin.framework.progress.FakeWorkProgress;
1718

1819
public class TestDirectoryTasks extends LCCoreAbstractTest {
1920

@@ -55,21 +56,24 @@ public void testDirectoryReader() throws Exception {
5556
Assert.assertTrue(dir.mkdir());
5657
File file = new File(root.toFile(), "titi");
5758
Assert.assertTrue(file.createNewFile());
59+
5860
DirectoryReader.Request request = new DirectoryReader.Request();
59-
request.getCreation = true;
60-
request.getIsSymbolicLink = true;
61-
request.getLastAccess = true;
62-
request.getLastModified = true;
63-
request.getSize = true;
61+
request.getCreation().getIsSymbolicLink().getLastAccess().getLastModified().getSize();
6462
DirectoryReader reader = new DirectoryReader(root.toFile(), Task.PRIORITY_NORMAL, request);
6563
Assert.assertEquals(root.toFile(), reader.getDirectory());
6664
reader.start().getOutput().blockThrow(0);
65+
6766
DirectoryReader.ListSubDirectories lister = new DirectoryReader.ListSubDirectories(root.toFile(), Task.PRIORITY_NORMAL);
6867
lister.start().getOutput().blockThrow(0);
68+
6969
new RemoveDirectoryContentTask(root.toFile(), null, 0, Task.PRIORITY_NORMAL, false).start().getOutput().blockThrow(0);
7070
request = new DirectoryReader.Request();
7171
reader = new DirectoryReader(root.toFile(), Task.PRIORITY_NORMAL, request);
7272
reader.start().getOutput().blockThrow(0);
73+
74+
request = new DirectoryReader.Request();
75+
reader = new DirectoryReader(root.toFile(), Task.PRIORITY_NORMAL, request, new FakeWorkProgress());
76+
reader.start().getOutput().blockThrow(0);
7377
}
7478

7579
}

net.lecousin.core/src/test/java/net/lecousin/framework/core/tests/math/TestFragmentedRangeLong.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,12 @@ public void test() throws Exception {
5959
// 9-31
6060
f.addValue(31);
6161
Assert.assertEquals(1, f.size());
62+
63+
Assert.assertFalse(f.containsOneValueIn(new RangeLong(0, 8)));
64+
Assert.assertFalse(f.containsOneValueIn(new RangeLong(35, 50)));
65+
Assert.assertTrue(f.containsOneValueIn(new RangeLong(10, 50)));
66+
Assert.assertTrue(f.containsOneValueIn(new RangeLong(5, 9)));
67+
Assert.assertTrue(f.containsOneValueIn(new RangeLong(31, 50)));
6268

6369
// 9-31, 100-150, 200-250
6470
f.addRanges(Arrays.asList(new RangeLong(100, 150), new RangeLong(200, 250)));

0 commit comments

Comments
 (0)