remove jsmin dependency

[RonnyPfannschmidt]

jsmin is a unmaintained since a while and breaks the build

im under the impression we practically only use it to remove some white-space from js code to send to selenium, so we should be fine without it

[codecov]

Codecov Report

Merging #204 (31b2b70) into master (839a81a) will decrease coverage by 0.09%.
The diff coverage is 100.00%.

❗ Current head 31b2b70 differs from pull request most recent head 6a52d41. Consider uploading reports for the commit 6a52d41 to get more accurate results

@@            Coverage Diff             @@
##           master     #204      +/-   ##
==========================================
- Coverage   86.63%   86.53%   -0.10%     
==========================================
  Files          18       18              
  Lines        2551     2547       -4     
==========================================
- Hits         2210     2204       -6     
- Misses        341      343       +2     

Flag	Coverage Δ
unittests	86.53% <100.00%> (-0.10%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
src/widgetastic/widget/input.py	96.61% <ø> (-0.06%)	⬇️
src/widgetastic/widget/table.py	89.16% <ø> (-0.17%)	⬇️
src/widgetastic/browser.py	75.90% <100.00%> (-0.24%)	⬇️
src/widgetastic/widget/select.py	91.97% <100.00%> (-0.06%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 839a81a...6a52d41. Read the comment docs.

[quarckster]

LGTM

[rsnyman]

LGTM

[mshriver]

Fine with removing jsmin dependency - thanks for cleaning that up!

There is a logical change here too with handling of kwargs in execute_script that I have some concern about.

[RonnyPfannschmidt]

I'll split those 2 changes and send a additional pr

[mshriver]

@RonnyPfannschmidt fine with it being in the same PR, just want to make sure unit tests are covering the call stack testing the execution of the script.

[mshriver]

Small rebase and I'm happy to merge @RonnyPfannschmidt