Skip to content

Add tests - #92

Open
Huliiiiii wants to merge 4 commits into
shssoichiro:masterfrom
Huliiiiii:test
Open

Add tests#92
Huliiiiii wants to merge 4 commits into
shssoichiro:masterfrom
Huliiiiii:test

Conversation

@Huliiiiii

Copy link
Copy Markdown
Contributor

#84

Comment thread src/lib.rs
let input = indoc!(
"
DROP INDEX IF EXISTS idx_a;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lines_between_queries:1 would add just a single new line, probably we should document it better or change its name.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I didn’t understand. Could you explain it in more detail?

@lu-zero

lu-zero commented Sep 23, 2025

Copy link
Copy Markdown
Contributor

I managed to forget to check this while I reworked some logic, could you please rebase and confirm now works?

@Huliiiiii

Copy link
Copy Markdown
Contributor Author

It still fails.

@lu-zero

lu-zero commented Oct 14, 2025

Copy link
Copy Markdown
Contributor

Now that I added the debug support cargo test -F debug shown the problem and hopefully now I have a solution.

@lu-zero

lu-zero commented Oct 15, 2025

Copy link
Copy Markdown
Contributor

Please test the current master, hopefully it should cover your needs.

@Huliiiiii

Copy link
Copy Markdown
Contributor Author

Sorry for the delay, but it still failed.

@Huliiiiii

Copy link
Copy Markdown
Contributor Author

@lu-zero Approval required to rerun CI.

@lu-zero

lu-zero commented May 15, 2026

Copy link
Copy Markdown
Contributor

@shssoichiro can you please approve the run?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants