BlockStorage/repertory
1
0
Fork 0
Code Issues 4 Pull Requests Releases 8 Wiki Activity

[unit test] Complete all providers unit tests #12
Some checks failed
Blockstorage/repertory/pipeline/head There was a failure building this commit
Details
BlockStorage/repertory/pipeline/head There was a failure building this commit
Details

Browse Source
This commit is contained in:
Scott E. Graves
2025-09-17 10:52:52 -05:00
parent 1214e777d5
commit 9875701f96
1 changed files with 9 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
repertory/repertory_test/src/providers_test.cpp
View File

@@ -1139,8 +1139,7 @@ static void get_pinned_files(i_provider &provider) {
app_config::get_provider_name(provider.get_provider_type()), app_config::get_provider_name(provider.get_provider_type()),
__FUNCTION__); __FUNCTION__);
if (provider.is_read_only()) { if (provider.is_read_only()) {
api_file_list pinned{}; auto pinned= provider.get_pinned_files());
EXPECT_EQ(api_error::success, provider.get_pinned_files(pinned));
EXPECT_TRUE(pinned.empty()); EXPECT_TRUE(pinned.empty());
return; return;
} }
@@ -1156,8 +1155,7 @@ static void get_pinned_files(i_provider &provider) {
EXPECT_EQ(api_error::success, EXPECT_EQ(api_error::success,
provider.set_item_meta("/nopin01.txt", META_PINNED, "false")); provider.set_item_meta("/nopin01.txt", META_PINNED, "false"));
api_file_list pinned{}; auto pinned = provider.get_pinned_files(pinned);
EXPECT_EQ(api_error::success, provider.get_pinned_files(pinned));
EXPECT_EQ(std::size_t(2U), pinned.size()); EXPECT_EQ(std::size_t(2U), pinned.size());
const auto has_path = [&](std::string_view path) { const auto has_path = [&](std::string_view path) {
@@ -1180,8 +1178,7 @@ static void remove_pin_updates_pinned_files(i_provider &provider) {
__FUNCTION__); __FUNCTION__);
if (provider.is_read_only()) { if (provider.is_read_only()) {
api_file_list pinned{}; auto pinned = provider.get_pinned_files();
EXPECT_EQ(api_error::success, provider.get_pinned_files(pinned));
EXPECT_TRUE(pinned.empty()); EXPECT_TRUE(pinned.empty());
return; return;
} }
@@ -1193,8 +1190,7 @@ static void remove_pin_updates_pinned_files(i_provider &provider) {
EXPECT_EQ(api_error::success, EXPECT_EQ(api_error::success,
provider.set_item_meta("/pin02.txt", META_PINNED, "true")); provider.set_item_meta("/pin02.txt", META_PINNED, "true"));
api_file_list pinned{}; auto pinned = provider.get_pinned_files();
EXPECT_EQ(api_error::success, provider.get_pinned_files(pinned));
EXPECT_EQ(std::size_t(2U), pinned.size()); EXPECT_EQ(std::size_t(2U), pinned.size());
const auto has_path = [&](std::string_view path) { const auto has_path = [&](std::string_view path) {
@@ -1204,16 +1200,14 @@ static void remove_pin_updates_pinned_files(i_provider &provider) {
EXPECT_EQ(api_error::success, EXPECT_EQ(api_error::success,
provider.set_item_meta("/pin02.txt", META_PINNED, "false")); provider.set_item_meta("/pin02.txt", META_PINNED, "false"));
pinned.clear(); pinned = provider.get_pinned_files();
EXPECT_EQ(api_error::success, provider.get_pinned_files(pinned));
EXPECT_EQ(std::size_t(1U), pinned.size()); EXPECT_EQ(std::size_t(1U), pinned.size());
EXPECT_TRUE(has_path("/pin01.txt")); EXPECT_TRUE(has_path("/pin01.txt"));
EXPECT_FALSE(has_path("/pin02.txt")); EXPECT_FALSE(has_path("/pin02.txt"));
EXPECT_EQ(api_error::success, EXPECT_EQ(api_error::success,
provider.set_item_meta("/pin01.txt", META_PINNED, "false")); provider.set_item_meta("/pin01.txt", META_PINNED, "false"));
pinned.clear(); pinned = provider.get_pinned_files();
EXPECT_EQ(api_error::success, provider.get_pinned_files(pinned));
EXPECT_TRUE(pinned.empty()); EXPECT_TRUE(pinned.empty());
EXPECT_EQ(api_error::success, provider.remove_file("/pin01.txt")); EXPECT_EQ(api_error::success, provider.remove_file("/pin01.txt"));
@@ -1226,8 +1220,7 @@ static void remove_file_updates_pinned_files(i_provider &provider) {
__FUNCTION__); __FUNCTION__);
if (provider.is_read_only()) { if (provider.is_read_only()) {
api_file_list pinned{}; auto pinned = provider.get_pinned_files();
EXPECT_EQ(api_error::success, provider.get_pinned_files(pinned));
EXPECT_TRUE(pinned.empty()); EXPECT_TRUE(pinned.empty());
return; return;
} }
@@ -1243,8 +1236,7 @@ static void remove_file_updates_pinned_files(i_provider &provider) {
EXPECT_EQ(api_error::success, EXPECT_EQ(api_error::success,
provider.set_item_meta("/nopin.txt", META_PINNED, "false")); provider.set_item_meta("/nopin.txt", META_PINNED, "false"));
api_file_list pinned{}; auto pinned = provider.get_pinned_files();
EXPECT_EQ(api_error::success, provider.get_pinned_files(pinned));
EXPECT_EQ(std::size_t(2U), pinned.size()); EXPECT_EQ(std::size_t(2U), pinned.size());
const auto has_path = [&](std::string_view path) { const auto has_path = [&](std::string_view path) {
@@ -1258,8 +1250,7 @@ static void remove_file_updates_pinned_files(i_provider &provider) {
EXPECT_EQ(api_error::success, provider.remove_file("/pin_delete.txt")); EXPECT_EQ(api_error::success, provider.remove_file("/pin_delete.txt"));
pinned.clear(); pinned = provider.get_pinned_files();
EXPECT_EQ(api_error::success, provider.get_pinned_files(pinned));
EXPECT_EQ(std::size_t(1U), pinned.size()); EXPECT_EQ(std::size_t(1U), pinned.size());
EXPECT_TRUE(has_path("/pin_keep.txt")); EXPECT_TRUE(has_path("/pin_keep.txt"));
EXPECT_FALSE(has_path("/pin_delete.txt")); EXPECT_FALSE(has_path("/pin_delete.txt"));