Add base and root bindings to pre test command

[Ni-g-3l]

Hey !

I found that on pre_test_command, "base" and "root" bindings were missing.

They are still available with "{this.base}" or "{this.root}", however I think it's important to keep consistency between available callback inside package.py

Feel free to comment or ask me question :)

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.33%. Comparing base (5c598c5) to head (816c9ae).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2032      +/-   ##
==========================================
+ Coverage   61.29%   61.33%   +0.04%     
==========================================
  Files         164      164              
  Lines       20568    20570       +2     
  Branches     3575     3575              
==========================================
+ Hits        12607    12617      +10     
+ Misses       7089     7081       -8     
  Partials      872      872              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: Ni-g-3l / name: Nig3l (5cfc2d9, c25be22)

[maxnbk]

@Ni-g-3l , I've added tests to the PR that it would be nice if you could evaluate yourself, and if you could sign the CLA, we would be able to move forward merging this PR I believe.

[Ni-g-3l]

Hi ! thanks ! I signed CLA but I don't know why it still doesn't work :/

[JeanChristopheMorinPerso]

@Ni-g-3l it looks like commit 9631300 was committed under a different email address. I think you can fix this using this:

git rebase --onto HEAD~2 --exec "git commit --amend --reset-author --no-edit" HEAD~2

(make sure that your git config is configured correctly. You can check using git config user.email and git config user.name).

[JeanChristopheMorinPerso]

Also, it might be worth rebasing on main. I think the failure to build the docs is related to the branch being tool old. Lastly, I think we need to update the docs to reflect that there are new variables available: https://rez.readthedocs.io/en/stable/package_commands.html#pre-test-commands.

[maxnbk]

@Ni-g-3l Just re-pinging to see if Jean-Christophe's command can help you with the CLA issue. I'd love to get this merged.

[Ni-g-3l]

Should be okay now :) sorry for late responding I was busy last week :)