remove unittest from pre-push hook

[diracdeltas]

fix #11081

[codecov-io]

Codecov Report

Merging #11084 into master will increase coverage by 0.89%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #11084      +/-   ##
==========================================
+ Coverage    52.6%    53.5%   +0.89%     
==========================================
  Files         268      251      -17     
  Lines       25237    21885    -3352     
  Branches     4024     3446     -578     
==========================================
- Hits        13277    11709    -1568     
+ Misses      11960    10176    -1784

Flag	Coverage Δ
#unittest	53.5% <ø> (+0.89%)	⬆️

Impacted Files	Coverage Δ
app/common/lib/bookmarkUtil.js	58.55% <0%> (-40.77%)	⬇️
app/renderer/components/common/browserButton.js	69.04% <0%> (-26.31%)	⬇️
app/renderer/components/tabs/content/tabIcon.js	45.83% <0%> (-13.26%)	⬇️
app/renderer/components/common/dropdown.js	90.62% <0%> (-9.38%)	⬇️
app/common/state/bookmarkFoldersState.js	67.15% <0%> (-7.66%)	⬇️
app/common/state/bookmarksState.js	80.79% <0%> (-7.04%)	⬇️
app/renderer/reducers/frameReducer.js	50.73% <0%> (-6.28%)	⬇️
...p/renderer/components/tabs/content/closeTabIcon.js	39.06% <0%> (-6.04%)	⬇️
app/common/state/tabUIState.js	25.2% <0%> (-5.49%)	⬇️
app/renderer/reducers/urlBarReducer.js	72.11% <0%> (-4.52%)	⬇️
... and 83 more

[luixxiul]

I've been not sure if this was a typo: #9690 (comment)

[diracdeltas]

cc @NejcZdovc on whether this is a typo

[bbondy]

These are sub-dirs of node_modules mostly so yes it's a typo. Thx.

[bsclifton]

@diracdeltas what do you think about updating the push hook to not run if you have a specific environment variable defined, like NO_PUSH_HOOK_TESTS. If that's defined, we can just noop

[diracdeltas]

@bsclifton that's a reasonable option

[diracdeltas]

@bsclifton done!

[bsclifton]

++ 😄