Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

test: added e2e test for iot systems #1750

Merged
merged 2 commits into from
Dec 13, 2023
Merged

test: added e2e test for iot systems #1750

merged 2 commits into from
Dec 13, 2023

Conversation

LavenderQAQ
Copy link
Member

@LavenderQAQ LavenderQAQ commented Oct 25, 2023 •
edited
Loading

What type of PR is this?

/kind enhancement
/sig iot

Fixes: #1545

What this PR does / why we need it:

Added e2e test for iot systems.

@openyurt-bot openyurt-bot added do-not-merge/work-in-progress do-not-merge/work-in-progress kind/enhancement kind/enhancement labels Oct 25, 2023
@openyurt-bot openyurt-bot added the sig/iot sig/iot label Oct 25, 2023
@openyurt-bot openyurt-bot requested a review from DrmagicE October 25, 2023 06:47
@openyurt-bot openyurt-bot added the size/M size/M: 30-99 label Oct 25, 2023
@openyurt-bot openyurt-bot added size/L size/L: 100-499 and removed size/M size/M: 30-99 labels Oct 25, 2023
@codecov
Copy link

codecov bot commented Oct 25, 2023 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (d67ebef) 52.11% compared to head (645b22e) 52.16%.
Report is 2 commits behind head on master.

Additional details and impacted files 
@@            Coverage Diff             @@
##           master    #1750      +/-   ##
==========================================
+ Coverage   52.11%   52.16%   +0.05%     
==========================================
  Files         172      172              
  Lines       20836    20836              
==========================================
+ Hits        10858    10870      +12     
+ Misses       9016     9005      -11     
+ Partials      962      961       -1
Flag Coverage Δ
unittests 52.16% <ø> (+0.05%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@LavenderQAQ LavenderQAQ force-pushed the test/iot-e2e branch 20 times, most recently from e9101d5 to 618afde Compare October 28, 2023 12:34
@LavenderQAQ LavenderQAQ force-pushed the test/iot-e2e branch 5 times, most recently from 52762b6 to a5bce7f Compare November 23, 2023 16:57
@LavenderQAQ
Copy link
Member Author

/rerun

@LavenderQAQ LavenderQAQ force-pushed the test/iot-e2e branch 8 times, most recently from f24aced to 6fdf798 Compare November 24, 2023 06:21
@LavenderQAQ
Copy link
Member Author

/rerun

@LavenderQAQ
Copy link
Member Author

/rerun

1 similar comment
@LavenderQAQ
Copy link
Member Author

/rerun

@sonarqubecloud SonarQubeCloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.4% 0.4% Duplication

@Rui-Gan
Copy link
Member

Rui-Gan commented Nov 25, 2023

/lgtm

@openyurt-bot openyurt-bot added the lgtm lgtm label Nov 25, 2023
@openyurt-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: LavenderQAQ, rambohe-ch, Rui-Gan, wangxye

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [LavenderQAQ,rambohe-ch]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@rambohe-ch rambohe-ch merged commit 74b0a47 into openyurtio:master Dec 13, 2023
14 checks passed
@LavenderQAQ LavenderQAQ deleted the test/iot-e2e branch January 30, 2024 05:35
zyjhtangtang pushed a commit to zyjhtangtang/openyurt that referenced this pull request Apr 16, 2024
@LavenderQAQ @zyjhtangtang
test: added e2e test for iot systems (openyurtio#1750)
69e3991 
* test: added e2e test for iot systems

Signed-off-by: LavenderQAQ <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rambohe-ch rambohe-ch rambohe-ch approved these changes

@Rui-Gan Rui-Gan Rui-Gan approved these changes

@wangxye wangxye wangxye approved these changes

@charleszheng44 charleszheng44 Awaiting requested review from charleszheng44

@DrmagicE DrmagicE Awaiting requested review from DrmagicE

Assignees

@Rui-Gan Rui-Gan

Labels
approved approved kind/enhancement kind/enhancement lgtm lgtm sig/iot sig/iot size/L size/L: 100-499
Projects
IoT-v1.5
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[feature request] Move e2e test for PlatformAdmin from yurt-edgex-manager to yurt-manager
5 participants
@LavenderQAQ @openyurt-bot @wangxye @Rui-Gan @rambohe-ch