Skip to content
This repository was archived by the owner on May 4, 2022. It is now read-only.

Fix tests#174

Open
dennybiasiolli wants to merge 6 commits into
ionic-team:masterfrom
dennybiasiolli:fix_tests
Open

Fix tests#174
dennybiasiolli wants to merge 6 commits into
ionic-team:masterfrom
dennybiasiolli:fix_tests

Conversation

@dennybiasiolli
Copy link
Copy Markdown
Contributor

@dennybiasiolli dennybiasiolli commented Jan 14, 2017

Short description of what this resolves:

This resolves errors in tests caused by commit 417997d and broken tests (at least) on MacOS
Here a working example.

Changes proposed in this pull request:

  • changing CircleCI's node version to 6.9.4 to avoid phantomjs installation error
  • adding bower to devDeps to avoid CircleCI bower install error
  • removed reference to inexistent file js/utils/list.js
  • sidemenu: fixing values causing tests error on osx
  • toggle: correcting click trigger

Ionic Version: 1.x

Fixes: #74

the error launching tests was the following:
WARN [watcher]: Pattern "./js/utils/list.js" does not match any file.
when passing 270/4 to openAmount() on osx, ctrl.getOpenAmount() it's equal to 65
but ctrl.left.width/4 it's 67.5. That causes error during karma tests
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant