fix bug: adding back prefix in service group creation

[rexf]

fix bug: adding back prefix in service group creation

Currently, the prefix will be missing if we are creating group from group.
e.g.

let sg = service.group("level1");
sg.endpoint("endpoint1") ==> we can expect to see "level1.endpoint1"

let g2 = sg.group("level2:);
g2.endpoint("endpoint2") ==> now we will see "level2.endpoint2"

this patch is to fix and apply the prefix for the g2 endpoint such that the topic will become
level1.level2.endpoint2

[Jarema]

Great catch!

Thanks!

Could you add a test for this case?

[Jarema]

This does not need to_string() call.

Suggested change

	prefix: format!("{}.{}", self.prefix.to_string(), prefix.to_string()),
	prefix: format!("{}.{}", self.prefix, prefix.to_string()),

[rexf]

updated

[rexf]

Great catch!

Thanks!

Could you add a test for this case?

test case added.

[Jarema]

Hm, I' not a fan of adding a test that starts a server in a unit test and we do not use this pattern across the client, but I can refactor it later myself based on your test, as you already did a lot.

Please fix fmt and it's good to merge.

[Jarema]

Please fix fmt.

[rexf]

Thansk Jarema, cargo fmt applied.

[Jarema]

LGTM!

Thanks for your contribution!