Skip to content

docs: plan for #298 — custom test extension scaffolding#299

Open
posit-vip-triage[bot] wants to merge 3 commits into
mainfrom
bot-plan-issue-298-df8e494718eeb8a6
Open

docs: plan for #298 — custom test extension scaffolding#299
posit-vip-triage[bot] wants to merge 3 commits into
mainfrom
bot-plan-issue-298-df8e494718eeb8a6

Conversation

@posit-vip-triage
Copy link
Copy Markdown
Contributor

@posit-vip-triage posit-vip-triage Bot commented May 29, 2026

This plan outlines adding a vip scaffold subcommand and a comprehensive example showing how to extend VIP with custom tests for cross-product validation scenarios (verifying specific R/Python versions and packages across Workbench and Connect).

The plan also addresses naming consistency across example directories and considers future extensibility for multi-template scaffolding support, including potential examples for SSL certificate validation, API performance testing, and user provisioning verification.

Merging this PR will trigger an implementation PR — comment to iterate on the plan first.

> Generated by Triage open issues on posit-dev/vip for issue #298 · ● 7.9M ·

Add plan for a vip scaffold subcommand and comprehensive GxP validation
example showing how to verify specific R/Python versions and packages
across Workbench and Connect.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

Comment on lines +72 to +73
- **UNCONFIRMED:** Should the example test directory be named `examples/gxp_validation/` or something more generic like `examples/cross_product_validation/`?
- **Leaning toward:** `examples/cross_product_validation/` because the pattern applies beyond GxP (any customer who needs to lock runtime/package versions).
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@copilot cross-product validation

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Updated the plan to use examples/cross_product_validation/ throughout — all gxp_validation references replaced and the naming question marked as CONFIRMED. (commit: docs: confirm cross_product_validation naming, update all references)

Comment on lines +75 to +76
- **UNCONFIRMED:** Should the scaffold command default to copying from `examples/cross_product_validation/` or should it be extensible (multiple scaffold templates)?
- **Leaning toward:** single template for now; extensibility can be added later if demand emerges.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@copilot single for now

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Confirmed — marked as single template for now with extensibility added later if needed.


## Out of scope

- Modifying the existing `examples/custom_tests/` directory (it serves a different purpose: minimal HTTP check). The new example is complementary, not a replacement.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@copilot it is weird to have one folder named custom_tests while another is named cross_product_validation. How many others example could we think of adding and if so we should add a way to support multiple.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Added new open question about renaming custom_testshttp_health_check for consistency. Listed potential future examples (SSL validation, API performance, user provisioning) and noted the Architecture section can support --template option for multi-template scaffolding.

Co-authored-by: ian-flores <18703558+ian-flores@users.noreply.github.com>
Co-authored-by: ian-flores <18703558+ian-flores@users.noreply.github.com>
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