@huchengtw-moz 

> > I'd feel less concerned if there was a unit test covering this.

> I had run all unit tests of SMS, this patch doesn't break any unit tests. And I don't think there is a unit test covering this. If it has, this bug should be found earlier.

What @gnarf37 is saying is that you should also include tests that prove this change