[BUGFIX canary] Remove test-loader from tests/index blueprint

[josemarluedke]

Since the test-loader was moved to NPM (#5885), calling test-loader.js from index is not necessary anymore.

[Turbo87]

@trentmwillis r?

[trentmwillis]

👍 test-loader should now get put into test-support so this seems good

[rwjblue]

We should likely do what we do for testem.js and have an empty file present for folks if they don't update their index.html. What do y'all think?

[trentmwillis]

How about logging/throwing an error in that file so they know to update?

[stefanpenner]

How about logging/throwing an error in that file so they know to update?

sounds good, ideally verbose warning/log. Eventually a throw?

[Turbo87]

the warning/log thing seems to be something that can be layered on top of this PR so I'm gonna go ahead and merge this.

since the testloader stuff is included in v2.7.0-beta.1 I'll cherry pick this down to the beta branch.

@homu r+

[homu]

📌 Commit 4687fe3 has been approved by Turbo87

[homu]

⌛ Testing commit 4687fe3 with merge 8a738a4...

[homu]

☀️ Test successful - status

[stefanpenner]

Hey guys, I'm having issues with test loader not being found.

@rwwagner90 can yo open a new issue (feel free to link back to this one) and if possible include a reproduction. That way we have something very actionable to work with.

[RobbieTheWagner]

@stefanpenner done. Sorry about that. Deleted my noise from this PR and opened an issue.

[stefanpenner]

@rwwagner90 not a problem, we often lose track of issues posted in other issues