search

vezel-dev / cathode

A terminal-centric replacement for the .NET console APIs.
https://docs.vezel.dev/cathode
BSD Zero Clause License
91 stars 7 forks source link

Investigate feasibility of testing the library #58

Closed alexrp closed 1 year ago

alexrp commented 2 years ago

Is unit testing the library actually doable and useful in practice?

Can we run a subset of the samples in CI at least?

alexrp commented 2 years ago

Some of the samples are now running in CI: https://github.com/alexrp/system-terminal/actions/runs/1645414546

alexrp commented 2 years ago

Some libraries that'll probably be useful for unit testing Cathode in particular:

alexrp commented 2 years ago

The testing utilities package is not yet shipping; see 2b7ed0c4e495ec67de1c850721b38889670517ed.

alexrp commented 1 year ago

Decided to revert the testing utilities package for now (8dca58a). The core terminal APIs still allow users to create their own derived types for testing purposes. Having thought about it, I'm just currently not convinced that an official package can add enough value for all users because different projects are likely going to have very different needs from the API surface of such types.

Might revisit this in the future if there's empirical evidence that an official package can make sense.