Просмотр исходного кода

wipfix: to be sure, also add check for hideBurnAfterReadingButtons

rugk 3 недель назад
Родитель
Сommit
3856891b3a
2 измененных файлов с 20 добавлено и 1 удалено
  1. 3 1
      js/privatebin.js
  2. 17 0
      js/test/TopNav.js

+ 3 - 1
js/privatebin.js

@@ -4496,7 +4496,9 @@ window.PrivateBin = (function () {
          * @function
          */
         me.hideQrCodeButton = function () {
-            qrCodeLink.classList.add('hidden');
+            if (qrCodeLink) {
+                qrCodeLink.classList.add('hidden');
+            }
         };
 
         /**

+ 17 - 0
js/test/TopNav.js

@@ -10,6 +10,23 @@ function query(selector) {
 }
 
 describe('TopNav', function () {
+    describe('hideBurnAfterReadingButtons', function () {
+        it(
+            'hides buttons when the QR code button is disabled',
+            function () {
+                document.documentElement.innerHTML =
+                    '<nav><button id="clonebutton"></button></nav>';
+                PrivateBin.TopNav.init();
+
+                assert.doesNotThrow(function () {
+                    PrivateBin.TopNav.hideBurnAfterReadingButtons();
+                });
+                assert.ok(query('#clonebutton').classList.contains('hidden'));
+                cleanup();
+            }
+        );
+    });
+
     describe('showViewButtons & hideViewButtons', function () {
         before(function () {
             cleanup();