Skip to content

Conversation

marcosptf
Copy link
Contributor

new test to don't covered function;

@bukka
Copy link
Member

bukka commented Oct 26, 2015

Please could you rewrite this and other openssl tests as well in the same way as #1600 . In this case, you can do for example something like:

var_dump(openssl_csr_export_to_file($csr, $crs_exported_file_notext_true, true));
var_dump(file_exists($crs_exported_file_notext_true));

Also don't forget to do --CLEAN-- if you create a new file to unlink it after the test.

And use tabs if you indent (no spaces)!

@marcosptf
Copy link
Contributor Author

@bukka
done!

:-)

@jerrygrey
Copy link

@marcosptf In --CLEAN-- you have unlink(__DIR__."/{$crs_exported_file_notext_true}"); which becomes __DIR__ ."/". __DIR__ . DIRECTORY_SEPARATOR . "crs-exported-file-notext-true.crs" which is wrong. And you are still using spaces instead of tabs as @bukka pointed out a few times now.

@marcosptf
Copy link
Contributor Author

@bukka @jerrygrey
fix done!
sorry, its will never happen again, thanks for everything!

@bukka
Copy link
Member

bukka commented Oct 27, 2015

The same here. I will take a look on this after PKCS12. There is some other stuff that needs to be tested.

@marcosptf
Copy link
Contributor Author

👍

@krakjoe
Copy link
Member

krakjoe commented Jan 6, 2017

@marcosptf @bukka can we get an update here please ?

@bukka
Copy link
Member

bukka commented Jan 6, 2017

Please close it. I will come up with different test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants