pw_kvs: Alignment handling adjustments

- Rename multi-span Write to WriteAligned to reflect that the number of
  bytes written may be larger than the input due to padding.
- Make alignment_bytes_ a member of FlashPartition instead of a virtual
  function.
- Enable tests that now pass.

Change-Id: Iad985988ef97607b38fb0c84589cabe752de71a2

1 files changed, 9 insertions, 7 deletions

diff --git a/pw_kvs/key_value_store_test.cc b/pw_kvs/key_value_store_test.cc
index 27ce09a59..c2848d7c3 100644
--- a/pw_kvs/key_value_store_test.cc
+++ b/pw_kvs/key_value_store_test.cc
@@ -282,8 +282,7 @@ uint16_t CalcTestPartitionCrc() {
 
 }  // namespace
 
-TEST_F(KeyValueStoreTest,
-       DISABLED_Put_SameKeySameValueRepeatedly_AlignedEntries) {
+TEST_F(KeyValueStoreTest, Put_SameKeySameValueRepeatedly_AlignedEntries) {
   std::array<char, 8> value{'v', 'a', 'l', 'u', 'e', '6', '7', '\0'};
 
   for (int i = 0; i < 1000; ++i) {
@@ -291,8 +290,7 @@ TEST_F(KeyValueStoreTest,
   }
 }
 
-TEST_F(KeyValueStoreTest,
-       DISABLED_Put_SameKeySameValueRepeatedly_UnalignedEntries) {
+TEST_F(KeyValueStoreTest, Put_SameKeySameValueRepeatedly_UnalignedEntries) {
   std::array<char, 7> value{'v', 'a', 'l', 'u', 'e', '6', '\0'};
 
   for (int i = 0; i < 1000; ++i) {
@@ -300,9 +298,13 @@ TEST_F(KeyValueStoreTest,
   }
 }
 
-TEST_F(KeyValueStoreTest, DISABLED_Put_SameKeyDifferentValueRepeatedly) {
-  for (uint64_t i = 0; i < 1000u; ++i) {
-    ASSERT_EQ(Status::OK, kvs_.Put("The Key!", i));
+TEST_F(KeyValueStoreTest, Put_SameKeyDifferentValuesRepeatedly) {
+  std::array<char, 10> value{'v', 'a', 'l', 'u', 'e', '6', '7', '8', '9', '\0'};
+
+  for (int i = 0; i < 100; ++i) {
+    for (unsigned size = 0; size < value.size(); ++size) {
+      ASSERT_EQ(Status::OK, kvs_.Put("The Key!", i));
+    }
   }
 }