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

🌱 Gitlab: Move tests that connect to gitlab.com out of unit-tests #3221

Merged
merged 4 commits into from
Jun 29, 2023

Conversation

raghavkaul
Copy link
Contributor

What kind of change does this PR introduce?

Moves tests that connect to GitLab instances out of the unit test framework and into E2E. Unit tests should not connect to the network.

@codecov
Copy link

codecov bot commented Jun 27, 2023

Codecov Report

Merging #3221 (3f94927) into main (b2bc681) will increase coverage by 2.03%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3221      +/-   ##
==========================================
+ Coverage   62.93%   64.97%   +2.03%     
==========================================
  Files         168      168              
  Lines       12600    12600              
==========================================
+ Hits         7930     8187     +257     
+ Misses       4238     3959     -279     
- Partials      432      454      +22     

Signed-off-by: Raghav Kaul <[email protected]>
Signed-off-by: Raghav Kaul <[email protected]>
@raghavkaul raghavkaul temporarily deployed to integration-test June 29, 2023 18:57 — with GitHub Actions Inactive
@raghavkaul raghavkaul merged commit c72cfd5 into ossf:main Jun 29, 2023
ashearin pushed a commit to kgangerlm/scorecard-gitlab that referenced this pull request Nov 13, 2023
…sf#3221)

* Move tests that connect to GitLab out of e2e

Signed-off-by: Raghav Kaul <[email protected]>

* update

Signed-off-by: Raghav Kaul <[email protected]>

* mark as pat test

Signed-off-by: Raghav Kaul <[email protected]>

---------

Signed-off-by: Raghav Kaul <[email protected]>
Signed-off-by: Allen Shearin <[email protected]>
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.

2 participants