add unit test

diff --git a/apps/files_external/js/settings.js b/apps/files_external/js/settings.js
index 6854b40e87d1a46a0bab71c760d79d10d4973644..1dae64216e1618198933bd2d9302423d3c877dd2 100644 (file)
--- 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 c5181737b31cd4a8ee39af24c29fb7fd809b905b..7cb86d7270b0394612cf8111d5fdce09ced219d5 100644 (file)
--- 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}));