Fix: add isDirty to setValue in checkster (check Editor v2)

[ckbedwell]

Problem

In Checkster (Check Editor v2) we have moved to using the native isDirty value when assessing if the form has changed or not. In several places we use setValue method to change the value in the form but by default this does not dirty the form.

This issue was originally noticed when simply modifying a check to enable a per-check alert.

Solution

I've gone through and added shouldDirty: true to all of the places where I believe it should dirty the form. There are a few places where I don't think it is necessary and have left them as they are, such as when we are closing and opening requests / assertions in the MultiHTTP check.

[github-actions]

Script size changes

Name	+/-	Main	This PR	Outcome
[502.js]	=	1,732.65 kB	1,732.65 kB	✅
[665.js]	=	936.23 kB	936.32 kB	✅
[datasource/module.js]	=	24.72 kB	24.72 kB	✅
[692.js]	=	16.80 kB	16.80 kB	✅
[663.js]	=	5.83 kB	5.83 kB	✅
[module.js]	=	4.92 kB	4.92 kB	✅

Totals

Name	+/-	Main	This PR	Outcome
[Scripts]	=	2,721.15 kB	2,721.24 kB	✅
[Non-script Assets]	=	2,613.55 kB	2,613.55 kB	✅
[All]	=	5,334.70 kB	5,334.78 kB	✅

Generated by 🚫 dangerJS against 3741893