Francisco Jordano [:arcturus] [:francisco] Assignee
	Description • 10 years ago

We get this error when we don't have any contact set as ICE.

Easily reproducible when doing a make reset-gaia

	Sergi Mansilla [:sergi] (Telenor) [PTO]
	Comment 2 • 10 years ago

Comment on attachment 8485714 [details] [review]
Pointer to PR 23805

Thanks Francisco! r+

	Francisco Jordano [:arcturus] [:francisco] Assignee
	Comment 3 • 10 years ago

Comment on attachment 8485714 [details] [review]
Pointer to PR 23805

Hei Zac,

had to modify the python ui tests, since we are creating now a new 'hidden' contact always that will lead us to the ICE section, when where selecting the number of contacts we were always having an extra one.

Just modify the selector to avoid including this hidden new row in the contact list.

	Francisco Jordano [:arcturus] [:francisco] Assignee
	Comment 4 • 10 years ago

Landed:

https://github.com/mozilla-b2g/gaia/commit/84fe3093b81a6eb6f461b08f2e79aa9684b273fc

Thanks!

	Francisco Jordano [:arcturus] [:francisco] Assignee
	Comment 5 • 10 years ago

Comment on attachment 8485714 [details] [review]
Pointer to PR 23805

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #):
ICE feature
[User impact] if declined:
We have a constant javascript error that can lead to major disfunction
[Testing completed]:
Smoke test, and more unit tests added to check this problem
[Risk to taking this patch] (and alternatives if risky):
Low, patchi is moving some lines above, and more unit test than proper patch
[String changes made]:

	bhavana bajaj [:bajaj]
	Comment 6 • 10 years ago

Comment on attachment 8485714 [details] [review]
Pointer to PR 23805

Approving the landing as the risk looks manageable at this point in the release cycle and given the added test

	Ryan VanderMeulen [:RyanVM]
	Comment 7 • 10 years ago

v2.1: https://github.com/mozilla-b2g/gaia/commit/186a163b0304dccc0ffc1918ae70c16c966291a0

	Ryan VanderMeulen [:RyanVM]
	Comment 8 • 10 years ago

Reverted from v2.1 for Gaia unit test failures.
v2.1: https://github.com/mozilla-b2g/gaia/commit/d3b1f6b47c8b18921a3a1bccbd2d4c0f4095d99b

https://tbpl.mozilla.org/php/getParsedLog.php?id=48136420&tree=Mozilla-Aurora

	Francisco Jordano [:arcturus] [:francisco] Assignee
	Comment 9 • 10 years ago

Hi Ryan,

something must have changed in our test harness, cause this was a problem on reseting the status of tests.

I just did a patch for 2.1, this patch compared with 2.2 has a line different, doing that reset automatically.

Could you uplift that one?

Thanks!

	Ryan VanderMeulen [:RyanVM]
	Comment 10 • 10 years ago

I can't get this patch to apply. Can you just submit a pull request?

	Francisco Jordano [:arcturus] [:francisco] Assignee
	Comment 11 • 10 years ago

Will do,

Thanks!

	Ryan VanderMeulen [:RyanVM]
	Comment 12 • 10 years ago

Nevermind, I figured it out.

v2.1: https://github.com/mozilla-b2g/gaia/commit/d6e06fc7378c3d25618b0e1f99715a96fa290f61

	Francisco Jordano [:arcturus] [:francisco] Assignee
	Comment 13 • 10 years ago

Unfortunately will need to backout last commit. As was the same that was merged originaly and producing the error in unit tests:


941cc6c93a64875bf88eb38d7327fb35db0a6ae2

	Francisco Jordano [:arcturus] [:francisco] Assignee
	Comment 14 • 10 years ago

And pushed the specific 2.1 patch version

https://github.com/mozilla-b2g/gaia/commit/987645cd189790e27ceb49497028ed32e8d00c90