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

🌱 Improve gitlabrepo.ListProgrammingLanguages e2e tests #3309

Merged
merged 3 commits into from
Jul 26, 2023

Conversation

naveensrinivasan
Copy link
Member

  • Add an end-to-end test for the gitlabrepo.ListProgrammingLanguages function
  • Include a test for repos without code to check that no programming languages are returned

[clients/gitlabrepo/languages_e2e_test.go]

  • Add an e2e test for the gitlabrepo.ListProgrammingLanguages function
  • Add a test for repos without code to check that no programming languages are returned

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.)


@naveensrinivasan naveensrinivasan self-assigned this Jul 25, 2023
@naveensrinivasan naveensrinivasan temporarily deployed to gitlab July 25, 2023 12:39 — with GitHub Actions Inactive
@naveensrinivasan naveensrinivasan temporarily deployed to integration-test July 25, 2023 12:40 — with GitHub Actions Inactive
@naveensrinivasan naveensrinivasan temporarily deployed to gitlab July 25, 2023 12:45 — with GitHub Actions Inactive
@codecov
Copy link

codecov bot commented Jul 25, 2023

Codecov Report

Merging #3309 (5394ad3) into main (81ed545) will decrease coverage by 11.06%.
The diff coverage is n/a.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3309       +/-   ##
===========================================
- Coverage   72.56%   61.50%   -11.06%     
===========================================
  Files         173      173               
  Lines       12686    12686               
===========================================
- Hits         9205     7803     -1402     
- Misses       2979     4456     +1477     
+ Partials      502      427       -75     

@naveensrinivasan naveensrinivasan temporarily deployed to integration-test July 25, 2023 12:45 — with GitHub Actions Inactive
@naveensrinivasan naveensrinivasan force-pushed the naveen/e2e/clients/gitlabrepo/languages.go branch from 3f1de9c to 0472227 Compare July 25, 2023 12:47
@naveensrinivasan naveensrinivasan temporarily deployed to gitlab July 25, 2023 12:47 — with GitHub Actions Inactive
@naveensrinivasan naveensrinivasan enabled auto-merge (squash) July 25, 2023 12:47
@naveensrinivasan naveensrinivasan temporarily deployed to integration-test July 25, 2023 12:47 — with GitHub Actions Inactive
@naveensrinivasan naveensrinivasan force-pushed the naveen/e2e/clients/gitlabrepo/languages.go branch from 0472227 to 928e42a Compare July 26, 2023 14:11
@naveensrinivasan naveensrinivasan temporarily deployed to gitlab July 26, 2023 14:11 — with GitHub Actions Inactive
@naveensrinivasan naveensrinivasan temporarily deployed to integration-test July 26, 2023 14:12 — with GitHub Actions Inactive
- Add an end-to-end test for the `gitlabrepo.ListProgrammingLanguages` function
- Include a test for repos without code to check that no programming languages are returned

[clients/gitlabrepo/languages_e2e_test.go]
- Add an e2e test for the `gitlabrepo.ListProgrammingLanguages` function
- Add a test for repos without code to check that no programming languages are returned

Signed-off-by: naveensrinivasan <[email protected]>
- Lower the numeric tolerance for e2e tests in the GitLab repository

[clients/gitlabrepo/languages_e2e_test.go]
- Lower numeric tolerance for test files

Signed-off-by: naveensrinivasan <[email protected]>
@naveensrinivasan naveensrinivasan force-pushed the naveen/e2e/clients/gitlabrepo/languages.go branch from 928e42a to 8807a13 Compare July 26, 2023 14:12
@naveensrinivasan naveensrinivasan temporarily deployed to gitlab July 26, 2023 14:12 — with GitHub Actions Inactive
@naveensrinivasan naveensrinivasan temporarily deployed to integration-test July 26, 2023 14:12 — with GitHub Actions Inactive
@spencerschrock spencerschrock temporarily deployed to gitlab July 26, 2023 20:21 — with GitHub Actions Inactive
@spencerschrock spencerschrock temporarily deployed to integration-test July 26, 2023 20:21 — with GitHub Actions Inactive
@naveensrinivasan naveensrinivasan merged commit 2e7d37a into main Jul 26, 2023
37 of 38 checks passed
@naveensrinivasan naveensrinivasan deleted the naveen/e2e/clients/gitlabrepo/languages.go branch July 26, 2023 20:27
ashearin pushed a commit to kgangerlm/scorecard-gitlab that referenced this pull request Nov 13, 2023
* 🌱 Improve `gitlabrepo.ListProgrammingLanguages` e2e tests

- Add an end-to-end test for the `gitlabrepo.ListProgrammingLanguages` function
- Include a test for repos without code to check that no programming languages are returned

[clients/gitlabrepo/languages_e2e_test.go]
- Add an e2e test for the `gitlabrepo.ListProgrammingLanguages` function
- Add a test for repos without code to check that no programming languages are returned

Signed-off-by: naveensrinivasan <[email protected]>

* Increase test accuracy for language e2e tests

- Lower the numeric tolerance for e2e tests in the GitLab repository

[clients/gitlabrepo/languages_e2e_test.go]
- Lower numeric tolerance for test files

Signed-off-by: naveensrinivasan <[email protected]>

---------

Signed-off-by: naveensrinivasan <[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