Yup just checked it and we'll make sure to exclude the fixture data. These dynamic urls will be checked by the tests written to check documentation loading and rendering from now on. Will push the change eod today. Apologies for the inconvenience caused with the first two results.
Will share the correct reports asap.
Thanks,
Akshat Jaimini
On Tue, Jul 25, 2023 at 1:01 PM Daniel Gustafsson <daniel@yesql.se> wrote:
I'm guessing this ran against the featurematrix/fixtures/data.json in the pgweb repo which does contain these links, but which doesn't reflect the actual website content due to being an initial dataset (in turn generated by the script pgweb/dump_initial_data.sh).
The data.json being served on the production site is based off the database and isn't committed to the repository. A testing harness for the website code should probably skip checking the validity of dynamic data unless it's running against the live site for those checks.