Skip to content

[CERT-188] fix: removed hardcoded Params#98

Merged
jpardogrowthaccelerationpartners merged 1 commit intomainfrom
CERT-188-bugfix
Dec 24, 2025
Merged

[CERT-188] fix: removed hardcoded Params#98
jpardogrowthaccelerationpartners merged 1 commit intomainfrom
CERT-188-bugfix

Conversation

@josephcampos-gap
Copy link
Contributor

This pull request updates the test parameterization in the Bruno test scripts for the MasterSchedule Sections, standardizing the use of placeholder variables and aligning the parameter names across multiple files. Additionally, it updates the script settings to include a timeout configuration.

Parameterization and consistency improvements:

  • Standardized all section validation test scripts (01 - Check first Section is valid.bru and 02 - Check second Section is valid.bru) to use the same placeholder variables (e.g., [ENTER SCHOOL ID], [ENTER SCHOOL YEAR], etc.) for both URL and query parameters, eliminating inconsistent or redundant variable names. [1] [2]

Settings updates:

  • Added timeout: 0 to the settings block in the post-response scripts for 02 - Check second Section is valid.bru, 03 - Check first Section availableCredits was Updated.bru, and 04 - Check second Section availableCredits was Updated.bru, ensuring explicit timeout configuration. [1] [2] [3]

@github-actions
Copy link

Bruno Lint Report

Files Scanned: 331
Problems: 0 (Errors: 0, Warnings: 0)
✅ No blocking errors.

Triggered by workflow: lint-bruno.yml

@github-actions
Copy link

QA Scenarios Summary

Status: PASS
Entities Processed: 1
Assertions: Passed 19, Failed 0, Total 19

Entity Passed Failed Total
BellSchedules 19 0 19

@github-actions
Copy link

Bruno Lint Report

Files Scanned: 331
Problems: 0 (Errors: 0, Warnings: 0)
✅ No blocking errors.

Triggered by workflow: lint-bruno.yml

@github-actions
Copy link

QA Scenarios Summary

Status: PASS
Entities Processed: 1
Assertions: Passed 19, Failed 0, Total 19

Entity Passed Failed Total
BellSchedules 19 0 19

@jpardogrowthaccelerationpartners jpardogrowthaccelerationpartners merged commit 4d3fec0 into main Dec 24, 2025
8 checks passed
@jpardogrowthaccelerationpartners jpardogrowthaccelerationpartners deleted the CERT-188-bugfix branch December 24, 2025 21:54
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