wegamekinglc / Derivatives-Algorithms-Lib

78%

master: 79%

LAST BUILD ON BRANCH feature/script-engine-test-coverage

branch: feature/script-engine-test-coverage

CHANGE BRANCH

x

Reset

Sync Branches

• feature/script-engine-test-coverage
• china_2025
• chore/remove-co-authored-by-from-guides
• ci/expand-ci-to-public-subprojects
• ci/matrix-workflow
• copilot/add-test-coverage-analysis
• copilot/analyze-test-coverage
• copilot/analyze-test-coverage-again
• copilot/design-web-ui-derivatives-portfolio-management
• copilot/enhance-yield-curve-framework
• copilot/explore-codebase-implementation-plan
• copilot/explore-codebase-yield-curve-modeling
• copilot/override-function-gradient-calibration
• copilot/refactor-script-engine
• dependabot/github_actions/actions/checkout-3
• dependabot/github_actions/actions/checkout-4
• dependabot/github_actions/actions/checkout-5
• dependabot/github_actions/actions/checkout-6
• dependabot/github_actions/actions/checkout-6.0.2
• dependabot/github_actions/actions/checkout-6.0.3
• dependabot/github_actions/astral-sh/setup-uv-8.2.0
• dependabot/github_actions/astral-sh/setup-uv-caf0cab7a618c569241d31dcd442f54681755d39
• dependabot/npm_and_yarn/webui/frontend/multi-f57e1e291f
• dependabot/npm_and_yarn/webui/frontend/vite-8.0.16
• docs/claude-config-sync
• docs/fix-documentation-consistency
• docs/improve-readme-formatting
• docs/markdown-table-alignment-and-cleanup
• docs/ptirds-curve-extension-plan
• docs/restructure-and-webui-rename
• experimental
• feature/aad-methodology-doc
• feature/adept-aad-backend
• feature/adept-expr-tape-tests
• feature/agent-team-design
• feature/agents-tdd-worktree
• feature/code-style-fixes
• feature/codipack-aad-backend
• feature/codipack-polish
• feature/coverage-build-flag
• feature/currency-unit-tests
• feature/dal-pr-review-agent
• feature/dal-python-uv-modernization
• feature/dal-refactor-docs
• feature/dal-refactor-full
• feature/dal-refactor-phase0-phase1
• feature/dal-refactor-phase2
• feature/dal-refactor-phase3
• feature/dal-refactor-phase4
• feature/dal-refactor-phase5
• feature/dal-unit-test-write-agent
• feature/dal-web-playwright-e2e
• feature/docs-table-style-guidance
• feature/docs-webui-startup
• feature/enforce-worktree-dal-dev-workflow
• feature/harden-dal-pr-review-agent
• feature/log-linear-interp-and-refactor
• feature/normal-expectation-quadrature
• feature/piecewise-linear-curve-tests
• feature/ptirds-log-discount-curve
• feature/ptirds-single-currency-curve
• feature/pybind11-migration
• feature/pybind11-submodule
• feature/remove-precompute-self-param
• feature/rename-skills-dal-prefix
• feature/reorg-miscs-to-subfolders
• feature/serialization-tests-and-skills
• feature/sync-agent-doc-guidance
• feature/webui-redesign-industrial-terminal
• feature/windows-ci-workflow
• feature/ycinstrument-polymorphic-calibration
• feature/yield-curve-calibration
• fix/adept-cmake-definitions
• fix/arrayn-dimension-validation
• fix/calibration-build-constant
• fix/machinist-cmake-compiler-id
• fix/msvc-ndarray-test-parsing
• fix/nodiscard-warnings-script-tests
• fix/readme-build-badge
• fix/underdetermined-guidance-and-xad-default
• fix/upgrade-vite-plugin-react-for-vite8
• fix/xccy-calibration-budget
• fix/xccy-calibration-coverage
• master
• old_virtual_dispatch
• refactor/rename-calibratedyieldcurve-to-curveblock
• refs/heads/master
• refs/pull/5/merge
• revert/ptirds-101
• use_codi
• worktree-agent-a344d3a27476b1053

Build # 26682887000

Build Type

Pull #73

github

Committed by wegamekinglc

Commit Message

test: address Copilot review feedback on script tests

- test_evaluator.cpp: iterate VarNames with size_t instead of a signed int
  to avoid signed/unsigned comparison against size().
- test_parser.cpp: the parser encodes < / > as the same NodeSup_ (and
  <= / >= as the same NodeSupEqual_), differing only by swapped NodeSub_
  operands. The condition tests previously only checked the wrapper type,
  so they could not tell < from >. Add an AssertCmpSubOrder helper and
  assert the inner subtraction operand order for the <, <=, and AND tests.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Pull Request Pull Request #73: test: expand script engine unit test coverage

Coverage Stats

5063 of 6496 relevant lines covered (77.94%)

3837405.66 hits per line