chore(helpers): deprecate invocations on non-functions for fakeEval

[ST-DDT]

This PR deprecates the current behavior of fake in preparion to be more consistent with standard JS behavior.

faker.helpers.fake('{{person.first_name()}}'); // 'Jenny'

Would now print a deprecation warning.

In v10 would fail with an exception:

FakerError: Cannot resolve expression 'person.first_name()'

---

This PR is part of an attempt to solve some of the TODOs in our source code.

[netlify]

✅ Deploy Preview for fakerjs ready!

Name	Link
🔨 Latest commit	4180bcf
🔍 Latest deploy log	https://app.netlify.com/sites/fakerjs/deploys/666bde267de6820008e86199
😎 Deploy Preview	https://deploy-preview-2913.fakerjs.dev
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.95%. Comparing base (a5ffca1) to head (4180bcf).

Additional details and impacted files

@@            Coverage Diff             @@
##             next    #2913      +/-   ##
==========================================
- Coverage   99.95%   99.95%   -0.01%     
==========================================
  Files        2987     2987              
  Lines      216037   216052      +15     
  Branches      601      948     +347     
==========================================
+ Hits       215944   215951       +7     
- Misses         93      101       +8     

Files	Coverage Δ
src/modules/helpers/eval.ts	96.77% <100.00%> (+0.20%)	⬆️

... and 2 files with indirect coverage changes

[matthewmayer]

I don't really like introducing this change suddenly as it's likely to only be found at runtime.

If we don't want to support this syntax any more I think we could log a warning in v9 and error in v10 instead.

[ST-DDT]

Team Decision

• We forgot about the deprecation -> deprecate now and "remove" in v10

[ST-DDT]

Fixed - ready for review

[Shinigami92]

Having a chore! is invalid 🤔 either it did not change runtime behavior or it is a breaking change 👀

[matthewmayer]

I think it's no longer breaking so ! Should be removed but it should get a brief write up at the bottom of the migration guide.

[ST-DDT]

Example log:

[@faker-js/faker]: Invoking expressions which are not functions is deprecated since v9.0 and will be removed in v10.0.
Please remove the parenthesis or replace the expression with an actual function.
person.first_name()
                 ^