Bug 500748 - [Tooling] Add missing unit tests for protocol message naming
Summary: [Tooling] Add missing unit tests for protocol message naming
Status: CLOSED FIXED
Alias: None
Product: Papyrus-rt
Classification: Modeling
Component: tool (show other bugs)
Version: 0.7.2   Edit
Hardware: PC Windows 7
: P3 normal
Target Milestone: 0.9.0   Edit
Assignee: smaoui asma CLA
QA Contact:
URL:
Whiteboard:
Keywords: test
Depends on:
Blocks:
 
Reported: 2016-09-02 09:58 EDT by Peter Cigehn CLA
Modified: 2017-01-09 04:29 EST (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Cigehn CLA 2016-09-02 09:58:54 EDT 
The improved protocol message naming in Bug 475904 did not get any unit tests in place to avoid future regressions in the protocol message naming. To be able to close Bug 475904, this Bugzilla is used to track that unit tests eventually gets added also for the the protocol message naming.
Comment 1 Charles Rivet CLA 2016-10-31 14:28:01 EDT 
Assigning to 0.9
Comment 2 Charles Rivet CLA 2016-11-10 15:42:47 EST 
Rémi, Could you please have a look and assign?
Comment 3 Eclipse Genie CLA 2016-12-02 07:08:10 EST 
New Gerrit change created: https://git.eclipse.org/r/86229
Comment 4 Peter Cigehn CLA 2016-12-02 07:15:33 EST 
Before putting it into resolved, I assume that we need to ensure that the Gerrit change actually have been merged as well. I suggest to not putting a bug into resolved until it the related Gerrit change(s) all have been merged.
Comment 6 Peter Cigehn CLA 2016-12-16 11:32:30 EST 
(In reply to Eclipse Genie from comment #5)
> Gerrit change https://git.eclipse.org/r/86229 was merged to [master].
> Commit:
> http://git.eclipse.org/c/papyrus-rt/org.eclipse.papyrus-rt.git/commit/
> ?id=895a241ab1062f8216613a00122e425966347534

Can this bug be considered resolved fixed now?
Comment 7 smaoui asma CLA 2017-01-05 10:46:30 EST 
Yes.
Comment 8 Peter Cigehn CLA 2017-01-09 04:29:16 EST 
The tests are now included in the core tests. Putting into state verified fixed.
Comment 9 Peter Cigehn CLA 2017-01-09 04:29:28 EST 
Closing as verified fixed.