Debug siatest

modules/renter/contractor/contractor.go

@@ -155,11 +155,11 @@ func (c *Contractor) ContractByPublicKey(pk types.SiaPublicKey) (modules.RenterC
 // CancelContract cancels the Contractor's contract by marking it !GoodForRenew
 // and !GoodForUpload
 func (c *Contractor) CancelContract(id types.FileContractID) error {
-	u := modules.ContractUtility{
+	return c.managedUpdateContractUtility(id, modules.ContractUtility{
 		GoodForRenew:  false,
 		GoodForUpload: false,
-	}
-	return c.managedUpdateContractUtility(id, u)
+		Locked:        true,
+	})
 }
 
 // Contracts returns the contracts formed by the contractor in the current

modules/renter/contractor/contracts.go

@@ -96,6 +96,11 @@ func (c *Contractor) managedMarkContractsUtility() error {
 	// Update utility fields for each contract.
 	for _, contract := range c.staticContracts.ViewAll() {
 		utility := func() (u modules.ContractUtility) {
+			// Record current utility of the contract
+			u.GoodForRenew = contract.Utility.GoodForRenew
+			u.GoodForUpload = contract.Utility.GoodForUpload
+			u.Locked = contract.Utility.Locked
+
 			// Start the contract in good standing if the utility wasn't
 			// locked.
 			if !u.Locked {

node/api/client/renter.go

@@ -12,12 +12,12 @@ import (
 	"github.com/NebulousLabs/Sia/types"
 )
 
-// RenterCancelContractPost uses the /renter/cancel/contract endpoint to cancel
+// RenterContractCancelPost uses the /renter/contract/cancel endpoint to cancel
 // a contract
-func (c *Client) RenterCancelContractPost(id types.FileContractID) error {
+func (c *Client) RenterContractCancelPost(id types.FileContractID) error {
 	values := url.Values{}
 	values.Set("id", id.String())
-	err := c.post("/renter/cancel/contract", values.Encode(), nil)
+	err := c.post("/renter/contract/cancel", values.Encode(), nil)
 	return err
 }
 

node/api/renter.go

@@ -275,10 +275,10 @@ func (api *API) renterHandlerPOST(w http.ResponseWriter, req *http.Request, _ ht
 	WriteSuccess(w)
 }
 
-// renterContractClearHandler handles the API call to cancel a specific Renter contract.
-func (api *API) renterContractClearHandler(w http.ResponseWriter, req *http.Request, _ httprouter.Params) {
+// renterContractCancelHandler handles the API call to cancel a specific Renter contract.
+func (api *API) renterContractCancelHandler(w http.ResponseWriter, req *http.Request, _ httprouter.Params) {
 	var fcid types.FileContractID
-	if err := fcid.LoadString(req.FormValue("ID")); err != nil {
+	if err := fcid.LoadString(req.FormValue("id")); err != nil {
 		return
 	}
 	err := api.renter.CancelContract(fcid)

node/api/routes.go

@@ -69,7 +69,7 @@ func (api *API) buildHTTPRoutes(requiredUserAgent string, requiredPassword strin
 	if api.renter != nil {
 		router.GET("/renter", api.renterHandlerGET)
 		router.POST("/renter", RequirePassword(api.renterHandlerPOST, requiredPassword))
-		router.GET("/renter/contract/cancel", api.renterContractClearHandler)
+		router.POST("/renter/contract/cancel", RequirePassword(api.renterContractCancelHandler, requiredPassword))
 		router.GET("/renter/contracts", api.renterContractsHandler)
 		router.GET("/renter/downloads", api.renterDownloadsHandler)
 		router.POST("/renter/downloads/clear", RequirePassword(api.renterClearDownloadsHandler, requiredPassword))

siatest/renter/renter_test.go

@@ -90,36 +90,35 @@ func testCancelContract(t *testing.T, tg *siatest.TestGroup) {
 	contract := rc.ActiveContracts[0]
 
 	// Cancel Contract
-	if err := r.RenterCancelContractPost(contract.ID); err != nil {
+	if err := r.RenterContractCancelPost(contract.ID); err != nil {
 		t.Fatal(err)
 	}
 
-	// Mine block to trigger threadedContractMaintenance
-	m := tg.Miners()[0]
-	if err := m.MineBlock(); err != nil {
-		t.Fatal(err)
-	}
-	tg.Sync()
-
-	// Check that Contract is now in inactive contracts and no longer in Active contracts
-	rc, err = r.RenterInactiveContractsGet()
-	if err != nil {
-		t.Fatal(err)
-	}
-	for _, c := range rc.ActiveContracts {
-		if c.ID == contract.ID {
-			t.Fatal("Contract not cancelled, contract found in Active Contracts")
+	err = build.Retry(200, 100*time.Millisecond, func() error {
+		// Check that Contract is now in inactive contracts and no longer in Active contracts
+		rc, err = r.RenterInactiveContractsGet()
+		if err != nil {
+			t.Fatal(err)
 		}
-	}
-	i := 1
-	for _, c := range rc.InactiveContracts {
-		if c.ID == contract.ID {
-			break
+		for _, c := range rc.ActiveContracts {
+			if c.ID == contract.ID {
+				return errors.New("Contract not cancelled, contract found in Active Contracts")
+			}
 		}
-		if i == len(rc.InactiveContracts) {
-			t.Fatal("Contract not found in Inactive Contracts")
+		i := 1
+		for _, c := range rc.InactiveContracts {
+			if c.ID == contract.ID {
+				break
+			}
+			if i == len(rc.InactiveContracts) {
+				return errors.New("Contract not found in Inactive Contracts")
+			}
+			i++
 		}
-		i++
+		return nil
+	})
+	if err != nil {
+		t.Fatal(err)
 	}
 }