search

compiler-research / xeus-cpp

Jupyter kernel for the C++ programming language
BSD 3-Clause "New" or "Revised" License
17 stars 24 forks source link

Add notebook tests #142

Closed tharun571 closed 1 month ago

tharun571 commented 3 months ago

Description

Using papermill library to execute the notebooks and check the necessary ouputs.

Tryng to fix #122

Type of change

Please tick all options which are relevant.

codecov-commenter commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@4745f72). Learn more about missing BASE report.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/compiler-research/xeus-cpp/pull/142/graphs/tree.svg?width=650&height=150&src=pr&token=9KM610P5A4&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research)](https://app.codecov.io/gh/compiler-research/xeus-cpp/pull/142?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research) ```diff @@ Coverage Diff @@ ## main #142 +/- ## ======================================= Coverage ? 82.72% ======================================= Files ? 17 Lines ? 625 Branches ? 61 ======================================= Hits ? 517 Misses ? 108 Partials ? 0 ```
vgvassilev commented 2 months ago

Is this pull request still struggling to show a good user message if a notebook execution ends up with unexpected results or we are seeing something else?

anutosh491 commented 1 month ago

Thanks @tharun571 for fixing this. Raised an issue to propagate the idea here to xeus https://github.com/jupyter-xeus/xeus/issues/407