Don't update packages in build/package-lock.json when running js-tests

[nickvergessen]

I find it quite distrubing to always end up with a modified build/package-lock.json file after running autotest-js.sh
Updating should be done explicitly (or automatically with dependabot) and then be commited like that to the repo, not running locally when you run tests

[juliusknorr]

Makes sense though we should also commit that properly then in the first run https://github.com/nextcloud/server/blob/master/build/package-lock.json and remove it from .gitignore

[nickvergessen]

Makes sense though we should also commit that properly then in the first run https://github.com/nextcloud/server/blob/master/build/package-lock.json and remove it from .gitignore

Done, please review

[nickvergessen]

Are failing JS tests related? Works locally

[skjnldsv]

jsunit fails ?

[nickvergessen]

As mentioned above, I have no idea.

[nickvergessen]

So anyone from @nextcloud/server-frontend wants to pick this up? Otherwise I would close it any we still have the updates happening

[artonge]

So anyone from @nextcloud/server-frontend wants to pick this up? Otherwise I would close it any we still have the updates happening

Should be good as of: #30567

[artonge]

The .gitignore part would be nice to have though