CURA-12386 G-Code replacement fails with single line code

[wawanbreton]

The regex was not specific enough and would catch the rest of the line, now we force stopping as soon as we see a { or }

CURA-12386
Fixes 20183

[github-actions]

Test Results

0 tests   - 23 701   0 ✅  - 23 699   0s ⏱️ -47s
0 suites  -      1   0 💤  -      2 
0 files    -      1   0 ❌ ±     0 

Results for commit 884983c. ± Comparison against base commit 9ac0f11.

♻️ This comment has been updated with latest results.

[rburema]

Looks good to me.

• Unit-tests! Yes! 🎈
• I briefly thought about [^}] instead of [^{}], but that wouldn't allow us to nest either (future features ...) and this does indeed make just as much sense (it's just wether you have a malformed bracket inside the replacement versus one outside -- but in the end it's the same syntax error).

[rburema]

Interestingly, it produces None in the GCode when you do {}, and just ignores any errors (so there's not error shown to the user) -- but I think it's quite fine for now!

[wawanbreton]

Yeah, the idea is just to have this working as best as possible within the current limitations, until we replace it by something more powerful (Jinja !!!)