feat: Add MongoDB ObjectId generation

[nhammond101]

Adds a function to generate MongoDB ObjectIds

The function aims to be deliberately simple so any combination of a-z and 0-9 is generated as a valid ObjectId, instead of implementing ObjectIds according to the official spec

Futhermore, I was unsure where to add the function given it's a datatype for a database, so happy to refactor/move if required.

[Shinigami92]

Would this fit better into /src/database.ts? I think we should move it to that module.

[Shinigami92]

Not related to this PR, but IMO we should try to prepare that the datatype module only defines functions for typeof "return" values.
So only string, number, boolean and so on. But this would be something for v7+

[nhammond101]

@Shinigami92 agree on the moving to database, and have done so now.

Is there anything I can add to the current PR to facilitate the future typeof improvements?

[Shinigami92]

Is there anything I can add to the current PR to facilitate the future typeof improvements?

Mh, currently I don't think so. You can always join our discord if you haven't already, so we can communicate better there.
But for now, we would tackle all the v6.1 stuff first anyways.
We expect to release 6.0.0 on monday evening.

[codecov]

Codecov Report

Merging #616 (1dff5dc) into main (5beac4b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 1dff5dc differs from pull request most recent head 892576e. Consider uploading reports for the commit 892576e to get more accurate results

@@           Coverage Diff           @@
##             main     #616   +/-   ##
=======================================
  Coverage   99.34%   99.34%           
=======================================
  Files        1921     1921           
  Lines      176656   176667   +11     
  Branches      905      906    +1     
=======================================
+ Hits       175503   175514   +11     
  Misses       1097     1097           
  Partials       56       56           

Impacted Files	Coverage Δ
src/database.ts	100.00% <100.00%> (ø)

Looks good for now, will approve after v6.1 released

[damienwebdev]

This feels more like a plugin than a method of the main library. @Shinigami92 @ST-DDT have you seen a way to add custom user-defined methods to the Faker API?

[ST-DDT]

have you seen a way to add custom user-defined methods to the Faker API?

This is possible, but it requires casting or subclassing the faker instance. While I agree this kind of custom/specific, moving this to a plugin seems overengeneered to me. We can consider this again, when we have more of these.
We could nest these in their own submodule if you want.
faker.database.mongodb.objectId()

[Shinigami92]

I'm with @ST-DDT, plugins would be over engineered.
We also already had the discussion about sub module, but IMO we can move it into that if really needed.