webpack-bundle-analyzer icon indicating copy to clipboard operation
webpack-bundle-analyzer copied to clipboard

plugin use infrastructureLogger and output the report.html to compiler.outputFileSystem

Open wood1986 opened this issue 3 years ago • 7 comments

fix #476

wood1986 avatar Oct 11 '21 08:10 wood1986

I hope you won't get discouraged by the dev-server tests hanging when running in GitHub actions CI 😓.

I spent a great deal of time in figuring out why the dev-server related tests were always hanging in CI and not on my own machine and the only fix that seemed to work was this one: 55f3478dbb8036d403ad5cd8b128ef4c58ef3417. If you check 36e960bac9010e4eb71947ba7091d6e70093a408 and the commits leading up to it, you can see my failed attempts at figuring out different alternatives which just didn't seem to work in CI.

If you manage to get the tests to not hang in CI, that would be amazing. I had a pretty bad time debugging those failures myself.

I cancelled the workflow run now as it was hanging.

valscion avatar Oct 12 '21 07:10 valscion

@valscion. I think the hanging up issue is because the second expect is failed. Then it stops executing the rest. If it can execute to the end, it will not hang up and finish quickly.

wood1986 avatar Oct 12 '21 07:10 wood1986

Even with classic function (done) it will still block from executing to the end if one of the expect is failed. Then it will still leave the never ended server. Jest cannot exit

wood1986 avatar Oct 12 '21 07:10 wood1986

Oh right, you removed the jest.setTimeout part.

valscion avatar Oct 12 '21 07:10 valscion

@wood1986 may I kindly ask you to:

  1. Fix PR review issues in a separate commits and don't use force-push as it's really difficult to track changes you've made
  2. Don't resolve my comments - I would like to do this myself.

th0r avatar Oct 13 '21 10:10 th0r

@wood1986 may I kindly ask you to:

  1. Fix PR review issues in a separate commits and don't use force-push as it's really difficult to track changes you've made
  2. Don't resolve my comments - I would like to do this myself.

No problem

  1. I thought this is a very tiny and it should be very quick to check all the check again
  2. I usually use the resolve to let comment owner know I have completed. Meanwhile it will be easy for me to see what things left.

I will do what you ask

wood1986 avatar Oct 13 '21 16:10 wood1986

@th0r @valscion What we need to finish it?

alexander-akait avatar Nov 05 '21 12:11 alexander-akait