Skip to content

Commit

Permalink
polish test
Browse files Browse the repository at this point in the history
  • Loading branch information
anthonykim1 committed Nov 12, 2024
1 parent 78ae35c commit 032741d
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions src/test/common/terminals/service.unit.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -189,8 +189,7 @@ suite('Terminal Service', () => {
terminal.verify((t) => t.sendText(TypeMoq.It.isValue(textToSend)), TypeMoq.Times.exactly(1));
});

// Ensure sendText is called when Python shell integration are disabled.
test('Ensure text is sent to terminal and it is shown when Python shell integration is disabled', async () => {
test('Ensure sendText is used when Python shell integration is disabled', async () => {
pythonConfig
.setup((p) => p.get('terminal.shellIntegration.enabled'))
.returns(() => false)
Expand All @@ -211,7 +210,6 @@ suite('Terminal Service', () => {
terminal.verify((t) => t.sendText(TypeMoq.It.isValue(textToSend)), TypeMoq.Times.exactly(1));
});

// Ensure sendText is called when terminal.shellIntegration is enabled but Python shell integration is disabled
test('Ensure sendText is called when terminal.shellIntegration enabled but Python shell integration disabled', async () => {
pythonConfig
.setup((p) => p.get('terminal.shellIntegration.enabled'))
Expand All @@ -233,8 +231,7 @@ suite('Terminal Service', () => {
terminal.verify((t) => t.sendText(TypeMoq.It.isValue(textToSend)), TypeMoq.Times.exactly(1));
});

// Ensure executeCommand is called when Python shell integration and terminal shell integration are both enabled
test('Ensure sendText is not called when Python shell integration and terminal shell integration are both enabled', async () => {
test('Ensure sendText is NOT called when Python shell integration and terminal shell integration are both enabled', async () => {
pythonConfig
.setup((p) => p.get('terminal.shellIntegration.enabled'))
.returns(() => true)
Expand Down

0 comments on commit 032741d

Please sign in to comment.