Skip to content

Commit

Permalink
adjust new tests with proper exception, re #11064
Browse files Browse the repository at this point in the history
  • Loading branch information
FedeMPouzols committed Apr 2, 2015
1 parent 8626f6f commit 4e1e937
Showing 1 changed file with 4 additions and 5 deletions.
Expand Up @@ -316,9 +316,8 @@ class SCARFLSFJobManagerTest : public CxxTest::TestSuite {
jm = boost::make_shared<MockedGoodLoginResponse_SCARFLSFJM>());

std::string tid;
TSM_ASSERT_THROWS_NOTHING(
"start transaction should throw when not logged in",
tid = jm->startRemoteTransaction());
TSM_ASSERT_THROWS("start transaction should throw when not logged in",
tid = jm->startRemoteTransaction(), std::runtime_error);

TSM_ASSERT_THROWS_NOTHING("successful authentication should not throw",
jm->authenticate("user", "pass"));
Expand Down Expand Up @@ -577,8 +576,8 @@ class SCARFLSFJobManagerTest : public CxxTest::TestSuite {
MockedErrorResponse_SCARFLSFJM err;
TSM_ASSERT_THROWS("authenticate with empty credentials should throw",
err.authenticate("", ""), std::runtime_error);
TSM_ASSERT_THROWS_NOTHING(
"logout with error response from server should throw", err.logout());
TSM_ASSERT_THROWS("logout with error response from server should throw",
err.logout(), std::runtime_error);

MockedGoodLoginResponse_SCARFLSFJM jm;
TSM_ASSERT_THROWS_NOTHING("successful authentication should not throw",
Expand Down

0 comments on commit 4e1e937

Please sign in to comment.