Merge lp:~mzanetti/unity8/coding-single-test into lp:unity8

Proposed by Michael Zanetti on 2015-05-18
Status: Merged
Approved by: Albert Astals Cid on 2015-05-18
Approved revision: 1779
Merged at revision: 1814
Proposed branch: lp:~mzanetti/unity8/coding-single-test
Merge into: lp:unity8
Diff against target: 13 lines (+4/-0)
1 file modified
CODING (+4/-0)
To merge this branch: bzr merge lp:~mzanetti/unity8/coding-single-test
Reviewer Review Type Date Requested Status
Daniel d'Andrada (community) Abstain on 2015-05-18
PS Jenkins bot continuous-integration Needs Fixing on 2015-05-18
Albert Astals Cid (community) 2015-05-18 Approve on 2015-05-18
Review via email: mp+259400@code.launchpad.net

Commit Message

add info on how to run a single function in make test

Description of the Change

 * Are there any related MPs required for this MP to build/function as expected? Please list.

no

 * Did you perform an exploratory manual test run of your code change and any related functionality?

no, just a change in a txt file

 * Did you make sure that your branch does not contain spurious tags?

yes

 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?

n/a

 * If you changed the UI, has there been a design review?

n/a

To post a comment you must log in.
Albert Astals Cid (aacid) :
review: Approve
Albert Astals Cid (aacid) wrote :

 * Did you perform an exploratory manual test run of the code change and any related functionality?
Yes

 * Did CI run pass? If not, please explain why.
N/A

 * Did you make sure that the branch does not contain spurious tags?
Yes

Daniel d'Andrada (dandrader) wrote :

It should go below

"""
12 So, for the DragHandle component that would be:
13
14 $ make testDragHandle
"""

Not above it.

review: Needs Fixing
Daniel d'Andrada (dandrader) wrote :

Btw, I didn't know about it. I always did "s/function test_/function notest_" when I wanted to isolate a single test :)

1779. By Michael Zanetti on 2015-05-18

move it a bit

Michael Zanetti (mzanetti) wrote :

> It should go below
>
> """
> 12 So, for the DragHandle component that would be:
> 13
> 14 $ make testDragHandle
> """
>
> Not above it.

done

Daniel d'Andrada (dandrader) wrote :

thanks

review: Approve
review: Abstain

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'CODING'
2--- CODING 2015-03-23 13:47:19 +0000
3+++ CODING 2015-05-18 21:08:29 +0000
4@@ -158,6 +158,10 @@
5
6 $ make gdbtestComponentName
7
8+To run a single test function of a single component/class:
9+
10+$ make testComponentName FUNCTION="TestCaseName::functionName"
11+
12 To run a test application that allow you to manually test or to simply try out
13 a given component:
14

Subscribers

People subscribed via source and target branches