@rancher/shell 0.3.13 → 0.3.15

package/plugins/steve/getters.js

@@ -94,7 +94,7 @@ export default {
     return url;
   },
 
-  defaultModel: state => (obj) => {
+  defaultModel: (state) => (obj) => {
     const which = state.config.modelBaseClass || STEVE_MODEL_TYPES.BY_TYPE.STEVE;
 
     if ( which === STEVE_MODEL_TYPES.BY_TYPE ) {
@@ -154,7 +154,7 @@ export default {
   },
 
   // Return all the pods for a given namespace
-  podsByNamespace: state => (namespace) => {
+  podsByNamespace: (state) => (namespace) => {
     const map = state.podsByNamespace[namespace];
 
     return map?.list || [];
@@ -164,7 +164,7 @@ export default {
     return GC_IGNORE_TYPES;
   },
 
-  currentGeneration: state => (type) => {
+  currentGeneration: (state) => (type) => {
     type = normalizeType(type);
 
     const cache = state.types[type];

package/plugins/steve/hybrid-class.js

@@ -9,7 +9,7 @@ import Resource from '@shell/plugins/dashboard-store/resource-class';
 export function cleanHybridResources(data) {
   const potentialNormanHoldovers = ['state', 'name', 'description', 'labels', 'annotations'];
 
-  potentialNormanHoldovers.forEach(key => delete data[key]);
+  potentialNormanHoldovers.forEach((key) => delete data[key]);
 
   return data;
 }

package/plugins/steve/mutations.js

@@ -66,7 +66,7 @@ function cleanPodsByNamespaceCache(state, resource) {
 
     // Extra defensive check that the cache exists for the namespace being removed
     if (cache) {
-      const inList = cache.list.findIndex(p => p.id === resource.id);
+      const inList = cache.list.findIndex((p) => p.id === resource.id);
 
       if ( inList >= 0 ) {
         cache.list.splice(inList, 1);

package/plugins/steve/performanceTesting.js

@@ -81,7 +81,7 @@ function replicate(data, config) {
   const templates = [];
   let j = 0;
 
-  data.forEach(d => templates.push(JSON.stringify(d)));
+  data.forEach((d) => templates.push(JSON.stringify(d)));
 
   const newData = [...data];
 

package/plugins/steve/resourceWatcher.js

@@ -40,7 +40,7 @@ export const keyForSubscribe = ({
   resourceType, type, namespace, id, selector
 } = {}) => {
   return [(resourceType || type), namespace, id, selector] // each watch param in an array
-    .filter(param => !!param) // filter out all the empty ones // the filter makes these keys neater
+    .filter((param) => !!param) // filter out all the empty ones // the filter makes these keys neater
     .join('/'); // join into a string so we can use it as an object key
 };
 

package/plugins/steve/subscribe.js

@@ -661,7 +661,7 @@ const defaultActions = {
       have = getters['all'](resourceType).slice();
 
       if ( namespace ) {
-        have = have.filter(x => x.metadata?.namespace === namespace);
+        have = have.filter((x) => x.metadata?.namespace === namespace);
       }
 
       want = await dispatch('findAll', {
@@ -1012,7 +1012,7 @@ const defaultMutations = {
   },
 
   setWatchStarted(state, obj) {
-    const existing = state.started.find(entry => equivalentWatch(obj, entry));
+    const existing = state.started.find((entry) => equivalentWatch(obj, entry));
 
     if ( !existing ) {
       addObject(state.started, obj);
@@ -1022,7 +1022,7 @@ const defaultMutations = {
   },
 
   setWatchStopped(state, obj) {
-    const existing = state.started.find(entry => equivalentWatch(obj, entry));
+    const existing = state.started.find((entry) => equivalentWatch(obj, entry));
 
     if ( existing ) {
       removeObject(state.started, existing);
@@ -1066,12 +1066,12 @@ const defaultMutations = {
  * Getters that cover cases 1 & 2 (see file description)
  */
 const defaultGetters = {
-  inError: state => (obj) => {
+  inError: (state) => (obj) => {
     return state.inError[keyForSubscribe(obj)];
   },
 
-  watchStarted: state => (obj) => {
-    return !!state.started.find(entry => equivalentWatch(obj, entry));
+  watchStarted: (state) => (obj) => {
+    return !!state.started.find((entry) => equivalentWatch(obj, entry));
   },
 
   nextResourceVersion: (state, getters) => (type, id) => {

package/plugins/steve/worker/web-worker.basic.js

@@ -105,7 +105,7 @@ const workerActions = {
   // Remove the cached schema
   removeSchema: (id) => {
     // Remove anything in the queue related to the schema - we don't want to send any pending updates later for a schema that has been removed
-    state.queue = state.queue.filter(schema => schema.id !== id);
+    state.queue = state.queue.filter((schema) => schema.id !== id);
 
     // Delete the schema from the map, so if it comes back we don't ignore it if the hash is the same
     delete state.schemas[id];

package/promptRemove/management.cattle.io.project.vue

@@ -52,7 +52,7 @@ export default {
 
     filteredNamespaces() {
       if (this.currentProject) {
-        return this.allNamespaces.filter(n => n.metadata.labels[PROJECT] === this.currentProject.metadata?.name);
+        return this.allNamespaces.filter((n) => n.metadata.labels[PROJECT] === this.currentProject.metadata?.name);
       }
 
       return [];
@@ -69,7 +69,7 @@ export default {
     },
 
     names() {
-      return this.filteredNamespaces.map(obj => obj.nameDisplay).slice(0, 5);
+      return this.filteredNamespaces.map((obj) => obj.nameDisplay).slice(0, 5);
     },
     // Only admins and cluster owners can see namespaces outside of projects
     canSeeProjectlessNamespaces() {
@@ -82,7 +82,7 @@ export default {
       // Delete all of thre namespaces and return false - this tells the prompt remove dialog to continue and delete the project
       // Delete all namespaces if the user wouldn't be able to see them after deleting the project
       if (this.deleteProjectNamespaces || !this.canSeeProjectlessNamespaces) {
-        return Promise.all(this.filteredNamespaces.map(n => n.remove())).then(() => false);
+        return Promise.all(this.filteredNamespaces.map((n) => n.remove())).then(() => false);
       }
 
       // Return false so that the main promptRemoval will continue to remove the project

package/promptRemove/mixin/roleDeletionCheck.js

@@ -17,7 +17,7 @@ export default {
     ...mapGetters({ t: 'i18n/t' }),
 
     names() {
-      return this.toRemove.map(obj => obj.nameDisplay).slice(0, 5);
+      return this.toRemove.map((obj) => obj.nameDisplay).slice(0, 5);
     },
 
     plusMore() {
@@ -82,14 +82,14 @@ export default {
 
         if (request.data && request.data.length) {
           rolesToRemove.forEach((toRemove) => {
-            const usedRoles = request.data.filter(item => item[propToMatch] === toRemove.id);
+            const usedRoles = request.data.filter((item) => item[propToMatch] === toRemove.id);
 
             if (usedRoles.length) {
-              const uniqueUsers = [...new Set(usedRoles.map(item => item.userName).filter(user => userMap[user]))];
+              const uniqueUsers = [...new Set(usedRoles.map((item) => item.userName).filter((user) => userMap[user]))];
 
               if (uniqueUsers.length) {
                 numberOfRolesWithBinds++;
-                uniqueUsers.forEach(user => uniqueUsersWithBinds.add(user));
+                uniqueUsers.forEach((user) => uniqueUsersWithBinds.add(user));
               }
             }
           });

package/promptRemove/pod.vue

@@ -79,7 +79,7 @@ export default {
       }
 
       try {
-        await Promise.all(this.value.map(resource => this.removePod(resource)));
+        await Promise.all(this.value.map((resource) => this.removePod(resource)));
         if ( goTo && !isEmpty(goTo) ) {
           parentComponent.currentRouter.push(goTo);
         }

package/rancher-components/StringList/StringList.test.ts

@@ -0,0 +1,80 @@
+import { mount } from '@vue/test-utils';
+import { StringList } from './index';
+
+describe('StringList.vue', () => {
+
+  it('is empty', () => {
+    const wrapper = mount(StringList, { 
+      propsData: { items: [] },
+    });
+    const box = wrapper.find('[data-testid="div-string-list-box"]').element as HTMLElement;
+
+    expect(box.children.length).toBe(0);
+  });
+
+  it('is showing one element', () => {
+    const wrapper = mount(StringList, { 
+      propsData: { items: ['test'] },
+    });
+    const box = wrapper.find('.string-list-box').element as HTMLElement;
+
+    expect(box.children.length).toBe(1);
+  });
+
+  it('action-buttons are visible', () => {
+    const wrapper = mount(StringList, { 
+      propsData: { items: ['test'] },
+    });
+    const actionButtons = wrapper.find('[data-testid="div-action-buttons"]').element as HTMLElement;
+
+    expect(actionButtons).not.toBe(undefined);
+  });
+
+  it('action-buttons are hidden when is view-only mode', () => {
+    const wrapper = mount(StringList, { 
+      propsData: {
+        items: ['test'],
+        readonly: true,
+      },
+    });
+    const actionButtons = wrapper.find('[data-testid="div-action-buttons"]').element as HTMLElement;
+
+    expect(actionButtons).toBe(undefined);
+  });
+
+  it('show new item when "items" property change', async () => {
+    const items = [ 'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'k'];
+
+    const wrapper = mount(StringList, { 
+      propsData: {
+        items,
+      },
+    });
+    const elements = wrapper.findAll('[data-testid^="div-item"]');
+
+    expect(elements.length).toBe(10);
+
+    await wrapper.setProps({ items: [ ...items, 'new' ] });
+
+    const newElements = wrapper.findAll('[data-testid^="div-item"]');
+    expect(newElements.length).toBe(11);
+  });
+
+  it('remove item when "items" property change', async () => {
+    const items = [ 'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'k'];
+
+    const wrapper = mount(StringList, { 
+      propsData: {
+        items,
+      },
+    });
+    const elements = wrapper.findAll('[data-testid^="div-item"]');
+    expect(elements.length).toBe(10);
+
+    await wrapper.setProps({ items: [ ...items.filter(f => f !== 'a') ] });
+
+    const newElements = wrapper.findAll('[data-testid^="div-item"]');
+    expect(newElements.length).toBe(9);
+  });
+
+});