mediaplayer-app

Merge lp:~renatofilho/mediaplayer-app/fix-1231418 into lp:mediaplayer-app

  1. fix-1231418
  2. Merge into trunk
Proposed by Renato Araujo Oliveira Filho
Status: Rejected
Rejected by: Robert Bruce Park
Proposed branch: lp:~renatofilho/mediaplayer-app/fix-1231418
Merge into: lp:mediaplayer-app
Diff against target: 50 lines (+16/-1)
2 files modified
debian/control (+7/-0)
tests/autopilot/mediaplayer_app/tests/test_player_with_video.py (+9/-1)
To merge this branch: bzr merge lp:~renatofilho/mediaplayer-app/fix-1231418
Reviewer Review Type Date Requested Status
Robert Bruce Park (community) Needs Resubmitting
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+188433@code.launchpad.net

Commit message

skipped tests due a bug on decode.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:146
http://jenkins.qa.ubuntu.com/job/mediaplayer-app-ci/120/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-saucy/4217
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-touch/1884
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-saucy-amd64-ci/56
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-saucy-armhf-ci/56
        deb: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-saucy-armhf-ci/56/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-saucy-i386-ci/56/console
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-saucy/510
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-amd64/85
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-amd64/85/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/1886
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/1886/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/1581
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/1594

Click here to trigger a rebuild:
http://s-jenkins:8080/job/mediaplayer-app-ci/120/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Autolanding.
More details in the following jenkins job:
http://jenkins.qa.ubuntu.com/job/mediaplayer-app-autolanding/55/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-saucy/4235/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-touch/1903
    FAILURE: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-saucy-amd64-autolanding/31/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-saucy-armhf-autolanding/31
        deb: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-saucy-armhf-autolanding/31/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-saucy-i386-autolanding/31/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-amd64/103/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/1905
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/1905/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/1600
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/1616

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:147
http://jenkins.qa.ubuntu.com/job/mediaplayer-app-ci/123/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-saucy/4239/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-touch/1907/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-saucy-amd64-ci/59/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-saucy-armhf-ci/59/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-saucy-i386-ci/59/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-amd64/107/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/1909/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/mediaplayer-app-ci/123/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Robert Bruce Park (robru) wrote :

Ok, I pushed an attempt at a fix here: https://code.launchpad.net/~robru/mediaplayer-app/fix-1231418/+merge/188502

review: Needs Fixing
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:147
http://10.97.0.26:8080/job/mediaplayer-app-ci/132/
Executed test runs:
    FAILURE: http://10.97.0.26:8080/job/generic-mediumtests-saucy/4680/console
    FAILURE: http://10.97.0.26:8080/job/generic-mediumtests-touch/2484/console
    FAILURE: http://10.97.0.26:8080/job/mediaplayer-app-saucy-amd64-ci/68/console
    FAILURE: http://10.97.0.26:8080/job/mediaplayer-app-saucy-armhf-ci/68/console
    FAILURE: http://10.97.0.26:8080/job/mediaplayer-app-saucy-i386-ci/68/console
    FAILURE: http://10.97.0.26:8080/job/generic-mediumtests-builder-saucy-amd64/555/console
    FAILURE: http://10.97.0.26:8080/job/generic-mediumtests-builder-saucy-armhf/2486/console

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/mediaplayer-app-ci/132/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Robert Bruce Park (robru) wrote :

I'm rejecting this because it's nearly 2 months old with no activity. Feel free to resubmit if you want to try this branch again.

review: Needs Resubmitting

Unmerged revisions

147. By Renato Araujo Oliveira Filho

Added necessary dependency to autopilot tests.

146. By Renato Araujo Oliveira Filho

skiped tests due a bug on decode.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/control'
--- debian/control 2013-09-25 23:08:08 +0000
+++ debian/control 2013-10-01 00:36:39 +0000
@@ -43,6 +43,13 @@
43 libqt5test5,43 libqt5test5,
44 libqt5widgets5,44 libqt5widgets5,
45 mediaplayer-app (>= ${source:Version}),45 mediaplayer-app (>= ${source:Version}),
46 libqt5multimedia5-touch,
47 libqt5multimedia5-touch-plugins,
48 libqt5multimediaquick-p5-touch,
49 qtdeclarative5-qtmultimedia-touch-plugin,
50 gstreamer1.0-plugins-good,
51 gstreamer1.0-hybris[armhf],
52 gstreamer1.0-plugins-bad
46Conflicts: media-player-autopilot53Conflicts: media-player-autopilot
47Replaces: media-player-autopilot54Replaces: media-player-autopilot
48Provides: media-player-autopilot55Provides: media-player-autopilot
4956
=== modified file 'tests/autopilot/mediaplayer_app/tests/test_player_with_video.py'
--- tests/autopilot/mediaplayer_app/tests/test_player_with_video.py 2013-09-25 21:19:34 +0000
+++ tests/autopilot/mediaplayer_app/tests/test_player_with_video.py 2013-10-01 00:36:39 +0000
@@ -132,11 +132,19 @@
132132
133 time_line = self.main_window.get_object("Slider", "TimeLine.Slider")133 time_line = self.main_window.get_object("Slider", "TimeLine.Slider")
134 time_label = self.main_window.get_object("Label", "TimeLine.TimeLabel")134 time_label = self.main_window.get_object("Label", "TimeLine.TimeLabel")
135
136 """
137 New backend work needed. bug 1231147
135 scene_selector = self.main_window.get_object("SceneSelector", "Controls.SceneSelector")138 scene_selector = self.main_window.get_object("SceneSelector", "Controls.SceneSelector")
139 """
136140
137 """ Seek to the midle of the movie """141 """ Seek to the midle of the movie """
138 self.pointing_device.click_object(time_line)142 self.pointing_device.click_object(time_line)
143
144 """
145 New backend work needed. bug 1231147
139 self.assertThat(scene_selector.opacity, Eventually(Equals(1)))146 self.assertThat(scene_selector.opacity, Eventually(Equals(1)))
147 """
140148
141 """ Time label must show the current video time (diff from zero or empty) """149 """ Time label must show the current video time (diff from zero or empty) """
142 self.assertNotEqual(time_label.text, "00:00:00")150 self.assertNotEqual(time_label.text, "00:00:00")
@@ -158,7 +166,7 @@
158166
159 """ avoid the test fails due the timeout """167 """ avoid the test fails due the timeout """
160 self.assertThat(time_label.text, Eventually(Equals("00:00:25"), timeout=35))168 self.assertThat(time_label.text, Eventually(Equals("00:00:25"), timeout=35))
161 169
162 controls = self.main_window.get_object("Controls", "controls")170 controls = self.main_window.get_object("Controls", "controls")
163 self.assertProperty(controls, visible=False)171 self.assertProperty(controls, visible=False)
164 self.assertThat(controls.visible, Eventually(Equals(True)))172 self.assertThat(controls.visible, Eventually(Equals(True)))

Subscribers

People subscribed via source and target branches