Chore: Mocha tests examples, Added getSublist for Record Module, Fixes (#13)

*  init branch

* installed mocha, created test folder structure

* work

* init mocha

* fixed one test

* Fixes, Added sublist for record (#12)

* Chore: Two fixes in N/Search, Utils/Record (#11)

* recordModule

Added new fuction `insertLine`

* Search-module

Fixes

* added sublistObjectMemeber

* Clean

Author: krolikowskidamian

Examples/1.0/exampleClientScript.js renamed to Examples/Jest/1.0/exampleClientScript.js

@@ -1,4 +1,4 @@
-require('../../index')({});
+require('../../../index')({});
 
 /**
    * Save Record Function

Examples/1.0/exampleClientScript.test.js renamed to Examples/Jest/1.0/exampleClientScript.test.js

@@ -11,7 +11,7 @@
 
 // Line below is required only becasue we are in the same repository as the library,
 // normally define is added to globally once you import our library and follow setup instructions
-const { define } = require('../../index')({});
+const { define } = require('../../../index')({});
 
 // eslint-disable-next-line import/no-amd
 define(['N/ui/message'], (msg) => {

Examples/2.0/exampleClientScript.js renamed to Examples/Jest/2.0/exampleClientScript.js

@@ -0,0 +1,53 @@
+/**
+ * @NApiVersion 2.x
+ * @NScriptType ClientScript
+ */
+
+/**
+  * This is an example of client script file.
+  * In corresponding .test.js file you can find a way how to test it
+  * of course with our ns-suitescript-mocks
+  */
+
+// Line below is required only becasue we are in the same repository as the library,
+// normally define is added to globally once you import our library and follow setup instructions
+const { define } = require('../../../index')({});
+
+// eslint-disable-next-line import/no-amd
+define(['N/ui/message'], (msg) => {
+  /**
+     * Page Init function to show message after form submit
+     *
+     * @param {N/context} context Module
+     * @returns {boolean}
+     */
+  function pageInit(context) {
+    // eslint-disable-next-line prefer-destructuring
+    const currentRecord = context.currentRecord;
+    const myMsg = msg.create({
+      title: 'Checkbox was checked',
+      message: 'Checkbox was checked!',
+      type: msg.Type.CONFIRMATION,
+    });
+    const showMessage = currentRecord.getValue({
+      fieldId: 'messageonpost',
+    });
+    if (showMessage) {
+      myMsg.show({
+        duration: 5000,
+      });
+    }
+
+    return true;
+  }
+  // required for node
+  if (typeof module !== 'undefined' && module.exports) {
+    module.exports = {
+      pageInit,
+      message: msg,
+    };
+  }
+  return {
+    pageInit,
+  };
+});

Examples/2.0/exampleClientScript.test.js renamed to Examples/Jest/2.0/exampleClientScript.test.js

@@ -0,0 +1,52 @@
+const assert = require('assert');
+const sinon = require('sinon');
+const fileUnderTest = require('./exampleClientScript');
+
+const defaultValues = {
+  tranid: '12121',
+  custbody_mycs_ship_method: 23,
+  shipstatus: 'B',
+  custbodysps_packagedetails: 'test',
+};
+const fulfillmentRecord = nlapiCreateRecord('itemfulfillment', defaultValues);
+const context = {
+  currentRecord: fulfillmentRecord,
+};
+
+describe('exampleClientScript.js', () => {
+  describe('pageInit', () => {
+    it('should return true', () => {
+      const result = fileUnderTest.pageInit(context);
+      assert.deepEqual(result, true);
+    });
+    it('should call create message method', () => {
+      // eslint-disable-next-line prefer-destructuring
+      const message = fileUnderTest.message;
+      const spyMessage = sinon.stub(message, 'create');
+      fileUnderTest.pageInit(context);
+      assert(spyMessage.called);
+      assert(spyMessage.calledWithExactly({
+        title: 'Checkbox was checked',
+        message: 'Checkbox was checked!',
+        type: 'confirmation',
+      }));
+      message.create.restore();
+    });
+
+    it('should show not message if checkbox is not checked', () => {
+      // eslint-disable-next-line prefer-destructuring
+      const message = fileUnderTest.message;
+      const createdMessage = message.create({
+        message: 'Email with CSV was sent to your mailbox!',
+        title: 'Email has been sent.',
+        type: 'confirmation',
+      });
+      context.currentRecord.setFieldValue('messageonpost', false);
+      const spyMesage = sinon.stub(message, 'create');
+      const spyShow = sinon.spy(createdMessage, 'show');
+      fileUnderTest.pageInit(context);
+      assert(spyMesage.called);
+      assert(spyShow.notCalled);
+    });
+  });
+});

Examples/Mocha/2.0/exampleClientScript.js

@@ -25,6 +25,7 @@ function createColumn(options) {
 function createFilter(options) { // eslint-disable-line no-unused-vars
   return `${options.name}${options.join ? `__${options.join}` : ''}`;
 }
+
 /**
  * Return NetSuite ResultSet type.
  * @returns {object} resultset
@@ -104,23 +105,6 @@ function ResultSet() {
   };
 }
 
-/**
- * NetSuite new column
- * @param {*} options
- * @returns {string}
- */
-function createColumn(options) {
-  return `${ options.name }${ options.join ? '__' + options.join : '' }`;
-}
-
-/**
- * NetSuite new filter
- * @param {*} options
- * @returns {string}
- */
-function createFilter(options) { // eslint-disable-line no-unused-vars
-  return `${ options.name }${ options.join ? '__' + options.join : '' }`;
-}
 /**
  * NetSuite - constructor of search module.
  * @param {*} options_
@@ -163,11 +147,11 @@ module.exports = {
   lookupFields: (options) => {
     const result = {};
     if (Array.isArray(options.columns)) {
-      options.columns.map((item, index) => {
-        result[item] = [ { value: index, text: item } ];
+      options.columns.forEach((item, index) => {
+        result[item] = [{ value: index, text: item }];
       });
     } else if (typeof options.column === 'string') {
-      result[options.column] = [ { value: Math.floor(Math.random() * 10000), text: options.column } ];
+      result[options.column] = [{ value: Math.floor(Math.random() * 10000), text: options.column }];
     }
     return result;
   },

Examples/Mocha/2.0/exampleClientScript.test.js

@@ -56,7 +56,7 @@ Before pushing your code, make sure that the tests are passing (at the moment, w
 
 ## Authors 😎
 
-* **Damian Królikowski** - *Mocks Creator* - [damiankrolikowskimycs](https://github.com/damiankrolikowskimycs)
+* **Damian Królikowski** - *Mocks Creator* - [krolikowskidamian](https://github.com/krolikowskidamian)
 * **Łukasz Kubasiewicz** - *Mocks Creator* - [lukaszMycs](https://github.com/lukaszmycs)
 * **Nicolas Ritouet** - *Automation setup, cleanup, documentation* - [nicolasritouet](https://github.com/nicolasritouet)
 

N/search.js

@@ -1,3 +1,5 @@
+const sublistObj = require('./sublistObjectMembers.jsx');
+
 /**
  * Return a Netsuite Record
  *
@@ -45,6 +47,7 @@ function NsRecord(name, defaultValues = {}) {
     getLineItemValue: (sublist, field, index) => this.values[sublist][index - 1][field],
     getSublistText: options => this.values[options.sublistId][options.line][options.fieldId],
     getSublistValue: options => this.values[options.sublistId][options.line][options.fieldId],
+    getSublists: sublistObj,
     getSubrecord: options => nlapiCreateRecord(options),
     getText: options => this.values[options.fieldId || options],
     getValue: options => this.values[options.fieldId || options],
@@ -68,7 +71,7 @@ function NsRecord(name, defaultValues = {}) {
     insertLineItem: (sublist, index) => {
       this.values[sublist].splice(index - 1, 0, {});
     },
-    insertLine: options => {
+    insertLine: (options) => {
       this.values[options.sublistId].splice(options.line, 0, {});
     },
     removeLine: (obj) => {

README.md

@@ -0,0 +1,23 @@
+/**
+ * Return a Netsuite Sublist Object
+ *
+ * @classDescription Record sublist bject
+ * @constructor
+ * @param {string} name
+ * @param {object} defaultValues
+ * @returns {nlobjRecord}
+ */
+function sublistObjectMembers(sublist, defaultValues = {}) {
+  const id = sublist.sublistId || sublist;
+  this.values = defaultValues;
+  this.values.item = defaultValues.item || [];
+  return {
+    getColumn: () => id,
+    id,
+    isChanged: this.values.isChanged || false,
+    isDisplay: this.values.isDisplay || false,
+    type: this.values.defaultValues.type || 'nlobjRecord',
+  };
+}
+
+module.exports = sublistObjectMembers;