Merge pull request #132 from skyflowapi/release/26.3.13

Release/26.3.13 - SK-2612: add cspell as spellcheck for react-native sdk

Author: skyflow-himanshu

.cspell.json

@@ -0,0 +1,95 @@
+{
+  "version": "0.2",
+  "language": "en",
+  "allowCompoundWords": true,
+  "words": [
+    "Skyflow",
+    "de-identify",
+    "re-identify",
+    "deidentify",
+    "reidentify",
+    "De-identify",
+    "Re-identify",
+    "Deidentify",
+    "Reidentify",
+    "detokenize",
+    "Detokenize",
+    "detokenization",
+    "de-identified",
+    "DEIDENTIFIED",
+    "de-identification",
+    "de-identifying",
+    "re-identified",
+    "Re-identified",
+    "re-identification",
+    "Blackbox",
+    "FORMDATA",
+    "typeof",
+    "Cobranded",
+    "HIPERCARD",
+    "UNIONPAY",
+    "BANCAIRES",
+    "CARTES",
+    "MAESTRO",
+    "vaultID",
+    "skyflowID",
+    "cardholder",
+    "CARDHOLDER",
+    "podspec",
+    "Podfile",
+    "gradlew",
+    "VAULTURL",
+    "PUREJS",
+    "APPLICATIONORJSON",
+    "TEXTORPLAIN",
+    "TEXTORXML",
+    "FORMURLENCODED",
+    "keyify",
+    "isvalid",
+    "cardnumber",
+    "alttext",
+    "fullname",
+    "revealerror",
+    "iamurl",
+    "Luhn",
+    "banceris",
+    "carter",
+    "gduye",
+    "rdsbe",
+    "ZSAE",
+    "XZYXX",
+    "gphone",
+    "myname",
+    "recordss",
+    "inputfield",
+    "showlist",
+    "Showlist",
+    "iframes"
+  ],
+  "ignorePaths": [
+    "node_modules/**",
+    "dist/**",
+    "build/**",
+    "lib/**",
+    "coverage/**",
+    "*.log",
+    "package-lock.json",
+    "yarn.lock",
+    ".git/**",
+    "android/**",
+    "ios/**",
+    "example/node_modules/**",
+    "example/android/**",
+    "example/ios/**",
+    "*.podspec",
+    "Podfile",
+    "*.gradle",
+    "gradlew",
+    "*.pbxproj"
+  ],
+  "ignoreRegExpList": [
+    "/\\b[A-Z0-9]{20,}\\b/g",
+    "/[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}/gi",
+    "/\\b[A-Za-z0-9_]{7,}\\b(?=])/g"
+  ]
+}

.github/workflows/CI.yml

@@ -31,6 +31,9 @@ jobs:
       - name: install node_modules
         run: npm install
 
+      - name: Run spellcheck
+        run: npm run spellcheck
+
       - name: Run tests
         run: npm run test
 

CHANGELOG.md

@@ -36,7 +36,7 @@ All notable changes to this project will be documented in this file.
 - Added Asterisk symbol for label of required collect element. 
 - `requiredAsterisk` custom styles to collect element labelStyles.
 ### Changed
-- OnFoucs of invalid collect element, removed errorText. 
+- OnFocus of invalid collect element, removed errorText. 
 ## [1.0.1] - 2022-11-01
 
 ### Changed
@@ -47,4 +47,4 @@ All notable changes to this project will be documented in this file.
 
 ## [1.0.0] - 2022-10-11
 
-- Inital Skyflow React Native SDK release
+- Initial Skyflow React Native SDK release

README.md

@@ -155,7 +155,7 @@ The following `props` can be passed to Skyflow Collection Element:
 
 ``` javascript
 {
-  conatiner: 'CollectContainer'  // Required, the collect container.
+  container: 'CollectContainer'  // Required, the collect container.
   table: 'string',               // Required, the table this data belongs to.
   column: 'string',              // Required, the column into which this data should be inserted.
   label: 'string',               // Optional, label for the form element.
@@ -590,7 +590,7 @@ The following `props` can be passed to Skyflow Collection Element:
 
 ``` javascript
 {
-  conatiner: 'CollectContainer'  // Required, the collect container.
+  container: 'CollectContainer'  // Required, the collect container.
   table: 'string',               // Required, the table this data belongs to.
   column: 'string',              // Required, the column into which this data should be inserted.
   label: 'string',               // Optional, label for the form element.
@@ -1421,7 +1421,7 @@ For non-PCI use-cases, retrieving data from the vault and revealing it in the mo
           "table": String,               // name of table from where records are to be fetched
           "redaction": RedactionType,    // redaction to be applied to retrieved data
           "columnName": String,          // a unique column name
-          "colunmnValues": Array         // Array of Column Values of the records to be fetched
+          "columnValues": Array         // Array of Column Values of the records to be fetched
         }
       ]
     }

__tests__/components/components.test.js

@@ -35,7 +35,7 @@ const onBlurMock = jest.fn();
 const onFocusMock = jest.fn();
 
 const changeTrigger = jest.fn();
-const foucsTrigger = jest.fn();
+const focusTrigger = jest.fn();
 const blurTrigger = jest.fn();
 
 describe('test Collect Elements Components', () => {
@@ -47,7 +47,7 @@ describe('test Collect Elements Components', () => {
     jest.spyOn(collectContainer, 'create').mockImplementation(() => ({
       getClientState: () => ({}),
       onChangeElement: changeTrigger,
-      onFocusElement: foucsTrigger,
+      onFocusElement: focusTrigger,
       onBlurElement: blurTrigger,
       updateLabelStyles: jest.fn(),
       updateInputStyles: jest.fn(),
@@ -84,7 +84,7 @@ describe('test Collect Elements Components', () => {
     expect(changeTrigger).toBeCalledWith('hello');
 
     fireEvent(textField, 'focus');
-    expect(foucsTrigger).toBeCalled();
+    expect(focusTrigger).toBeCalled();
 
     fireEvent(textField, 'blur');
     expect(blurTrigger).toBeCalled();
@@ -146,7 +146,7 @@ describe('test Collect Elements Components', () => {
     expect(changeTrigger).toBeCalledWith('1234');
 
     fireEvent(textField, 'focus');
-    expect(foucsTrigger).toBeCalled();
+    expect(focusTrigger).toBeCalled();
 
     fireEvent(textField, 'blur');
     expect(blurTrigger).toBeCalled();
@@ -204,7 +204,7 @@ describe('test Collect Elements Components', () => {
     expect(changeTrigger).toBeCalledWith('123');
 
     fireEvent(textField, 'focus');
-    expect(foucsTrigger).toBeCalled();
+    expect(focusTrigger).toBeCalled();
 
     fireEvent(textField, 'blur');
     expect(blurTrigger).toBeCalled();
@@ -262,7 +262,7 @@ describe('test Collect Elements Components', () => {
     expect(changeTrigger).toBeCalledWith('test');
 
     fireEvent(textField, 'focus');
-    expect(foucsTrigger).toBeCalled();
+    expect(focusTrigger).toBeCalled();
 
     fireEvent(textField, 'blur');
     expect(blurTrigger).toBeCalled();
@@ -324,7 +324,7 @@ describe('test Collect Elements Components', () => {
     expect(changeTrigger).toBeCalledWith('4111111111111111');
 
     fireEvent(textField, 'focus');
-    expect(foucsTrigger).toBeCalled();
+    expect(focusTrigger).toBeCalled();
 
     fireEvent(textField, 'blur');
     expect(blurTrigger).toBeCalled();
@@ -394,7 +394,7 @@ describe('test Collect Elements Components', () => {
     expect(changeTrigger).toBeCalledWith('12/2032');
 
     fireEvent(textField, 'focus');
-    expect(foucsTrigger).toBeCalled();
+    expect(focusTrigger).toBeCalled();
 
     fireEvent(textField, 'blur');
     expect(blurTrigger).toBeCalled();
@@ -458,7 +458,7 @@ describe('test Collect Elements Components', () => {
     expect(changeTrigger).toBeCalledWith('12');
 
     fireEvent(textField, 'focus');
-    expect(foucsTrigger).toBeCalled();
+    expect(focusTrigger).toBeCalled();
 
     fireEvent(textField, 'blur');
     expect(blurTrigger).toBeCalled();
@@ -521,7 +521,7 @@ describe('test Collect Elements Components', () => {
     expect(changeTrigger).toBeCalledWith('2032');
 
     fireEvent(textField, 'focus');
-    expect(foucsTrigger).toBeCalled();
+    expect(focusTrigger).toBeCalled();
 
     fireEvent(textField, 'blur');
     expect(blurTrigger).toBeCalled();

__tests__/core-utils/collect.test.js

@@ -103,7 +103,7 @@ describe('test collect utils class', () => {
       });
   });
 
-  it('test tokenize with addtional fields', (done) => {
+  it('test tokenize with additional fields', (done) => {
     const successResponse = {
       responses: [
         { records: [{ skyflow_id: 'test_skyflow_id' }] },
@@ -268,7 +268,7 @@ describe('test collect utils class', () => {
       });
   });
 
-  it('test tokenize with addtional fields duplicate columns', (done) => {
+  it('test tokenize with additional fields duplicate columns', (done) => {
     const testSkyflowClient = new Skyflow({
       vaultID: '1234',
       vaultURL: 'https://url.com',

__tests__/core-utils/element-validations.test.js

@@ -94,7 +94,7 @@ describe('test validateUpsertOptions', () => {
     }
   });
 
-  it('should throw error for inavlid upsert object type', () => {
+  it('should throw error for invalid upsert object type', () => {
     try {
       validateUpsertOptions([true, 123]);
     } catch (err) {
@@ -181,7 +181,7 @@ describe('test validatePin function', () => {
     expect(validatePin('1234')).toBe(true);
     expect(validatePin('123456789')).toBe(true);
   });
-  it('should return true for value length outof range', () => {
+  it('should return true for value length out of range', () => {
     expect(validatePin('12')).toBe(false);
     expect(validatePin('12345678901234677')).toBe(false);
   });
@@ -710,7 +710,7 @@ describe('test validateGetInput', () => {
         records: [
           {
             columnValues: [],
-            columnName: 'cloumn',
+            columnName: 'column',
             table: 'test',
             redaction: RedactionType.PLAIN_TEXT,
           },

__tests__/core-utils/reveal.test.js

@@ -570,7 +570,7 @@ describe('fetchRecordGET fn test', () => {
       });
   });
 
-  it('should give success reponse in case of success', (done) => {
+  it('should give success response in case of success', (done) => {
     jest.spyOn(ClientModule, 'default').mockImplementation(() => ({
       request: () => Promise.resolve(getSuccessResponse),
     }));

__tests__/core/CollectContainer.test.js

@@ -20,7 +20,7 @@ jest.mock('../../src/core-utils/collect', () => ({
   __esModule: true,
   tokenize: jest.fn(),
 }));
-describe('test CollectConatiner Class', () => {
+describe('test CollectContainer Class', () => {
   let collectContainer;
   beforeEach(() => {
     collectContainer = new CollectContainer(testSkyflowClient);

__tests__/core/RevealContainer.test.js

@@ -13,7 +13,7 @@ const testSkyflowClient = new Skyflow({
   getBearerToken: () => Promise.resolve('valid_auth_token'),
 });
 
-describe('test RevealConatiner Class', () => {
+describe('test RevealContainer Class', () => {
   let revealContainer;
   beforeEach(() => {
     revealContainer = new RevealContainer(testSkyflowClient);
@@ -75,7 +75,7 @@ describe('test RevealConatiner Class', () => {
     });
     revealElement2.setMethods(setValueMock2, setErrorMock2);
 
-    const revealFailedVaule = {
+    const revealFailedValue = {
       records: [
         {
           token: 'random_token',
@@ -96,7 +96,7 @@ describe('test RevealConatiner Class', () => {
     };
     jest
       .spyOn(revealUtils, 'fetchRecordsByTokenId')
-      .mockRejectedValue(revealFailedVaule);
+      .mockRejectedValue(revealFailedValue);
 
     revealContainer
       .reveal()
@@ -111,7 +111,7 @@ describe('test RevealConatiner Class', () => {
           expect(setValueMock2).toBeCalledTimes(0);
           expect(err).toEqual({
             success: [{ token: 'random_token', valueType: 'STRING' }],
-            errors: revealFailedVaule.errors,
+            errors: revealFailedValue.errors,
           });
           done();
         } catch (error) {