[sales] Pass along request and slot index in onProve callback

codex/node.nim

@@ -339,8 +339,8 @@ proc start*(node: CodexNodeRef) {.async.} =
     contracts.sales.onClear = proc(availability: Availability, request: StorageRequest) =
       # TODO: remove data from local storage
       discard
-
-    contracts.sales.onProve = proc(cid: string): Future[seq[byte]] {.async.} =
+    contracts.sales.onProve = proc(request: StorageRequest,
+                                   slot: UInt256): Future[seq[byte]] {.async.} =
       # TODO: generate proof
       return @[42'u8]
 

codex/sales.nim

@@ -57,7 +57,8 @@ type
   OnStore = proc(request: StorageRequest,
                  slot: UInt256,
                  availability: Availability): Future[void] {.gcsafe, upraises: [].}
-  OnProve = proc(cid: string): Future[seq[byte]] {.gcsafe, upraises: [].}
+  OnProve = proc(request: StorageRequest,
+                 slot: UInt256): Future[seq[byte]] {.gcsafe, upraises: [].}
   OnClear = proc(availability: Availability, request: StorageRequest) {.gcsafe, upraises: [].}
   OnSale = proc(availability: Availability,
                 request: StorageRequest,
@@ -187,7 +188,7 @@ proc start(agent: SalesAgent) {.async.} =
     agent.waiting = some agent.waitForExpiry()
 
     await onStore(request, slotIndex, availability)
-    let proof = await onProve(request.content.cid)
+    let proof = await onProve(request, slotIndex)
     await market.fillSlot(request.id, slotIndex, proof)
   except CancelledError:
     raise

tests/codex/testsales.nim

@@ -37,7 +37,8 @@ suite "Sales":
                          slot: UInt256,
                          availability: Availability) {.async.} =
       discard
-    sales.onProve = proc(cid: string): Future[seq[byte]] {.async.} =
+    sales.onProve = proc(request: StorageRequest,
+                         slot: UInt256): Future[seq[byte]] {.async.} =
       return proof
     await sales.start()
     request.expiry = (clock.now() + 42).u256
@@ -113,11 +114,16 @@ suite "Sales":
     check sales.available == @[availability]
 
   test "generates proof of storage":
-    var provingCid: string
-    sales.onProve = proc(cid: string): Future[seq[byte]] {.async.} = provingCid = cid
+    var provingRequest: StorageRequest
+    var provingSlot: UInt256
+    sales.onProve = proc(request: StorageRequest,
+                         slot: UInt256): Future[seq[byte]] {.async.} =
+      provingRequest = request
+      provingSlot = slot
     sales.add(availability)
-    discard await market.requestStorage(request)
-    check provingCid == request.content.cid
+    let requested = await market.requestStorage(request)
+    check provingRequest == requested
+    check provingSlot < request.ask.slots.u256
 
   test "fills a slot":
     sales.add(availability)
@@ -145,7 +151,8 @@ suite "Sales":
     check soldSlotIndex < request.ask.slots.u256
 
   test "calls onClear when storage becomes available again":
-    sales.onProve = proc(cid: string): Future[seq[byte]] {.async.} =
+    sales.onProve = proc(request: StorageRequest,
+                         slot: UInt256): Future[seq[byte]] {.async.} =
       raise newException(IOError, "proof failed")
     var clearedAvailability: Availability
     var clearedRequest: StorageRequest