<p>In theory I have nothing against this.  In practice though, I see a few problems:</p>
<ul>
<li>
<p>as <a href="https://github.com/codebrainz" class="user-mention">@codebrainz</a> pointed out, with the current code classic unit tests will likely not be able to do test much, as we have way too many state everywhere, and that reproducing it basically means starting Geany.</p>
</li>
<li>
<p>I have nothing against Criterion, apart that it's apparently not currently in any distribution, which means it'll have to be manually installed, and thus</p>
<ol>
<li>we can't hard-depend on it</li>
<li>very few people will actually run the tests</li>
</ol>
<p>which is kind of sad for testing purposes.</p>
</li>
</ul>
<p>So IMO it's not a great idea to add an "uncommon" test framework, no matter how great it is, if we won't have much test to add to it and won't have much people running them.<br>
So I'd rather add tests using something we already have available (maybe GLib's thing) even if it's slightly less fun -- at least until we actually have a strong use-case, as again I don't expect much useful tests to be runnable that way.</p>
<p>Just my ¢2</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/1427#issuecomment-287548353">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ2mbhXYoXzzmHuL-aJZ23Yt66XIVks5rm-RQgaJpZM4Ma5hm">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ7lp6ncA1vnv9gzqXzNHX3N989bjks5rm-RQgaJpZM4Ma5hm.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany/pull/1427#issuecomment-287548353"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/geany/geany","title":"geany/geany","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@b4n in #1427: In theory I have nothing against this.  In practice though, I see a few problems:\r\n* as @codebrainz pointed out, with the current code classic unit tests will likely not be able to do test much, as we have way too many state everywhere, and that reproducing it basically means starting Geany.\r\n* I have nothing against Criterion, apart that it's apparently not currently in any distribution, which means it'll have to be manually installed, and thus\r\n  1. we can't hard-depend on it\r\n  2. very few people will actually run the tests\r\n  \r\n  which is kind of sad for testing purposes.\r\n\r\nSo IMO it's not a great idea to add an \"uncommon\" test framework, no matter how great it is, if we won't have much test to add to it and won't have much people running them.\r\nSo I'd rather add tests using something we already have available (maybe GLib's thing) even if it's slightly less fun -- at least until we actually have a strong use-case, as again I don't expect much useful tests to be runnable that way.\r\n\r\nJust my ¢2"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1427#issuecomment-287548353"}}}</script>