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 runner + example for dataplane tests #507

Merged
merged 2 commits into from
Dec 13, 2024
Merged

Test runner + example for dataplane tests #507

merged 2 commits into from
Dec 13, 2024

Conversation

DanG100
Copy link
Collaborator

@DanG100 DanG100 commented Dec 12, 2024

No description provided.

@DanG100 DanG100 requested a review from guoshiuan December 12, 2024 22:09
@DanG100 DanG100 changed the base branch from main to sub-port December 12, 2024 22:17
@DanG100 DanG100 changed the base branch from sub-port to main December 12, 2024 22:17
@DanG100 DanG100 changed the base branch from main to sub-port December 12, 2024 22:24
guoshiuan
guoshiuan reviewed Dec 13, 2024
View reviewed changes
integration_tests/saiutil/saiutil.go Show resolved Hide resolved
integration_tests/saiutil/saiutil.go Show resolved Hide resolved
integration_tests/saiutil/saiutil.go Outdated Show resolved Hide resolved
integration_tests/saiutil/saiutil.go Outdated Show resolved Hide resolved
integration_tests/dataplane/subintf/subintf_test.go Outdated Show resolved Hide resolved
Base automatically changed from sub-port to main December 13, 2024 18:42
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 13, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 12321358535

Details

  • 145 of 164 (88.41%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 2.755%
Changes Missing Coverage Covered Lines Changed/Added Lines %
integration_tests/saiutil/runner.go 38 43 88.37%
integration_tests/saiutil/saiutil.go 107 121 88.43%
Totals Coverage Status
Change from base Build 12321221542: 0.02%
Covered Lines: 19921
Relevant Lines: 723084
💛 - Coveralls

@DanG100 DanG100 merged commit f16ef38 into main Dec 13, 2024
10 checks passed
morrowc pushed a commit to morrowc/lemming that referenced this pull request Dec 21, 2024
@DanG100 @morrowc
Test runner + example for dataplane tests (openconfig#507)
308dbb8 
* Test runner + example for dataplane tests

* feedback
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@guoshiuan guoshiuan guoshiuan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DanG100 @guoshiuan