Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test only #723

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Prev Previous commit
Next Next commit
fix bugs in test framework.
  • Loading branch information
RTEnzyme committed Sep 25, 2024

Partially verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
We cannot verify signatures from co-authors, and some of the co-authors attributed to this commit require their commits to be signed.
commit a06bf22b0b7dee60e120dded4b14ff33383d976e
4 changes: 2 additions & 2 deletions test/test_query_compilation.cpp
Original file line number Diff line number Diff line change
@@ -47,7 +47,7 @@ std::string execute(const std::string& command) {

std::string execute_func(std::string &func_body) {
const std::string file_name = "test_add.cpp";
const std::string output_name = "test_add"
const std::string output_name = "test_add";
std::ofstream out_file(file_name);
if (!out_file) {
std::cerr << "Failed to open file for writing!" << std::endl;
@@ -65,7 +65,7 @@ std::string execute_func(std::string &func_body) {
// define and execute command
std::string output = execute("./a");
// delete files
if (std::remove(file_name) && std::remove(output_name)) {
if (std::remove(file_name.c_str()) && std::remove(output_name.c_str())) {
std::cerr << "Failed to delete files: " << file_name
<< ", " << output_name << std::endl;
}