http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/AsynchronousDataStoreTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/AsynchronousDataStoreTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/AsynchronousDataStoreTest.java index 3c02261..40da190 100644 --- a/wicket-core/src/test/java/org/apache/wicket/pageStore/AsynchronousDataStoreTest.java +++ b/wicket-core/src/test/java/org/apache/wicket/pageStore/AsynchronousDataStoreTest.java @@ -21,25 +21,22 @@ import java.util.concurrent.CountDownLatch; import java.util.concurrent.ExecutorService; import java.util.concurrent.Executors; -import org.apache.wicket.versioning.InMemoryPageStore; +import org.apache.wicket.MockPage; +import org.apache.wicket.page.IManageablePage; import org.junit.jupiter.api.Test; /** - * Tests for {@link AsynchronousDataStore} + * Tests for {@link AsynchronousPageStore} */ public class AsynchronousDataStoreTest { -// private static final IDataStore WRAPPED_DATA_STORE = new DiskDataStore("asyncDataStoreApp", new StoreSettings(null).getFileStoreFolder(), Bytes.kilobytes(1)); - private static final IDataStore WRAPPED_DATA_STORE = new InMemoryPageStore(); + private static final IPageStore WRAPPED_PAGE_STORE = new InMemoryPageStore("test", Integer.MAX_VALUE); /** the data store under test */ - private static final IDataStore DATA_STORE = new AsynchronousDataStore(WRAPPED_DATA_STORE, 100); - - /** the data for each page */ - private static final byte[] DATA = new byte[] { 1, 2, 3 }; + private static final IPageStore ASYNC_PAGE_STORE = new AsynchronousPageStore(WRAPPED_PAGE_STORE, 100); /** the used jsessionid's */ - private static final String[] SESSIONS = new String[] { "s1", "s2", "s3" }; + private static final IPageContext[] CONTEXT = new IPageContext[] { createContext("s1"), createContext("s2"), createContext("s3")}; /** the ids for the stored/removed pages */ private static final int[] PAGE_IDS = new int[] { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 }; @@ -57,13 +54,13 @@ public class AsynchronousDataStoreTest private static final SecureRandom RND = new SecureRandom(); /** - * Executes random mutator and accessor operations on {@link AsynchronousDataStore} validating + * Executes random mutator and accessor operations on {@link AsynchronousPageStore} validating * that the used data structures can be used simultaneously. * * @throws Exception */ @Test - void randomOperations() throws Exception + void randomOperations() throws Exception { ExecutorService executorService = Executors.newFixedThreadPool(50); @@ -74,7 +71,12 @@ public class AsynchronousDataStoreTest } LATCH.await(); executorService.shutdown(); - DATA_STORE.destroy(); + ASYNC_PAGE_STORE.destroy(); + } + + private static IPageContext createContext(String sessionId) + { + return new DummyPageContext(sessionId); } private static abstract class AbstractTask implements Runnable @@ -94,15 +96,20 @@ public class AsynchronousDataStoreTest } } - String getSessionId() + protected IPageContext getPageContext() { - return SESSIONS[RND.nextInt(SESSIONS.length)]; + return CONTEXT[RND.nextInt(CONTEXT.length)]; } - int getPageId() + protected int getPageId() { return PAGE_IDS[RND.nextInt(PAGE_IDS.length)]; } + + protected IManageablePage getPage() + { + return new MockPage(getPageId()); + } } private static class StoreTask extends AbstractTask @@ -110,7 +117,7 @@ public class AsynchronousDataStoreTest @Override public void r() { - DATA_STORE.storeData(getSessionId(), getPageId(), DATA); + ASYNC_PAGE_STORE.addPage(getPageContext(), getPage()); } } @@ -119,7 +126,7 @@ public class AsynchronousDataStoreTest @Override public void r() { - DATA_STORE.getData(getSessionId(), getPageId()); + ASYNC_PAGE_STORE.getPage(getPageContext(), getPageId()); } } @@ -128,7 +135,7 @@ public class AsynchronousDataStoreTest @Override public void r() { - DATA_STORE.removeData(getSessionId(), getPageId()); + ASYNC_PAGE_STORE.removePage(getPageContext(), getPage()); } } @@ -137,7 +144,7 @@ public class AsynchronousDataStoreTest @Override public void r() { - DATA_STORE.removeData(getSessionId()); + ASYNC_PAGE_STORE.removeAllPages(getPageContext()); } } }
http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/AsynchronousPageStoreTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/AsynchronousPageStoreTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/AsynchronousPageStoreTest.java index b4fb451..0717230 100644 --- a/wicket-core/src/test/java/org/apache/wicket/pageStore/AsynchronousPageStoreTest.java +++ b/wicket-core/src/test/java/org/apache/wicket/pageStore/AsynchronousPageStoreTest.java @@ -17,20 +17,23 @@ package org.apache.wicket.pageStore; import static org.junit.jupiter.api.Assertions.assertEquals; -import static org.junit.jupiter.api.Assertions.assertNotEquals; import static org.junit.jupiter.api.Assertions.assertTrue; -import static org.mockito.Mockito.never; -import static org.mockito.Mockito.spy; -import static org.mockito.Mockito.times; -import static org.mockito.Mockito.verify; +import static org.junit.jupiter.api.Assertions.fail; import java.io.IOException; +import java.io.Serializable; import java.util.ArrayList; import java.util.List; import java.util.concurrent.CountDownLatch; +import java.util.concurrent.Semaphore; import java.util.concurrent.TimeUnit; +import java.util.concurrent.atomic.AtomicBoolean; import org.apache.commons.lang3.RandomUtils; +import org.apache.wicket.MetaDataKey; +import org.apache.wicket.MockPage; +import org.apache.wicket.WicketRuntimeException; +import org.apache.wicket.mock.MockPageStore; import org.apache.wicket.page.IManageablePage; import org.apache.wicket.serialize.ISerializer; import org.apache.wicket.serialize.java.DeflatedJavaSerializer; @@ -41,6 +44,7 @@ import org.junit.jupiter.api.Tag; import org.junit.jupiter.api.Test; import org.slf4j.Logger; import org.slf4j.LoggerFactory; + import com.google.common.base.Stopwatch; @@ -50,7 +54,7 @@ import com.google.common.base.Stopwatch; * @author manuelbarzi */ @Tag(WicketTestTag.SLOW) -class AsynchronousPageStoreTest +public class AsynchronousPageStoreTest { /** Log for reporting. */ private static final Logger log = LoggerFactory.getLogger(AsynchronousPageStoreTest.class); @@ -155,31 +159,51 @@ class AsynchronousPageStoreTest @Test void storeReturnsSameInstanceOnClosePageRequest() throws InterruptedException { + final Semaphore semaphore = new Semaphore(0); + + IPageStore store = new NoopPageStore() { + + @Override + public synchronized void addPage(IPageContext context, IManageablePage page) + { + try + { + // wait until the page was get below + semaphore.acquire(); + } + catch (InterruptedException e) + { + } + + super.addPage(context, page); + } + + @Override + public IManageablePage getPage(IPageContext context, int id) + { + fail(); + return null; + } + }; - ISerializer serializer = new DeflatedJavaSerializer("applicationKey"); - // ISerializer serializer = new DummySerializer(); - - IDataStore dataStore = new DiskDataStore("applicationName", new File("./target"), - Bytes.bytes(10000l)); - - // IPageStore pageStore = new DummyPageStore(new File("target/store")); - IPageStore pageStore = spy(new DefaultPageStore(serializer, dataStore, 0)); - - IPageStore asyncPageStore = new AsynchronousPageStore(pageStore, 100); + IPageStore asyncPageStore = new AsynchronousPageStore(store, 100); int pageId = 0; + String sessionId = "sessionId"; + + IPageContext context = new DummyPageContext(sessionId); DummyPage page = new DummyPage(pageId, 1000, 1000, sessionId); - asyncPageStore.storePage(sessionId, page); + asyncPageStore.addPage(context, page); - Thread.sleep(500); + IManageablePage pageBack = asyncPageStore.getPage(context, pageId); - IManageablePage pageBack = asyncPageStore.getPage(sessionId, pageId); - - verify(pageStore, never()).getPage(sessionId, pageId); + semaphore.release(); assertEquals(page, pageBack); + + store.destroy(); } /** @@ -191,31 +215,55 @@ class AsynchronousPageStoreTest @Test void storeReturnsRestoredInstanceOnDistantPageRequest() throws InterruptedException { + final Semaphore semaphore = new Semaphore(0); + + final AtomicBoolean got = new AtomicBoolean(false); + + IPageStore store = new MockPageStore() { + + @Override + public synchronized void addPage(IPageContext context, IManageablePage page) + { + super.addPage(context, page); + + semaphore.release(); + } + + @Override + public IManageablePage getPage(IPageContext context, int id) + { + got.set(true); + + return super.getPage(context, id); + } + }; - ISerializer serializer = new DeflatedJavaSerializer("applicationKey"); - // ISerializer serializer = new DummySerializer(); - - IDataStore dataStore = new DiskDataStore("applicationName", new File("./target"), - Bytes.bytes(10000l)); - - // IPageStore pageStore = new DummyPageStore(new File("target/store")); - IPageStore pageStore = spy(new DefaultPageStore(serializer, dataStore, 0)); - - IPageStore asyncPageStore = new AsynchronousPageStore(pageStore, 100); + IPageStore asyncPageStore = new AsynchronousPageStore(store, 100); int pageId = 0; + String sessionId = "sessionId"; + + IPageContext context = new DummyPageContext(sessionId); DummyPage page = new DummyPage(pageId, 1000, 1000, sessionId); - asyncPageStore.storePage(sessionId, page); + asyncPageStore.addPage(context, page); - Thread.sleep(1500); + try + { + semaphore.acquire(); + } + catch (InterruptedException e) + { + } - IManageablePage pageBack = asyncPageStore.getPage(sessionId, pageId); + IManageablePage pageBack = asyncPageStore.getPage(context, pageId); - verify(pageStore, times(1)).getPage(sessionId, pageId); + semaphore.release(); - assertNotEquals(page, pageBack); + assertEquals(page, pageBack); + + store.destroy(); } /** @@ -288,6 +336,121 @@ class AsynchronousPageStoreTest assertTrue(sync > 0); } + private MetaDataKey<Serializable> KEY1 = new MetaDataKey<Serializable>() + { + }; + + private MetaDataKey<Serializable> KEY2 = new MetaDataKey<Serializable>() + { + }; + + /** + * Store does not allow modifications when pages are added asynchronously. + */ + @Test + public void storeAsynchronousContextClosed() throws InterruptedException + { + IPageStore store = new MockPageStore() { + + @Override + public boolean canBeAsynchronous(IPageContext context) + { + // can bind and get session id + context.bind(); + context.getSessionId(); + + // can access request data + context.getRequestData(KEY1); + context.setRequestData(KEY1, "value1"); + + // can access session data + context.getSessionData(KEY1); + context.setSessionData(KEY1, "value1"); + + // can access session + context.getSessionAttribute("key1"); + context.setSessionAttribute("key1", "value1"); + + return true; + } + + @Override + public synchronized void addPage(IPageContext context, IManageablePage page) + { + // can bind and get session id + context.bind(); + context.getSessionId(); + + // cannot access request + try { + context.getRequestData(KEY1); + fail(); + } catch (WicketRuntimeException expected) { + } + try { + context.setRequestData(KEY1, "value1"); + fail(); + } catch (WicketRuntimeException expected) { + } + try { + context.getRequestData(KEY2); + fail(); + } catch (WicketRuntimeException expected) { + } + try { + context.setRequestData(KEY2, "value2"); + fail(); + } catch (WicketRuntimeException expected) { + } + + // can read session data + context.getSessionData(KEY1); + context.getSessionData(KEY2); + // .. but cannot set + try { + context.setSessionData(KEY1, "value1"); + fail(); + } catch (WicketRuntimeException expected) { + } + try { + context.setSessionData(KEY2, "value2"); + fail(); + } catch (WicketRuntimeException expected) { + } + + // can read session already read + context.getSessionAttribute("key1"); + // .. but nothing new + try { + context.getSessionAttribute("key2"); + fail(); + } catch (WicketRuntimeException expected) { + } + // .. but cannot set + try { + context.setSessionAttribute("key1", "value1"); + fail(); + } catch (WicketRuntimeException expected) { + } + try { + context.setSessionAttribute("key2", "value2"); + fail(); + } catch (WicketRuntimeException expected) { + } + } + }; + + IPageStore asyncPageStore = new AsynchronousPageStore(store, 100); + + MockPage page = new MockPage(); + + IPageContext context = new DummyPageContext(); + + asyncPageStore.addPage(context , page); + + store.destroy(); + } + // test run private class Metrics @@ -315,18 +478,13 @@ class AsynchronousPageStoreTest // ISerializer serializer = new DummySerializer(); ISerializer serializer = new DeflatedJavaSerializer("applicationKey"); - IDataStore dataStore = new DiskDataStore("applicationName", new File("./target"), - Bytes.bytes(10000l)); - - // IPageStore pageStore = new DummyPageStore(new File("target/store")) { - IPageStore pageStore = new DefaultPageStore(serializer, dataStore, 0) - { - + IPageStore pageStore = new DiskPageStore("applicationName", new File("./target"), + Bytes.bytes(10000l), serializer) { @Override - public void storePage(String sessionId, IManageablePage page) + public void addPage(IPageContext context, IManageablePage page) { - super.storePage(sessionId, page); + super.addPage(context, page); lock.countDown(); } @@ -341,20 +499,21 @@ class AsynchronousPageStoreTest for (int i = 1; i <= sessions; i++) { String sessionId = String.valueOf(i); + IPageContext context = new DummyPageContext(sessionId); Metrics metrics = new Metrics(); stopwatch.reset(); DummyPage page = new DummyPage(pageId, around(writeMillis), around(readMillis), sessionId); stopwatch.start(); - asyncPageStore.storePage(sessionId, page); + asyncPageStore.addPage(context, page); metrics.storedPage = page; metrics.storingMillis = stopwatch.elapsed(TimeUnit.MILLISECONDS); stopwatch.reset(); stopwatch.start(); metrics.restoredPage = DummyPage.class - .cast(asyncPageStore.getPage(sessionId, pageId)); + .cast(asyncPageStore.getPage(context, pageId)); metrics.restoringMillis = stopwatch.elapsed(TimeUnit.MILLISECONDS); results.add(metrics); @@ -363,6 +522,8 @@ class AsynchronousPageStoreTest lock.await(pages * sessions * (writeMillis + readMillis), TimeUnit.MILLISECONDS); + pageStore.destroy(); + return results; } http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/DefaultPageStoreTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/DefaultPageStoreTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/DefaultPageStoreTest.java deleted file mode 100644 index 2ffa376..0000000 --- a/wicket-core/src/test/java/org/apache/wicket/pageStore/DefaultPageStoreTest.java +++ /dev/null @@ -1,31 +0,0 @@ -/* - * Licensed to the Apache Software Foundation (ASF) under one or more - * contributor license agreements. See the NOTICE file distributed with - * this work for additional information regarding copyright ownership. - * The ASF licenses this file to You under the Apache License, Version 2.0 - * (the "License"); you may not use this file except in compliance with - * the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ -package org.apache.wicket.pageStore; - -import org.apache.wicket.serialize.ISerializer; - -/** - * Tests for DefaultPageStore - */ -public class DefaultPageStoreTest extends AbstractPageStoreTest -{ - @Override - protected IPageStore createPageStore(ISerializer serializer, IDataStore dataStore, int maxEntries) - { - return new DefaultPageStore(serializer, dataStore, maxEntries); - } -} http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/DiskDataStoreTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/DiskDataStoreTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/DiskDataStoreTest.java deleted file mode 100644 index 9187f79..0000000 --- a/wicket-core/src/test/java/org/apache/wicket/pageStore/DiskDataStoreTest.java +++ /dev/null @@ -1,413 +0,0 @@ -/* - * Licensed to the Apache Software Foundation (ASF) under one or more - * contributor license agreements. See the NOTICE file distributed with - * this work for additional information regarding copyright ownership. - * The ASF licenses this file to You under the Apache License, Version 2.0 - * (the "License"); you may not use this file except in compliance with - * the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ -package org.apache.wicket.pageStore; - -import static org.junit.jupiter.api.Assertions.assertEquals; -import static org.junit.jupiter.api.Assertions.assertFalse; -import static org.junit.jupiter.api.Assertions.assertTrue; - -import java.util.ArrayList; -import java.util.List; -import java.util.Map; -import java.util.Random; -import java.util.UUID; -import java.util.concurrent.ConcurrentHashMap; -import java.util.concurrent.ConcurrentLinkedQueue; -import java.util.concurrent.atomic.AtomicBoolean; -import java.util.concurrent.atomic.AtomicInteger; - -import org.apache.wicket.settings.StoreSettings; -import org.apache.wicket.util.WicketTestTag; -import org.apache.wicket.util.lang.Bytes; -import org.junit.jupiter.api.Tag; -import org.junit.jupiter.api.Test; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; - -/** - */ -@Tag(WicketTestTag.SLOW) -public class DiskDataStoreTest -{ - /** Log for reporting. */ - private static final Logger log = LoggerFactory.getLogger(DiskDataStoreTest.class); - - /** - * Construct. - */ - DiskDataStoreTest() - { - } - - private static final Random random = new Random(); - private static final int FILE_SIZE_MIN = 1024 * 200; - private static final int FILE_SIZE_MAX = 1024 * 300; - private static final Bytes MAX_SIZE_PER_SESSION = Bytes.megabytes(10); - private static final int SESSION_COUNT = 50; - private static final int FILES_COUNT = 1000; - private static final int SLEEP_MAX = 10; - private static final int THREAD_COUNT = 20; - private static final int READ_MODULO = 100; - - private static class File - { - private final String sessionId; - private final int id; - - private byte first; - private byte last; - private int length; - - File(String sessionId, int id) - { - this.sessionId = sessionId; - this.id = id; - } - - String getSessionId() - { - return sessionId; - } - - int getId() - { - return id; - } - - byte[] generateData() - { - length = FILE_SIZE_MIN + random.nextInt(FILE_SIZE_MAX - FILE_SIZE_MIN); - byte data[] = new byte[length]; - random.nextBytes(data); - first = data[0]; - last = data[data.length - 1]; - return data; - } - - boolean checkData(byte data[]) - { - if (data == null) - { - log.error("data[] should never be null"); - return false; - } - if (data.length != length) - { - log.error("data.length != length"); - return false; - } - if (first != data[0]) - { - log.error("first != data[0]"); - return false; - } - if (last != data[data.length - 1]) - { - log.error("last != data[data.length - 1]"); - return false; - } - return true; - } - } - - private final Map<String, AtomicInteger> sessionCounter = new ConcurrentHashMap<String, AtomicInteger>(); - private final ConcurrentLinkedQueue<File> filesToSave = new ConcurrentLinkedQueue<File>(); - private final ConcurrentLinkedQueue<File> filesToRead1 = new ConcurrentLinkedQueue<File>(); - private final ConcurrentLinkedQueue<File> filesToRead2 = new ConcurrentLinkedQueue<File>(); - - private final AtomicInteger read1Count = new AtomicInteger(0); - private final AtomicInteger read2Count = new AtomicInteger(0); - private final AtomicInteger saveCount = new AtomicInteger(0); - - private final AtomicBoolean saveDone = new AtomicBoolean(false); - private final AtomicBoolean read1Done = new AtomicBoolean(false); - private final AtomicBoolean read2Done = new AtomicBoolean(false); - - private final AtomicInteger failures = new AtomicInteger(); - - private final AtomicInteger bytesWritten = new AtomicInteger(0); - private final AtomicInteger bytesRead = new AtomicInteger(0); - - private final AtomicInteger saveTime = new AtomicInteger(0); - - private RuntimeException exceptionThrownByThread; - - private String randomSessionId() - { - List<String> s = new ArrayList<String>(sessionCounter.keySet()); - return s.get(random.nextInt(s.size())); - } - - private int nextSessionId(String sessionId) - { - AtomicInteger i = sessionCounter.get(sessionId); - return i.incrementAndGet(); - } - - private void generateFiles() - { - for (int i = 0; i < SESSION_COUNT; ++i) - { - sessionCounter.put(UUID.randomUUID().toString(), new AtomicInteger(0)); - } - for (int i = 0; i < FILES_COUNT; ++i) - { - String session = randomSessionId(); - File file = new File(session, nextSessionId(session)); - long now = System.nanoTime(); - filesToSave.add(file); - long duration = System.nanoTime() - now; - saveTime.addAndGet((int)duration); - } - } - - private IDataStore dataStore; - - /** - * Stores RuntimeException into a field. - */ - private abstract class ExceptionCapturingRunnable implements Runnable - { - @Override - public final void run() - { - try - { - doRun(); - } - catch (RuntimeException e) - { - exceptionThrownByThread = e; - } - } - - /** - * Called by {@link #run()}. Thrown RuntimeExceptions are stores into a field for later - * check. - */ - protected abstract void doRun(); - } - - // Store/Save data in DataStore - private class SaveRunnable extends ExceptionCapturingRunnable - { - @Override - protected void doRun() - { - File file; - - while ((file = filesToSave.poll()) != null || saveCount.get() < FILES_COUNT) - { - if (file != null) - { - byte data[] = file.generateData(); - dataStore.storeData(file.getSessionId(), file.getId(), data); - - if (saveCount.get() % READ_MODULO == 0) - { - filesToRead1.add(file); - } - saveCount.incrementAndGet(); - bytesWritten.addAndGet(data.length); - } - - try - { - Thread.sleep(random.nextInt(SLEEP_MAX)); - } - catch (InterruptedException e) - { - log.error(e.getMessage(), e); - } - } - - saveDone.set(true); - } - }; - - // Read data from DataStore - private class Read1Runnable extends ExceptionCapturingRunnable - { - @Override - protected void doRun() - { - File file; - while ((file = filesToRead1.poll()) != null || !saveDone.get()) - { - if (file != null) - { - byte bytes[] = dataStore.getData(file.getSessionId(), file.getId()); - if (file.checkData(bytes) == false) - { - failures.incrementAndGet(); - log.error("Detected error number: " + failures.get()); - } - filesToRead2.add(file); - read1Count.incrementAndGet(); - bytesRead.addAndGet(bytes.length); - } - - try - { - Thread.sleep(random.nextInt(SLEEP_MAX)); - } - catch (InterruptedException e) - { - log.error(e.getMessage(), e); - } - } - - read1Done.set(true); - } - }; - - private class Read2Runnable extends ExceptionCapturingRunnable - { - @Override - protected void doRun() - { - File file; - while ((file = filesToRead2.poll()) != null || !read1Done.get()) - { - if (file != null) - { - byte bytes[] = dataStore.getData(file.getSessionId(), file.getId()); - if (file.checkData(bytes) == false) - { - failures.incrementAndGet(); - log.error("Detected error number: " + failures.get()); - } - read2Count.incrementAndGet(); - bytesRead.addAndGet(bytes.length); - } - - try - { - Thread.sleep(random.nextInt(SLEEP_MAX)); - } - catch (InterruptedException e) - { - log.error(e.getMessage(), e); - } - } - - read2Done.set(true); - } - } - - private void doTestDataStore() - { - log.info("Starting..."); - long start = System.currentTimeMillis(); - - for (int i = 0; i < THREAD_COUNT; ++i) - { - new Thread(new Read1Runnable()).start(); - } - - for (int i = 0; i < THREAD_COUNT; ++i) - { - new Thread(new Read2Runnable()).start(); - } - - for (int i = 0; i < THREAD_COUNT; ++i) - { - new Thread(new SaveRunnable()).start(); - } - - while (!(read1Done.get() && read2Done.get() && saveDone.get())) - { - try - { - Thread.sleep(50); - } - catch (InterruptedException e) - { - log.error(e.getMessage(), e); - } - } - - if (exceptionThrownByThread != null) - { - throw new RuntimeException("One of the worker threads failed.", exceptionThrownByThread); - } - - long duration = System.currentTimeMillis() - start; - - log.info("Took: " + duration + " ms"); - log.info("Save: " + saveCount.intValue() + " files, " + bytesWritten.get() + " bytes"); - log.info("Read: " + (read1Count.get() + read2Count.get()) + " files, " + bytesRead.get() + - " bytes"); - - log.info("Average save time (ns): " + (double)saveTime.get() / (double)saveCount.get()); - - assertEquals(0, failures.get()); - - for (String s : sessionCounter.keySet()) - { - dataStore.removeData(s); - } - } - - /** - * store() - */ - @Test - void store() - { - generateFiles(); - - StoreSettings storeSettings = new StoreSettings(null); - java.io.File fileStoreFolder = storeSettings.getFileStoreFolder(); - - dataStore = new DiskDataStore("app1", fileStoreFolder, MAX_SIZE_PER_SESSION); - int asynchronousQueueCapacity = storeSettings.getAsynchronousQueueCapacity(); - dataStore = new AsynchronousDataStore(dataStore, asynchronousQueueCapacity); - - doTestDataStore(); - - dataStore.destroy(); - } - - /** - * https://issues.apache.org/jira/browse/WICKET-4478 - * - * Tests that the folder where a session data is put is partitioned, i.e. - * it is put in folders which names are automatically calculated on the fly. - */ - @Test - void sessionFolderName() - { - StoreSettings storeSettings = new StoreSettings(null); - java.io.File fileStoreFolder = storeSettings.getFileStoreFolder(); - DiskDataStore store = new DiskDataStore("sessionFolderName", fileStoreFolder, MAX_SIZE_PER_SESSION); - - String sessionId = "abcdefg"; - java.io.File sessionFolder = store.getSessionFolder(sessionId, true); - String absolutePath = sessionFolder.getAbsolutePath(); - assertTrue(absolutePath.contains("sessionFolderName-filestore")); - assertTrue(absolutePath.contains("7141")); - assertTrue(absolutePath.contains("1279")); - assertTrue(absolutePath.contains("abcdefg")); - - DiskDataStore.SessionEntry sessionEntry = new DiskDataStore.SessionEntry(store, sessionId); - sessionEntry.unbind(); - // assert that the 'sessionId' folder and the parents two levels up are removed - assertFalse(sessionFolder.getParentFile().getParentFile().exists()); - - } -} http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/DiskPageStoreTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/DiskPageStoreTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/DiskPageStoreTest.java new file mode 100644 index 0000000..dc9d534 --- /dev/null +++ b/wicket-core/src/test/java/org/apache/wicket/pageStore/DiskPageStoreTest.java @@ -0,0 +1,52 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.wicket.pageStore; + +import java.io.File; +import java.io.IOException; +import java.nio.file.Files; + +import org.apache.wicket.serialize.java.JavaSerializer; +import org.apache.wicket.util.WicketTestTag; +import org.apache.wicket.util.lang.Bytes; +import org.junit.jupiter.api.Tag; +import org.junit.jupiter.api.Test; + +/** + * Test for {@link DiskPageStore}. + */ +@Tag(WicketTestTag.SLOW) +public class DiskPageStoreTest extends AbstractConcurrentPageStoreTest +{ + + private static final Bytes MAX_SIZE_PER_SESSION = Bytes.megabytes(10); + + /** + * @throws IOException + */ + @Test + void store() throws IOException + { + File fileStoreFolder = Files.createTempDirectory(null).toFile(); + + IPageStore pageStore = new DiskPageStore("app1", fileStoreFolder, MAX_SIZE_PER_SESSION, new JavaSerializer("app1")); + + doTestStore(pageStore); + + pageStore.destroy(); + } +} \ No newline at end of file http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/DummyPageContext.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/DummyPageContext.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/DummyPageContext.java new file mode 100644 index 0000000..88fbb06 --- /dev/null +++ b/wicket-core/src/test/java/org/apache/wicket/pageStore/DummyPageContext.java @@ -0,0 +1,110 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.wicket.pageStore; + +import java.io.Serializable; +import java.util.HashMap; +import java.util.Map; + +import org.apache.wicket.MetaDataEntry; +import org.apache.wicket.MetaDataKey; +import org.apache.wicket.pageStore.IPageContext; + +/** + * Dummy implementation of a page context - suitable for a single session only. + */ +public class DummyPageContext implements IPageContext +{ + + final String sessionId; + + MetaDataEntry<?>[] requestData; + + MetaDataEntry<?>[] sessionData; + + Map<String, Object> sessionAttributes = new HashMap<>(); + + public DummyPageContext() + { + this("dummy_id"); + } + + public DummyPageContext(String sessionId) + { + this.sessionId = sessionId; + } + + @Override + public <T> void setRequestData(MetaDataKey<T> key, T value) + { + requestData = key.set(requestData, value); + } + + @Override + public <T> T getRequestData(MetaDataKey<T> key) + { + return key.get(requestData); + } + + @Override + public <T extends Serializable> void setSessionAttribute(String key, T value) + { + sessionAttributes.put(key, value); + } + + @SuppressWarnings("unchecked") + @Override + public <T extends Serializable> T getSessionAttribute(String key) + { + return (T)sessionAttributes.get(key); + } + + @Override + public <T extends Serializable> T setSessionData(MetaDataKey<T> key, T value) + { + sessionData = key.set(sessionData, value); + + return value; + } + + @Override + public <T extends Serializable> T getSessionData(MetaDataKey<T> key) + { + return key.get(sessionData); + } + + @Override + public void bind() + { + } + + @Override + public String getSessionId() + { + return sessionId; + } + + public void clearRequest() + { + requestData = null; + } + + public void clearSession() { + sessionAttributes.clear(); + sessionData = null; + } +} \ No newline at end of file http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/FilePageStoreTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/FilePageStoreTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/FilePageStoreTest.java new file mode 100644 index 0000000..86f393a --- /dev/null +++ b/wicket-core/src/test/java/org/apache/wicket/pageStore/FilePageStoreTest.java @@ -0,0 +1,52 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.wicket.pageStore; + +import java.io.File; +import java.io.IOException; +import java.nio.file.Files; + +import org.apache.wicket.serialize.java.JavaSerializer; +import org.apache.wicket.util.WicketTestTag; +import org.apache.wicket.util.lang.Bytes; +import org.junit.jupiter.api.Tag; +import org.junit.jupiter.api.Test; + +/** + * Test for {@link FilePageStore}. + */ +@Tag(WicketTestTag.SLOW) +public class FilePageStoreTest extends AbstractConcurrentPageStoreTest +{ + + private static final Bytes MAX_SIZE_PER_SESSION = Bytes.megabytes(10); + + /** + * @throws IOException + */ + @Test + void store() throws IOException + { + File folder = Files.createTempDirectory(null).toFile(); + + IPageStore pageStore = new FilePageStore("app1", folder, MAX_SIZE_PER_SESSION, new JavaSerializer("app1")); + + doTestStore(pageStore); + + pageStore.destroy(); + } +} \ No newline at end of file http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/GroupingPageStoreTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/GroupingPageStoreTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/GroupingPageStoreTest.java new file mode 100644 index 0000000..e96aa92 --- /dev/null +++ b/wicket-core/src/test/java/org/apache/wicket/pageStore/GroupingPageStoreTest.java @@ -0,0 +1,112 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.wicket.pageStore; + +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertFalse; +import static org.junit.jupiter.api.Assertions.assertTrue; + +import java.io.Serializable; + +import org.apache.wicket.MetaDataKey; +import org.apache.wicket.MockPage; +import org.apache.wicket.mock.MockPageStore; +import org.apache.wicket.page.IManageablePage; +import org.junit.jupiter.api.Test; + +/** + * Test for {@link GroupingPageStore}. + * + * @author svenmeier + */ +public class GroupingPageStoreTest +{ + + private static MetaDataKey<Serializable> KEY = new MetaDataKey<Serializable>() + { + }; + + private static Serializable VALUE = new Serializable() + { + }; + + @Test + void test() + { + String sessionId = "foo"; + + IPageStore store = new MockPageStore() { + + public void addPage(IPageContext context, IManageablePage page) { + + context.setSessionAttribute("attribute", "value"); + context.setSessionData(KEY, VALUE); + + assertEquals(sessionId + "_" + group(page), context.getSessionId()); + + super.addPage(context, page); + } + + @Override + public void removeAllPages(IPageContext context) + { + assertEquals(sessionId + "_group1", context.getSessionId()); + + super.removeAllPages(context); + } + }; + + IPageStore groupingStore = new GroupingPageStore(store, 2) { + @Override + protected String getGroup(IManageablePage page) + { + return group(page); + } + }; + + DummyPageContext context = new DummyPageContext(sessionId) { + @Override + public <T extends Serializable> T setSessionData(MetaDataKey<T> key, T value) + { + assertFalse(value == VALUE, "group session data not set directly in session"); + + return super.setSessionData(key, value); + } + + @Override + public <T extends Serializable> void setSessionAttribute(String key, T value) + { + assertTrue(key.startsWith("attribute_group"), "group session attribute starts with group"); + + super.setSessionAttribute(key, value); + } + }; + + groupingStore.addPage(context, new MockPage(0)); // group 0 + groupingStore.addPage(context, new MockPage(1)); // group 0 + groupingStore.addPage(context, new MockPage(10)); // group 1 + groupingStore.addPage(context, new MockPage(11)); // group 1 + groupingStore.addPage(context, new MockPage(2)); // group 0 + groupingStore.addPage(context, new MockPage(21)); // group 2, expels oldest group 1 + + } + + protected String group(IManageablePage page) + { + return "group" + page.getPageId() / 10; + } +} http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/InMemoryPageStoreTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/InMemoryPageStoreTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/InMemoryPageStoreTest.java new file mode 100644 index 0000000..ab1dcec --- /dev/null +++ b/wicket-core/src/test/java/org/apache/wicket/pageStore/InMemoryPageStoreTest.java @@ -0,0 +1,31 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.wicket.pageStore; + +import org.apache.wicket.serialize.ISerializer; + +/** + * Tests for {@link InMemoryPageStore} + */ +public class InMemoryPageStoreTest extends AbstractPageStoreTest +{ + @Override + protected IPageStore createPageStore(ISerializer serializer, int maxEntries) + { + return new InMemoryPageStore("test", maxEntries); + } +} http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/InSessionPageStoreTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/InSessionPageStoreTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/InSessionPageStoreTest.java new file mode 100644 index 0000000..4904c9b --- /dev/null +++ b/wicket-core/src/test/java/org/apache/wicket/pageStore/InSessionPageStoreTest.java @@ -0,0 +1,32 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.wicket.pageStore; + +import org.apache.wicket.serialize.ISerializer; + +/** + * Test for {@link InSessionPageStore}. + */ +public class InSessionPageStoreTest extends AbstractPageStoreTest +{ + @Override + protected IPageStore createPageStore(ISerializer serializer, int maxEntries) + { + return new InSessionPageStore(new NoopPageStore(), maxEntries, serializer); + } + +} http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/NoopDataStore.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/NoopDataStore.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/NoopDataStore.java deleted file mode 100644 index 8242d4f..0000000 --- a/wicket-core/src/test/java/org/apache/wicket/pageStore/NoopDataStore.java +++ /dev/null @@ -1,61 +0,0 @@ -/* - * Licensed to the Apache Software Foundation (ASF) under one or more - * contributor license agreements. See the NOTICE file distributed with - * this work for additional information regarding copyright ownership. - * The ASF licenses this file to You under the Apache License, Version 2.0 - * (the "License"); you may not use this file except in compliance with - * the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ -package org.apache.wicket.pageStore; - -/** - * An implementation of IDataStore that does nothing - */ -class NoopDataStore implements IDataStore -{ - @Override - public byte[] getData(String sessionId, int id) - { - return null; - } - - @Override - public void removeData(String sessionId, int id) - { - } - - @Override - public void removeData(String sessionId) - { - } - - @Override - public void storeData(String sessionId, int id, byte[] data) - { - } - - @Override - public void destroy() - { - } - - @Override - public boolean isReplicated() - { - return false; - } - - @Override - public boolean canBeAsynchronous() - { - return false; - } -} http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/PerSessionPageStoreTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/PerSessionPageStoreTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/PerSessionPageStoreTest.java deleted file mode 100644 index 7205659..0000000 --- a/wicket-core/src/test/java/org/apache/wicket/pageStore/PerSessionPageStoreTest.java +++ /dev/null @@ -1,55 +0,0 @@ -/* - * Licensed to the Apache Software Foundation (ASF) under one or more - * contributor license agreements. See the NOTICE file distributed with - * this work for additional information regarding copyright ownership. - * The ASF licenses this file to You under the Apache License, Version 2.0 - * (the "License"); you may not use this file except in compliance with - * the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ -package org.apache.wicket.pageStore; - -import static org.junit.jupiter.api.Assertions.assertNotNull; - -import org.apache.wicket.MockPage; -import org.apache.wicket.serialize.ISerializer; -import org.junit.jupiter.api.Test; - -/** - * Tests for PerSessionPageStore - */ -public class PerSessionPageStoreTest extends AbstractPageStoreTest -{ - @Override - IPageStore createPageStore(ISerializer serializer, IDataStore dataStore, int maxEntries) - { - return new PerSessionPageStore(serializer, dataStore, maxEntries); - } - - /** - * Verify that it is OK to store more pages than {@code maxEntries} - * if they are in different sessions - */ - @Test - @Override - void maxSizeDifferentSessions() - { - String sessionId2 = "0987654321"; - - pageStore.storePage(sessionId, new MockPage(pageId)); - - assertNotNull(pageStore.getPage(sessionId, pageId)); - - pageStore.storePage(sessionId2, new MockPage(pageId)); - - assertNotNull(pageStore.getPage(sessionId, pageId)); - assertNotNull(pageStore.getPage(sessionId2, pageId)); - } -} http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/RequestPageStoreTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/RequestPageStoreTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/RequestPageStoreTest.java new file mode 100644 index 0000000..b7fdf7a --- /dev/null +++ b/wicket-core/src/test/java/org/apache/wicket/pageStore/RequestPageStoreTest.java @@ -0,0 +1,64 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.wicket.pageStore; + +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertNull; +import static org.junit.jupiter.api.Assertions.assertTrue; + +import org.apache.wicket.MockPage; +import org.apache.wicket.mock.MockPageStore; +import org.junit.jupiter.api.Test; + +/** + * Test for {@link RequestPageStore} + * + * @author svenmeier + */ +public class RequestPageStoreTest +{ + + @Test + void test() + { + MockPageStore mockStore = new MockPageStore(); + + DummyPageContext context = new DummyPageContext(); + + RequestPageStore store = new RequestPageStore(mockStore); + + MockPage page1 = new MockPage(2); + MockPage page2 = new MockPage(3); + MockPage page3 = new MockPage(4); + + store.addPage(context, page1); + store.addPage(context, page2); + store.addPage(context, page3); + + assertTrue(mockStore.getPages().isEmpty(), "no pages delegated before detach"); + + store.detach(context); + + assertEquals(3, mockStore.getPages().size(), "pages delegated on detach"); + + mockStore.getPages().clear(); + + assertNull(store.getPage(context, 2), "no page in request store"); + assertNull(store.getPage(context, 3), "no page in request store"); + assertNull(store.getPage(context, 4), "no page in request store"); + } +} http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/SerializingPageStoreTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/SerializingPageStoreTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/SerializingPageStoreTest.java new file mode 100644 index 0000000..a72a514 --- /dev/null +++ b/wicket-core/src/test/java/org/apache/wicket/pageStore/SerializingPageStoreTest.java @@ -0,0 +1,52 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.wicket.pageStore; + +import static org.junit.jupiter.api.Assertions.assertEquals; + +import org.apache.wicket.MockPage; +import org.apache.wicket.mock.MockPageStore; +import org.apache.wicket.page.IManageablePage; +import org.apache.wicket.serialize.java.JavaSerializer; +import org.junit.jupiter.api.Test; + +/** + * Test for {@link SerializingPageStore}. + */ +public class SerializingPageStoreTest +{ + + @Test + void test() + { + DummyPageContext context = new DummyPageContext(); + + MockPageStore mockStore = new MockPageStore(); + + IManageablePage original = new MockPage(1); + + SerializingPageStore store = new SerializingPageStore(mockStore, new JavaSerializer("test")); + + store.addPage(context, original); + + SerializedPage serialized = (SerializedPage)mockStore.getPage(context, 1); + assertEquals(1, serialized.getPageId(), "page was serialized"); + + MockPage deserialized = (MockPage)store.getPage(context, 1); + assertEquals(1, deserialized.getPageId(), "page was deserialized"); + } +} http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/disk/NestedFoldersTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/disk/NestedFoldersTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/disk/NestedFoldersTest.java new file mode 100644 index 0000000..5d65390 --- /dev/null +++ b/wicket-core/src/test/java/org/apache/wicket/pageStore/disk/NestedFoldersTest.java @@ -0,0 +1,96 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.wicket.pageStore.disk; + +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertFalse; +import static org.junit.jupiter.api.Assertions.assertTrue; + +import java.io.File; +import java.io.IOException; +import java.nio.file.Files; + +import org.junit.jupiter.api.Test; + +/** + * Test for {@link NestedFolders}. + */ +public class NestedFoldersTest +{ + + @Test + public void escapedName() throws IOException + { + NestedFolders folders = new NestedFolders(Files.createTempDirectory(null).toFile()); + + File sessionFolder = folders.get("x*x/x:x", false); + + assertEquals("x_x_x_x", sessionFolder.getName()); + } + + /** + * https://issues.apache.org/jira/browse/WICKET-4478 + * + * Tests that the folder where a session data is put is partitioned, i.e. it is put in folders + * which names are automatically calculated on the fly. + * + * @throws IOException + */ + @Test + public void parentFoldersAreRemovedIfEmpty() throws IOException + { + NestedFolders folders = new NestedFolders(Files.createTempDirectory(null).toFile()); + + String sessionId = "abcdefg"; + File sessionFolder = folders.get(sessionId, true); + + assertEquals("abcdefg", sessionFolder.getName()); + assertEquals("1279", sessionFolder.getParentFile().getName()); + assertEquals("7141", sessionFolder.getParentFile().getParentFile().getName()); + assertEquals(folders.getBase(), sessionFolder.getParentFile().getParentFile().getParentFile()); + + folders.remove(sessionId); + + // assert that the 'sessionId' folder and the parents two levels up are removed + assertFalse(sessionFolder.getParentFile().getParentFile().exists()); + } + + @Test + public void parentFoldersAreKeptWhenNotEmpty() throws IOException + { + NestedFolders folders = new NestedFolders(Files.createTempDirectory(null).toFile()); + + String sessionId = "hijklmn"; + File sessionFolder = folders.get(sessionId, true); + + assertEquals("hijklmn", sessionFolder.getName()); + assertEquals("2326", sessionFolder.getParentFile().getName()); + assertEquals("7548", sessionFolder.getParentFile().getParentFile().getName()); + assertEquals(folders.getBase(), sessionFolder.getParentFile().getParentFile().getParentFile()); + + // create additional folder inside the nested folders + new File(sessionFolder.getParentFile().getParentFile(), "2345").mkdirs(); + + folders.remove(sessionId); + + // assert that the 'sessionId' folder and the parent one level up are removed + assertFalse(sessionFolder.getParentFile().exists()); + // ... but the parent two levels up still exists + assertTrue(sessionFolder.getParentFile().getParentFile().exists()); + + } +} http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/disk/PageWindowManagerTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/disk/PageWindowManagerTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/disk/PageWindowManagerTest.java new file mode 100644 index 0000000..714738c --- /dev/null +++ b/wicket-core/src/test/java/org/apache/wicket/pageStore/disk/PageWindowManagerTest.java @@ -0,0 +1,306 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.wicket.pageStore.disk; + +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertNotNull; +import static org.junit.jupiter.api.Assertions.assertNull; +import static org.junit.jupiter.api.Assertions.assertTrue; + +import java.security.SecureRandom; +import java.util.concurrent.CountDownLatch; +import java.util.concurrent.ExecutorService; +import java.util.concurrent.Executors; + +import org.apache.wicket.pageStore.disk.PageWindowManager.FileWindow; +import org.junit.jupiter.api.Test; + +/** + * Test for {@link PageWindowManager}. + * + * @author Matej Knopp + */ +public class PageWindowManagerTest +{ + /** + * https://issues.apache.org/jira/browse/WICKET-4572 + */ + @Test + void removeObsoleteIndices() + { + int page0id = 0, + page1id = 1, + page2id = 2; + int maxSize = 10; + + PageWindowManager manager = new PageWindowManager(maxSize); + + // Add few pages. + // All of them fully occupy the max space in the pageWindowManager. + // So adding N+1st page removes the Nth page. + manager.createPageWindow(page0id, "foo", maxSize); + FileWindow page0Window = manager.getPageWindow(page0id); + assertWindow(page0Window, page0id, page0Window.getFilePartOffset(), page0Window.getFilePartSize()); + + manager.createPageWindow(page1id, "foo", maxSize); + FileWindow page1Window = manager.getPageWindow(page1id); + assertWindow(page1Window, page1id, page1Window.getFilePartOffset(), page1Window.getFilePartSize()); + + // Try to get a page which has been lost with the adding of page1 + assertNull(manager.getPageWindow(page0id), "Page0 must be lost when Page1 has been added."); + + manager.createPageWindow(page2id, "foo", maxSize); + FileWindow page2Window = manager.getPageWindow(page2id); + assertWindow(page2Window, page2id, page2Window.getFilePartOffset(), page2Window.getFilePartSize()); + + // Try to get a page which has been lost with the adding of page2 + assertNull(manager.getPageWindow(page1id), "Page1 must be lost when Page2 has been added."); + } + + /** + * + */ + @Test + void addRemove() + { + PageWindowManager manager = new PageWindowManager(300); + FileWindow window; + + window = manager.createPageWindow(1, "foo", 50); + assertWindow(window, 1, 0, 50); + + window = manager.createPageWindow(2, "foo", 40); + assertWindow(window, 2, 50, 40); + + assertEquals(manager.getTotalSize(), 90); + + window = manager.createPageWindow(2, "foo", 30); + assertWindow(window, 2, 50, 30); + assertEquals(manager.getTotalSize(), 80); + + manager.removePage(2); + assertEquals(manager.getTotalSize(), 50); + + window = manager.createPageWindow(3, "foo", 30); + assertWindow(window, 3, 50, 30); + assertEquals(manager.getTotalSize(), 80); + } + + /** + * + */ + @Test + public void pageWindowCycle() + { + PageWindowManager manager = new PageWindowManager(100); + FileWindow window; + + window = manager.createPageWindow(1, "foo", 30); + + window = manager.createPageWindow(2, "foo", 30); + + window = manager.createPageWindow(3, "foo", 30); + + assertWindow(window, 3, 60, 30); + + window = manager.createPageWindow(4, "foo", 30); + + assertWindow(window, 4, 90, 30); + + // should start at the beginging + + window = manager.createPageWindow(5, "foo", 20); + + assertWindow(window, 5, 0, 20); + + assertNull(manager.getPageWindow(1)); + + window = manager.getPageWindow(2); + assertWindow(window, 2, 30, 30); + + window = manager.createPageWindow(6, "foo", 10); + + assertWindow(window, 6, 20, 10); + + window = manager.getPageWindow(2); + assertWindow(window, 2, 30, 30); + + window = manager.createPageWindow(6, "foo", 30); + assertWindow(window, 6, 20, 30); + + assertNull(manager.getPageWindow(2)); + assertNotNull(manager.getPageWindow(3)); + + window = manager.createPageWindow(6, "foo", 60); + assertWindow(window, 6, 20, 60); + + assertNull(manager.getPageWindow(3)); + + window = manager.createPageWindow(7, "foo", 20); + assertWindow(window, 7, 80, 20); + + assertNotNull(manager.getPageWindow(7)); + + // should start at the beginning again + + window = manager.createPageWindow(8, "foo", 10); + assertWindow(window, 8, 0, 10); + + assertNull(manager.getPageWindow(5)); + assertNotNull(manager.getPageWindow(6)); + + window = manager.createPageWindow(9, "foo", 20); + assertWindow(window, 9, 10, 20); + + assertNull(manager.getPageWindow(6)); + assertNotNull(manager.getPageWindow(7)); + + window = manager.createPageWindow(10, "foo", 20); + assertWindow(window, 10, 30, 20); + + assertNull(manager.getPageWindow(6)); + assertNotNull(manager.getPageWindow(7)); + + // make sure when replacing a page that's not last the old "instance" is + // not valid anymore + + manager.createPageWindow(8, "foo", 10); + + window = manager.getPageWindow(8); + assertWindow(window, 8, 50, 10); + } + + + private void assertWindow(FileWindow window, int pageId, int filePartOffset, int filePartSize) + { + assertTrue(window.getPageId() == pageId && window.getFilePartOffset() == filePartOffset && + window.getFilePartSize() == filePartSize); + } + + /** how many operations to execute */ + private static final int EXECUTIONS = 10000; + + /** used to wait the executions */ + private static final CountDownLatch LATCH = new CountDownLatch(EXECUTIONS); + + private final PageWindowManager pageWindowManager = new PageWindowManager(1000L); + + /** the execution types */ + private final Runnable[] TASKS = new Runnable[] + { + new CreatePageWindowTask(pageWindowManager), + new GetPageWindowTask(pageWindowManager), + new RemovePageInSessionTask(pageWindowManager) + }; + + private static final SecureRandom RND = new SecureRandom(); + + /** + * Executes random mutator and accessor operations on {@link org.apache.wicket.pageStore.AsynchronousPageStore} validating + * that the used data structures can be used simultaneously. + * + * @throws Exception + */ + @Test + public void randomOperations() throws Exception + { + ExecutorService executorService = Executors.newFixedThreadPool(50); + + for (int i = 0; i < EXECUTIONS; i++) + { + Runnable task = TASKS[RND.nextInt(TASKS.length)]; + executorService.submit(task); + } + LATCH.await(); + executorService.shutdown(); + } + + private static abstract class AbstractTask implements Runnable + { + /** the ids for the stored/removed pages */ + private static final int[] PAGE_IDS = new int[] { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 }; + + protected final PageWindowManager pageWindowManager; + + private AbstractTask(PageWindowManager pageWindowManager) + { + this.pageWindowManager = pageWindowManager; + } + + protected abstract void r(); + + @Override + public void run() + { + try + { + r(); + } + finally + { + LATCH.countDown(); + } + } + + protected int getPageId() + { + return PAGE_IDS[RND.nextInt(PAGE_IDS.length)]; + } + } + + private static class CreatePageWindowTask extends AbstractTask + { + private CreatePageWindowTask(PageWindowManager pageWindowManager) + { + super(pageWindowManager); + } + + @Override + public void r() + { + pageWindowManager.createPageWindow(getPageId(), "foo", 1000); + } + } + + private static class GetPageWindowTask extends AbstractTask + { + private GetPageWindowTask(PageWindowManager pageWindowManager) + { + super(pageWindowManager); + } + + @Override + public void r() + { + pageWindowManager.getPageWindow(getPageId()); + } + } + + private static class RemovePageInSessionTask extends AbstractTask + { + private RemovePageInSessionTask(PageWindowManager pageWindowManager) + { + super(pageWindowManager); + } + + @Override + public void r() + { + pageWindowManager.removePage(getPageId()); + } + } +} http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/DummyPageManagerContext.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/DummyPageManagerContext.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/DummyPageManagerContext.java deleted file mode 100644 index 6018ded..0000000 --- a/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/DummyPageManagerContext.java +++ /dev/null @@ -1,66 +0,0 @@ -/* - * Licensed to the Apache Software Foundation (ASF) under one or more - * contributor license agreements. See the NOTICE file distributed with - * this work for additional information regarding copyright ownership. - * The ASF licenses this file to You under the Apache License, Version 2.0 - * (the "License"); you may not use this file except in compliance with - * the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ -package org.apache.wicket.pageStore.memory; - -import java.io.Serializable; - -import org.apache.wicket.page.IPageManagerContext; - -/** - */ -public class DummyPageManagerContext implements IPageManagerContext -{ - - private Serializable attribute = null; - private Object requestData; - - @Override - public void setRequestData(Object data) - { - requestData = data; - } - - @Override - public Object getRequestData() - { - return requestData; - } - - @Override - public void setSessionAttribute(String key, Serializable value) - { - attribute = value; - } - - @Override - public Serializable getSessionAttribute(String key) - { - return attribute; - } - - @Override - public void bind() - { - } - - @Override - public String getSessionId() - { - return "dummy_id"; - } - -} \ No newline at end of file http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/HttpSessionDataStoreTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/HttpSessionDataStoreTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/HttpSessionDataStoreTest.java deleted file mode 100644 index a1010c4..0000000 --- a/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/HttpSessionDataStoreTest.java +++ /dev/null @@ -1,114 +0,0 @@ -/* - * Licensed to the Apache Software Foundation (ASF) under one or more - * contributor license agreements. See the NOTICE file distributed with - * this work for additional information regarding copyright ownership. - * The ASF licenses this file to You under the Apache License, Version 2.0 - * (the "License"); you may not use this file except in compliance with - * the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ -package org.apache.wicket.pageStore.memory; - -import static org.junit.jupiter.api.Assertions.assertArrayEquals; -import static org.junit.jupiter.api.Assertions.assertNotNull; -import static org.junit.jupiter.api.Assertions.assertNull; - -import org.apache.wicket.util.tester.WicketTestCase; -import org.junit.jupiter.api.AfterEach; -import org.junit.jupiter.api.BeforeEach; -import org.junit.jupiter.api.Test; - -/** - */ -class HttpSessionDataStoreTest extends WicketTestCase -{ - private final String sessionId = "anything"; - - private final int pageId = 1; - - private final byte[] PAGE1 = new byte[] { 1 }; - final byte[] PAGE2 = new byte[] { 2 }; - - private HttpSessionDataStore store; - - /** - * before() - */ - @BeforeEach - void before() - { - store = new HttpSessionDataStore(new DummyPageManagerContext(), new NoopEvictionStrategy()); - } - - /** - * after() - */ - @AfterEach - void after() - { - store.destroy(); - } - - /** - * storePage() - */ - @Test - void storePage() - { - assertNull(store.getData(sessionId, pageId)); - - store.storeData(sessionId, pageId, PAGE1); - assertArrayEquals(PAGE1, store.getData(sessionId, pageId)); - } - - /** - * removePage1() - */ - @Test - void removePage1() - { - assertNull(store.getData(sessionId, pageId)); - - store.storeData(sessionId, pageId, PAGE1); - - assertNotNull(store.getData(sessionId, pageId)); - - store.removeData(sessionId, pageId); - - assertNull(store.getData(sessionId, pageId)); - } - - /** - * removePage2() - */ - @Test - void removePage2() - { - assertNull(store.getData(sessionId, pageId)); - - store.storeData(sessionId, pageId, PAGE1); - - assertNotNull(store.getData(sessionId, pageId)); - - store.removeData(sessionId); - - assertNull(store.getData(sessionId, pageId)); - } - - - private static final class NoopEvictionStrategy implements IDataStoreEvictionStrategy - { - - @Override - public void evict(PageTable pageTable) - { - } - } -} http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/MemorySizeEvictionStrategyTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/MemorySizeEvictionStrategyTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/MemorySizeEvictionStrategyTest.java deleted file mode 100644 index daacc62..0000000 --- a/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/MemorySizeEvictionStrategyTest.java +++ /dev/null @@ -1,64 +0,0 @@ -/* - * Licensed to the Apache Software Foundation (ASF) under one or more - * contributor license agreements. See the NOTICE file distributed with - * this work for additional information regarding copyright ownership. - * The ASF licenses this file to You under the Apache License, Version 2.0 - * (the "License"); you may not use this file except in compliance with - * the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ -package org.apache.wicket.pageStore.memory; - -import static org.junit.jupiter.api.Assertions.assertEquals; -import static org.junit.jupiter.api.Assertions.assertTrue; - -import org.apache.wicket.core.util.lang.WicketObjects; -import org.apache.wicket.util.lang.Bytes; -import org.junit.jupiter.api.Test; - -/***/ -class MemorySizeEvictionStrategyTest -{ - private final byte[] PAGE1 = new byte[] { 1 }; - private final byte[] PAGE2 = new byte[] { 2, 3 }; - - /***/ - @Test - void evict() - { - PageTable pageTable = new PageTable(); - - long sizeOfEmptyPageTable = WicketObjects.sizeof(pageTable); - - // evict to empty page table - MemorySizeEvictionStrategy strategy = new MemorySizeEvictionStrategy( - Bytes.bytes(sizeOfEmptyPageTable)); - pageTable.storePage(PAGE1.length, PAGE1); - assertEquals(1, pageTable.size()); - strategy.evict(pageTable); - assertEquals(0, pageTable.size()); - long currentSize = WicketObjects.sizeof(pageTable); - assertTrue(currentSize <= sizeOfEmptyPageTable, "Current size: |" + currentSize + "|, strategy size: |" + sizeOfEmptyPageTable + - "|"); - - // evict to page table with size: empty + PAGE2 - pageTable.storePage(PAGE2.length, PAGE2); - long sizeOfWithPage2 = WicketObjects.sizeof(pageTable); - strategy = new MemorySizeEvictionStrategy(Bytes.bytes(sizeOfWithPage2)); - pageTable.storePage(PAGE1.length, PAGE1); - assertEquals(2, pageTable.size()); - strategy.evict(pageTable); - // the following assertion depends on the fact that PAGE2 has - // bigger size than PAGE1 - assertEquals(1, pageTable.size()); - currentSize = WicketObjects.sizeof(pageTable); - assertTrue(currentSize <= sizeOfWithPage2, "Current size: |" + currentSize + "|, strategy size: |" + sizeOfWithPage2 + "|"); - } -} http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/PageNumberEvictionStrategyTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/PageNumberEvictionStrategyTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/PageNumberEvictionStrategyTest.java deleted file mode 100644 index f0ca167..0000000 --- a/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/PageNumberEvictionStrategyTest.java +++ /dev/null @@ -1,66 +0,0 @@ -/* - * Licensed to the Apache Software Foundation (ASF) under one or more - * contributor license agreements. See the NOTICE file distributed with - * this work for additional information regarding copyright ownership. - * The ASF licenses this file to You under the Apache License, Version 2.0 - * (the "License"); you may not use this file except in compliance with - * the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ -package org.apache.wicket.pageStore.memory; - -import static org.junit.jupiter.api.Assertions.assertEquals; -import static org.junit.jupiter.api.Assertions.assertNotNull; -import static org.junit.jupiter.api.Assertions.assertNull; -import static org.junit.jupiter.api.Assertions.assertThrows; - -import org.junit.jupiter.api.Test; - -/***/ -class PageNumberEvictionStrategyTest -{ - - private final byte[] PAGE1 = new byte[] { 1 }; - private final byte[] PAGE2 = new byte[] { 2, 3 }; - - /***/ - @Test - void evict() - { - // evict to page table with one page only - PageNumberEvictionStrategy strategy = new PageNumberEvictionStrategy(1); - - PageTable pageTable = new PageTable(); - - pageTable.storePage(PAGE1.length, PAGE1); - assertEquals(1, pageTable.size()); - strategy.evict(pageTable); - assertEquals(1, pageTable.size()); - assertNotNull(pageTable.getPage(PAGE1.length)); - - pageTable.storePage(PAGE2.length, PAGE2); - assertEquals(2, pageTable.size()); - strategy.evict(pageTable); - assertEquals(1, pageTable.size()); - assertNotNull(pageTable.getPage(PAGE2.length)); - assertNull(pageTable.getPage(PAGE1.length)); - } - - /** - * The number of pages must be at least '1' - */ - @Test - void greaterThanZero() - { - assertThrows(IllegalArgumentException.class, () -> { - new PageNumberEvictionStrategy(0); - }); - } -} http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/PageTableTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/PageTableTest.java b/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/PageTableTest.java deleted file mode 100644 index 9c03545..0000000 --- a/wicket-core/src/test/java/org/apache/wicket/pageStore/memory/PageTableTest.java +++ /dev/null @@ -1,58 +0,0 @@ -/* - * Licensed to the Apache Software Foundation (ASF) under one or more - * contributor license agreements. See the NOTICE file distributed with - * this work for additional information regarding copyright ownership. - * The ASF licenses this file to You under the Apache License, Version 2.0 - * (the "License"); you may not use this file except in compliance with - * the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ -package org.apache.wicket.pageStore.memory; - -import static org.junit.jupiter.api.Assertions.assertEquals; -import static org.junit.jupiter.api.Assertions.assertNull; - -import org.junit.jupiter.api.Test; - -/***/ -class PageTableTest -{ - - private final byte[] data = new byte[] { 1 }; - - /***/ - @Test - void getOldest() - { - PageTable pageTable = new PageTable(); - - assertNull(pageTable.getOldest()); - - pageTable.storePage(1, data); - // index: 1 - assertEquals(Integer.valueOf(1), pageTable.getOldest()); - - pageTable.storePage(2, data); - // index: 2, 1 - assertEquals(Integer.valueOf(1), pageTable.getOldest()); - - pageTable.storePage(3, data); - // index: 3, 2, 1 - assertEquals(Integer.valueOf(1), pageTable.getOldest()); - - pageTable.getPage(1); - // index: 1, 3, 2 - assertEquals(Integer.valueOf(2), pageTable.getOldest()); - - pageTable.removePage(2); - // index: 1, 3 - assertEquals(Integer.valueOf(3), pageTable.getOldest()); - } -} http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/request/handler/PageIdPoliticTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/request/handler/PageIdPoliticTest.java b/wicket-core/src/test/java/org/apache/wicket/request/handler/PageIdPoliticTest.java index 71b0dc5..190abf8 100644 --- a/wicket-core/src/test/java/org/apache/wicket/request/handler/PageIdPoliticTest.java +++ b/wicket-core/src/test/java/org/apache/wicket/request/handler/PageIdPoliticTest.java @@ -29,17 +29,16 @@ import org.apache.wicket.markup.IMarkupResourceStreamProvider; import org.apache.wicket.markup.html.WebComponent; import org.apache.wicket.markup.html.WebPage; import org.apache.wicket.mock.MockApplication; +import org.apache.wicket.page.IManageablePage; import org.apache.wicket.page.IPageManager; -import org.apache.wicket.page.IPageManagerContext; -import org.apache.wicket.page.PageStoreManager; -import org.apache.wicket.pageStore.DefaultPageStore; -import org.apache.wicket.pageStore.IPageStore; +import org.apache.wicket.page.PageManager; +import org.apache.wicket.pageStore.IPageContext; +import org.apache.wicket.pageStore.InMemoryPageStore; +import org.apache.wicket.pageStore.RequestPageStore; import org.apache.wicket.request.Url; -import org.apache.wicket.serialize.java.JavaSerializer; import org.apache.wicket.util.resource.IResourceStream; import org.apache.wicket.util.resource.StringResourceStream; import org.apache.wicket.util.tester.WicketTester; -import org.apache.wicket.versioning.InMemoryPageStore; import org.junit.jupiter.api.AfterEach; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; @@ -47,10 +46,10 @@ import org.junit.jupiter.api.Test; /** * @author Pedro Santos */ -class PageIdPoliticTest +public class PageIdPoliticTest { private WicketTester tester; - private InMemoryPageStore dataStore; + private InMemoryPageStore pageStore; private MockApplication application; private int storeCount; @@ -89,12 +88,12 @@ class PageIdPoliticTest void setUp() throws Exception { application = new MockApplication(); - dataStore = new InMemoryPageStore() + pageStore = new InMemoryPageStore("test", Integer.MAX_VALUE) { @Override - public void storeData(String sessionId, int pageId, byte[] pageAsBytes) + public void addPage(IPageContext context, IManageablePage page) { - super.storeData(sessionId, pageId, pageAsBytes); + super.addPage(context, page); storeCount++; } }; @@ -106,12 +105,9 @@ class PageIdPoliticTest return new IPageManagerProvider() { @Override - public IPageManager apply(IPageManagerContext pageManagerContext) + public IPageManager get() { - IPageStore pageStore = new DefaultPageStore(new JavaSerializer( - application.getApplicationKey()), dataStore, 4); - return new PageStoreManager(application.getName(), pageStore, - pageManagerContext); + return new PageManager(new RequestPageStore(pageStore)); } }; } @@ -135,7 +131,7 @@ class PageIdPoliticTest /** * Construct. */ - public TestPage() + public TestPage() { WebComponent component; component = new WebComponent("component"); @@ -157,7 +153,7 @@ class PageIdPoliticTest * @param encoding * @return ajaxUrl */ - Url getAjaxUrl(String encoding) + public Url getAjaxUrl(String encoding) { return Url.parse(eventBehavior.getCallbackUrl().toString(), Charset.forName(encoding)); } http://git-wip-us.apache.org/repos/asf/wicket/blob/f6485517/wicket-core/src/test/java/org/apache/wicket/request/handler/PageProviderTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/request/handler/PageProviderTest.java b/wicket-core/src/test/java/org/apache/wicket/request/handler/PageProviderTest.java index b7ad145..b5c2aef 100644 --- a/wicket-core/src/test/java/org/apache/wicket/request/handler/PageProviderTest.java +++ b/wicket-core/src/test/java/org/apache/wicket/request/handler/PageProviderTest.java @@ -65,8 +65,8 @@ class PageProviderTest extends WicketTestCase // storing test page TestMapperContext mapperContext = new TestMapperContext(); - mapperContext.getPageManager().touchPage(testPage); - mapperContext.getPageManager().commitRequest(); + mapperContext.getPageManager().addPage(testPage); + mapperContext.getPageManager().detach(); // by cleaning session cache we make sure of not being testing the same in-memory instance mapperContext.cleanSessionCache(); @@ -80,8 +80,8 @@ class PageProviderTest extends WicketTestCase // changing some sate StatefullMockPage providedPage = (StatefullMockPage)pageProvider.getPageInstance(); providedPage.state = newState; - mapperContext.getPageManager().touchPage(providedPage); - mapperContext.getPageManager().commitRequest(); + mapperContext.getPageManager().addPage(providedPage); + mapperContext.getPageManager().detach(); mapperContext.cleanSessionCache(); @@ -215,8 +215,8 @@ class PageProviderTest extends WicketTestCase { TestMapperContext mapperContext = new TestMapperContext(); Page page = new TestPage(); - mapperContext.getPageManager().touchPage(page); - mapperContext.getPageManager().commitRequest(); + mapperContext.getPageManager().addPage(page); + mapperContext.getPageManager().detach(); // by cleaning session cache we make sure of not being testing the same in-memory instance mapperContext.cleanSessionCache(); @@ -238,8 +238,8 @@ class PageProviderTest extends WicketTestCase { TestMapperContext mapperContext = new TestMapperContext(); Page page = new TestPage(); - mapperContext.getPageManager().touchPage(page); - mapperContext.getPageManager().commitRequest(); + mapperContext.getPageManager().addPage(page); + mapperContext.getPageManager().detach(); // by cleaning session cache we make sure of not being testing the same in-memory instance mapperContext.cleanSessionCache(); @@ -255,8 +255,8 @@ class PageProviderTest extends WicketTestCase { TestMapperContext mapperContext = new TestMapperContext(); Page page = new TestPage(); - mapperContext.getPageManager().touchPage(page); - mapperContext.getPageManager().commitRequest(); + mapperContext.getPageManager().addPage(page); + mapperContext.getPageManager().detach(); PageProvider pageProvider = new PageProvider(page.getPageId(), page.getRenderCount()); JavaSerializer javaSerializer = new JavaSerializer("app");