Added tests for template generation

[santacodes]

Description

• Added basic tests for template generation using pytest-cookies.
• Added cookiecutter as a dependency and setup ___init.py__ for the src layout.

[Saransh-cpp]

Thanks, @santacodes! Looks good to me. Could you please add a minimal CI job to check if the tests are working fine?

[arjxn-py]

Thanks @santacodes, Good Start I must admit :)
Some comments below to propose some improvements

[santacodes]

Thanks, @santacodes! Looks good to me. Could you please add a minimal CI job to check if the tests are working fine?

Sorry for the delay @Saransh-cpp, I was travelling. I added CI for the tests, created a nox session for testing the template generation, and tested it on my branch. Could you please review and look at it? Also, I added uv to the nox backend, which seems to be faster.

[Saransh-cpp]

Thanks, @santacodes! Few minor changes below -

[Saransh-cpp]

TODO:

• Update branch protection rule to include the new CI (status checks must pass before merging)

[santacodes]

TODO:

• Update branch protection rule to include the new CI (status checks must pass before merging)

I think I can only create a branch rule allowing merges only after status checks when the status checks themselves have been merged to the main branch.

[Saransh-cpp]

Hi, don't worry about the rules. I'll set them up.

[Saransh-cpp]

This looks good to me, thanks @santacodes! I'll wait for @agriyakhetarpal and @arjxn-py's reviews.

[arjxn-py]

Hi, I'll review this in a couple of hours.

[arjxn-py]

Looks much better @santacodes 🙂, Some comments below for minor improvements, feel free to ping me again.
Before we merge, we would want that the workflow starts running on this same PR.
Plus, I really appreciate that we're also adapting to uv in this.
xref PyBaMM#3825

[agriyakhetarpal]

P.S. I'm sorry for the late review! I have been a bit out of touch, plus I noticed that Saransh and Arjun had already put in their reviews. I hope this is helpful.

[santacodes]

@arjxn-py can you please review my last commit, I actually modified those test cases completely because I think they're better in terms of assertions and we don't have to play with paths as pytest-cookies mostly takes care of it.
So one of the test cases tests template generation with default contexts from cookiecutter.json while the other tests template generation with a custom context passed into the bake() function. If you feel anything is wrong feel free to point it out so that I could revert it back if needed.

[arjxn-py]

Thanks @santacodes, the tests look perfect to me now.

[arjxn-py]

Before we merge, I'd just want to confirm if we can safely resolve the conversation about uv caching plus can you maybe link the CI run on the latest commit from your fork here?

[agriyakhetarpal]

This looks good to me as long as the tests pass – resolved the previous conversation. Thanks for adding this, @santacodes!

xrefs for uv's caching to look at later: astral-sh/uv#2231, yezz123/setup-uv#25, actions/setup-python#822, actions/setup-python#818

[agriyakhetarpal]

(I think we should disable the setting that dismisses reviews on further commits)

[arjxn-py]

I'm merging this as @santacodes needs to start working on entrypoints as well.
Although everything looks good but in case something fails we can look into it later as well

[arjxn-py]

(I think we should disable the setting that dismisses reviews on further commits)

Yes, I was going to mention that reviews were getting dismissed automatically