Hi @nacx Thanks for your comments - have reworked the pagination tests in InfrastructureApiMockTest, and saw it fail because of the bug you spotted with the optional.
Have added an IHookable in BaseDimensionDataCloudControlMockTest that checks after each test runs that we've assertted all of the requests received by the mock web server - hopefully this will help us write better tests going forward. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/377#issuecomment-289446793
