From 954c8edf7689844a6be298b3cfddaea491497f7f Mon Sep 17 00:00:00 2001
From: Eric <5089238+emizzle@users.noreply.github.com>
Date: Thu, 21 Dec 2023 09:12:24 +1100
Subject: [PATCH] comment out, instead of skipping, intermittently failing test
 (#661)

`skip` seems to working only sometimes. It intermittently fails on Windows in CI. The reason for skipping the test in the first place was to prevent intermittent failures of the test itself, so adding `skip` did not improve the situation, unfortunately.
---
 tests/integration/testIntegration.nim | 45 +++++++++++++--------------
 1 file changed, 22 insertions(+), 23 deletions(-)

diff --git a/tests/integration/testIntegration.nim b/tests/integration/testIntegration.nim
index eed181f5..b41f2c93 100644
--- a/tests/integration/testIntegration.nim
+++ b/tests/integration/testIntegration.nim
@@ -247,30 +247,29 @@ twonodessuite "Integration tests", debug1 = false, debug2 = false:
   # reason. This test has been completely refactored in
   # https://github.com/codex-storage/nim-codex/pull/607 in which it will be
   # reintroduced.
-  test "expired request partially pays out for stored time":
-    skip
-    let marketplace = Marketplace.new(Marketplace.address, ethProvider.getSigner())
-    let tokenAddress = await marketplace.token()
-    let token = Erc20Token.new(tokenAddress, ethProvider.getSigner())
-    let reward = 400.u256
-    let duration = 100.u256
+  # test "expired request partially pays out for stored time":
+  #   let marketplace = Marketplace.new(Marketplace.address, ethProvider.getSigner())
+  #   let tokenAddress = await marketplace.token()
+  #   let token = Erc20Token.new(tokenAddress, ethProvider.getSigner())
+  #   let reward = 400.u256
+  #   let duration = 100.u256
 
-    # client 2 makes storage available
-    let startBalanceClient2 = await token.balanceOf(account2)
-    discard client2.postAvailability(size=140000.u256, duration=200.u256, minPrice=300.u256, maxCollateral=300.u256).get
+  #   # client 2 makes storage available
+  #   let startBalanceClient2 = await token.balanceOf(account2)
+  #   discard client2.postAvailability(size=140000.u256, duration=200.u256, minPrice=300.u256, maxCollateral=300.u256).get
 
-    # client 1 requests storage but requires two nodes to host the content
-    let startBalanceClient1 = await token.balanceOf(account1)
-    let expiry = (await ethProvider.currentTime()) + 10
-    let cid = client1.upload(exampleString(100000)).get
-    let id = client1.requestStorage(cid, duration=duration, reward=reward, proofProbability=3.u256, expiry=expiry, collateral=200.u256, nodes=2).get
+  #   # client 1 requests storage but requires two nodes to host the content
+  #   let startBalanceClient1 = await token.balanceOf(account1)
+  #   let expiry = (await ethProvider.currentTime()) + 10
+  #   let cid = client1.upload(exampleString(100000)).get
+  #   let id = client1.requestStorage(cid, duration=duration, reward=reward, proofProbability=3.u256, expiry=expiry, collateral=200.u256, nodes=2).get
 
-    # We have to wait for Client 2 fills the slot, before advancing time.
-    # Until https://github.com/codex-storage/nim-codex/issues/594 is implemented nothing better then
-    # sleeping some seconds is available.
-    await sleepAsync(2.seconds)
-    await ethProvider.advanceTimeTo(expiry+1)
-    check eventually(client1.purchaseStateIs(id, "cancelled"), 20000)
+  #   # We have to wait for Client 2 fills the slot, before advancing time.
+  #   # Until https://github.com/codex-storage/nim-codex/issues/594 is implemented nothing better then
+  #   # sleeping some seconds is available.
+  #   await sleepAsync(2.seconds)
+  #   await ethProvider.advanceTimeTo(expiry+1)
+  #   check eventually(client1.purchaseStateIs(id, "cancelled"), 20000)
 
-    check eventually ((await token.balanceOf(account2)) - startBalanceClient2) > 0 and ((await token.balanceOf(account2)) - startBalanceClient2) < 10*reward
-    check eventually (startBalanceClient1 - (await token.balanceOf(account1))) == ((await token.balanceOf(account2)) - startBalanceClient2)
+  #   check eventually ((await token.balanceOf(account2)) - startBalanceClient2) > 0 and ((await token.balanceOf(account2)) - startBalanceClient2) < 10*reward
+  #   check eventually (startBalanceClient1 - (await token.balanceOf(account1))) == ((await token.balanceOf(account2)) - startBalanceClient2)