Skip to content

Commit

Permalink
fixed hole in unit test and windows test
Browse files Browse the repository at this point in the history
  • Loading branch information
ic3man5 committed Aug 15, 2023
1 parent 09e0d30 commit 8297cfe
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/ice_library.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ Library::Library(std::string name, bool nothrow)
LocalFree(buffer);
m_last_error = err.str();
m_has_error = true;
if (m_nothrow) {
if (!m_nothrow) {
throw Exception(err.str());
}
}
Expand Down
4 changes: 2 additions & 2 deletions tests/test_ice_library.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -115,8 +115,8 @@ TEST_CASE("LibraryManagerBasic", "[librarymanager]")
REQUIRE(library->_library() != NULL);
// Test a failure library being added
try {
mgr.add("failure", "This_Should_Always_Fail", true);
// auto failure_library = ice::Library("This_Should_Always_Fail");
mgr.add("failure", "This_Should_Always_Fail", false);
REQUIRE(false);
} catch (ice::Exception& ex) {
auto error_msg = ex.whatString();
INFO(error_msg);
Expand Down

0 comments on commit 8297cfe

Please sign in to comment.