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

🌱 Included tests for accessor #3178

Merged
merged 2 commits into from
Jun 21, 2023

Conversation

naveensrinivasan
Copy link
Member

  • Add test file for Token Accessor

What kind of change does this PR introduce?

(Is it a bug fix, feature, docs update, something else?)

What is the current behavior?

What is the new behavior (if this is a feature change)?**

  • Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes

Special notes for your reviewer

Does this PR introduce a user-facing change?

For user-facing changes, please add a concise, human-readable release note to
the release-note

(In particular, describe what changes users might need to make in their
application as a result of this pull request.)


@codecov
Copy link

codecov bot commented Jun 18, 2023

Codecov Report

Merging #3178 (cdd721b) into main (b4c197c) will increase coverage by 0.07%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3178      +/-   ##
==========================================
+ Coverage   62.74%   62.82%   +0.07%     
==========================================
  Files         168      168              
  Lines       12586    12586              
==========================================
+ Hits         7897     7907      +10     
+ Misses       4259     4248      -11     
- Partials      430      431       +1     

@naveensrinivasan naveensrinivasan force-pushed the naveen/unit-tests/tokens/accessor branch from a9d337e to c22be31 Compare June 21, 2023 15:59
@naveensrinivasan naveensrinivasan temporarily deployed to integration-test June 21, 2023 15:59 — with GitHub Actions Inactive
- Add test file for Token Accessor

Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
@naveensrinivasan naveensrinivasan force-pushed the naveen/unit-tests/tokens/accessor branch from c22be31 to cdd721b Compare June 21, 2023 21:47
@naveensrinivasan naveensrinivasan enabled auto-merge (squash) June 21, 2023 21:47
@naveensrinivasan naveensrinivasan temporarily deployed to integration-test June 21, 2023 21:48 — with GitHub Actions Inactive
@naveensrinivasan naveensrinivasan merged commit 4edb078 into main Jun 21, 2023
37 checks passed
@naveensrinivasan naveensrinivasan deleted the naveen/unit-tests/tokens/accessor branch June 21, 2023 21:57
ashearin pushed a commit to kgangerlm/scorecard-gitlab that referenced this pull request Nov 13, 2023
* 🌱 Included tests for accessor

- Add test file for Token Accessor

Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>

* Code review comments.

Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>

---------

Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
Signed-off-by: Allen Shearin <allen.p.shearin@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants