442
POV: You maintain a JS library that 80% of modern web infrastructure uses as a dependency. (lemmy.blahaj.zone)
submitted 8 months ago by to c/programmer_humor@programming.dev
49 comments fedilink hide all child comments
🤔 1
 
you are viewing a single comment's thread
view the rest of the comments
[–] 31 points 8 months ago (2 children)

There are several legit reasons why you'd do this. Unit tests, for example: override getRandom() with an implementation that always returns the same series of numbers, and now you have repeatable tests without touching the production code.

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 2 child comments
    •
  • [–] 3 points 8 months ago* (1 child)

    Can you override Math.random within a local scope?

    At my shop we do create generic covers for vendor specific functionality, for the reasons you stated. Though the practice was started in case we ever needed to swap vendors.

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 1 child comment
    •
  • [–] 12 points 8 months ago

    You can, but you shouldn’t. You don’t know what else relies on Math.random. That’s why there’s the wrapper function. That you can override in unit tests without worrying about other, unrelated code.

  • permalink
  • fedilink
  • source
  • parent