dynein icon indicating copy to clipboard operation
dynein copied to clipboard

Add integration test for each command

Open ryota-sakamoto opened this issue 1 year ago • 2 comments

Background

We need to check compatibility through test before pull request is merged. For now, some integration tests are implemented but we haven't covered all of the command. So we need to add integration test for all of the command.

Tasks

  • [ ] admin:
    • [ ] admin create: WIP (@KIrie-0217)
    • [ ] admin delete: #194
    • [x] admin desc: #161
    • [x] admin list: #166
    • [ ] admin update:
  • [ ] backup: WIP (@ryota-sakamoto)
  • [x] bootstrap: #153
  • [x] bwrite: #190
  • [x] config: #148
  • [x] del: #144
  • [x] desc: #151
  • [ ] export: #181
  • [x] get: #142
  • [ ] import:
  • [x] list: #158
    • [x] show current table: #180
  • [ ] put: WIP (@tmyoda )
  • [ ] query: #218
  • [ ] restore:
  • [x] scan: #217
  • [ ] upd: WIP (@StoneDot)
  • [ ] use:

ryota-sakamoto avatar Jul 07 '23 10:07 ryota-sakamoto

@kzym-w, Could you assess the quality of the test for the query command? I have added tests in #187, so new tests may no longer be needed.

StoneDot avatar Apr 15 '24 01:04 StoneDot

I have contacted kzym-w and I will take over the tests for the query.

StoneDot avatar Apr 29 '24 09:04 StoneDot

We have added integration tests for all commands. I will close this.

StoneDot avatar May 17 '24 08:05 StoneDot