concord-consortium / codap

CODAP (Common Online Data Analysis Platform)
MIT License
94 stars 38 forks source link

182089941 log events #1378

Closed eireland closed 1 month ago

eireland commented 1 month ago

Adds storing document metadata in the properties property of the document model. Adds logger library to send log messages to the log server (currently using concordqa server) Adds some log messages to some actions for testing (adding a tile, changing a tile title, and closing a tile), though the actual language is still different from V2

Things not included:

  1. Adding run_remote-endpoint to the log message
  2. Adding parameters to the log message
cypress[bot] commented 1 month ago



Test summary

200 0 29 0Flakiness 0


Run details

Project codap-v3
Status Passed
Commit 0dc1a6cf0c
Started Jul 30, 2024 2:53 PM
Ended Jul 30, 2024 3:02 PM
Duration 08:54 💡
OS Linux Ubuntu -
Browser Chrome 126

View run in Cypress Cloud ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Cloud

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 87.23404% with 18 lines in your changes missing coverage. Please review.

Project coverage is 82.90%. Comparing base (fcdf2bf) to head (805f5cf). Report is 19 commits behind head on main.

Files Patch % Lines
v3/src/lib/logger.ts 87.27% 7 Missing :warning:
v3/src/models/app-state.ts 63.63% 3 Missing and 1 partial :warning:
v3/src/v2/codap-v2-document.ts 84.21% 3 Missing :warning:
v3/src/components/slider/slider-thumb.tsx 0.00% 1 Missing :warning:
v3/src/components/slider/use-slider-animation.tsx 66.66% 1 Missing :warning:
...rc/data-interactive/handlers/collection-handler.ts 0.00% 1 Missing :warning:
v3/src/data-interactive/handlers/item-handler.ts 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1378 +/- ## ========================================== - Coverage 85.74% 82.90% -2.85% ========================================== Files 522 523 +1 Lines 25582 25672 +90 Branches 6534 6999 +465 ========================================== - Hits 21935 21283 -652 - Misses 3493 4075 +582 - Partials 154 314 +160 ``` | [Flag](https://app.codecov.io/gh/concord-consortium/codap/pull/1378/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=concord-consortium) | Coverage Δ | | |---|---|---| | [cypress](https://app.codecov.io/gh/concord-consortium/codap/pull/1378/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=concord-consortium) | `69.10% <83.33%> (-3.48%)` | :arrow_down: | | [jest](https://app.codecov.io/gh/concord-consortium/codap/pull/1378/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=concord-consortium) | `53.41% <69.50%> (-0.15%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=concord-consortium#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.