MB-21539: Use correct expiry value for test 'MB-16357' 65/69365/3
authorDave Rigby <daver@couchbase.com>
Tue, 1 Nov 2016 14:22:54 +0000 (14:22 +0000)
committerDave Rigby <daver@couchbase.com>
Wed, 2 Nov 2016 15:04:09 +0000 (15:04 +0000)
During the refactor done in b4c858e, the expiration value of 1s was
lost for this test. Restore it.

Change-Id: I0b1bf35548cffcd6867a29dcbb9a843b0f06b4c8
Reviewed-on: http://review.couchbase.org/69365
Reviewed-by: Trond Norbye <trond.norbye@gmail.com>
Tested-by: buildbot <build@couchbase.com>
tests/ep_test_apis.cc
tests/ep_test_apis.h
tests/ep_testsuite_dcp.cc

index 07a02ee..e3cc8ce 100644 (file)
@@ -1459,10 +1459,9 @@ void validate_store_resp(ENGINE_ERROR_CODE ret, int& num_items)
     }
 }
 
-/* Helper function to write unique "num_items" starting from keyXX
-   (XX is start_seqno) */
 void write_items(ENGINE_HANDLE *h, ENGINE_HANDLE_V1 *h1, int num_items,
-                 int start_seqno, const char *key_prefix, const char *value)
+                 int start_seqno, const char *key_prefix, const char *value,
+                 uint32_t expiry)
 {
     int j = 0;
     while (1) {
@@ -1472,7 +1471,8 @@ void write_items(ENGINE_HANDLE *h, ENGINE_HANDLE_V1 *h1, int num_items,
         item *i = nullptr;
         std::string key(key_prefix + std::to_string(j + start_seqno));
         ENGINE_ERROR_CODE ret = store(h, h1, nullptr, OPERATION_SET,
-                                      key.c_str(), value, &i);
+                                      key.c_str(), value, &i, /*cas*/0, /*vb*/0,
+                                      expiry);
         h1->release(h, nullptr, i);
         validate_store_resp(ret, j);
     }
index 7e64af3..c728a2c 100644 (file)
@@ -401,11 +401,19 @@ void set_degraded_mode(ENGINE_HANDLE *h,
                        const void* cookie,
                        bool enable);
 
-/* Helper function to write unique "num_items" starting from keyXX
-   (XX is start_seqno) */
+/**
+ * Helper function to write unique "num_items" starting from {key_prefix}XX,
+ * where XX is start_seqno.
+ * @param num_items Number of items to write
+ * @param start_seqno Sequence number to start from (inclusive).
+ * @param key_prefix Prefix for key names
+ * @param value Value for each item
+ * @param expiry Expiration time for each item.
+ */
 void write_items(ENGINE_HANDLE *h, ENGINE_HANDLE_V1 *h1,
                  int num_items, int start_seqno = 0,
-                 const char *key_prefix = "key", const char *value = "data");
+                 const char *key_prefix = "key", const char *value = "data",
+                 uint32_t expiry = 0);
 
 /* Helper function to write unique items starting from keyXX until memory usage
    hits "mem_thresh_perc" (XX is start_seqno) */
index cb259dc..6ab679a 100644 (file)
@@ -5008,7 +5008,7 @@ static enum test_result test_mb16357(ENGINE_HANDLE *h,
     // Load up vb0 with n items, expire in 1 second
     const int num_items = 1000;
 
-    write_items(h, h1, num_items, 0, "key-");
+    write_items(h, h1, num_items, 0, "key-", "value", /*expiration*/1);
 
     wait_for_flusher_to_settle(h, h1);
     testHarness.time_travel(3617); // force expiry pushing time forward.