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

Document execution of testing in golang/redisConn subdirectory #102

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rilma
Copy link
Contributor

@rilma rilma commented Apr 8, 2023

This PR improves documentation in golang/README.md, specifically for the execution of testing script in golang/redisConn subdirectory. No more confusions in the proper execution, this issue #101 must be closed.

@rilma ➜ /workspaces/redis-in-action/golang (issue/101-bug-in-redisConn-test) $ docker exec -it redis-in-action-golang go test ./redisConn/redisConn_test.go ./redisConn/redisConn.go -v
=== RUN   TestCheckVersion
--- PASS: TestCheckVersion (0.00s)
=== RUN   TestTxPipeline
=== RUN   TestTxPipeline/Tx_Example1:_tx.TxPipeline()
    redisConn_test.go:27: before tx incr key: 1
    redisConn_test.go:29: inside tx incr key: 0
    redisConn_test.go:39: after tx get key: 1
=== RUN   TestTxPipeline/Tx_Example2:_tx.Pipelined()
    redisConn_test.go:47: before tx incr key: 1
    redisConn_test.go:49: inside tx incr key: 0
    redisConn_test.go:58: after tx get key: 1
=== RUN   TestTxPipeline/Tx_a_wrong_example
    redisConn_test.go:68: before tx incr key: 1
    redisConn_test.go:69: inside tx incr key: 0
    redisConn_test.go:80: after tx get key: 2
--- PASS: TestTxPipeline (0.01s)
    --- PASS: TestTxPipeline/Tx_Example1:_tx.TxPipeline() (0.00s)
    --- PASS: TestTxPipeline/Tx_Example2:_tx.Pipelined() (0.00s)
    --- PASS: TestTxPipeline/Tx_a_wrong_example (0.00s)
PASS
ok      command-line-arguments  0.017s
@rilma ➜ /workspaces/redis-in-action/golang (issue/101-bug-in-redisConn-test) $ 

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant