Ubuntu UI Toolkit

Merge lp://staging/~ubuntu-sdk-team/ubuntu-ui-toolkit/noScores into lp://staging/ubuntu-ui-toolkit/staging

noScores
Merge into staging

Proposed by Cris Dywan on 2014-07-21

Status:

Merged

Approved by:

Zoltan Balogh on 2014-07-23

Approved revision:

1147

Merged at revision:

1161

Proposed branch:

lp://staging/~ubuntu-sdk-team/ubuntu-ui-toolkit/noScores

Merge into:

lp://staging/ubuntu-ui-toolkit/staging

Diff against target:

49 lines (+16/-4)

2 files modified

tests/autopilot/ubuntuuitoolkit/fixture_setup.py (+12/-4)
tests/autopilot/ubuntuuitoolkit/tests/__init__.py (+4/-0)

To merge this branch:

bzr merge lp://staging/~ubuntu-sdk-team/ubuntu-ui-toolkit/noScores

Critical

Fix Released

Link a bug report

Reviewer	Review Type	Date Requested	Status
PS Jenkins bot	continuous-integration		Needs Fixing on 2014-07-22
Leo Arias (community)			Approve on 2014-07-22
Zoltan Balogh		2014-07-21	Approve on 2014-07-21
Review via email: mp+227541@code.staging.launchpad.net

Commit message

Wrap tempfile.NamedTemporaryFile to discard underscored filenames

Revision history for this message

Zoltan Balogh (bzoltan) wrote on 2014-07-21:

review: Approve

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-21:

FAILED: Continuous integration, rev:1144
http://jenkins.qa.ubuntu.com/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-ci/625/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/2230/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-utopic/1857/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-utopic-amd64-ci/457/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-utopic-armhf-ci/457/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-utopic-i386-ci/457/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/3412/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/2076/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-ci/625/rebuild

review: Needs Fixing (continuous-integration)

lp://staging/~ubuntu-sdk-team/ubuntu-ui-toolkit/noScores updated on 2014-07-21

1145. By Cris Dywan on 2014-07-21: Make pep8 happy again
1146. By Cris Dywan on 2014-07-21: Always remove discarded temporary filename with underscores

Revision history for this message

Leo Arias (elopio) wrote on 2014-07-21:

15 + def _named_temporary_file(self, dir=None, mode='w+t',

Please add a comment on this method about why we are striping the underscores. A link to the bug would be good too.

55 + desktop_file = fixture_setup._named_temporary_file(
56 + suffix='.desktop', dir=desktop_file_dir)

This won't work because it's a method in an object, not in the module. What we need to do is to refactor this test case to use the same fixture. Let me know if you want me to take over that part, using your branch as a prerequisite.

review: Approve

Revision history for this message

Leo Arias (elopio) on 2014-07-21:

review: Needs Fixing

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-21: