add unit test

2 files changed, 13 insertions, 0 deletions

diff --git a/apps/files_external/js/settings.js b/apps/files_external/js/settings.js
index 6854b40e87d..1dae64216e1 100644
--- a/apps/files_external/js/settings.js
+++ b/apps/files_external/js/settings.js
@@ -509,6 +509,8 @@ MountOptionsDropdown.prototype = {
 			var optionId = $row.find('input, select').attr('name');
 			if (enabledOptions.indexOf(optionId) === -1) {
 				$row.hide();
+			} else {
+				$row.show();
 			}
 		});
 	}
diff --git a/apps/files_external/tests/js/settingsSpec.js b/apps/files_external/tests/js/settingsSpec.js
index c5181737b31..7cb86d7270b 100644
--- a/apps/files_external/tests/js/settingsSpec.js
+++ b/apps/files_external/tests/js/settingsSpec.js
@@ -205,6 +205,17 @@ describe('OCA.External.Settings tests', function() {
 				expect($td.find('.dropdown').length).toEqual(0);
 			});
 
+			it('doesnt show the encryption option when encryption is disabled', function () {
+				view._encryptionEnabled = false;
+				$td.find('img').click();
+
+				expect($td.find('.dropdown [name=encrypt]:visible').length).toEqual(0);
+
+				$('body').mouseup();
+
+				expect($td.find('.dropdown').length).toEqual(0);
+			});
+
 			it('reads config from mountOptions field', function() {
 				$tr.find('input.mountOptions').val(JSON.stringify({previews:false}));