Skip to content

Update multiple commands to clear context under certain circumstances.#208

Merged
Nadegamra merged 12 commits intomainfrom
nadegamra/auth-clear-context
Aug 11, 2025
Merged

Update multiple commands to clear context under certain circumstances.#208
Nadegamra merged 12 commits intomainfrom
nadegamra/auth-clear-context

Conversation

@Nadegamra
Copy link
Copy Markdown
Collaborator

  • Updated auth login command to clear context unless user logs in with the same service client.
  • Updated auth logout command to always clear context
  • Updated itwin delete command to clear context if the deleted iTwin is currently saved in context.
  • Updated imodel delete command to remove iModelId from context if the deleted iModel is currently saved in context.

@Nadegamra Nadegamra merged commit 4f6f576 into main Aug 11, 2025
14 checks passed
@Nadegamra Nadegamra deleted the nadegamra/auth-clear-context branch August 11, 2025 12:45
linasburneika pushed a commit that referenced this pull request Aug 19, 2025
#208)

* Updated auth commands to clear context when appropriate.

* Updated 'itwin delete' and 'imodel delete' commands to update context when saved itwin/imodel is deleted.

* Added integration tests.

* Added more mocked integration tests.

* Added missing copyright notices to files.

* Removed nock usage from integration tests.

* Updated a test.

* Fixed auth integration tests.

* Attempt to fix auth integration tests.

* Updated context clear on logout tests.

* Updated auth integration tests.

* Updated auth integration tests.
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