alibaba / open-local

cloud-native local storage management system for stateful workload, low-latency with simplicity
Apache License 2.0
464 stars 81 forks source link

feat: add e2e unittest framework #251

Closed dongjiang1989 closed 9 months ago

dongjiang1989 commented 10 months ago

Add e2e unittest framework and add One case for CRD Install

fix #248

codecov-commenter commented 10 months ago

Codecov Report

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

Comparison is base (833c992) 32.02% compared to head (dfbb975) 32.30%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #251 +/- ## ========================================== + Coverage 32.02% 32.30% +0.28% ========================================== Files 41 41 Lines 6426 6426 ========================================== + Hits 2058 2076 +18 + Misses 4079 4060 -19 - Partials 289 290 +1 ``` | [Flag](https://app.codecov.io/gh/alibaba/open-local/pull/251/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=alibaba) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/alibaba/open-local/pull/251/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=alibaba) | `32.30% <100.00%> (+0.28%)` | :arrow_up: | 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=alibaba#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.

dongjiang1989 commented 10 months ago

@peter-wangxu PTAL

peter-wangxu commented 10 months ago

Do you plan adding a example test case other than install CRD?

dongjiang1989 commented 10 months ago

Do you plan adding a example test case other than install CRD?

Yes. I will continue to add e2e cases.

dongjiang1989 commented 9 months ago

@peter-wangxu It's need to update token?

[2024-01-08T11:40:03.070Z] ['info'] Pinging Codecov: https://codecov.io/upload/v4?package=github-action-2.1.0-uploader-0.7.1&token=*******&branch=e2e-framework&build=7446893003&build_url=https%3A%2F%2Fgithub.com%2Falibaba%2Fopen-local%2Factions%2Fruns%2F7446893003%2Fjob%2F20258074232&commit=d8411a0ab6dfa9eb3ef6c75eca8cbc54579387b5&job=Build&pr=251&service=github-actions&slug=alibaba%2Fopen-local&name=&tag=&flags=unittests&parent=
[2024-01-08T11:40:03.374Z] ['error'] There was an error running the uploader: Error uploading to [https://codecov.io:](https://codecov.io/) Error: There was an error fetching the storage URL during POST: 404 - {'detail': ErrorDetail(string='Unable to locate build via Github Actions API. Please upload with the Codecov repository upload token to resolve issue.', code='not_found')}
Error: Codecov: Failed to properly upload: The process '/home/runner/work/_actions/codecov/codecov-action/v2/dist/codecov' failed with exit code 255
peter-wangxu commented 9 months ago

Did you pass local testing? If yes i am ok to merge this now.

dongjiang1989 commented 9 months ago

Did you pass local testing? If yes i am ok to merge this now.

Yes. Please merge it. I will continue to add e2e cases.