@rancher/shell 0.3.14 → 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.vue

@@ -87,7 +87,7 @@ export default Vue.extend({
     return {
       value:             null as string | null,
       selected:          null as string | null,
-      editedItem:        null as string | null,
+      isEditItem:        null as string | null,
       isCreateItem:      false,
       errors:            { duplicate: false } as Record<Error, boolean>
     };
@@ -100,7 +100,7 @@ export default Vue.extend({
      */
     errorMessagesArray(): string[] {
       return (Object.keys(this.errors) as Error[])
-        .filter(f => this.errors[f] && this.errorMessages[f])
+        .filter(f => !!(this.errors)[f])
         .map(k => this.errorMessages[k]);
     },
   },
@@ -113,32 +113,23 @@ export default Vue.extend({
       this.toggleEditMode(false);
       this.toggleCreateMode(false);
     },
-    errors: {
-      handler(val) {
-        this.$emit('errors', val);
-      },
-      deep: true
-    }
   },
 
   methods: {
     onChange(value: string) {
       this.value = value;
-
-      const items = [
-        ...this.items,
-        this.value
-      ];
-
+      /**
+       * Remove duplicate error when a new value is typed
+       */
       this.toggleError(
         'duplicate',
-        hasDuplicatedStrings(items, this.caseSensitive),
-        this.isCreateItem ? INPUT.create : INPUT.edit
+        false,
+        this.isCreateItem ? INPUT.create : INPUT.edit,
       );
     },
 
     onSelect(item: string) {
-      if (this.readonly || this.isCreateItem || this.editedItem === item) {
+      if (this.isCreateItem || this.isEditItem === item) {
         return;
       }
       this.selected = item;
@@ -169,7 +160,7 @@ export default Vue.extend({
     },
 
     onClickEmptyBody() {
-      if (!this.isCreateItem && !this.editedItem) {
+      if (!this.isCreateItem && !this.isEditItem) {
         this.toggleCreateMode(true);
       }
     },
@@ -185,7 +176,7 @@ export default Vue.extend({
 
         return;
       }
-      if (this.editedItem) {
+      if (this.isEditItem) {
         this.toggleEditMode(false);
 
         return;
@@ -238,7 +229,6 @@ export default Vue.extend({
      */
     toggleError(type: Error, val: boolean, refId?: string) {
       this.errors[type] = val;
-
       if (refId) {
         this.toggleErrorClass(refId, val);
       }
@@ -260,11 +250,7 @@ export default Vue.extend({
      * Show/Hide the input line to create new item
      */
     toggleCreateMode(show: boolean) {
-      if (this.readonly) {
-        return;
-      }
       if (show) {
-        this.toggleEditMode(false);
         this.value = '';
 
         this.isCreateItem = true;
@@ -282,21 +268,18 @@ export default Vue.extend({
      * Show/Hide the in-line editing to edit an existing item
      */
     toggleEditMode(show: boolean, item?: string) {
-      if (this.readonly) {
-        return;
-      }
       if (show) {
         this.toggleCreateMode(false);
-        this.value = this.editedItem;
+        this.value = this.isEditItem;
 
-        this.editedItem = item || '';
+        this.isEditItem = item || '';
         this.setFocus(INPUT.edit);
       } else {
         this.value = null;
         this.toggleError('duplicate', false);
         this.onSelectLeave();
 
-        this.editedItem = null;
+        this.isEditItem = null;
       }
     },
 
@@ -309,7 +292,7 @@ export default Vue.extend({
     /**
      * Create a new item and insert in the items list
      */
-    saveItem(closeInput = true) {
+    saveItem() {
       const value = this.value?.trim();
 
       if (value) {
@@ -318,20 +301,21 @@ export default Vue.extend({
           value,
         ];
 
-        if (!hasDuplicatedStrings(items, this.caseSensitive)) {
-          this.updateItems(items);
+        if (hasDuplicatedStrings(items, this.caseSensitive)) {
+          this.toggleError('duplicate', true, INPUT.create);
+
+          return;
         }
-      }
 
-      if (closeInput) {
-        this.toggleCreateMode(false);
+        this.updateItems(items);
       }
+      this.toggleCreateMode(false);
     },
 
     /**
      * Update an existing item in the items list
      */
-    updateItem(item: string, closeInput = true) {
+    updateItem(item: string) {
       const value = this.value?.trim();
 
       if (value) {
@@ -342,14 +326,15 @@ export default Vue.extend({
           items[index] = value;
         }
 
-        if (!hasDuplicatedStrings(items, this.caseSensitive)) {
-          this.updateItems(items);
+        if (hasDuplicatedStrings(items, this.caseSensitive)) {
+          this.toggleError('duplicate', true, INPUT.edit);
+
+          return;
         }
-      }
 
-      if (closeInput) {
-        this.toggleEditMode(false);
+        this.updateItems(items);
       }
+      this.toggleEditMode(false);
     },
 
     /**
@@ -402,19 +387,19 @@ export default Vue.extend({
         @blur="onSelectLeave(item)"
       >
         <span
-          v-if="!editedItem || editedItem !== item"
+          v-if="!isEditItem || isEditItem !== item"
           class="label static"
         >
           {{ item }}
         </span>
         <LabeledInput
-          v-if="editedItem && editedItem === item"
+          v-if="isEditItem && isEditItem === item"
           ref="item-edit"
           class="edit-input static"
           :value="value != null ? value : item"
           @input="onChange($event)"
-          @blur.prevent="updateItem(item)"
-          @keydown.native.enter="updateItem(item, !errors.duplicate)"
+          @blur.prevent="toggleEditMode(false)"
+          @keydown.native.enter="updateItem(item)"
         />
       </div>
       <div
@@ -428,8 +413,7 @@ export default Vue.extend({
           :value="value"
           :placeholder="placeholder"
           @input="onChange($event)"
-          @blur.prevent="saveItem"
-          @keydown.native.enter="saveItem(!errors.duplicate)"
+          @keydown.native.enter="saveItem"
         />
       </div>
     </div>
@@ -444,14 +428,14 @@ export default Vue.extend({
       >
         <button
           class="btn btn-sm role-tertiary remove-button"
-          :disabled="!selected && !isCreateItem && !editedItem"
+          :disabled="!selected && !isCreateItem && !isEditItem"
           @mousedown.prevent="onClickMinusButton"
         >
           <span class="icon icon-minus icon-sm" />
         </button>
         <button
           class="btn btn-sm role-tertiary add-button"
-          :disabled="isCreateItem || editedItem"
+          :disabled="isCreateItem"
           @click.prevent="onClickPlusButton"
         >
           <span class="icon icon-plus icon-sm" />

package/rancher-components/components/Form/Checkbox/Checkbox.test.ts

@@ -1,7 +1,7 @@
 import { shallowMount, Wrapper } from '@vue/test-utils';
 import { Checkbox } from './index';
 
-describe('Checkbox.vue', () => {
+describe('checkbox.vue', () => {
   const event = {
     target:          { tagName: 'input', href: null },
     stopPropagation: () => { },
@@ -63,6 +63,6 @@ describe('Checkbox.vue', () => {
     wrapper.vm.clicked(event);
     await wrapper.vm.$nextTick();
 
-    expect(wrapper.emitted().input?.[0][0]).toBe(null);
+    expect(wrapper.emitted().input?.[0][0]).toBeNull();
   });
 });

package/rancher-components/components/Form/Checkbox/Checkbox.vue

@@ -202,7 +202,7 @@ export default Vue.extend({
      * @param value A collection of values for the checkbox.
      */
     findTrueValues(value: boolean[]): boolean {
-      return value.find(v => v === this.valueWhenTrue) || false;
+      return value.find((v) => v === this.valueWhenTrue) || false;
     }
   }
 });

package/rancher-components/components/Form/Radio/RadioButton.test.ts

@@ -2,7 +2,7 @@ import { shallowMount } from '@vue/test-utils';
 import { RadioButton } from './index';
 import { cleanHtmlDirective } from '@shell/plugins/clean-html-directive';
 
-describe('RadioButton.vue', () => {
+describe('radioButton.vue', () => {
   it('renders label slot contents', () => {
     const wrapper = shallowMount(RadioButton, { slots: { label: 'Test Label' } });
 

package/rancher-components/components/Form/Radio/RadioButton.vue

@@ -17,7 +17,7 @@ export default Vue.extend({
      */
     val: {
       required:  true,
-      validator: x => true
+      validator: () => true
     },
 
     /**
@@ -25,7 +25,7 @@ export default Vue.extend({
      */
     value: {
       required:  true,
-      validator: x => true
+      validator: () => true
     },
 
     /**

package/rancher-components/components/Form/Radio/RadioGroup.vue

@@ -152,7 +152,7 @@ export default Vue.extend({
      */
     clickNext(direction: number): void {
       const opts = this.normalizedOptions;
-      const selected = opts.find(x => x.value === this.value);
+      const selected = opts.find((x) => x.value === this.value);
       let newIndex = (selected ? opts.indexOf(selected) : -1) + direction;
 
       if (newIndex >= opts.length) {
@@ -209,7 +209,7 @@ export default Vue.extend({
         <slot
           :listeners="$listeners"
           :option="option"
-          :isDisabled="isDisabled"
+          :is-disabled="isDisabled"
           :name="i"
         >
           <RadioButton

package/rancher-components/components/Form/TextArea/TextAreaAutoGrow.vue

@@ -155,6 +155,7 @@ export default Vue.extend({
 <template>
   <textarea
     ref="ta"
+    data-testid="text-area-auto-grow"
     :disabled="isDisabled"
     :style="style"
     :placeholder="placeholder"

package/rancher-components/components/Form/ToggleSwitch/ToggleSwitch.test.ts

@@ -1,7 +1,7 @@
 import { shallowMount, Wrapper } from '@vue/test-utils';
 import { ToggleSwitch } from './index';
 
-describe('ToggleSwitch.vue', () => {
+describe('toggleSwitch.vue', () => {
   it('renders falsy by default', () => {
     const wrapper = shallowMount(ToggleSwitch);
 

package/rancher-components/components/Form/ToggleSwitch/ToggleSwitch.vue

@@ -31,13 +31,6 @@ export default Vue.extend({
     return { state: false as boolean | string | number };
   },
 
-  methods: {
-    toggle(neu: boolean | string | number) {
-      this.state = neu === null ? !this.state : neu;
-      this.$emit('input', this.state ? this.onValue : this.offValue);
-    }
-  },
-
   watch: {
     value: {
       handler() {
@@ -45,6 +38,13 @@ export default Vue.extend({
       },
       immediate: true
     }
+  },
+
+  methods: {
+    toggle(neu: boolean | string | number) {
+      this.state = neu === null ? !this.state : neu;
+      this.$emit('input', this.state ? this.onValue : this.offValue);
+    }
   }
 });
 </script>

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

@@ -1,18 +1,19 @@
+/* eslint-disable jest/no-hooks */
 import { mount, Wrapper } from '@vue/test-utils';
 import { StringList } from './index';
 
-describe('StringList.vue', () => {
+describe('stringList.vue', () => {
   let wrapper: Wrapper<InstanceType<typeof StringList>>;
 
   beforeEach(() => {
     wrapper = mount(StringList, { propsData: { items: [] } });
   });
 
-  describe('List box', () => {
+  describe('list box', () => {
     it('is empty', () => {
       const box = wrapper.find('[data-testid="div-string-list-box"]').element as HTMLElement;
 
-      expect(box.children.length).toBe(0);
+      expect(box.children).toHaveLength(0);
     });
 
     it('show multiple items', async() => {
@@ -22,7 +23,7 @@ describe('StringList.vue', () => {
 
       const elements = wrapper.findAll('[data-testid^="div-item"]');
 
-      expect(elements.length).toBe(10);
+      expect(elements).toHaveLength(10);
     });
 
     it('double click triggers inline edit mode', async() => {
@@ -130,7 +131,7 @@ describe('StringList.vue', () => {
     });
   });
 
-  describe('Buttons', () => {
+  describe('buttons', () => {
     it('are visible by default', () => {
       const actionButtons = wrapper.find('[data-testid="div-action-buttons"]');
 
@@ -144,7 +145,7 @@ describe('StringList.vue', () => {
       expect(actionButtons.element).toBeUndefined();
     });
 
-    describe('Add button', () => {
+    describe('add button', () => {
       it('is enabled by default', () => {
         const addButton = wrapper.find('[data-testid="button-add"]')?.element as HTMLButtonElement;
 
@@ -176,7 +177,7 @@ describe('StringList.vue', () => {
       });
     });
 
-    describe('Remove button', () => {
+    describe('remove button', () => {
       it('is disabled by default', () => {
         const removeButton = wrapper.find('[data-testid="button-remove"]');
         const buttonElem = removeButton.element as HTMLButtonElement;
@@ -279,7 +280,7 @@ describe('StringList.vue', () => {
     });
   });
 
-  describe('List edit', () => {
+  describe('list edit', () => {
     const validItem = '    item name   ';
     const emptyItem = '  ';
 
@@ -397,7 +398,7 @@ describe('StringList.vue', () => {
     });
   });
 
-  describe('Errors handling', () => {
+  describe('errors handling', () => {
     it('show duplicate warning icon when errorMessages is defined', async() => {
       const items = ['test'];
 

package/rancher-components/components/StringList/StringList.vue

@@ -30,9 +30,9 @@ const CLASS = {
  * Manage a list of strings
  */
 export default Vue.extend({
-  components: { LabeledInput },
 
-  name: 'StringList',
+  name:       'StringList',
+  components: { LabeledInput },
 
   props: {
     /**
@@ -100,8 +100,8 @@ export default Vue.extend({
      */
     errorMessagesArray(): string[] {
       return (Object.keys(this.errors) as Error[])
-        .filter(f => this.errors[f] && this.errorMessages[f])
-        .map(k => this.errorMessages[k]);
+        .filter((f) => this.errors[f] && this.errorMessages[f])
+        .map((k) => this.errorMessages[k]);
     },
   },
 
@@ -364,7 +364,7 @@ export default Vue.extend({
      * Remove an item from items list
      */
     deleteItem(item?: string) {
-      const items = this.items.filter(f => f !== item);
+      const items = this.items.filter((f) => f !== item);
 
       this.updateItems(items);
     },

package/scripts/extension/helm/charts/ui-plugin-server/Chart.yaml

@@ -1,6 +1,6 @@
 annotations:
   catalog.cattle.io/certified: rancher # Any application we are adding as a helm chart
-  catalog.cattle.io/kube-version: '>= 1.16.0-0 < 1.26.0-0'
+  catalog.cattle.io/kube-version: '>= 1.16.0-0 < 1.28.0-0'
   catalog.cattle.io/namespace: cattle-ui-plugin-system # Must prefix with cattle- and suffix with -system=
   catalog.cattle.io/os: linux
   catalog.cattle.io/permits-os: linux, windows

package/store/action-menu.js

@@ -22,10 +22,10 @@ export const state = function() {
 };
 
 export const getters = {
-  showing:   state => state.show,
-  elem:      state => state.elem,
-  event:     state => state.event,
-  resources: state => state.resources,
+  showing:   (state) => state.show,
+  elem:      (state) => state.elem,
+  event:     (state) => state.event,
+  resources: (state) => state.resources,
 
   options(state) {
     let selected = state.resources;