Bug 500748

Summary:	[Tooling] Add missing unit tests for protocol message naming
Product:	[Modeling] Papyrus-rt	Reporter:	Peter Cigehn <peter.cigehn>
Component:	tool	Assignee:	smaoui asma <asma.smaoui>
Status:	CLOSED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	charles
Version:	0.7.2	Keywords:	test
Target Milestone:	0.9.0
Hardware:	PC
OS:	Windows 7
See Also:	https://git.eclipse.org/r/86229 https://git.eclipse.org/c/papyrus-rt/org.eclipse.papyrus-rt.git/commit/?id=895a241ab1062f8216613a00122e425966347534
Whiteboard:

Description Peter Cigehn

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

2016-10-31 14:28:01 EDT

Assigning to 0.9

Comment 2 Charles Rivet

2016-11-10 15:42:47 EST

Rémi, Could you please have a look and assign?

Comment 3 Eclipse Genie

2016-12-02 07:08:10 EST

New Gerrit change created: https://git.eclipse.org/r/86229

Comment 4 Peter Cigehn

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 5 Eclipse Genie

2016-12-16 10:50:28 EST

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

Comment 6 Peter Cigehn

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

2017-01-05 10:46:30 EST

Yes.

Comment 8 Peter Cigehn

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

2017-01-09 04:29:28 EST

Closing as verified fixed.