Skip to content

Update integration test submodule and fix current-time property#6

Merged
jkebinger merged 7 commits into
mainfrom
update-integration-test-submodule
Sep 19, 2025
Merged

Update integration test submodule and fix current-time property#6
jkebinger merged 7 commits into
mainfrom
update-integration-test-submodule

Conversation

@jkebinger
Copy link
Copy Markdown
Contributor

Summary

  • Updated git submodule URL from ReforgeHQ/sdk-integration-test-data to ReforgeHQ/integration-test-data
  • Fixed criteria evaluator to use 'reforge.current-time' instead of 'reforge-sdk.current-time' to match integration test data expectations
  • Synchronized submodule configuration to point to the correct repository

Test plan

  • Verify submodule points to correct repository
  • Updated virtual current-time property to work with integration tests
  • Ensure integration tests pass with these changes

🤖 Generated with Claude Code

jkebinger and others added 4 commits September 19, 2025 16:20
🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Use 'reforge.current-time' instead of 'reforge-sdk.current-time' to match integration test data expectations.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@jdwyah jdwyah left a comment

Choose a reason for hiding this comment

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

thanks

jkebinger and others added 3 commits September 19, 2025 17:07
- Remove deprecated prefab_api_url parameter from Options init
- Add reforge_api_url parameter with deprecation warning
- Update integration test parser to handle reforge_api_url instead of prefab_api_url
- Both parameters convert to sources array for backward compatibility

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@jkebinger jkebinger merged commit d1196ff into main Sep 19, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants