Skip to content

Clear environment variable when initializationProcedure is no longer set#565

Merged
kenherring merged 4 commits into
mainfrom
fix-cached-initializationProcedure
Apr 15, 2026
Merged

Clear environment variable when initializationProcedure is no longer set#565
kenherring merged 4 commits into
mainfrom
fix-cached-initializationProcedure

Conversation

@kenherring
Copy link
Copy Markdown
Owner

Fixes #494

@kenherring kenherring marked this pull request as ready for review April 15, 2026 09:06
Copilot AI review requested due to automatic review settings April 15, 2026 09:06
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes a regression where initializationProcedure could continue to be used after being removed from the test profile by explicitly clearing the related environment variable, and improves runner-side handling when an initialization procedure can’t be located.

Changes:

  • Clear ABLUNIT_INITIALIZATION_PROCEDURE when no initializationProcedure is configured.
  • Add an ABL-side search() check and throw a clearer error if the initialization procedure isn’t found.
  • Bump extension version and update changelog entries.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/ABLUnitRun.ts Clears the initialization procedure env var when the config no longer provides it.
resources/VSCodeTestRunner/ABLUnitCore.p Validates the initialization procedure exists before running it; adds a safer CallStack display check.
package.json Version bump to 1.4.33.
package-lock.json Lockfile version bump to 1.4.33.
CHANGELOG.md Adds release notes for 1.4.33 and updates compare link text/link.

Comment thread CHANGELOG.md Outdated
Comment thread src/ABLUnitRun.ts
Comment thread src/ABLUnitRun.ts
@sonarqubecloud
Copy link
Copy Markdown

@kenherring kenherring merged commit d8ba052 into main Apr 15, 2026
10 checks passed
@kenherring kenherring deleted the fix-cached-initializationProcedure branch April 15, 2026 10:00
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.

🐛 [Bug] Issue with new initializationProcedure option

3 participants