Android Test Failures #41

Closed
kitsonk opened this Issue Apr 11, 2016 · 1 comment

Projects

None yet

2 participants

@kitsonk
Member
kitsonk commented Apr 11, 2016

Now that I have everything merged and updated (as well as moved over to SauceLabs for testing), there are some additional failures on Android 4.4:

× android 4.4 on Linux - unit tests - addCssRule - addCssRule - add rule - multiple properties (0.005s)
AssertionError: expected '8px' to equal '4px'
  at Assertion.assertEqual  <__intern/browser_modules/chai/chai.js:776:12>
  at Assertion.ctx.(anonymous function) [as equal]  <__intern/browser_modules/chai/chai.js:4192:25>
  at Function.assert.strictEqual  <__intern/browser_modules/chai/chai.js:2329:32>
  at Test.registerSuite.addCssRule.add rule.multiple properties [as test]  <tests/unit/addCssRule.ts:46:12>
  at <__intern/lib/Test.js:181:24>
  at <__intern/browser_modules/dojo/Promise.js:156:41>
  at runCallbacks  <__intern/browser_modules/dojo/Promise.js:19:22>
  at <__intern/browser_modules/dojo/Promise.js:103:21>
  at run  <__intern/browser_modules/dojo/Promise.js:51:33>
  at CallbackQueue.drain  <__intern/browser_modules/dojo/CallbackQueue.js:39:35>
× android 4.4 on Linux - unit tests - addCssRule - addCssRule - add multiple rules (0.001s)
AssertionError: expected '8px' to equal '4px'
  at Assertion.assertEqual  <__intern/browser_modules/chai/chai.js:776:12>
  at Assertion.ctx.(anonymous function) [as equal]  <__intern/browser_modules/chai/chai.js:4192:25>
  at Function.assert.strictEqual  <__intern/browser_modules/chai/chai.js:2329:32>
  at Test.registerSuite.addCssRule.add multiple rules [as test]  <tests/unit/addCssRule.ts:57:11>
  at <__intern/lib/Test.js:181:24>
  at <__intern/browser_modules/dojo/Promise.js:156:41>
  at runCallbacks  <__intern/browser_modules/dojo/Promise.js:19:22>
  at <__intern/browser_modules/dojo/Promise.js:103:21>
  at run  <__intern/browser_modules/dojo/Promise.js:51:33>
  at CallbackQueue.drain  <__intern/browser_modules/dojo/CallbackQueue.js:39:35>
× android 4.4 on Linux - unit tests - addCssRule - addCssRule - #destroy - destroy out of order (0.002s)
AssertionError: expected '8px' to equal '6px'
  at Assertion.assertEqual  <__intern/browser_modules/chai/chai.js:776:12>
  at Assertion.ctx.(anonymous function) [as equal]  <__intern/browser_modules/chai/chai.js:4192:25>
  at Function.assert.strictEqual  <__intern/browser_modules/chai/chai.js:2329:32>
  at Test.registerSuite.addCssRule.#destroy.destroy out of order [as test]  <tests/unit/addCssRule.ts:111:12>
  at <__intern/lib/Test.js:181:24>
  at <__intern/browser_modules/dojo/Promise.js:156:41>
  at runCallbacks  <__intern/browser_modules/dojo/Promise.js:19:22>
  at <__intern/browser_modules/dojo/Promise.js:103:21>
  at run  <__intern/browser_modules/dojo/Promise.js:51:33>
  at CallbackQueue.drain  <__intern/browser_modules/dojo/CallbackQueue.js:39:35>
× android 4.4 on Linux - unit tests - addCssRule - addCssRule - #set - single (0.001s)
AssertionError: expected '8px' to equal '4px'
  at Assertion.assertEqual  <__intern/browser_modules/chai/chai.js:776:12>
  at Assertion.ctx.(anonymous function) [as equal]  <__intern/browser_modules/chai/chai.js:4192:25>
  at Function.assert.strictEqual  <__intern/browser_modules/chai/chai.js:2329:32>
  at Test.registerSuite.addCssRule.#set.single [as test]  <tests/unit/addCssRule.ts:150:12>
  at <__intern/lib/Test.js:181:24>
  at <__intern/browser_modules/dojo/Promise.js:156:41>
  at runCallbacks  <__intern/browser_modules/dojo/Promise.js:19:22>
  at <__intern/browser_modules/dojo/Promise.js:103:21>
  at run  <__intern/browser_modules/dojo/Promise.js:51:33>
  at CallbackQueue.drain  <__intern/browser_modules/dojo/CallbackQueue.js:39:35>
× android 4.4 on Linux - unit tests - addCssRule - addCssRule - #set - multiple (0.002s)
AssertionError: expected '8px' to equal '4px'
  at Assertion.assertEqual  <__intern/browser_modules/chai/chai.js:776:12>
  at Assertion.ctx.(anonymous function) [as equal]  <__intern/browser_modules/chai/chai.js:4192:25>
  at Function.assert.strictEqual  <__intern/browser_modules/chai/chai.js:2329:32>
  at Test.registerSuite.addCssRule.#set.multiple [as test]  <tests/unit/addCssRule.ts:163:12>
  at <__intern/lib/Test.js:181:24>
  at <__intern/browser_modules/dojo/Promise.js:156:41>
  at runCallbacks  <__intern/browser_modules/dojo/Promise.js:19:22>
  at <__intern/browser_modules/dojo/Promise.js:103:21>
  at run  <__intern/browser_modules/dojo/Promise.js:51:33>
  at CallbackQueue.drain  <__intern/browser_modules/dojo/CallbackQueue.js:39:35>
>> android 4.4 on Linux: 5/129 tests failed
@vansimke vansimke was assigned by kitsonk Apr 11, 2016
@kitsonk kitsonk added bug tests labels Apr 11, 2016
@kitsonk kitsonk added this to the 2016.04 milestone Apr 11, 2016
@vansimke
Contributor

I've looked into this a bit, and it seems like something specific to the font-size property for Android 4.4 on Sauce Labs. This doesn't repeat on a local VM or on Browserstack. It also doesn't happen on other CSS properties. Based on that, I'm thinking about switching over to another CSS property such as fill. It should still test the same goals as font-size, but will work around the apparent bug.

@vansimke vansimke added a commit to vansimke/dom that referenced this issue Apr 20, 2016
@vansimke vansimke Changed tests in addCssRule to use the display property instead of th…
…e font-size CSS property in order to workaround an apparent bug on Sauce Labs Android 4.4 browser that prevented the font-size from being changed.

Addresses issue #41
fbba0cf
@vansimke vansimke added a commit to vansimke/dom that referenced this issue Apr 20, 2016
@vansimke vansimke Changed tests in addCssRule to use the display property instead of th…
…e font-size CSS property in order to workaround an apparent bug on Sauce Labs Android 4.4 browser that prevented the font-size from being changed.

Addresses issue #41
45f36b3
@vansimke vansimke closed this Apr 29, 2016
@kitsonk kitsonk added a commit that referenced this issue Jun 1, 2016
@vansimke @kitsonk vansimke + kitsonk Change tests to resolve issue on Android (#44)
Changed tests in addCssRule to use the display property instead of the font-size CSS property in order to workaround an apparent bug on Sauce Labs Android 4.4 browser that prevented the font-size from being changed.

Fixes #41
e19c837
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment