From c9eb37a2778ab51be044cfce1115bc93d7b9d955 Mon Sep 17 00:00:00 2001
From: Vit Stanislav <vit@lightcurve.io>
Date: Wed, 17 Jan 2018 08:22:57 +0100
Subject: [PATCH] Hide unfinished UI elements - Closes #206

---
 src/components/header/header.js           |  7 -------
 src/components/header/header.test.js      |  4 ++--
 src/components/mainMenu/index.test.js     |  4 ++--
 src/components/mainMenu/mainMenu.js       |  4 ++++
 src/components/mainMenu/mainMenu.test.js  |  9 +++++----
 src/components/setting/setting.js         |  7 +++----
 src/components/setting/setting.test.js    |  6 ++++--
 test/e2e/forging.feature                  |  2 +-
 test/e2e/registerDelegate.feature         |  8 ++++----
 test/e2e/registerSecondPassphrase.feature | 22 ++++++----------------
 test/e2e/settings.feature                 |  2 ++
 11 files changed, 33 insertions(+), 42 deletions(-)

diff --git a/src/components/header/header.js b/src/components/header/header.js
index 60085d0d48e..aa52e5cf036 100644
--- a/src/components/header/header.js
+++ b/src/components/header/header.js
@@ -49,13 +49,6 @@ const Header = props => (
                 menuRipple
                 theme={styles}
               >
-                {
-                  !props.account.isDelegate &&
-              <MenuItem theme={styles}>
-                <RelativeLink className={`register-as-delegate ${styles.link}`}
-                  to='register-delegate'>{props.t('Register as delegate')}</RelativeLink>
-              </MenuItem>
-                }
                 <MenuItem theme={styles}>
                   <RelativeLink className={`sign-message ${styles.link}`} to='sign-message'>{props.t('Sign message')}</RelativeLink>
                 </MenuItem>
diff --git a/src/components/header/header.test.js b/src/components/header/header.test.js
index 7892ee32383..7aa7299a515 100644
--- a/src/components/header/header.test.js
+++ b/src/components/header/header.test.js
@@ -38,7 +38,7 @@ describe('Header', () => {
     propsMock.restore();
   });
 
-  it('renders 9 RelativeLink components', () => {
-    expect(wrapper.find(RelativeLink)).to.have.length(7);
+  it('renders 6 RelativeLink components', () => {
+    expect(wrapper.find(RelativeLink)).to.have.length(6);
   });
 });
diff --git a/src/components/mainMenu/index.test.js b/src/components/mainMenu/index.test.js
index 14f0ef669cf..2585773c47c 100644
--- a/src/components/mainMenu/index.test.js
+++ b/src/components/mainMenu/index.test.js
@@ -49,7 +49,7 @@ describe('MainMenuHOC', () => {
     expect(wrapper.find('MainMenu')).to.have.lengthOf(1);
   });
 
-  it('should mount 7 Tab inside MainMenu', () => {
-    expect(wrapper.find('Tab')).to.have.lengthOf(6);
+  it('should mount 5 Tab inside MainMenu', () => {
+    expect(wrapper.find('Tab')).to.have.lengthOf(5);
   });
 });
diff --git a/src/components/mainMenu/mainMenu.js b/src/components/mainMenu/mainMenu.js
index c7cc23774ba..6c603ab9c0d 100644
--- a/src/components/mainMenu/mainMenu.js
+++ b/src/components/mainMenu/mainMenu.js
@@ -55,18 +55,22 @@ class MainMenu extends React.Component {
         id: 'transactions',
         image: menuLogos.wallet,
       }, {
+      /* TODO: uncomment when the page is updated
         label: t('Buy Lisk'),
         id: 'butLisk',
         image: menuLogos.buyLisk,
       }, {
+      */
         label: t('Delegates'),
         id: 'voting',
         image: menuLogos.delegates,
       }, {
+      /* TODO: uncomment when the page is updated
         label: t('Forging'),
         id: 'forging',
         image: menuLogos.sidechains,
       }, {
+      */
         label: t('Sidechains'),
         id: 'sidechains',
         image: menuLogos.sidechains,
diff --git a/src/components/mainMenu/mainMenu.test.js b/src/components/mainMenu/mainMenu.test.js
index b130979601e..dec39ab0064 100644
--- a/src/components/mainMenu/mainMenu.test.js
+++ b/src/components/mainMenu/mainMenu.test.js
@@ -72,18 +72,19 @@ describe('MainMenu', () => {
     expect(wrapper.find(ToolboxTabs).exists()).to.equal(true);
   });
 
-  it('should render 8 Button components if props.isDelegate', () => {
+  // TODO: re-enable if the functionality is re-enabled 
+  it.skip('should render 6 Button components if props.isDelegate', () => {
     const wrapper = mount(<MemoryRouter>
       <MainMenu store={store} isDelegate={true} history={history} t={t} />
     </MemoryRouter>, options);
-    expect(wrapper.find(Tab)).to.have.lengthOf(7);
+    expect(wrapper.find(Tab)).to.have.lengthOf(6);
   });
 
-  it('should render 7 menu item components if !props.isDelegate', () => {
+  it('should render 5 menu item components if !props.isDelegate', () => {
     const wrapper = mount(<MemoryRouter>
       <MainMenu store={store} isDelegate={false} history={history} t={t} />
     </MemoryRouter>, options);
-    expect(wrapper.find(Tab)).to.have.lengthOf(6);
+    expect(wrapper.find(Tab)).to.have.lengthOf(5);
   });
 
   it('should allow to change active menu item', () => {
diff --git a/src/components/setting/setting.js b/src/components/setting/setting.js
index 0883ee79782..45cbf6e369d 100644
--- a/src/components/setting/setting.js
+++ b/src/components/setting/setting.js
@@ -1,12 +1,9 @@
 import React from 'react';
 import ReactSwipe from 'react-swipe';
 import styles from './setting.css';
-import languageSwitcherTheme from './languageSwitcher.css';
 // eslint-disable-next-line import/no-named-as-default
 import SliderCheckbox from '../toolbox/checkbox';
-import RelativeLink from '../relativeLink';
 import i18n from '../../i18n';
-import { FontIcon } from '../fontIcon';
 
 class Setting extends React.Component {
   constructor() {
@@ -33,7 +30,7 @@ class Setting extends React.Component {
 
   render() {
     this.language = (i18n.language === 'de');
-    const { t, hasSecondPassphrase } = this.props;
+    const { t } = this.props;
     return <footer className={styles.wrapper}>
       <ReactSwipe
         className={styles.carousel}
@@ -70,6 +67,7 @@ class Setting extends React.Component {
             <p>{t('Delegate section will be displayed.')}</p>
           </article>
         </div>
+        {/* TODO: will be re-enabled when the functionality is updated
         <div>
           <SliderCheckbox
             theme={languageSwitcherTheme}
@@ -109,6 +107,7 @@ class Setting extends React.Component {
             <p>{t('Register 2nd passphrase')}</p>
           </article>
         </div>
+        */}
       </ReactSwipe>
       <ul className={ styles.carouselNav } id='carouselNav'>
         {[...Array(4)].map((x, i) =>
diff --git a/src/components/setting/setting.test.js b/src/components/setting/setting.test.js
index 5b06ff57133..febde2095eb 100644
--- a/src/components/setting/setting.test.js
+++ b/src/components/setting/setting.test.js
@@ -66,14 +66,16 @@ describe('Setting', () => {
     expect(wrapper.find('#carouselNav li').at(2).props().className).to.be.include('activeSlide');
   });
 
-  it('should click on "languageSwitcher" change the language to "de"', () => {
+  // TODO: will be re-enabled when the functionality is re-enabled
+  it.skip('should click on "languageSwitcher" change the language to "de"', () => {
     // const languageSpy = sinon.spy(i18n, 'changeLanguage');
     wrapper.find('.language-switcher .circle').simulate('click');
     wrapper.update();
     expect(i18n.language).to.be.equal('de');
   });
 
-  it('should second click on "languageSwitcher" change the language to "en"', () => {
+  // TODO: will be re-enabled when the functionality is re-enabled
+  it.skip('should second click on "languageSwitcher" change the language to "en"', () => {
     wrapper.find('.language-switcher .circle').simulate('click');
     wrapper.update();
     expect(i18n.language).to.be.equal('de');
diff --git a/test/e2e/forging.feature b/test/e2e/forging.feature
index 8748e3f4a90..552492ec10b 100644
--- a/test/e2e/forging.feature
+++ b/test/e2e/forging.feature
@@ -1,7 +1,7 @@
 Feature: Forging page
   Scenario: should allow to view forging center if account is delegate and should show more blocks on scroll
     Given I'm logged in as "delegate"
-    When I click "forging" menu
+    When I go to "main/forging/"
     Then I should see forging center
     And I should see table with 20 lines
     And I scroll to the bottom of "forging"
diff --git a/test/e2e/registerDelegate.feature b/test/e2e/registerDelegate.feature
index 05559c93fb5..081d8e1f483 100644
--- a/test/e2e/registerDelegate.feature
+++ b/test/e2e/registerDelegate.feature
@@ -1,7 +1,7 @@
 Feature: Register delegate
   Scenario: should allow to register a delegate
     Given I'm logged in as "delegate candidate"
-    When I click "register as delegate" in main menu
+    When I go to "main/dashboard/register-delegate"
     And I fill in "test" to "username" field
     And I click "register button"
     Then I should see alert dialog with title "Success" and text "Delegate registration was successfully submitted with username: "test". It can take several seconds before it is processed."
@@ -12,7 +12,7 @@ Feature: Register delegate
 
   Scenario: should allow to register a delegate with second passphrase
     Given I'm logged in as "second passphrase account"
-    When I click "register as delegate" in main menu
+    When I go to "main/dashboard/register-delegate"
     And I fill in "test2" to "username" field
     And I fill in second passphrase of "second passphrase account" to "second passphrase" field
     And I click "register button"
@@ -21,14 +21,14 @@ Feature: Register delegate
   @integration
   Scenario: should allow to exit delegate registration dialog
     Given I'm logged in as "genesis"
-    When I click "register as delegate" in main menu
+    When I go to "main/dashboard/register-delegate"
     And I click "cancel button"
     Then I should see no "modal dialog"
 
   @integration
   Scenario: should not allow to register delegate if not enough funds for the fee
     Given I'm logged in as "empty account"
-    When I click "register as delegate" in main menu
+    When I go to "main/dashboard/register-delegate"
     Then I should see "Insufficient funds for 25 LSK fee" error message
     And "register button" should be disabled
 
diff --git a/test/e2e/registerSecondPassphrase.feature b/test/e2e/registerSecondPassphrase.feature
index cc2783f6b7e..91cd6750d3c 100644
--- a/test/e2e/registerSecondPassphrase.feature
+++ b/test/e2e/registerSecondPassphrase.feature
@@ -1,4 +1,6 @@
 Feature: Register second passphrase
+  # TODO: will be re-enabled when the functionality is re-enabled
+  @pending
   Scenario: should allow to set 2nd passphrase
     Given I'm logged in as "second passphrase candidate"
     When I click "more menu"
@@ -30,11 +32,7 @@ Feature: Register second passphrase
     And I wait 1 seconds
     And I refresh the page
     And I wait 2 seconds
-    When I click "more menu"
-    And I wait 0.4 seconds
-    And I click 4 item in setting carousel
-    And I wait 0.4 seconds
-    And I click "register second passphrase"
+    When I go to "main/dashboard/register-second-passphrase"
     And I fill in passphrase of "empty account" to "passphrase" field
     And I click "authenticate button"
     Then I should see "Insufficient funds for 5 LSK fee" error message
@@ -43,21 +41,13 @@ Feature: Register second passphrase
   @integration
   Scenario: should not allow to set 2nd passphrase if not enough funds for the fee
     Given I'm logged in as "empty account"
-    When I click "more menu"
-    And I wait 0.4 seconds
-    And I click 4 item in setting carousel
-    And I wait 0.4 seconds
-    And I click "register second passphrase"
+    When I go to "main/dashboard/register-second-passphrase"
     Then I should see "Insufficient funds for 5 LSK fee" error message
     And "next button" should be disabled
 
   @integration
   Scenario: should allow to exit 2nd passphrase registration dialog
     Given I'm logged in as "genesis"
-    When I click "more menu"
-    And I wait 0.4 seconds
-    And I click 4 item in setting carousel
-    And I wait 0.4 seconds
-    And I click "register second passphrase"
+    When I go to "main/dashboard/register-second-passphrase"
     And I click "cancel button"
-    Then I should see no "modal dialog"
\ No newline at end of file
+    Then I should see no "modal dialog"
diff --git a/test/e2e/settings.feature b/test/e2e/settings.feature
index 00cb923d301..66c2f7bdd02 100644
--- a/test/e2e/settings.feature
+++ b/test/e2e/settings.feature
@@ -1,4 +1,6 @@
 Feature: Settings dialog
+  # TODO: will be re-enabled when the functionality is re-enabled
+  @pending
   Scenario: should allow to change language
     Given I'm logged in as "any account"
     When I click "more menu"