-
Notifications
You must be signed in to change notification settings - Fork 201
feat(cli): Add missing database integrations, add tests for all CLI integrations add and edit commands #321
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 1 commit
Commits
Show all changes
73 commits
Select commit
Hold shift + click to select a range
a5b381d
feat(cli): Add integrations add and edit sub commads
tkislan 768f54d
Fix test
tkislan 95537c2
Fix spell check issues
tkislan fc09494
Update prettierignore to ignore deepnote env files
tkislan 5b404ec
Revert changes to gitignore file
tkislan f0531b3
Refactor add and edit commands for better typescript, and reusability
tkislan c8be239
Update vitest config
tkislan ac5a8db
Add mongodb
tkislan 78c2b68
Revert .gitignore
tkislan 2412e9b
Refactor CLI mongodb integration
tkislan d879126
Fix build and test issues around xterm
tkislan add2602
Update TypeScript configuration and add spell check entry for RAWCONN…
tkislan c98d84c
Remove unused code
tkislan 794e66b
Update TypeScript configuration to use ESNext module and bundler reso…
tkislan 2beecc8
Preserve itegrations file top comment
tkislan 3758deb
Use instanbul coverage
tkislan 98d1e1c
Remove unused packages
tkislan c8eb442
Update package dependencies and configuration; switch coverage provid…
tkislan 362dd18
Update TypeScript configuration to switch module to CommonJS and reso…
tkislan a6657ba
Remove instanbul coverage package
tkislan 8591577
Update tsconfig
tkislan ca29210
Update tsconfig
tkislan 0e1ac5a
Update vitest config
tkislan 80fff95
Update package.json and pnpm-lock.yaml to include patched dependencie…
tkislan 37f2417
Minor changes
tkislan c127d57
Add tests for encodeConnectionString function in mongodb module
tkislan 3df2540
Merge remote-tracking branch origin/main into tk/cli-integrations-add…
tkislan ea7e14c
Fix spellcheck
tkislan 93339b5
Add cli prompt implementetion for rest of the database integrations
tkislan 0dc5eb1
Revert tsconfig change
tkislan 9a8e06a
Merge remote-tracking branch 'origin/tk/cli-integrations-add-edit' in…
tkislan 0ef61ee
Merge remote-tracking branch 'origin/main' into tk/cli-integrations-a…
tkislan cb77284
Merge remote-tracking branch 'origin/tk/cli-integrations-add-edit' in…
tkislan 3a57740
Remove ts-expect-error
tkislan 7519d5b
Merge remote-tracking branch 'origin/tk/cli-integrations-add-edit' in…
tkislan 23db8c8
Add missing database integratios, add tests for all CLI integrations …
tkislan 9f50745
Minor fixes
tkislan 4d7b9ca
Update tests
tkislan 737bf5e
Refactor edit integration to not load dotenv into process environment
tkislan a8bd851
Fix editing secret with default value
tkislan 9786553
Remove debug console logs
tkislan 1e859be
Merge remote-tracking branch 'origin/main' into tk/cli-integrations-a…
tkislan 7b16bb8
Merge remote-tracking branch 'origin/tk/cli-integrations-add-edit' in…
tkislan 1236b34
Refactor mongodb url parse
tkislan fc708a6
No dynamic type imports
tkislan 300f4e1
Merge branch 'main' into tk/cli-integrations-add-edit
tkislan fc921ca
Merge remote-tracking branch 'origin/tk/cli-integrations-add-edit' in…
tkislan ec5839f
Merge remote-tracking branch origin/main into tk/cli-integrations-add…
tkislan 4461826
Extract promptForSslFields into a separate function
tkislan c07b213
Extract promptForSshFields into a separate function
tkislan 776ce04
Update sql server prompts
tkislan 93097ab
Refactor snowflake integration prompts for better typescript support
tkislan fb792c1
Update tests
tkislan 52ca3ed
Fix test
tkislan 0e24ecc
Update tests
tkislan d7590c8
Add test for clearing Okta defaults when switching to native Snowflak…
tkislan 6826d4d
Minor test changes
tkislan e2a2dcd
Refactor tests and integration functions for improved clarity and fun…
tkislan 10ebfd8
Merge remote-tracking branch 'origin/main' into tk/cli-integrations-a…
tkislan 7ebf215
refactor(integrations): enhance authentication handling and prompts f…
tkislan 41067eb
Add some more tests
tkislan 1765c6a
Fix test
tkislan 9fd62cd
Update tests
tkislan 379d565
Fix tests
tkislan 3bedeb0
Multiple test improvements
tkislan 58efea5
Extract integrations process env into a helper function for test mocking
tkislan 5dbc8b8
Improve tests
tkislan c47d7d9
Update tests
tkislan 9024f4f
Update tests
tkislan c881194
Update test
tkislan a75a152
Merge branch 'main' into tk/cli-integrations-add-edit-full
tkislan 5d42df5
Merge branch 'main' into tk/cli-integrations-add-edit-full
tkislan ee97dc1
Merge branch 'main' into tk/cli-integrations-add-edit-full
tkislan File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Multiple test improvements
- Loading branch information
commit 3bedeb00948a39985759c5681ad6d05ffed060b4
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
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
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
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
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
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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.