Improve the error message for a missing file

[PeterJCLaw]

This now attempts to detect misplaced files and provide suggestions about how to resolve the problem if such files are found.

Previously it would have been like:

ZIP file must contain a file named exactly 'robot.py'.

Found the following files:
 face.py
 sub/robot.py

Now would be like:

ZIP file must contain a file named exactly 'robot.py'.

Found the following files:
 face.py
 sub/robot.py

It looks like you have included a similar file at 'sub/robot.py', perhaps you meant to include that file at 'robot.py' rather than within 'sub/'?

[PeterJCLaw]

@ShadowLord2005 would be good to get your thoughts on the error message here.

[ShadowLord2005]

It looks like it should be alright however do you have some example archives and what they actually output to just see how it reads when looking at a folder structure alongside it?

[PeterJCLaw]

The example messages are for an archive with two files -- one in the root called face.py and a robot.py which is within a folder called sub. Hrm, your question suggests that perhaps that part of the message could be clearer though 🤔

[ShadowLord2005]

Ok, that wasn't what I was thinking it did however it could be that it is clearer with a structure to look at. I think the first part of the message is that is says it couldn't find the file robot.py and then a list of all files that it found.
It was the specific error message that was confusing.
What exactly is the single specific error message trying to achieve?

[PeterJCLaw]

In case useful, here's the archive I used for testing: my.zip

[PeterJCLaw]

What exactly is the single specific error message trying to achieve?

I'm not sure what you mean by "single specific error message". The goal here is to help the user understand what's wrong with the archive they've tried to submit for their code, by adding some guidance for what I believe is the most common error people make -- putting their code in a folder inside the archive rather than just in the root.

[ShadowLord2005]

Right, I was thinking that you were going to be doing more case specific error messages.

[PeterJCLaw]

Oh, I see. If there's other potential causes for why the file might be missing that we can detect I'm up for adding those too. (Though I can't think of any right now)