spyne 0.18.3 → 0.19.1

package/src/spyne/views/view-stream.js

@@ -1,6 +1,7 @@
 import { baseCoreMixins } from '../utils/mixins/base-core-mixins';
 import {SpyneAppProperties} from '../utils/spyne-app-properties';
 import { deepMerge } from '../utils/deep-merge';
+import {safeClone} from '../utils/safe-clone';
 import {
   findStrOrRegexMatchStr,
   getConstructorName
@@ -445,10 +446,11 @@ export class ViewStream {
     const elIsRendered = el => document.body.contains(el);
     const elIsReadyBool = propSatisfies(
         allPass([elIsRendered, elIsDomElement]), 'el');
-
     if (elIsReadyBool(this.props)) {
       this.updatePropsToMatchEl();
       this.postRender();
+    } else if(this.props.el === null){
+      console.error(`Spyne Error: The defined element for this ViewStream instance, ${this.constructor.name}, appears to not exist.`)
     }
   }
 
@@ -890,6 +892,13 @@ export class ViewStream {
     this.renderViewAndAttachToDom(node, 'dom', 'appendChild');
   }
 
+  appendToDomAfter(node){
+    if (this.props.el !== undefined){
+      console.warn(`Spyne Warning: The ViewStream, ${this.props.name}, has an element, ${this.props.el}, that is already rendered and does not need to be appendedToDomAfter. This may create unsusual side effects!`)
+    }
+    this.renderViewAndAttachToDom(node, 'dom', 'after');
+  }
+
   /**
    * Prepends the current ViewStream object to an existing dom element.
    * @param {HTMLElement} node the ViewStream child that is to be attached.
@@ -931,6 +940,10 @@ export class ViewStream {
     this.exchangeViewsWithChild(v, this.setAttachData('appendChild', query));
   }
 
+  appendViewAfter(v, query) {
+    this.exchangeViewsWithChild(v, this.setAttachData('after', query));
+  }
+
   /**
    * This method appends a child ViewStream object to a parent ViewStream object.
    * @param {ViewStream} v the ViewStream parent.
@@ -1286,7 +1299,7 @@ export class ViewStream {
       if (/CHANNEL_LIFECYCLE/.test(action)===false){
         Object.defineProperties(data, {
           payload: {
-            get: ()=> clone(pl)
+            get: ()=> safeClone(pl)
 
           }
         })

package/src/tests/channels/channel.test.js

@@ -31,8 +31,14 @@ describe('should test channel instance', () => {
     expect(props.sendCachedPayload).to.be.true;
     expect(actionsObj).to.deep.equal(finalActionsObj);
 
+  })
 
-
+  it('should update SpyneAppProperties doNotTrackList if doNotTrack is true', ()=>{
+    const channelName="CHANNEL_MY_TEST_CHANNEL";
+    const doNotTrack = true;
+    Channel.checkForNotTrackFlag({channelName, doNotTrack});
+    const untrackedChannelsArr = SpyneAppProperties.getUntrackedChannelsList()
+    expect(untrackedChannelsArr).to.deep.eq([channelName]);
   })
 
 });

package/src/tests/channels/route-utils.test.js

@@ -64,6 +64,7 @@ describe('it should compare two objects for updated keys', () => {
 
   it('first comparison should be done against empty obj', () => {
     let compareStart = checkUpdatedKeys.compare(obj1);
+    console.log("compare start is ",compareStart);
     expect(compareStart.pathsChanged).to.deep.equal(['pageId', 'section']);
   });
 

package/src/tests/utils/channel-data-packet-generator.unused.js

@@ -39,7 +39,7 @@ describe('should test the ChannelDataPacketGenerator', () => {
     map.set(label, ChannelPayloadToTestFilters);
 
     const filterGateway = ChannelDataPacketGenerator.createFilterGateway(label, map);
-    const pred = R.propEq('action', 'CHANNEL_ROUTE_CHANGE_EVENT');
+    const pred = R.propEq('CHANNEL_ROUTE_CHANGE_EVENT', 'action');
 
     const isCorrectAction = filterGateway(pred)
 

package/src/tests/views/dom-el-template.test.js

@@ -31,7 +31,7 @@ describe('DomElTemplate', () => {
     let data = {dog: {
         sound: 'woof'
       }};
-    let template = "<h1>The dog says {{#dog}}{{sound}}{{/dog}}";
+    let template = "<h1>The dog says {{#dog}}{{sound}}{{/dog}}</h1>";
     let domElTemplate = new DomElementTemplate(template, data);
     let render = domElTemplate.renderDocFrag();
     expect(render.firstElementChild.innerText).to.equal('The dog says woof');
@@ -163,6 +163,23 @@ describe('DomElTemplate', () => {
     return true;
   });
 
+  it('DomElementTemplate should render index string value of an array',() =>{
+    const article = ["Usu at illum porro audire. Nam at ubique latine, vidit ocurreret pri ea, cu elitr nonumes mediocritatem nam.",
+      "Eu delenit meliore graecis sea. Sit id ubique commune, ius ne recusabo oportere similique, error putant usu ei.",
+      "Nam mutat saperet detracto eu, te ubique utamur aliquando pro. Ut verear probatus sea. Porro sonet euripidis ex est.",
+      "Mucius platonem eu per. Te utroque persecuti pro, error verterem scribentur no est.",
+      "An pro nibh salutatus, ea rebum aeterno complectitur has."];
+
+    const template = `<li>{{article.1}}</li>`;
+    const data = {article};
+    const domElTemplate = new DomElementTemplate(template, data);
+
+    const result = domElTemplate.renderToString();
+
+    expect(result).to.eq("<li>Eu delenit meliore graecis sea. Sit id ubique commune, ius ne recusabo oportere similique, error putant usu ei.</li>");
+
+  })
+
 
 
 });