onflow / kitty-items

Kitty Items: CryptoKitties Sample App
https://kitty-items.onflow.org
Apache License 2.0
408 stars 213 forks source link

Cadence error running purchase tx on testnet #304

Closed chasefleming closed 1 year ago

chasefleming commented 2 years ago

Describe the bug

Cannot purchase item after mint on Testnet

To Reproduce

Expected behavior Should purchase

Screenshots/Logs/Output

{
    "block_id": "3503f5b9821cc43b491afc89da92f5589219ed4cf2fc55a31629a43a70a0b943",
    "execution": "Failure",
    "status": "Sealed",
    "status_code": 1,
    "error_message": "[Error Code: 1101] cadence runtime error Execution failed:\n  --\u003e ad9a4d57e0db5c76cea962c71727a199f65a6b6c31dfafe13c93038931be03a3:50:32\n   |\n50 |     self.kittyItemsCollection = getOrCreateCollection(account: account)\n   |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n\nerror: unexpectedly found non-`KittyItems.Collection` while force-casting value\n --\u003e ad9a4d57e0db5c76cea962c71727a199f65a6b6c31dfafe13c93038931be03a3:8:25\n  |\n8 |   if let collectionRef = account.borrow\u003c\u0026KittyItems.Collection\u003e(from: KittyItems.CollectionStoragePath) {\n  |                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n",
    "computation_used": "0",
    "events": [
        {
            "type": "A.7e60df042a9c0868.FlowToken.TokensWithdrawn",
            "transaction_id": "ad9a4d57e0db5c76cea962c71727a199f65a6b6c31dfafe13c93038931be03a3",
            "transaction_index": "0",
            "event_index": "0",
            "payload": "eyJ0eXBlIjoiRXZlbnQiLCJ2YWx1ZSI6eyJpZCI6IkEuN2U2MGRmMDQyYTljMDg2OC5GbG93VG9rZW4uVG9rZW5zV2l0aGRyYXduIiwiZmllbGRzIjpbeyJuYW1lIjoiYW1vdW50IiwidmFsdWUiOnsidHlwZSI6IlVGaXg2NCIsInZhbHVlIjoiMC4wMDAwMDM5OSJ9fSx7Im5hbWUiOiJmcm9tIiwidmFsdWUiOnsidHlwZSI6Ik9wdGlvbmFsIiwidmFsdWUiOnsidHlwZSI6IkFkZHJlc3MiLCJ2YWx1ZSI6IjB4ZjA4NmE1NDVjZTNjNTUyZCJ9fX1dfX0K"
        },
        {
            "type": "A.7e60df042a9c0868.FlowToken.TokensDeposited",
            "transaction_id": "ad9a4d57e0db5c76cea962c71727a199f65a6b6c31dfafe13c93038931be03a3",
            "transaction_index": "0",
            "event_index": "1",
            "payload": "eyJ0eXBlIjoiRXZlbnQiLCJ2YWx1ZSI6eyJpZCI6IkEuN2U2MGRmMDQyYTljMDg2OC5GbG93VG9rZW4uVG9rZW5zRGVwb3NpdGVkIiwiZmllbGRzIjpbeyJuYW1lIjoiYW1vdW50IiwidmFsdWUiOnsidHlwZSI6IlVGaXg2NCIsInZhbHVlIjoiMC4wMDAwMDM5OSJ9fSx7Im5hbWUiOiJ0byIsInZhbHVlIjp7InR5cGUiOiJPcHRpb25hbCIsInZhbHVlIjp7InR5cGUiOiJBZGRyZXNzIiwidmFsdWUiOiIweDkxMmQ1NDQwZjdlMzc2OWUifX19XX19Cg=="
        },
        {
            "type": "A.912d5440f7e3769e.FlowFees.FeesDeducted",
            "transaction_id": "ad9a4d57e0db5c76cea962c71727a199f65a6b6c31dfafe13c93038931be03a3",
            "transaction_index": "0",
            "event_index": "2",
            "payload": "eyJ0eXBlIjoiRXZlbnQiLCJ2YWx1ZSI6eyJpZCI6IkEuOTEyZDU0NDBmN2UzNzY5ZS5GbG93RmVlcy5GZWVzRGVkdWN0ZWQiLCJmaWVsZHMiOlt7Im5hbWUiOiJhbW91bnQiLCJ2YWx1ZSI6eyJ0eXBlIjoiVUZpeDY0IiwidmFsdWUiOiIwLjAwMDAwMzk5In19LHsibmFtZSI6ImluY2x1c2lvbkVmZm9ydCIsInZhbHVlIjp7InR5cGUiOiJVRml4NjQiLCJ2YWx1ZSI6IjEuMDAwMDAwMDAifX0seyJuYW1lIjoiZXhlY3V0aW9uRWZmb3J0IiwidmFsdWUiOnsidHlwZSI6IlVGaXg2NCIsInZhbHVlIjoiMC4wMDAwMDA2MCJ9fV19fQo="
        }
    ],
    "_links": {
        "_self": "/v1/transaction_results/ad9a4d57e0db5c76cea962c71727a199f65a6b6c31dfafe13c93038931be03a3"
    }
}

Environment (Optional - Version, OS, Browser, etc.) Testnet

chasefleming commented 2 years ago

Did the storage path change? I made a new blocto account and it worked.

10thfloor commented 1 year ago

@chasefleming Yes, this could be the issue.

codingone21 commented 1 year ago

Yep, it did change: https://github.com/onflow/kitty-items/commit/e2a5c9148cb031c2d1c9f9465e8ceb2a173bd666

I've confirmed that we are able to purchase an item without running into this specific error with a new blocto account! I will be closing this issue, and update the README to include this as a warning