D18464: [autotests] Rewrite testShellClientRules

davidedmundson added inline comments.

INLINE COMMENTS

> zzag wrote in shell_client_rules_test.cpp:987
> Yeah, it's a very good idea. Whould it be enough to expose only
> 
>   // All these values are saved to the cfg file, and are also used in kstart!
>   enum {
>       Unused = 0,
>       DontAffect, // use the default value
>       Force,      // force the given value
>       Apply,      // apply only after initial mapping
>       Remember,   // like apply, and remember the value when the window is withdrawn
>       ApplyNow,   // apply immediatelly, then forget the setting
>       ForceTemporarily // apply and force until the window is withdrawn
>   };
> 
> ?

And StringMatch

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D18464

To: zzag, #kwin
Cc: davidedmundson, graesslin, kwin, jraleigh, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, alexeymin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart