j0k3r / graby / 884

97%

master: 95%

Pull #207

travis-ci

Change the way to merge config for find & replace string

`find_string` & `replace_string` had special case when merging them.
But it failed when merging multiple times.

Update it to a bit more complex solution to ensure `find_string` & `replace_string` aren't duplicated when merging config multiple times.

We can't perform an `array_unique` on these values (looks like fabrizio got the point years ago) mostly because `replace_string` can have same values, example:

```
find_string: <amp-img
replace_string: <img

find_string: <other-img
replace_string: <img
```

To fix that issue, we combine find & replace as key & value in one array, we merge them (so we can't have duplicates keys) and then rebuild find & replace string in the current config

8 of 8 new or added lines in 1 file covered. (100.0%)

1280 of 1326 relevant lines covered (96.53%)

44.26 hits per line