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

test: mock npm data provider #72

Merged
merged 2 commits into from
Nov 1, 2023
Merged

test: mock npm data provider #72

merged 2 commits into from
Nov 1, 2023

Conversation

zvigrinberg
Copy link
Collaborator

Description

Need to mock interactions with npm package manager in order to achieve the following benefits:

  • in order to eliminate the need to change expected sbom files (due to versions bump)
  • in order to focus merely on API logic and not on integrations of API with npm package manager
  • in order to reduce tests times

Checklist

  • I have followed this repository's contributing guidelines.
  • I will adhere to the project's code of conduct.

- in order to eliminate the need to change expected sbom files (due to versions bump)
- in order to focus merely on API logic and not on integrations of API with npm package manager
- in order to reduce tests times

Signed-off-by: Zvi Grinberg <zgrinber@redhat.com>
Copy link

github-actions bot commented Nov 1, 2023

Test Results

76 tests   76 ✔️  39s ⏱️
  8 suites    0 💤
  8 files      0

Results for commit a7f1eb0.

♻️ This comment has been updated with latest results.

… versions

Signed-off-by: Zvi Grinberg <zgrinber@redhat.com>
@zvigrinberg zvigrinberg merged commit 702c77e into main Nov 1, 2023
6 checks passed
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.

1 participant