Skip to content

feat(4454): add asset API tests for locale query params#145

Merged
OMpawar-21 merged 1 commit intodevelopmentfrom
enhc/DX-4454
Mar 25, 2026
Merged

feat(4454): add asset API tests for locale query params#145
OMpawar-21 merged 1 commit intodevelopmentfrom
enhc/DX-4454

Conversation

@OMpawar-21
Copy link

feat: add asset API tests for locale query params

  • Add Test031–Test036 in Contentstack013_AssetTest.cs.
  • Positive: locale=en-us via ParameterCollection on Fetch, FetchAsync, and Query().Find(...).
  • Negative: invalid locale on fetch, invalid asset UID with locale, query with invalid locale (handles API success or ContentstackErrorException).
  • Add using Contentstack.Management.Core.Queryable;.

Add Test031–Test036 in Contentstack013_AssetTest covering
ParameterCollection with locale=en-us on Fetch, FetchAsync, and
Query.Find, plus negative cases (invalid locale, invalid UID+locale,
query with invalid locale). Import Contentstack.Management.Core.Queryable.
@OMpawar-21 OMpawar-21 requested a review from a team as a code owner March 25, 2026 06:31
@github-actions
Copy link

We regret to inform you that you are currently not able to merge your changes into the master branch due to restrictions applied by our SRE team. To proceed with merging your changes, we kindly request that you create a pull request from the next branch. Our team will then review the changes and work with you to ensure a successful merge into the master branch.

@github-actions
Copy link

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

@OMpawar-21 OMpawar-21 changed the base branch from main to development March 25, 2026 06:34
@github-actions
Copy link

We regret to inform you that you are currently not able to merge your changes into the master branch due to restrictions applied by our SRE team. To proceed with merging your changes, we kindly request that you create a pull request from the next branch. Our team will then review the changes and work with you to ensure a successful merge into the master branch.

@github-actions
Copy link

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

Copy link

@sunil-lakshman sunil-lakshman left a comment

Choose a reason for hiding this comment

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

LGTM

@OMpawar-21 OMpawar-21 merged commit 83ea7a3 into development Mar 25, 2026
15 of 17 checks passed
@OMpawar-21 OMpawar-21 deleted the enhc/DX-4454 branch March 25, 2026 06:57
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.

3 participants