tarantool / go-tarantool

79%

master: 74%

LAST BUILD ON BRANCH askalt/gh-330-support-operation-data

branch: askalt/gh-330-support-operation-data

CHANGE BRANCH

x

Reset

• askalt/gh-330-support-operation-data
• ligurio/code-coverage
• master
• ligurio/gh-123-documentation
• ligurio/gh-121-release-process
• ligurio/gofmt
• vr009/gh-62-minimal-sql-support
• vr009/gh-122-add-perf-tests
• vr009/gh-142-fix-lint-warnings
• ligurio/gh-142-workflow
• ligurio/gh-118-datetime-support
• oleg-jukovec/gh-126-request-object-select
• ligurio/gh-96-decimal-support
• ligurio/better-code-coverage
• AnaNek/master-discovery
• oleg-jukovec/gh-155-ssl-support
• ligurio/review-checklist
• DifferentialOrange/fix-readme-go-1.13
• ligurio/documentation-fixes
• AnaNek/gh-125-rename-default-call
• vr009/gh-48-add-context-support
• gh-67-iproto-push-support
• Totktonada/remove-trailing-whilespaces
• oleg-jukovec/gh-124-msgpack-v5
• ligurio/fix-coverage
• vr009/gh-117-add-prepared-statements
• oleg-jukovec/golangci-lint-goimports
• oleg-jukovec/release-1.6.0
• refs/tags/v1.6.0
• AnaNek/gh-101-interactive-transactions
• oleg-jukovec/2.10-workaround-delete
• oleg-jukovec/connect-validation-helper
• oleg-jukovec/only-do-request
• oleg-jukovec/remove-NewErrorFuture
• oleg-jukovec/gh-191-fix-datetime-range
• oleg-jukovec/ci-tnt-1.6
• oleg-jukovec/fix-centos7-build
• oleg-jukovec/race-condition-fixes
• oleg-jukovec/release-1.7.0
• oleg-jukovec/gh-163-datetime-tz
• refs/tags/v1.7.0
• oleg-jukovec/gh-201-fix-decimal-godoc
• oleg-jukovec/gofmt-1.19
• oleg-jukovec/gh-165-interval-support
• oleg-jukovec/gh-177-add-queue-missing
• oleg-jukovec/release-1.8.0
• refs/tags/v1.8.0
• oleg-jukovec/connection_pool_fixes
• oleg-jukovec/gh-178-connection-pool-callbacks
• oleg-jukovec/gh-176-queue-loves-pool
• oleg-jukovec/gettyped-after-get
• vr009/fix-typo
• oleg-jukovec/datetime-non-fixed
• oleg-jukovec/usage-test-func
• oleg-jukovec/release-1.9.0
• oleg-jukovec/fix-flaky-queue-pool
• oleg-jukovec/datetime-interval-offsets
• oleg-jukovec/gh-223-invalid-streamid
• oleg-jukovec/gh-196-get-typed-examples
• refs/tags/v1.9.0
• DifferentialOrange/gh-120-iproto-id
• DifferentialOrange/gh-209-error-type
• better0fdead/gh-157-macos
• oleg-jukovec/gh-119-watchers-support
• oleg-jukovec/fix-ee-tests-ref
• oleg-jukovec/fix-decimal-export
• DifferentialOrange/gh-215-session-settings
• DifferentialOrange/gh-214-graceful-shutdown
• DifferentialOrange/gh-214-graceful-shutdown-v2
• oleg-jukovec/gh-243-pap-sha256
• oleg-jukovec/v2
• v2_WIP
• oleg-jukovec/1.10-support
• oleg-jukovec/gh-244-fix-flaky-withcontext
• oleg-jukovec/gh-231-golint-ci-output
• oleg-jukovec/gh-234-flaky-multi-TestDisconnectAll
• oleg-jukovec/release-1.10.0
• refs/tags/v1.10.0
• oleg-jukovec/gh-218-schema-data-race
• oleg-jukovec/gh-246-pagination
• AnaNek/gh-108-crud-api
• psergee/no-gh-test-branch
• oleg-jukovec/connection-dialer
• AnaNek/gh-108-alternative-crud-api
• oleg-jukovec/gh-260-fix-M1-build
• oleg-jukovec/gh-271-crud-improvments
• oleg-jukovec/gh-272-pool-reconnect-bug
• oleg-jukovec/crud-conditions-fix
• oleg-jukovec/release-1.11.0
• better0fdead/flaky-macOS
• oleg-jukovec/fix-ci
• oleg-jukovec/gh-288-fix-flaky-crud
• oleg-jukovec/gh-284-fix-newwatcher
• refs/tags/v1.11.0
• oleg-jukovec/gh-293-replicaset-uuid
• oleg-jukovec/gh-257-graceful-close
• oleg-jukovec/fix-seq-scan
• oleg-jukovec/gh-290-pool-add-remove
• oleg-jukovec/release-1.12.0
• oleg-jukovec/fix-flaky-closegraceful
• oleg-jukovec/flaky-decimal-testselect
• refs/tags/v1.12.0
• grafin/is_nullable_test
• oleg-jukovec/contibuting-update
• oleg-jukovec/forks-disable-ee
• oleg-jukovec/gh-241-deprecated
• oleg-jukovec/gh-238-immutable
• oleg-jukovec/merge-close
• i-cod/bump-action-version
• oleg-jukovec/linters
• oleg-jukovec/gh-129-unexpected
• oleg-jukovec/gh-309-round-robin-race
• locker/queue-identify-arg-fix
• oleg-jukovec/decimal-fix
• DifferentialOrange/ssl-key-decryption
• oleg-jukovec/test-go-openssl
• oleg-jukovec/gh-322-fix-after
• oleg-jukovec/gh-321-fix-after
• oleg-jukovec/gh-320-fix-after
• release/1.12
• oleg-jukovec/release-1.12.1
• refs/tags/v1.12.1
• DerekBum-gh-136-add-context-to-connection-create
• DifferentialOrange/crud-improvements
• DifferentialOrange/no-gh-crud-schema
• DerekBum/gh-337-support-iproto-watch-once
• oleg-jukovec/gh-no-sql-case-sensitive
• DifferentialOrange/gh-342-float-timeout
• DerekBum/gh-338-support-space-and-index-names
• oleg-jukovec/fix-master-cache
• fix-typo
• DifferentialOrange/no-gh-crud-yield-every
• DifferentialOrange/no-gh-crud-schema-opts
• askalt/gh-321-socket-fd
• DerekBum/gh-348-fix-splice-update-operation
• DerekBum/gh-7-connection-schema-getter
• oleg-jukovec/gh-324-deprecate-session-push
• oleg-jukovec/gh-357-fix-ssl-diable-build
• better0fdead/no-gh-svacer
• DerekBum/gh-237-ability-to-mock-connections
• oleg-jukovec/bump-go-iproto
• oleg-jukovec/release-1.12.2
• refs/tags/v1.12.2
• 0x501D/ci-use-tt
• DerekBum/gh-301-disable-ssl-by-default
• oleg-jukovec/gh-356-pool-connect-update-interface
• oleg-jukovec/gh-372-pool-connect-improve
• oleg-jukovec/gh-365-crud-typed-batches
• oleg-jukovec/gh-376-pool-do-by-name
• DerekBum/gh-301-remove-ssl
• DerekBum/gh-378-bump-go-version
• oleg-jukovec/remove-future-err
• oleg-jukovec/release-2.0.0
• refs/tags/v2.0.0
• oleg-jukovec/pool-remove-does-not-call-deactivated
• DerekBum/gh-380-get-status-from-watch-once
• oleg-jukovec/release-2.1.0
• refs/tags/v2.1.0
• ylobankov/bump-actions-version
• nshi/fix-error-payload-tests
• nshi/fix-error-payload-tests-more
• nshi/fix-compare-box-errors-invocation
• install_crud_master
• dmyger/gh-399-add-support-IPROTO_INSERT_ARROW
• dmyger/gh-412-api-update-Arrow-constants-with-iproto
• oleg-jukovec/gh-415-fix-arrow-error

Build # 5948293621

Build Type

Pull #331

github

Committed by askalt

Commit Message

crud: support `operation_data` field in errors

This patch adds `operation_data` decoding for the `crud.Error`.

The `operation_data` type is determined as `rowType` in `crud.Result`.

Also, according to [1], an error can contain one of the following:
- an error
- an array of errors
- nil

So the error decoding logic has been modified to consider each case,
in order to avoid comparing an error to nil.

1. https://github.com/tarantool/crud/tree/master#api

Closes #330

Pull Request Pull Request #331: crud: support `operation_data` field in errors

Run Details

26 of 26 new or added lines in 2 files covered. (100.0%)

5336 of 6718 relevant lines covered (79.43%)

8235.89 hits per line