SmetDenis / Clojure-SICP / 7864893950

93%

Build Type

push

github

Committed by web-flow

Commit Message

Enhance test coverage (#164)

* Enhance test coverage

* Remove splint from lint sequence in bb.edn

The changes of this commit are made to the lint sequence inside bb.edn file. The 'splint' style check was removed from the sequencing. Now, the lint sequence now includes only 'cljfmt', 'cljstyle', 'eastwood', and 'kondo'. This simplifies the sequence and prioritizes the rest of the style checks.

* Refactor complex number functionality and add tests

The code was restructured to reorder, rename, and modify complex number functions in Clojure for two models ("Ben's way" and "Alyssa's way"). Float typecasting was added to some functions to ensure accurate calculations. Corresponding tests were also implemented in a new test file to validate and ensure the correctness of these revised functions. These changes enhance code clarity and testing comprehensiveness in handling complex numbers.

* Reformat imports in book_2_4_test.clj

The :require block in the testing file book_2_4_test.clj has been reformatted to improve readability. No functionality was changed in this commit, just the way the code is organized for aesthetically pleasing and more manageable structure.

* Refactor exception handling across multiple test files

Refactored exception handling in tests across multiple modules, replacing previous implementations with the centralized and enhanced `is-exception?` macro. This macro captures the exception type and message and eliminates repeated code, enhancing code cleanliness and maintainability.

* Modify exception checking in tests

Refactored the way exceptions are checked in tests across various modules. This was done by changing from direct exception checks to using the `is-exception?` macro. This improves code readability and makes the test scripts more maintainable.

* Correct naming for test coverage workflow

Fixed a typo and improved the naming pattern for the "test coverage" workflow in the main GitHub actions YAML... (continued)

Run Details

54 of 57 new or added lines in 6 files covered. (94.74%)

1 existing line in 1 file now uncovered.

2298 of 2458 relevant lines covered (93.49%)

3.68 hits per line