Add highWaterMarkBytes option to tests where it matters

The `classic-level` implementation will have this option, replacing the `highWaterMark` option of `leveldown` which conflicts with streams.
Level · Feb 6, 2022 · 6b25a91 · 6b25a91
1 parent 2e90b05
commit 6b25a91
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/test/iterator-no-snapshot-test.js b/test/iterator-no-snapshot-test.js
@@ -17,9 +17,10 @@ exports.noSnapshot = function (test, testCommon) {
           t.ifError(err, 'no batch error')
 
           // For this test it is important that we don't read eagerly.
-          // NOTE: highWaterMark is not an abstract option atm, but
-          // it is supported by leveldown, rocksdb and others.
-          const it = db.iterator({ highWaterMark: 0 })
+          // NOTE: highWaterMarkBytes is not an abstract option, but
+          // it is supported by classic-level and others. Also set the
+          // old & equivalent leveldown highWaterMark option for compat.
+          const it = db.iterator({ highWaterMarkBytes: 0, highWaterMark: 0 })
 
           run(db, function (err) {
             t.ifError(err, 'no run error')

diff --git a/test/iterator-snapshot-test.js b/test/iterator-snapshot-test.js
@@ -12,9 +12,10 @@ exports.snapshot = function (test, testCommon) {
           t.ifError(err, 'no put error')
 
           // For this test it is important that we don't read eagerly.
-          // NOTE: highWaterMark is not an abstract option atm, but
-          // it is supported by leveldown, rocksdb and others.
-          const it = db.iterator({ highWaterMark: 0 })
+          // NOTE: highWaterMarkBytes is not an abstract option, but
+          // it is supported by classic-level and others. Also set the
+          // old & equivalent leveldown highWaterMark option for compat.
+          const it = db.iterator({ highWaterMarkBytes: 0, highWaterMark: 0 })
 
           run(t, db, it, function end (err) {
             t.ifError(err, 'no run error')