Update OCI Model URI onChange to not to reset the connection type (opendatahub-io#3919)

Author: ppadti

frontend/src/__tests__/cypress/cypress/pages/modelRegistry/modelVersionDeployModal.ts

@@ -10,7 +10,8 @@ class ModelVersionDeployModal extends Modal {
   }
 
   selectProjectByName(name: string) {
-    this.findProjectSelector().findSelectOption(name).click();
+    this.findProjectSelector().click();
+    cy.findByRole('option', { name, hidden: true }).click();
   }
 }
 

frontend/src/__tests__/cypress/cypress/pages/modelServing.ts

@@ -101,6 +101,12 @@ class InferenceServiceModal extends Modal {
     super(`${edit ? 'Edit' : 'Deploy'} model`);
   }
 
+  findConnectionType(name: string | RegExp) {
+    return this.findExistingConnectionSelect()
+      .findByRole('button', { name: 'Typeahead menu toggle' })
+      .findSelectOption(name);
+  }
+
   findSubmitButton() {
     return this.findFooter().findByTestId('modal-submit-button');
   }
@@ -191,10 +197,9 @@ class InferenceServiceModal extends Modal {
     return this.find().findByTestId('model-uri');
   }
 
-  findConnectionType(name: string | RegExp) {
-    return this.findExistingConnectionSelect()
-      .findByRole('button', { name: 'Typeahead menu toggle' })
-      .findSelectOption(name);
+  selectConnectionType(name: string) {
+    this.findExistingConnectionSelect().click();
+    cy.findByRole('option', { name, hidden: true }).click();
   }
 
   selectExistingConnectionSelectOptionByResourceName() {
@@ -233,6 +238,10 @@ class InferenceServiceModal extends Modal {
     return this.find().findByTestId('field AWS_S3_BUCKET');
   }
 
+  findBaseURL() {
+    return this.find().findByTestId('field OCI_HOST');
+  }
+
   findLocationRegionInput() {
     return this.find().findByTestId('field AWS_DEFAULT_REGION');
   }

frontend/src/__tests__/cypress/cypress/tests/mocked/modelRegistry/modelVersion/modelVersionDeploy.cy.ts

@@ -499,17 +499,25 @@ describe('Deploy model version', () => {
     kserveModal.findLocationAccessKeyInput().type('test-access-key');
     kserveModal.findLocationSecretKeyInput().type('test-secret-key');
 
-    kserveModal
-      .findConnectionType('URI - v1 Connection type description Category: existing-category')
-      .click();
+    kserveModal.selectConnectionType(
+      'URI - v1 Connection type description Category: existing-category',
+    );
+
     kserveModal.findConnectionFieldInput('URI').type('http://test-uri');
 
+    // switch the connection type to OCI and fill data
+    kserveModal.selectConnectionType(
+      'OCI compliant registry - v1 Connection type description Category: Database, Testing',
+    );
+
+    kserveModal.findBaseURL().type('oci://test');
+    kserveModal.findModelURITextBox().type('test.io/test/private:test');
+
     // switch the connection type to s3 to check whether all the data is still persistent
-    kserveModal
-      .findConnectionType(
-        'S3 compatible object storage - v1 description 2 Category: existing-category',
-      )
-      .click();
+    kserveModal.selectConnectionType(
+      'S3 compatible object storage - v1 description 2 Category: existing-category',
+    );
+
     kserveModal.findLocationBucketInput().should('have.value', 'test-bucket');
     kserveModal.findLocationEndpointInput().should('have.value', 'test-endpoint');
     kserveModal.findLocationRegionInput().should('have.value', 'test-region');
@@ -518,10 +526,18 @@ describe('Deploy model version', () => {
     kserveModal.findLocationSecretKeyInput().should('have.value', 'test-secret-key');
 
     //switch it back to uri
-    kserveModal
-      .findConnectionType('URI - v1 Connection type description Category: existing-category')
-      .click();
+    kserveModal.selectConnectionType(
+      'URI - v1 Connection type description Category: existing-category',
+    );
+
     kserveModal.findConnectionFieldInput('URI').should('have.value', 'http://test-uri');
+    // oci-connection
+    kserveModal.selectConnectionType(
+      'OCI compliant registry - v1 Connection type description Category: Database, Testing',
+    );
+
+    kserveModal.findModelURITextBox().should('have.value', 'test.io/test/private:test');
+    kserveModal.findBaseURL().should('have.value', 'oci://test');
   });
 
   it('Prefills when there is one s3 matching connection', () => {

frontend/src/concepts/modelRegistry/apiHooks/__tests__/usePrefillDeployModalFromModelRegistry.spec.ts

@@ -1,3 +1,8 @@
+import { LabeledConnection } from '~/pages/modelServing/screens/types';
+import {
+  ConnectionTypeConfigMapObj,
+  ConnectionTypeValueType,
+} from '~/concepts/connectionTypes/types';
 import { ModelVersion, ModelState, RegisteredModel } from './types';
 
 export type ObjectStorageFields = {
@@ -13,6 +18,14 @@ export type RegisteredModelLocation = {
   ociUri: string | null;
 } | null;
 
+export type PrefilledConnection = {
+  initialNewConnectionType: ConnectionTypeConfigMapObj | undefined;
+  initialNewConnectionValues: { [key: string]: ConnectionTypeValueType };
+  connections: LabeledConnection[];
+  connectionsLoaded: boolean;
+  connectionsLoadError: Error | undefined;
+};
+
 export const objectStorageFieldsToUri = (fields: ObjectStorageFields): string | null => {
   const { endpoint, bucket, region, path } = fields;
   if (!endpoint || !bucket || !path) {