From d88edb6a1806bfd0225c7235e6fc401d1e6a106d Mon Sep 17 00:00:00 2001 From: David LJ Date: Wed, 27 Nov 2024 20:11:10 +0100 Subject: [PATCH] refactor: rename selected -> isSelected --- src/app/resume-page/chip/chip.component.spec.ts | 2 +- src/app/resume-page/chip/chip.component.ts | 14 +++++++------- .../chipped-content/chipped-content.component.html | 4 ++-- .../chipped-content.component.spec.ts | 10 +++++----- 4 files changed, 15 insertions(+), 15 deletions(-) diff --git a/src/app/resume-page/chip/chip.component.spec.ts b/src/app/resume-page/chip/chip.component.spec.ts index df6be55e8..b1af2279c 100644 --- a/src/app/resume-page/chip/chip.component.spec.ts +++ b/src/app/resume-page/chip/chip.component.spec.ts @@ -66,7 +66,7 @@ describe('ChipComponent', () => { beforeEach(() => { ;[fixture, component] = componentTestSetup(ChipComponent) fixture.componentRef.setInput('selected', false) - subscription = component.selectedChange + subscription = component.isSelectedChange .pipe(first()) .subscribe((isSelectedReceived) => (isSelected = isSelectedReceived)) fixture.detectChanges() diff --git a/src/app/resume-page/chip/chip.component.ts b/src/app/resume-page/chip/chip.component.ts index b53b3a735..187dbea42 100644 --- a/src/app/resume-page/chip/chip.component.ts +++ b/src/app/resume-page/chip/chip.component.ts @@ -6,22 +6,22 @@ import { Component, EventEmitter, input, Output } from '@angular/core' styleUrls: ['./chip.component.scss'], standalone: true, host: { - '[class.selected]': 'selected()', - '[class.selectable]': 'selectedChange.observed', - '[attr.role]': "selectedChange.observed ? 'button': undefined", - '[attr.tabindex]': 'selectedChange.observed ? 0 : undefined', + '[class.selected]': 'isSelected()', + '[class.selectable]': 'isSelectedChange.observed', + '[attr.role]': "isSelectedChange.observed ? 'button': undefined", + '[attr.tabindex]': 'isSelectedChange.observed ? 0 : undefined', '(click)': 'emitToggledSelected()', '(keydown.enter)': 'emitToggledSelected()', '(keydown.space)': 'emitToggledSelected()', }, }) export class ChipComponent { - readonly selected = input() + readonly isSelected = input() @Output() - selectedChange = new EventEmitter() + isSelectedChange = new EventEmitter() protected emitToggledSelected() { - this.selectedChange.emit(!this.selected()) + this.isSelectedChange.emit(!this.isSelected()) } } diff --git a/src/app/resume-page/chipped-content/chipped-content.component.html b/src/app/resume-page/chipped-content/chipped-content.component.html index f439e583c..43669d8be 100644 --- a/src/app/resume-page/chipped-content/chipped-content.component.html +++ b/src/app/resume-page/chipped-content/chipped-content.component.html @@ -1,8 +1,8 @@
@for (content of contents(); track content) { {{ content.displayName }} diff --git a/src/app/resume-page/chipped-content/chipped-content.component.spec.ts b/src/app/resume-page/chipped-content/chipped-content.component.spec.ts index 77125e33a..6698b12bc 100644 --- a/src/app/resume-page/chipped-content/chipped-content.component.spec.ts +++ b/src/app/resume-page/chipped-content/chipped-content.component.spec.ts @@ -41,7 +41,7 @@ describe('ChippedContentComponent', () => { chipElements.forEach((chipElement, index) => { const content = CONTENTS[index] - expect(getComponentInstance(chipElement, ChipComponent).selected()) + expect(getComponentInstance(chipElement, ChipComponent).isSelected()) .withContext(`chip ${index} is unselected`) .toBeFalse() @@ -100,7 +100,7 @@ describe('ChippedContentComponent', () => { it('should mark the chip as selected', () => { expect( - getComponentInstance(firstChipElement, ChipComponent).selected(), + getComponentInstance(firstChipElement, ChipComponent).isSelected(), ).toBe(true) }) @@ -126,7 +126,7 @@ describe('ChippedContentComponent', () => { it('should mark the chip as unselected', () => { expect( - getComponentInstance(firstChipElement, ChipComponent).selected(), + getComponentInstance(firstChipElement, ChipComponent).isSelected(), ).toBeFalse() }) @@ -152,11 +152,11 @@ describe('ChippedContentComponent', () => { it('should mark the previous chip as unselected and just tapped chip as selected', () => { expect( - getComponentInstance(firstChipElement, ChipComponent).selected(), + getComponentInstance(firstChipElement, ChipComponent).isSelected(), ).toBeFalse() expect( - getComponentInstance(secondChipElement, ChipComponent).selected(), + getComponentInstance(secondChipElement, ChipComponent).isSelected(), ).toBeTrue() })