Noop message store implementation

[guidomedina]

Noop message store implementation.

[guidomedina]

Would it be nice to replace MemoryStore by this one at some tests to make it faster?

[chrjohn]

If the tests don't rely on messages being resent (i.e. PersistMessages=N) then it might be an option.

[chrjohn]

@guidomedina The unit tests fail with this:

Tests run: 4, Failures: 0, Errors: 3, Skipped: 0, Time elapsed: 0.016 sec <<< FAILURE! - in quickfix.NoopStoreTest
testMessageStorageOutOfSequence(quickfix.NoopStoreTest)  Time elapsed: 0.015 sec  <<< ERROR!
java.lang.NullPointerException

testMessageStorageMessages(quickfix.NoopStoreTest)  Time elapsed: 0 sec  <<< ERROR!
java.lang.NullPointerException

testRefreshableMessageStore(quickfix.NoopStoreTest)  Time elapsed: 0 sec  <<< ERROR!
java.lang.NullPointerException

This is from the AbstractMessageStoreTest.

[guidomedina]

I think I know what it is, no default constructor, adding it and testing.

[guidomedina]

Failed again, I can't figure out what's the trick, do I need to declare exceptions?

Tests in error: 
  NoopStoreTest>AbstractMessageStoreTest.testMessageStorageMessages:107 » NullPointer
  NoopStoreTest>AbstractMessageStoreTest.testMessageStorageOutOfSequence:124 » NullPointer
  NoopStoreTest>AbstractMessageStoreTest.testRefreshableMessageStore:160 » NullPointer

[chrjohn]

I'll have a look

[guidomedina]

I look these tests and I think we should delete NoopStoreTest as these tests expect messages to be stored where NoopStore is all the oppossite, sending PR.

[chrjohn]

The session that is tried to lookup is NULL in the NoopStore.refresh() method. But I don't know why this works for the other tests.