tarantool / go-tarantool / 25767043469

75%

master: 76%

Build Type

Pull #588

github

Committed by bigbes

Commit Message

api: unexport Stream fields

Make Stream.Id and Stream.Conn unexported, turning Stream into an
opaque handle. This enforces encapsulation and prevents unsafe direct
mutation (e.g. stream.Conn = nil). Neither the stream identifier nor
the underlying connection is reachable from outside the package —
callers should hold their own *Connection reference if they need it.

- Rename Stream.Id/Stream.Conn to Stream.id/Stream.conn.
- Drop TestStream_IdValues: it only existed to push synthetic stream
  ids through a Stream; MP_UINT encoding is covered elsewhere.
- Document the breaking change in CHANGELOG.md and MIGRATION.md.

Closes #471

Pull Request Pull Request #588: api: unexport Stream fields, add Id() accessor

Coverage Stats

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

7 existing lines in 1 file now uncovered.

3118 of 4136 relevant lines covered (75.39%)

10295.84 hits per line