<p>Friendly Automation <strong>submitted</strong> this change.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/19555">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span></span><br></pre><div style="white-space:pre-wrap">Approvals:
  George Joseph: Looks good to me, approved
  Friendly Automation: Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">test_json: Remove duplicated static function.<br><br>Removes the function mkstemp_file and uses<br>ast_file_mkftemp from file.h instead.<br><br>ASTERISK-30295 #close<br><br>Change-Id: I7412ec06f88c39ee353bcdb8c976c2fcac546609<br>---<br>M tests/test_json.c<br>1 file changed, 18 insertions(+), 24 deletions(-)<br><br></pre>
<pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/tests/test_json.c b/tests/test_json.c</span><br><span>index a14ac3a..e1fc0ba 100644</span><br><span>--- a/tests/test_json.c</span><br><span>+++ b/tests/test_json.c</span><br><span>@@ -41,6 +41,7 @@</span><br><span> #include "asterisk/json.h"</span><br><span> #include "asterisk/module.h"</span><br><span> #include "asterisk/test.h"</span><br><span style="color: hsl(120, 100%, 40%);">+#include "asterisk/file.h"</span><br><span> </span><br><span> #include <stdio.h></span><br><span> #include <unistd.h></span><br><span>@@ -1269,27 +1270,6 @@</span><br><span>   return 0;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static FILE *mkstemp_file(char *template, const char *mode)</span><br><span style="color: hsl(0, 100%, 40%);">-{</span><br><span style="color: hsl(0, 100%, 40%);">-  int fd = mkstemp(template);</span><br><span style="color: hsl(0, 100%, 40%);">-     FILE *file;</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-     if (fd < 0) {</span><br><span style="color: hsl(0, 100%, 40%);">-                ast_log(LOG_ERROR, "Failed to create temp file: %s\n",</span><br><span style="color: hsl(0, 100%, 40%);">-                        strerror(errno));</span><br><span style="color: hsl(0, 100%, 40%);">-               return NULL;</span><br><span style="color: hsl(0, 100%, 40%);">-    }</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-       file = fdopen(fd, mode);</span><br><span style="color: hsl(0, 100%, 40%);">-        if (!file) {</span><br><span style="color: hsl(0, 100%, 40%);">-            ast_log(LOG_ERROR, "Failed to create temp file: %s\n",</span><br><span style="color: hsl(0, 100%, 40%);">-                        strerror(errno));</span><br><span style="color: hsl(0, 100%, 40%);">-               return NULL;</span><br><span style="color: hsl(0, 100%, 40%);">-    }</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-       return file;</span><br><span style="color: hsl(0, 100%, 40%);">-}</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span> AST_TEST_DEFINE(json_test_dump_load_file)</span><br><span> {</span><br><span>      RAII_VAR(struct ast_json *, uut, NULL, ast_json_unref);</span><br><span>@@ -1312,7 +1292,7 @@</span><br><span> </span><br><span>  /* dump/load file */</span><br><span>         expected = ast_json_pack("{ s: i }", "one", 1);</span><br><span style="color: hsl(0, 100%, 40%);">-     file = mkstemp_file(filename, "w");</span><br><span style="color: hsl(120, 100%, 40%);">+ file = ast_file_mkftemp(filename, 0644);</span><br><span>     ast_test_validate(test, NULL != file);</span><br><span>       uut_res = ast_json_dump_file(expected, file);</span><br><span>        ast_test_validate(test, 0 == uut_res);</span><br><span>@@ -1347,7 +1327,7 @@</span><br><span> </span><br><span>   /* dump/load filename */</span><br><span>     expected = ast_json_pack("{ s: i }", "one", 1);</span><br><span style="color: hsl(0, 100%, 40%);">-     file = mkstemp_file(filename, "w");</span><br><span style="color: hsl(120, 100%, 40%);">+ file = ast_file_mkftemp(filename, 0644);</span><br><span>     ast_test_validate(test, NULL != file);</span><br><span>       uut_res = ast_json_dump_new_file(expected, filename);</span><br><span>        ast_test_validate(test, 0 == uut_res);</span><br><span>@@ -1378,7 +1358,7 @@</span><br><span>       /* dump/load NULL tests */</span><br><span>   uut = ast_json_load_string("{ \"one\": 1 }", NULL);</span><br><span>      ast_test_validate(test, NULL != uut);</span><br><span style="color: hsl(0, 100%, 40%);">-   file = mkstemp_file(filename, "w");</span><br><span style="color: hsl(120, 100%, 40%);">+ file = ast_file_mkftemp(filename, 0644);</span><br><span>     ast_test_validate(test, NULL != file);</span><br><span>       ast_test_validate(test, NULL == ast_json_dump_string(NULL));</span><br><span>         ast_test_validate(test, -1 == ast_json_dump_file(NULL, file));</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/19555">change 19555</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/19555"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 18 </div>
<div style="display:none"> Gerrit-Change-Id: I7412ec06f88c39ee353bcdb8c976c2fcac546609 </div>
<div style="display:none"> Gerrit-Change-Number: 19555 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: N A <asterisk@phreaknet.org> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>