Merged
Conversation
- Rename runtime object structure from interface to class-based implementation - Add session properties (activeProfile, activeTenant, outputMode) to c8ctl object - Implement getters/setters with modern TypeScript syntax - Update config.ts to use c8ctl runtime object instead of session file - Update commands/session.ts to use c8ctl runtime directly - Update index.ts to initialize logger from c8ctl.outputMode - Session state now lives in memory on c8ctl object (no file persistence) Co-authored-by: vobu <6573426+vobu@users.noreply.github.com>
- Add tests for session properties on c8ctl runtime object - Update integration tests to verify c8ctl.activeProfile, activeTenant, outputMode - Reset c8ctl runtime state in beforeEach hook - Tests now verify both c8ctl object and loadSessionState() consistency Co-authored-by: vobu <6573426+vobu@users.noreply.github.com>
- Fix type checking in runtime tests for activeProfile and activeTenant - Update outputMode test to not set value before testing - Improve comment for saveSessionState function Co-authored-by: vobu <6573426+vobu@users.noreply.github.com>
…ion tests for mode switching
…ersisting changes
The logger.info() method was not outputting anything in JSON mode, causing the output-mode integration test to fail. This fix ensures that info messages are properly formatted as JSON when in JSON mode. Fixes the failing test: "output mode changes are reflected in list process instances" Co-authored-by: vobu <6573426+vobu@users.noreply.github.com>
- Remove push trigger for non-main branches - Tests now only run on pull_request events - Eliminates duplicate workflow runs for PR branches
- Update test to expect JSON output in JSON mode - Logger.info() now outputs JSON in JSON mode instead of being silent - All unit tests now passing (120/120)
- Update workflow to test only with Node 24 (current LTS) - Update .nvmrc to v22 for local development testing - All integration tests passing on Node 22.22.0
- Add Node 22 back to test matrix - Tests will run on Node 22.x and 24.x with Camunda 8.8 and 8.9
…integration tests
|
This has been released in 2.0.0-alpha.1. |
|
This has been released in 1.1.0. |
Collaborator
Author
|
ping back to https://github.com/camunda/product-hub/issues/2638 |
|
This has been released in 2.0.0. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.