♻️ Changes done after code review

LiskHQ · Nov 15, 2018 · adb53a7 · adb53a7
1 parent 6c3c05e
commit adb53a7
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 51 deletions.
diff --git a/karma.conf.js b/karma.conf.js
@@ -75,8 +75,7 @@ module.exports = function (config) {
             'src/utils/rawTransactionWrapper.js',
             'src/utils/to.js',
             'src/components/newsFeed/index.js', // This component doesn't meet the setted tresholds for mocha but in jest
-            'src/components/dashboard/currencyGraph.js', // This component should be fixed later, right now the test is being skip (canvas issue)
-            'test/integration/wallet.js', // integration wallet throw random error that should be check it
+            'src/components/dashboard/currencyGraph.js', // This should be unskipped in issue #1499
             'src/components/passphraseCreation/index.js',
           ],
           overrides: {

diff --git a/src/components/newsFeed/index.test.js b/src/components/newsFeed/index.test.js
diff --git a/test/integration/wallet.test.js b/test/integration/wallet.test.js
@@ -260,6 +260,7 @@ describe('@integration: Wallet', () => {
       step(`Then I should see text ${successMessage} in "result box message" element`, () => helper.haveTextOf('.result-box-message', successMessage));
     });
 
+    // This should be unskipped in issue #1500
     describe.skip('Scenario: should allow to send LSK from unlocked account with second passphrase', () => {
       const { secondPassphrase } = accounts['second passphrase account'];
       step('Given I\'m on "wallet" as "second passphrase account"', () => setupStep('second passphrase account'));
@@ -391,6 +392,7 @@ describe('@integration: Wallet', () => {
       step('Then I should be redirected to transactoinDetails step', () => helper.checkRedirectionToDetails('123456'));
     });
 
+    // This should be unskipped in issue #1500
     describe.skip('Scenario: should allow to filter transactions', () => {
       step('Given I\'m on "wallet" as "genesis" account', () => setupStep('genesis'));
       step('Then the "All" filter should be selected by default', () => helper.checkSelectedFilter('all'));