Skip to content
New issue

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

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

Already on GitHub? # to your account

Move Rust API unit tests to a separate directory #1227

Conversation

vinnamkim
Copy link
Contributor

@vinnamkim vinnamkim commented Dec 14, 2023

Summary

How to test

$ cd rust
$ cargo test

Checklist

  • I have added unit tests to cover my changes.​
  • I have added integration tests to cover my changes.​
  • I have added the description of my changes into CHANGELOG.​
  • I have updated the documentation accordingly

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below).
# Copyright (C) 2023 Intel Corporation
#
# SPDX-License-Identifier: MIT

@vinnamkim vinnamkim marked this pull request as ready for review December 14, 2023 05:45
@vinnamkim vinnamkim requested review from a team as code owners December 14, 2023 05:45
@vinnamkim vinnamkim requested review from jihyeonyi and removed request for a team December 14, 2023 05:45
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Copy link

codecov bot commented Dec 14, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (6b567b7) 80.44% compared to head (049d952) 80.44%.
Report is 1 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1227   +/-   ##
========================================
  Coverage    80.44%   80.44%           
========================================
  Files          269      269           
  Lines        29915    29915           
  Branches      5850     5850           
========================================
  Hits         24066    24066           
  Misses        4487     4487           
  Partials      1362     1362           
Flag Coverage Δ
macos-11_Python-3.8 79.10% <ø> (ø)
ubuntu-20.04_Python-3.8 80.44% <ø> (ø)
windows-2022_Python-3.8 80.41% <ø> (ø)

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.

@vinnamkim vinnamkim added the ENHANCE Enhancement of existing features label Dec 14, 2023
@vinnamkim vinnamkim added this to the 1.6.0 milestone Dec 14, 2023
Copy link
Contributor

@wonjuleee wonjuleee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the quick update.

@vinnamkim vinnamkim merged commit 76467bc into openvinotoolkit:develop Dec 15, 2023
7 checks passed
@yunchu yunchu modified the milestones: 1.6.0, 2.0.0 Mar 28, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
ENHANCE Enhancement of existing features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants