add test for listartifacts v4 filter

[ChristopherHX]

• fixes a defect of last change

I reviewed my own code today and saw || instead of && resulting in broken artifact matching.

After fixing this I noticed that nodejs truncates the int64 ids so switch to 32bit that fitd in double.

Additionally added an actual test what this change should fix

[github-actions]

🦙 MegaLinter status: ✅ SUCCESS

Descriptor	Linter	Files	Fixed	Errors	Elapsed time
✅ EDITORCONFIG	editorconfig-checker	1		0	0.01s
✅ REPOSITORY	gitleaks	yes		no	2.43s
✅ REPOSITORY	git_diff	yes		no	0.01s
✅ REPOSITORY	grype	yes		no	9.82s
✅ REPOSITORY	secretlint	yes		no	1.21s
✅ REPOSITORY	trivy-sbom	yes		no	0.11s
✅ REPOSITORY	trufflehog	yes		no	3.43s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.01%. Comparing base (5a80a04) to head (d07f371).
Report is 126 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff             @@
##           master    #2507       +/-   ##
===========================================
+ Coverage   61.56%   75.01%   +13.44%     
===========================================
  Files          53       62        +9     
  Lines        9002    10013     +1011     
===========================================
+ Hits         5542     7511     +1969     
+ Misses       3020     1936     -1084     
- Partials      440      566      +126     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ChristopherHX]

@wolfogre my last artifacts change got merged too fast. I'm good in programming bugs :). After reviewing my own code I had to fix problems, writing code on a phone has side effects.

Now this is covered by tests and ci logs looks good