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.

Sign up for GitHub

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

Jump to bottom

[ENT] Complete ent pagination and update backend tests #1870

Merged
merged 18 commits into from
Apr 29, 2024
Merged

[ENT] Complete ent pagination and update backend tests #1870

merged 18 commits into from
Apr 29, 2024

Conversation

pxp928
Copy link
Collaborator

@pxp928 pxp928 commented Apr 27, 2024

Description of the PR

Complete ent pagination and update backend tests.

Part of issue #1525

PR Checklist

  • All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
  • All new changes are covered by tests
  • If GraphQL schema is changed, make generate has been run
  • If OpenAPI spec is changed, make generate has been run
  • If collectsub protobuf has been changed, make proto has been run
  • All CI checks are passing (tests and formatting)
  • All dependent PRs have already been merged

pxp928 added 18 commits April 25, 2024 17:52
@pxp928
add more descriptive error messages and update hasSBOM query
a25b1d7 
Signed-off-by: pxp928 <[email protected]>
@pxp928
remove unnecessary limit with pagination
f7065ec 
Signed-off-by: pxp928 <[email protected]>
@pxp928
add certifyBad pagination
a651756 
Signed-off-by: pxp928 <[email protected]>
@pxp928
add certifyGood pagination
e5598b6 
Signed-off-by: pxp928 <[email protected]>
@pxp928
add certifyLegal pagination
127810f 
Signed-off-by: pxp928 <[email protected]>
@pxp928
add vex pagination
0a985ce 
Signed-off-by: pxp928 <[email protected]>
@pxp928
add certifyVuln and isDep pagination
3663cfa 
Signed-off-by: pxp928 <[email protected]>
@pxp928
add hashequal pagination
bab01b9 
Signed-off-by: pxp928 <[email protected]>
@pxp928
add hasMetadata and license pagination
660ddd6 
Signed-off-by: pxp928 <[email protected]>
@pxp928
add occurrence pagination
0d0aeb2 
Signed-off-by: pxp928 <[email protected]>
@pxp928
add package and pkgEqual pagination
9c646d3 
Signed-off-by: pxp928 <[email protected]>
@pxp928
add point of contact pagination
dc09727 
Signed-off-by: pxp928 <[email protected]>
@pxp928
add sbom and scorecard pagination
e93c817 
Signed-off-by: pxp928 <[email protected]>
@pxp928
add slsa pagination
01cdc94 
Signed-off-by: pxp928 <[email protected]>
@pxp928
add hasSourceAt pagination
0a3d150 
Signed-off-by: pxp928 <[email protected]>
@pxp928
add source pagination
22181e6 
Signed-off-by: pxp928 <[email protected]>
@pxp928
add vulnEqual, vuln and vulnMeta pagination
3009581 
Signed-off-by: pxp928 <[email protected]>
@pxp928
update backend unit tests for pagination
b655e9f 
Signed-off-by: pxp928 <[email protected]>
@pxp928 pxp928 marked this pull request as ready for review April 27, 2024 21:56
@pxp928 pxp928 requested a review from mihaimaruseac as a code owner April 27, 2024 21:56
mihaimaruseac
mihaimaruseac approved these changes Apr 28, 2024
View reviewed changes
Copy link
Collaborator

@mihaimaruseac mihaimaruseac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

mrizzi
mrizzi approved these changes Apr 29, 2024
View reviewed changes
Copy link
Collaborator

@mrizzi mrizzi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pxp928 thanks 👏

@kodiakhq kodiakhq bot merged commit 8189495 into guacsec:main Apr 29, 2024
8 checks passed
arorasoham9 pushed a commit to arorasoham9/guac that referenced this pull request May 17, 2024
@pxp928 @arorasoham9
[ENT] Complete ent pagination and update backend tests (guacsec#1870)
188d667 
* add more descriptive error messages and update hasSBOM query

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

* remove unnecessary limit with pagination

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

* add certifyBad pagination

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

* add certifyGood pagination

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

* add certifyLegal pagination

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

* add vex pagination

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

* add certifyVuln and isDep pagination

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

* add hashequal pagination

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

* add hasMetadata and license pagination

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

* add occurrence pagination

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

* add package and pkgEqual pagination

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

* add point of contact pagination

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

* add sbom and scorecard pagination

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

* add slsa pagination

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

* add hasSourceAt pagination

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

* add source pagination

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

* add vulnEqual, vuln and vulnMeta pagination

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

* update backend unit tests for pagination

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

---------

Signed-off-by: pxp928 <[email protected]>
Signed-off-by: Soham Arora <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mihaimaruseac mihaimaruseac mihaimaruseac approved these changes

@mrizzi mrizzi mrizzi approved these changes

Assignees
No one assigned
Labels
size/XXL
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pxp928 @mihaimaruseac @mrizzi