Update form validations and tests for acceleration

common/constants/index.ts

@@ -32,7 +32,16 @@ export const ACCELERATION_TIME_INTERVAL = [
   { text: 'week(s)', value: 'week' },
 ];
 
+export const SKIPPING_INDEX_ACCELERATION_METHODS = [
+  { value: 'PARTITION', text: 'Partition' },
+  { value: 'VALUE_SET', text: 'Value Set' },
+  { value: 'MIN_MAX', text: 'Min Max' },
+];
+
 export const ACCELERATION_ADD_FIELDS_TEXT = '(add fields here)';
+export const ACCELERATION_INDEX_NAME_REGEX = /^[a-z][a-z_\-]*$/;
+export const ACCELERATION_S3_URL_REGEX = /^(s3|s3a):\/\/[a-zA-Z0-9.\-]+\/.*/;
+export const ACCELERATION_DEFUALT_SKIPPING_INDEX_NAME = 'skipping';
 
 export const ACCELERATION_INDEX_NAME_INFO = `All OpenSearch acceleration indices have a naming format of pattern: \`prefix_<index name>_suffix\`. They share a common prefix structure, which is \`flint_<data source name>_<database name>_<table name>_\`. Additionally, they may have a suffix that varies based on the index type. 
 ##### Skipping Index

common/types/index.ts

@@ -12,11 +12,13 @@ export interface MaterializedViewColumn {
   fieldAlias?: string;
 }
 
+export type SkippingIndexAccMethodType = 'PARTITION' | 'VALUE_SET' | 'MIN_MAX';
+
 export interface SkippingIndexRowType {
   id: string;
   fieldName: string;
   dataType: string;
-  accelerationMethod: 'PARTITION' | 'VALUE_SET' | 'MIN_MAX';
+  accelerationMethod: SkippingIndexAccMethodType;
 }
 
 export interface DataTableFieldsType {
@@ -43,6 +45,20 @@ export interface materializedViewQueryType {
   groupByTumbleValue: GroupByTumbleType;
 }
 
+export interface FormErrorsType {
+  dataSourceError: string[];
+  databaseError: string[];
+  dataTableError: string[];
+  skippingIndexError: string[];
+  coveringIndexError: string[];
+  materializedViewError: string[];
+  indexNameError: string[];
+  primaryShardsError: string[];
+  replicaShardsError: string[];
+  refreshIntervalError: string[];
+  checkpointLocationError: string[];
+}
+
 export interface CreateAccelerationForm {
   dataSource: string;
   database: string;
@@ -57,5 +73,6 @@ export interface CreateAccelerationForm {
   replicaShardsCount: number;
   refreshType: 'interval' | 'auto';
   checkpointLocation: string | undefined;
-  refreshIntervalOptions: RefreshIntervalType | undefined;
+  refreshIntervalOptions: RefreshIntervalType;
+  formErrors: FormErrorsType;
 }

package.json

@@ -27,12 +27,15 @@
   },
   "devDependencies": {
     "@testing-library/user-event": "^13.1.9",
+    "@types/enzyme-adapter-react-16": "^1.0.6",
+    "@types/react-test-renderer": "^16.9.1",
     "cypress": "^5.0.0",
     "eslint": "^6.8.0",
     "husky": "^4.2.5",
     "jest-raw-loader": "^1.0.1",
     "lint-staged": "^10.2.0",
     "mutationobserver-shim": "^0.3.3",
+    "jest-dom": "^4.0.0",
     "ts-jest": "^29.1.0"
   },
   "resolutions": {

public/ace-themes/sql_console.js

@@ -3,10 +3,13 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-
 import * as ace from 'brace';
 
-ace.define('ace/theme/sql_console', ['require', 'exports', 'module', 'ace/lib/dom'], function (acequire, exports, module) {
+ace.define('ace/theme/sql_console', ['require', 'exports', 'module', 'ace/lib/dom'], function (
+  acequire,
+  exports,
+  module
+) {
   exports.isDark = false;
   exports.cssClass = 'ace-sql-console';
   exports.cssText = require('../index.scss');