Skip to content
This repository has been archived by the owner on Dec 19, 2024. It is now read-only.

Test bug: --layout-vertical #109

Open
1 of 8 tasks
dimwell-git opened this issue Feb 15, 2017 · 0 comments
Open
1 of 8 tasks

Test bug: --layout-vertical #109

dimwell-git opened this issue Feb 15, 2017 · 0 comments

Comments

@dimwell-git
Copy link

Description

Wrong initial state of container.

Expected outcome

The initial layout of container should be different to one after adding class "vertical".
This means that if we don't add class "vertical" to the container the test should fail.
To my understanding the main idea of test - is to check that by applying property "--layout-vertical" it changes the layout to vertical and thus passing a test.

Actual outcome

Currently initial layout is the same as after we add class "vertical" to container.
You can simply check this by commenting or removing line 166 -> container.classList.add('vertical');
And the test still would pass.

Steps to reproduce

  1. Open test page in a web browser.
  2. Put a break-point at line 166.
  3. Comment or remove line 166 -> container.classList.add('vertical');
  4. Refresh page/test.
  5. Test Result Status: passed.

Browsers Affected

Checked only at chrome.

  • Chrome
  • Firefox
  • Safari 9
  • Safari 8
  • Safari 7
  • Edge
  • IE 11
  • IE 10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant