Convert pytest installation prompt to modal dialog

Co-authored-by: eleanorjboyd <[email protected]>

Author: Copilot

src/client/testing/configuration/pytestInstallationHelper.ts

@@ -24,7 +24,12 @@ export class PytestInstallationHelper {
         const installOption = l10n.t('Install pytest');
         const ignoreOption = l10n.t('Ignore');
 
-        const selection = await this.appShell.showInformationMessage(message, installOption, ignoreOption);
+        const selection = await this.appShell.showInformationMessage(
+            message,
+            { modal: true },
+            installOption,
+            ignoreOption,
+        );
 
         if (selection === installOption) {
             return this.installPytest(workspaceUri);

src/test/testing/configuration/pytestInstallationHelper.unit.test.ts

@@ -33,7 +33,7 @@ suite('PytestInstallationHelper', () => {
 
     test('promptToInstallPytest should return false if user selects ignore', async () => {
         appShell
-            .setup((a) => a.showInformationMessage(TypeMoq.It.isAny(), TypeMoq.It.isAny(), TypeMoq.It.isAny()))
+            .setup((a) => a.showInformationMessage(TypeMoq.It.isAny(), TypeMoq.It.isAny(), TypeMoq.It.isAny(), TypeMoq.It.isAny()))
             .returns(() => Promise.resolve('Ignore'))
             .verifiable(TypeMoq.Times.once());
 
@@ -45,7 +45,7 @@ suite('PytestInstallationHelper', () => {
 
     test('promptToInstallPytest should return false if user cancels', async () => {
         appShell
-            .setup((a) => a.showInformationMessage(TypeMoq.It.isAny(), TypeMoq.It.isAny(), TypeMoq.It.isAny()))
+            .setup((a) => a.showInformationMessage(TypeMoq.It.isAny(), TypeMoq.It.isAny(), TypeMoq.It.isAny(), TypeMoq.It.isAny()))
             .returns(() => Promise.resolve(undefined))
             .verifiable(TypeMoq.Times.once());
 
@@ -68,7 +68,7 @@ suite('PytestInstallationHelper', () => {
         useEnvExtensionStub.returns(false);
 
         appShell
-            .setup((a) => a.showInformationMessage(TypeMoq.It.isAny(), TypeMoq.It.isAny(), TypeMoq.It.isAny()))
+            .setup((a) => a.showInformationMessage(TypeMoq.It.isAny(), TypeMoq.It.isAny(), TypeMoq.It.isAny(), TypeMoq.It.isAny()))
             .returns(() => Promise.resolve('Install pytest'))
             .verifiable(TypeMoq.Times.once());
 
@@ -95,6 +95,7 @@ suite('PytestInstallationHelper', () => {
                     TypeMoq.It.is((msg: string) => msg.includes('pytest selected but not installed')),
                     TypeMoq.It.isAny(),
                     TypeMoq.It.isAny(),
+                    TypeMoq.It.isAny(),
                 ),
             )
             .returns(() => Promise.resolve('Install pytest'))