cube-js / cube

πŸ“Š Cube β€” Universal semantic layer platform for AI, BI, spreadsheets, and embedded analytics
https://cube.dev
Other
17.96k stars 1.78k forks source link

chore(cubesql): E2E - check that env variables are not empty #8965

Closed ovr closed 3 days ago

ovr commented 3 days ago

I figured out that CI is broken for contributors. I am not totally sure, but I belive it's related to that env variables are defined, while being empty.

image

https://github.com/cube-js/cube/pull/8947

codecov[bot] commented 3 days ago

Codecov Report

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

Project coverage is 82.65%. Comparing base (5fd13d1) to head (10b39e1). Report is 6 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #8965 +/- ## ========================================== - Coverage 82.66% 82.65% -0.01% ========================================== Files 221 221 Lines 78303 78303 ========================================== - Hits 64726 64722 -4 - Misses 13577 13581 +4 ``` | [Flag](https://app.codecov.io/gh/cube-js/cube/pull/8965/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cube-js) | Coverage Ξ” | | |---|---|---| | [cubesql](https://app.codecov.io/gh/cube-js/cube/pull/8965/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cube-js) | `82.65% <ΓΈ> (-0.01%)` | :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=cube-js#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.


🚨 Try these New Features: