Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Possible bug in Virtual Objects #2533

Closed
katelynsills opened this issue Feb 24, 2021 · 1 comment
Closed

Possible bug in Virtual Objects #2533

katelynsills opened this issue Feb 24, 2021 · 1 comment
Assignees
Labels
bug Something isn't working SwingSet package: SwingSet

Comments

@katelynsills
Copy link
Contributor

Describe the bug

#2526 makes payments virtual objects. I think I'm doing everything right, but in the one test that is in a vat, I get an error in the bowels of SwingSet (printed below). I did notice that we are testing the identity of payments in issuer.js so that needs to be changed, but I think that should not produce this error.

Temporary logging of sent error (Error#1)
Nested error
  Error#1: syscall vatstoreSet suffered error, shutdown commencing
     at doSyscall (packages/SwingSet/src/kernel/vatManager/syscall.js:88:13)
    at Object.vatstoreSet (packages/SwingSet/src/kernel/vatManager/syscall.js:104:31)
    at Object.delete (packages/SwingSet/src/kernel/virtualObjectManager.js:303:19)
    at eval (packages/ERTP/src/issuer.js:175:53)
    at Array.forEach (<anonymous>)
    at reallocate (packages/ERTP/src/issuer.js:175:14)
    at eval (packages/ERTP/src/issuer.js:250:29)
    at eval (packages/eventual-send/src/track-turns.js:64:18)
  
  Error#1 ERROR_NOTE: Thrown from: (Error#2) : 2 . 0
  Error#1 ERROR_NOTE: Rejection from: (Error#3) : 13 . 0
  Error#1 ERROR_NOTE: Sent as error:liveSlots:v2#1
  Nested 2 errors under Error#1
    Error#2: Event: 1.1
       at Object.trackTurns (packages/eventual-send/src/track-turns.js:47:24)
      at Function.E.when (packages/eventual-send/src/E.js:102:47)
      at Alleged: moola issuer.split (packages/ERTP/src/issuer.js:241:23)
    
    Error#3: Event: 12.1
       at Object.deliver (packages/SwingSet/src/kernel/liveSlots.js:480:28)
      at eval (packages/SwingSet/src/kernel/vatManager/deliver.js:51:48)
    
##### KERNEL PANIC: kp40.policy panic: rejected {"body":"{\"@qclass\":\"error\",\"name\":\"Error\",\"message\":\"vat terminated\"}","slots":[]} #####
  swingsetTests › splitPayments › splitPayments › test splitPayments

  Rejected promise returned by test. Reason:

  Error {
    message: 'kernel panic kp40.policy panic: rejected {"body":"{\\"@qclass\\":\\"error\\",\\"name\\":\\"Error\\",\\"message\\":\\"vat terminated\\"}","slots":[]}',
  }

  › panic (packages/SwingSet/src/kernel/kernel.js:260:26)
  › doResolve (packages/SwingSet/src/kernel/kernel.js:332:9)
  › resolveToError (packages/SwingSet/src/kernel/kernel.js:338:5)
  › terminateVat (packages/SwingSet/src/kernel/kernel.js:354:9)
  › processQueueMessage (packages/SwingSet/src/kernel/kernel.js:573:9)
  › async Object.run (packages/SwingSet/src/kernel/kernel.js:926:7)
  › async main (packages/ERTP/test/swingsetTests/splitPayments/test-splitPayments.js:9:3)
  › async packages/ERTP/test/swingsetTests/splitPayments/test-splitPayments.js:21:16

To Reproduce

Steps to reproduce the behavior:

  1. Check out branch 1830-ERTP-virtual-objects-payments
  2. yarn install and yarn build at root
  3. cd packages/ERTP and yarn test -m "test splitPayments"

Expected behavior

The test should pass.

@katelynsills katelynsills added the bug Something isn't working label Feb 24, 2021
@katelynsills katelynsills added the SwingSet package: SwingSet label Feb 24, 2021
@katelynsills katelynsills added this to the Beta Initial Launch milestone Feb 24, 2021
@FUDCo
Copy link
Contributor

FUDCo commented Feb 25, 2021

Closed by #2539

@FUDCo FUDCo closed this as completed Feb 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working SwingSet package: SwingSet
Projects
None yet
Development

No branches or pull requests

3 participants