Remove sourceMappingURL lines from tinymce static files

[marius-mather]

Closes #460 . This is just a workaround, the "ideal" solution is that TinyMCE either ships the .map files in their releases or removes references to them. But probably the easiest fix for now.

[claudep]

Thanks, it's probably the best move for now. Do you think we could add a regression test for that, so we don't forget to reapply this in future lib updates, if the upstream project doesn't fix this issue?

[marius-mather]

I've added a test that just tries to run collectstatic with the built-in ManifestStaticFilesStorage - it fails when the source map issue is encountered.

[claudep]

Thanks a lot for the test! Now to be absolutely clean, I think we should delete the temp_dir at the end of the test, something like self.addCleanup(shutil.rmtree, local_path). Or even better use the with tempfile.TemporaryDirectory() as temp_dir context manager.

[marius-mather]

Yes, good point, I've done that now.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.14%. Comparing base (f04e1c8) to head (e0f4b9f).

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #461   +/-   ##
=======================================
  Coverage   87.14%   87.14%           
=======================================
  Files           6        6           
  Lines         249      249           
  Branches       45       45           
=======================================
  Hits          217      217           
  Misses         15       15           
  Partials       17       17           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[claudep]

Thanks!