Skip to content

[WIP] Fix test gaps in compile/common.rs and compile/onees.rs#163

Merged
jamesadevine merged 2 commits into
mainfrom
copilot/fix-test-gaps-in-compile-common
Apr 13, 2026
Merged

[WIP] Fix test gaps in compile/common.rs and compile/onees.rs#163
jamesadevine merged 2 commits into
mainfrom
copilot/fix-test-gaps-in-compile-common

Conversation

Copilot AI commented Apr 13, 2026

Copy link
Copy Markdown
Contributor
  • Add replace_with_indent tests in compile/common.rs (multi-line; placeholder not at line start; trailing newline)
  • Add format_step_yaml / format_step_yaml_indented tests in compile/common.rs (single-line; multi-line; --- stripped; custom base indent; zero base indent)
  • Add generate_acquire_ado_token tests in compile/common.rs (with SC; None → empty; write token variable)
  • Add generate_copilot_ado_env / generate_executor_ado_env tests in compile/common.rs (present when SC configured; empty when None; read token vs write token isolation)
  • Add generate_agent_context_root / generate_mcp_configuration / generate_inline_steps / generate_setup_job / generate_teardown_job tests in compile/onees.rs
  • Add invalid hour interval error path tests in fuzzy_schedule.rs (every 5h, every 7h, every 0h)
  • Run tests to validate all new tests pass (638 unit tests, up from 605)

… and fuzzy_schedule.rs

Agent-Logs-Url: https://github.com/githubnext/ado-aw/sessions/cc9ca027-17fc-4675-abd6-a6e5ff2e8b2c

Co-authored-by: jamesadevine <4742697+jamesadevine@users.noreply.github.com>
@jamesadevine jamesadevine marked this pull request as ready for review April 13, 2026 12:21
@jamesadevine jamesadevine merged commit d06a059 into main Apr 13, 2026
10 of 13 checks passed
Copilot stopped work on behalf of jamesadevine due to an error April 13, 2026 12:23
Copilot AI requested a review from jamesadevine April 13, 2026 12:23
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.

🧪 Test gap analysis — 6 gaps found in compile/common.rs, compile/onees.rs, fuzzy_schedule.rs

2 participants