Skip to content

feat: Adding Hybrid Testing#35

Draft
Kronprinz03 wants to merge 8 commits intomainfrom
creating-hybride-setup
Draft

feat: Adding Hybrid Testing#35
Kronprinz03 wants to merge 8 commits intomainfrom
creating-hybride-setup

Conversation

@Kronprinz03
Copy link
Contributor

Have you...

  • Added relevant entry to the change log?

@hyperspace-insights
Copy link
Contributor

Summary

The following content is AI-generated and provides a summary of the pull request:


Enable Hybrid Integration Testing with Cloud Foundry and HANA Cloud

New Features

✨ Activates hybrid integration testing by uncommenting and configuring previously disabled CI/CD workflow steps. Tests can now run against real BTP services (SAP Build Process Automation) and HANA Cloud, using Cloud Foundry for deployment and binding.

Changes

  • .github/actions/integration-tests/action.yml: Uncommented and activated all previously disabled steps:

    • Cloud Foundry authentication with retry logic
    • HANA Cloud HDI container creation and CDS model deployment
    • Service binding for the process service against sbpa-service-instance
    • HDI container cleanup on completion (key deletion and service deletion with retry)
    • Removed the CDS version-specific dependency override step
  • .github/workflows/test.yml: Enabled the integration-tests job (previously commented out), running against Node.js 20.x and 22.x with the latest CDS version, using CF secrets for hybrid test execution.

  • package.json: Added test:hybrid script targeting the lifecycle annotation integration test file; added [hybrid] profile for ProcessService pointing to deployed-process-service.

  • tests/bookshop/package.json: Added SAP Cloud SDK dependencies (connectivity, http-client, resilience) and registered the new external Lifecycle_Test_ProcessService CDS model.

  • tests/bookshop/srv/external/eu12.cdsmunich.capprocesspluginhybridtest.lifecycle_Test_Process.cds: New generated CDS service definition for the hybrid test process, exposing start/suspend/resume/cancel actions.

  • tests/bookshop/srv/lifecycle-annotation-test.cds: New CDS service with 6 lifecycle annotation scenarios (basic workflow, status-based cancellation, suspend/resume, full lifecycle, conditional start/cancel, external workflow management) backed by the Car entity.

  • tests/bookshop/workflows/eu12.cdsmunich.capprocesspluginhybridtest.lifecycle_Test_Process.json: Added workflow definition JSON for the hybrid lifecycle test process.

  • tests/integration/annotations/lifeCycleAnnotation.test.ts: Updated all OData endpoint paths from /odata/v4/annotation/ to /odata/v4/lifecycle-annotation/ and updated definitionId assertions to use the real BTP process ID (eu12.cdsmunich.capprocesspluginhybridtest.lifecycle_Test_Process).


  • 🔄 Regenerate and Update Summary
  • ✏️ Insert as PR Description (deletes this comment)
  • 🗑️ Delete comment
PR Bot Information

Version: 1.18.4 | 📖 Documentation | 🚨 Create Incident | 💬 Feedback

  • Correlation ID: 96ceae70-1e15-11f1-87d2-4610f8c07a06
  • Summary Prompt: Default Prompt
  • Output Template: Default Template
  • LLM: anthropic--claude-4.6-sonnet
  • Event Trigger: pull_request.opened

💌 Have ideas or want to contribute? Create an issue and share your thoughts with us!
📑 Check out the documentation for more information.
📬 Subscribe to the Hyperspace PR Bot DL to get the latest announcements and pilot features!

Made with ❤️ by Hyperspace.

Copy link
Contributor

@hyperspace-insights hyperspace-insights bot left a comment

Choose a reason for hiding this comment

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

The PR introduces several substantive issues: a YAML syntax error that will break the entire action (bare text Deploy model to HANA at line 73), the repository checkout happening after CF login rather than before it, a hardcoded BTP service instance name that will silently bind to the wrong service in other projects, and a missing cf create-service-key step that makes the cleanup step reference a key that was never created. The workflow-level npm run test step following the composite action will also likely fail due to missing dependencies at the workflow scope.

PR Bot Information

Version: 1.18.4 | 📖 Documentation | 🚨 Create Incident | 💬 Feedback

  • Correlation ID: 96ceae70-1e15-11f1-87d2-4610f8c07a06
  • LLM: anthropic--claude-4.6-sonnet
  • Event Trigger: pull_request.opened

@Kronprinz03 Kronprinz03 force-pushed the creating-hybride-setup branch from 36c43b6 to d2ef219 Compare March 12, 2026 15:28
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.

1 participant