Skip to content

Improve unit test coverage for the pkl package #222

Description

@NawafSwe

Hi! I'd like to contribute additional unit tests to improve code coverage for the pkl package.

Current coverage

Package Coverage
pkl 63.9%
pkl/internal 63.4%
pkl/internal/msgapi 0.0%

Areas with low or no coverage

  • external_reader.go — 0% coverage across all functions (Run, Close, listen, handleReadResource, handleReadModule, etc.)
  • errors.goIs, Error, and Unwrap methods at 0%
  • logger.goFormatLogMessage, Trace, Warn at 0%
  • values.goGoDuration, String methods at 0–30%
  • decode_primitives.godecodeInt64, decodeUint64 at 0%
  • evaluator_manager.goGetVersion, interrupt, killProcess at 0–33%
  • internal/msgapi — entire package at 0%

I'd like to open a PR adding unit tests targeting the gaps listed above. Happy to align on priorities or testing conventions before submitting. 🙌

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions