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.

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

First draft of register tests #149

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

First draft of register tests #149

wants to merge 3 commits into from

Conversation

pudelkoM
Copy link
Member

@pudelkoM pudelkoM commented Jan 8, 2021

Adds a simple read write test for the register in the int profile.

Also introduces the Stratum specific annotation for registers: @stratum("clear_interval_ms: 1")

TODO:

  • Dataplane tests

@codecov
Copy link

codecov bot commented Jan 8, 2021

Codecov Report

Merging #149 (e7540b7) into main (971bcb3) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #149   +/-   ##
=========================================
  Coverage     69.91%   69.91%           
  Complexity      223      223           
=========================================
  Files            17       17           
  Lines          1622     1622           
  Branches        133      133           
=========================================
  Hits           1134     1134           
  Misses          414      414           
  Partials         74       74           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 971bcb3...c528a56. Read the comment docs.

@ccascone ccascone added the WIP Do not merge! label Jan 13, 2021
@onf-bot
Copy link
Collaborator

onf-bot commented Oct 28, 2021

Can one of the admins verify this patch? For help please reach out to [email protected]. The list of admins can be updated in the Jenkins configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP Do not merge!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants