njord-rs / njord

A versatile, feature-rich Rust ORM ⛵
https://njord.rs
BSD 3-Clause "New" or "Revised" License
409 stars 21 forks source link

Create better tests structure #141

Closed mjovanc closed 1 month ago

mjovanc commented 1 month ago

We are starting to get a lot of tests and eventually it is going to be a huge file for sqlite. What I propose is that we do something like this instead:

tests
    sqlite
        common_test.rs
        insert_test.rs
        update_test.rs
        delete_test.rs        
        select_test.rs
    postgres
        ...
Hiccup-za commented 1 month ago

This is definitely the way to go for structuring tests as you build out support for other DBs 🚀

mjovanc commented 1 month ago

Thanks! Are you perhaps interested in fixing this? 😎

Hiccup-za commented 1 month ago

I'd be more than happy to give it a shot!

Hiccup-za commented 1 month ago

Got a branch ready to go

Screenshot 2024-10-07 at 19 42 28

mjovanc commented 1 month ago

Hey man, you need to fork the repo first and clone it. Then you push to that forked repo and create PR from your repo to this repo. That's how you do it.