944162 - Add ispixel assertions to AssertionHelper

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Testing, defect)

Description

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

This should just be a wrapper around Assert.ispixel and Assert.isnotpixel, named in the junit style (e.g. assertIsPixel).

Also, remove the associated TODO in AssertionHelper.

Comment 1

[Isura Edirisinghe]

Attachment: 944162.patch

Comment 2

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Comment on attachment 8344221 [details] [diff] [review]
944162.patch

lgtm! Thanks for the help!

Comment 3

[Isura Edirisinghe]

Thanks. My first submission to open source. 

Could you commit this patch for me?

Comment 4

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Congratulations! :)

You can add "checkin-needed" to the keywords field to have someone do it for you (so your reviewer doesn't always have to). I'll let you do that now for practice!

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/97d17b777f68

Comment 6

[Ryan VanderMeulen [:RyanVM]]

Backed out for Android bustage.
https://tbpl.mozilla.org/php/getParsedLog.php?id=31660764&tree=Mozilla-Inbound

https://hg.mozilla.org/integration/mozilla-inbound/rev/7713c80c1028

Comment 7

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

It seems the "String" keyword is lower-cased. Isura, do you mind fixing that?

Make sure you're also compiling the Robocop tests in addition to the Fennec package. You can find the steps at [1]. Please let me know if you have any questions.

[1]: https://wiki.mozilla.org/Auto-tools/Projects/Robocop

Comment 8

[Isura Edirisinghe]

Attachment: 944162.patch

I have fixed the type string -> String. 

However I'm not able to compile robocop on ubuntu 13 following the instructions.
I've added android SDK to the path as suggested. 

isura@ubuntu:~/Devel/mozilla-central$ ./mach build build/mobile/robocop
 0:00.33 /usr/bin/make -C /home/isura/Devel/mozilla-central/obj-arm-linux-androideabi -j4 -s backend.RecursiveMakeBackend
 0:00.34 make: /home/isura/opt/adt-bundle-linux/sdk/platforms/android-19/../..//platform-tools/adb: Command not found
 0:00.45 /usr/bin/make -C build/mobile/robocop -j4 -s
 0:00.61 make[1]: /home/isura/opt/adt-bundle-linux/sdk/platforms/android-19/../..//build-tools/17.0.0/aapt: Command not found
 0:00.61 make[1]: *** [.aapt.deps] Error 127
 0:00.61 make: *** [default] Error 2

Comment 9

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Using the android binary, make sure you have the latest "Android SDK Tools", "Android SDK Platform-tools", and "Android SDK Build-tools" packages installed. It seems certain binaries from these packages cannot be found:

>  0:00.34 make: /home/isura/opt/adt-bundle-linux/sdk/platforms/android-19/../..//platform-tools/adb: Command
> not found
>  0:00.45 /usr/bin/make -C build/mobile/robocop -j4 -s
>  0:00.61 make[1]: /home/isura/opt/adt-bundle-linux/sdk/platforms/android-19/../..//build-tools/17.0.0/aapt:
> Command not found

Please let me know if you still have issues or the patch compiles successfully!

Comment 10

[Isura Edirisinghe]

I used built-tools version 17 as suggested in the instructions. I then tried with the newest version 19 (so everything is the newest).  The files aapt and adb are located in that location. 

Any ideas? Thanks.

Comment 11

[Isura Edirisinghe]

I'm able to build Robocop. The problem was with ia32-lib. From docs

"If you're using a 64-bit Debian or Ubuntu install, you'll need ia32-libs to allow the toolchain binaries to run. Otherwise you may get a "bash: file not found" error when trying to use any of the SDK/NDK tools."

This package is not available on Ubuntu 13 (obsolete). So I used the workaround at http://ubuntuforums.org/showthread.php?t=2182653 (post 4) to install the package.

Comment 12

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

> I'm able to build Robocop.

Great!

> This package is not available on Ubuntu 13 (obsolete). So I used the workaround at
> http://ubuntuforums.org/showthread.php?t=2182653 (post 4) to install the package.

The Mozilla wiki is actually editable by anyone so do you want to add your workaround to the page? If not, let me know and I can take care of it for you!

Comment 13

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Comment on attachment 8345274 [details] [diff] [review]
944162.patch

Review of attachment 8345274 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm!

(Just a reminder that usually you should add the r? flag when you upload subsequent patch versions for review - I got it this time though)

Don't forget to add the "checkin-needed" keyword!

Comment 14

[Isura Edirisinghe]

(In reply to Michael Comella (:mcomella) from comment #12)
> > I'm able to build Robocop.
> 
> Great!
> 
> > This package is not available on Ubuntu 13 (obsolete). So I used the workaround at
> > http://ubuntuforums.org/showthread.php?t=2182653 (post 4) to install the package.
> 
> The Mozilla wiki is actually editable by anyone so do you want to add your
> workaround to the page? If not, let me know and I can take care of it for
> you!

I updated the wiki. Thanks for the help.

Comment 15

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

(In reply to Isura Edirisinghe from comment #14)
> I updated the wiki. Thanks for the help.

Great, thanks!

One last thing is that I would also include the source of the thread you got the workaround from in the Wiki text (though I suppose it is not strictly necessary).

Comment 16

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a3b8fdff3d1e

Comment 17

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/a3b8fdff3d1e