Fix JS tests, rename stuff

JammingBen · JammingBen · commit b51abaa9c5f5 · 2020-12-15T09:47:04.000+01:00
diff --git a/apps/files/js/fileactionsapplicationselectmenu.js b/apps/files/js/fileactionsapplicationselectmenu.js
diff --git a/apps/files/js/filelist.js b/apps/files/js/filelist.js
@@ -675,8 +675,8 @@
 
 						// don't show app drawer for directories as we want to open them per default
 						if (Object.keys(actionsWithoutAll).length > 1 && type !== 'dir') {
-							var appDrawer = new OCA.Files.FileActionsApplicationSelectMenu();
-							appDrawer.show(context, $tr.find('td.filename'));
+							var appSelectMenu = new OCA.Files.FileActionsApplicationSelectMenu();
+							appSelectMenu.show(context, $tr.find('td.filename'));
 							event.preventDefault();
 							return;
 						}
diff --git a/apps/files/lib/Controller/ViewController.php b/apps/files/lib/Controller/ViewController.php
@@ -186,7 +186,7 @@ public function index($dir = '', $view = '', $fileid = null, $details = null) {
 
 		\OCP\Util::addScript('files', 'fileactions');
 		\OCP\Util::addScript('files', 'fileactionsmenu');
-		\OCP\Util::addScript('files', 'fileactionsappdrawer');
+		\OCP\Util::addScript('files', 'fileactionsapplicationselectmenu');
 		\OCP\Util::addScript('files', 'files');
 		\OCP\Util::addScript('files', 'keyboardshortcuts');
 		\OCP\Util::addScript('files', 'navigation');
diff --git a/apps/files/tests/js/filelistSpec.js b/apps/files/tests/js/filelistSpec.js
@@ -2378,9 +2378,9 @@ describe('OCA.Files.FileList tests', function() {
 			var $tr = fileList.findFileEl('One.txt');
 			$tr.find('td.filename .nametext').click();
 			expect(actionStub.calledOnce).toEqual(false);
-			expect($tr.find('td.filename .fileActionsAppDrawer').length).toEqual(1);
-			expect($tr.find('td.filename .fileActionsAppDrawer a').length).toEqual(2);
-			var firstAction = $tr.find('td.filename .fileActionsAppDrawer a')[0];
+			expect($tr.find('td.filename .fileActionsApplicationSelectMenu').length).toEqual(1);
+			expect($tr.find('td.filename .fileActionsApplicationSelectMenu a').length).toEqual(2);
+			var firstAction = $tr.find('td.filename .fileActionsApplicationSelectMenu a')[0];
 			firstAction.click();
 			expect(actionStub.calledOnce).toEqual(true);
 		});
