Skip to content

Conversation

@ericelliott
Copy link
Collaborator

  • Comprehensive TDD workflow implementation

  • Follows RITE Way testing principles

  • Includes proper Red-Green-Refactor cycle

  • Defines assert function with 5 essential test questions

  • Provides clear guidance for test framework usage

- Comprehensive TDD workflow implementation

- Follows RITE Way testing principles

- Includes proper Red-Green-Refactor cycle

- Defines assert function with 5 essential test questions

- Provides clear guidance for test framework usage
@ericelliott ericelliott requested a review from Copilot August 15, 2025 18:34
Copy link
Contributor

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

This PR adds a comprehensive TDD Engineer SudoLang program that provides systematic guidance for implementing Test-Driven Development workflows. The program establishes a structured approach to TDD with clear testing principles and process guidelines.

  • Implements a complete TDD workflow with Red-Green-Refactor cycle guidance
  • Defines an assert function type with 5 essential test validation questions
  • Establishes clear testing principles following RITE Way methodology

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ericelliott ericelliott merged commit b3da1f7 into main Aug 15, 2025
@ericelliott ericelliott deleted the tdd branch August 15, 2025 18:39
ericelliott pushed a commit that referenced this pull request Dec 8, 2025
Bug #1: Missing exports
- Add handleForm, createWithCSRF, withCSRF to src/server/index.js

Bug #2: CSRF body token breaks form validation
- Strip _csrf from request.body before schema validation
- Allows withCSRF + handleForm to work together with strict schemas

Bug #3: Missing parameter validation
- Add fail-fast validation for name, schema, processSubmission
- Throws clear errors at factory creation time, not runtime

Added 4 new tests for the fixes.
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