fix(danger-button): correct event handling comment for touchend trigger

Author: andy1li

app/components/danger-button-with-timed-confirmation.ts

@@ -28,7 +28,7 @@ export default class DangerButtonWithTimedConfirmation extends Component<Signatu
 
   @action
   startProgress(event: Event) {
-    event.preventDefault(); // Prevent menu from popping up, which triggers touchup
+    event.preventDefault(); // Prevent menu from popping up, which triggers touchend
     this.stopProgress();
 
     this.shouldShowProgressBar = true;

tests/acceptance/concept-admin/delete-concept-test.js

@@ -37,7 +37,6 @@ module('Acceptance | concept-admin | delete-concept-test', function (hooks) {
     await basicDetailsPage.deleteConceptModal.deleteConceptButton.mousedown();
     assert.ok(basicDetailsPage.deleteConceptModal.deleteConceptButton.progressIndicator.isVisible, 'progress indicator should be visible');
 
-    // assert progressIndicator.width is greater than 0
     await waitUntil(() => basicDetailsPage.deleteConceptModal.deleteConceptButton.progressIndicator.width > 0, { timeout: 10 });
     assert.ok(basicDetailsPage.deleteConceptModal.deleteConceptButton.progressIndicator.width > 0, 'progress indicator should have a width');
 
@@ -76,7 +75,6 @@ module('Acceptance | concept-admin | delete-concept-test', function (hooks) {
     await basicDetailsPage.deleteConceptModal.deleteConceptButton.mousedown();
     assert.ok(basicDetailsPage.deleteConceptModal.deleteConceptButton.progressIndicator.isVisible, 'progress indicator should be visible');
 
-    // assert progressIndicator.width is greater than 0
     await waitUntil(() => basicDetailsPage.deleteConceptModal.deleteConceptButton.progressIndicator.width > 0, { timeout: 10 });
     assert.ok(basicDetailsPage.deleteConceptModal.deleteConceptButton.progressIndicator.width > 0, 'progress indicator should have a width');
 

tests/acceptance/course-page/delete-repository-test.js

@@ -126,7 +126,7 @@ module('Acceptance | course-page | delete-repository-test', function (hooks) {
     );
   });
 
-  test('can delete repository after mouse down', async function (assert) {
+  test('can delete repository via mouse down', async function (assert) {
     testScenario(this.server, ['dummy']);
     signInAsStaff(this.owner, this.server);
 
@@ -153,7 +153,6 @@ module('Acceptance | course-page | delete-repository-test', function (hooks) {
     await coursePage.deleteRepositoryModal.deleteRepositoryButton.mousedown();
     assert.ok(coursePage.deleteRepositoryModal.deleteRepositoryButton.progressIndicator.isVisible, 'progress indicator should be visible');
 
-    // assert progressIndicator.width is greater than 0
     await waitUntil(() => coursePage.deleteRepositoryModal.deleteRepositoryButton.progressIndicator.width > 0, { timeout: 10 });
     assert.ok(coursePage.deleteRepositoryModal.deleteRepositoryButton.progressIndicator.width > 0, 'progress indicator should have a width');
 
@@ -165,7 +164,7 @@ module('Acceptance | course-page | delete-repository-test', function (hooks) {
     assert.notOk(coursePage.repositoryDropdown.content.text.includes('Delete Repository'), 'delete repository action should not be available');
   });
 
-  test('can delete repository after touch start', async function (assert) {
+  test('can delete repository via touch start', async function (assert) {
     testScenario(this.server, ['dummy']);
     signInAsStaff(this.owner, this.server);
 
@@ -192,7 +191,6 @@ module('Acceptance | course-page | delete-repository-test', function (hooks) {
     await coursePage.deleteRepositoryModal.deleteRepositoryButton.touchstart();
     assert.ok(coursePage.deleteRepositoryModal.deleteRepositoryButton.progressIndicator.isVisible, 'progress indicator should be visible');
 
-    // assert progressIndicator.width is greater than 0
     await waitUntil(() => coursePage.deleteRepositoryModal.deleteRepositoryButton.progressIndicator.width > 0, { timeout: 10 });
     assert.ok(coursePage.deleteRepositoryModal.deleteRepositoryButton.progressIndicator.width > 0, 'progress indicator should have a width');