diff options
| author | Henning Baldersheim <balder@yahoo-inc.com> | 2019-07-30 22:02:53 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-07-30 22:02:53 +0200 |
| commit | 0beb5f054f3662b07b76bb0cd9d3311abafe63cb (patch) | |
| tree | fa4d031538ffbf212f5aabbe89c95925f185499d /storage/src/tests/persistence/provider_error_wrapper_test.cpp | |
| parent | 753e9f322ee8a1dccd598c0dbab2c937e3c552cc (diff) | |
| parent | 0d58172a97648bd6823b675fcd7df49be3729562 (diff) | |
Merge pull request #10132 from vespa-engine/toregge/use-enum-class-for-storage-spi-result-errortype
Use enum class for storage::spi::Result::ErrorType.
Diffstat (limited to 'storage/src/tests/persistence/provider_error_wrapper_test.cpp')
| -rw-r--r-- | storage/src/tests/persistence/provider_error_wrapper_test.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/storage/src/tests/persistence/provider_error_wrapper_test.cpp b/storage/src/tests/persistence/provider_error_wrapper_test.cpp index 36238abb238..2ba5218003b 100644 --- a/storage/src/tests/persistence/provider_error_wrapper_test.cpp +++ b/storage/src/tests/persistence/provider_error_wrapper_test.cpp @@ -64,7 +64,7 @@ TEST_F(ProviderErrorWrapperTest, fatal_error_invokes_listener) { Fixture f(getPersistenceProvider()); auto listener = std::make_shared<MockErrorListener>(); f.errorWrapper.register_error_listener(listener); - f.providerWrapper.setResult(spi::Result(spi::Result::FATAL_ERROR, "eject! eject!")); + f.providerWrapper.setResult(spi::Result(spi::Result::ErrorType::FATAL_ERROR, "eject! eject!")); EXPECT_FALSE(listener->_seen_fatal_error); f.perform_spi_operation(); @@ -78,7 +78,7 @@ TEST_F(ProviderErrorWrapperTest, resource_exhaustion_error_invokes_listener) { Fixture f(getPersistenceProvider()); auto listener = std::make_shared<MockErrorListener>(); f.errorWrapper.register_error_listener(listener); - f.providerWrapper.setResult(spi::Result(spi::Result::RESOURCE_EXHAUSTED, "out of juice")); + f.providerWrapper.setResult(spi::Result(spi::Result::ErrorType::RESOURCE_EXHAUSTED, "out of juice")); EXPECT_FALSE(listener->_seen_resource_exhaustion_error); f.perform_spi_operation(); @@ -103,8 +103,8 @@ TEST_F(ProviderErrorWrapperTest, listener_not_invoked_on_regular_errors) { auto listener = std::make_shared<MockErrorListener>(); f.errorWrapper.register_error_listener(listener); - EXPECT_NO_FATAL_FAILURE(f.check_no_listener_invoked_for_error(*listener, spi::Result::TRANSIENT_ERROR)); - EXPECT_NO_FATAL_FAILURE(f.check_no_listener_invoked_for_error(*listener, spi::Result::PERMANENT_ERROR)); + EXPECT_NO_FATAL_FAILURE(f.check_no_listener_invoked_for_error(*listener, spi::Result::ErrorType::TRANSIENT_ERROR)); + EXPECT_NO_FATAL_FAILURE(f.check_no_listener_invoked_for_error(*listener, spi::Result::ErrorType::PERMANENT_ERROR)); } TEST_F(ProviderErrorWrapperTest, multiple_listeners_can_be_registered) { @@ -114,7 +114,7 @@ TEST_F(ProviderErrorWrapperTest, multiple_listeners_can_be_registered) { f.errorWrapper.register_error_listener(listener1); f.errorWrapper.register_error_listener(listener2); - f.providerWrapper.setResult(spi::Result(spi::Result::RESOURCE_EXHAUSTED, "out of juice")); + f.providerWrapper.setResult(spi::Result(spi::Result::ErrorType::RESOURCE_EXHAUSTED, "out of juice")); f.perform_spi_operation(); EXPECT_TRUE(listener1->_seen_resource_exhaustion_error); |