Skip to content

test: add comprehensive unit tests for dependency_updater helper func…#1065

Open
Kewe63 wants to merge 1 commit into
base:mainfrom
Kewe63:test/dependency-updater-coverage
Open

test: add comprehensive unit tests for dependency_updater helper func…#1065
Kewe63 wants to merge 1 commit into
base:mainfrom
Kewe63:test/dependency-updater-coverage

Conversation

@Kewe63
Copy link
Copy Markdown

@Kewe63 Kewe63 commented May 11, 2026

  • Add tests for generateGithubRepoUrl (valid + nil guard)
  • Add tests for writeToVersionsJson (success + invalid path)
  • Add tests for createVersionsEnv (basic, branch tracking, sorted output, invalid path)
  • Add tests for writeToGithubOutput (normal + missing env var)
  • Add tests for createCommitMessage in githubAction mode
  • Add tests for updateVersionTagAndCommit (in-memory + file write)
  • Add nil-safety guard to generateGithubRepoUrl for unknown keys

Increases test coverage from 0 to 389 lines across 12 test functions.

…tions

- Add tests for generateGithubRepoUrl (valid + nil guard)
- Add tests for writeToVersionsJson (success + invalid path)
- Add tests for createVersionsEnv (basic, branch tracking, sorted output, invalid path)
- Add tests for writeToGithubOutput (normal + missing env var)
- Add tests for createCommitMessage in githubAction mode
- Add tests for updateVersionTagAndCommit (in-memory + file write)
- Add nil-safety guard to generateGithubRepoUrl for unknown keys

Increases test coverage from 0 to 389 lines across 12 test functions.
@cb-heimdall
Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

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