File Recognition - Add support for files without extensions

[abujeda]

What this PR does / why we need it:
Use the full filename to determine the file type when the file do not have an extension.

Which issue(s) this PR closes:

• Closes Feature Request/Idea: File type recognition based on filename #8740

Special notes for your reviewer:
Supported files in this PR are Makefile and Snakemake

Suggestions on how to test this:
Upload a file without extension as the ones suggested above.
Files with extension work as before.

Does this PR introduce a user interface change? If mockups are available, please link/include them here:
No

Is there a release notes update needed for this change?:
No

Additional documentation:
No

[coveralls]

Coverage increased (+0.006%) to 19.77% when pulling 017137d on adaybujeda:8740-file-recognition-based-on-filename into 567e506 on IQSS:develop.

[pdurbin]

@adaybujeda can you please resolve the merge conflicts in this branch?

[abujeda]

@pdurbin conflicts resolved - ready to go!

[pdurbin]

@adaybujeda thanks. A couple things.

FilesIT.testAddTinyFile is failing. Here's a screenshot from Jenkins:

Is this expected? Dataverse can no longer figure out that a text file called "1char" is of type text/plain? Should we get this working again?

Less important (no need for you to fix this) but worth mentioning, in Netbeans I'm having a strange problem where I can't use "Test File" or "Run Focused Test Method" in FileUtilTest.java. However, this works fine in Intellij. This isn't specific to your changes at all. It seems like this is happening because the tests are within subclasses.

[abujeda]

Looking into the FilesIT.testAddTinyFile error...

[abujeda]

The problem was that the type recognized by JHove was being overriden by MIME_TYPE_MAP.getContentType(fileName); when the file has no extension. This was not happening before.

I updated to only override the file type when Dataverse has a content type for the file name.

The failing test is now working locally.

[pdurbin]

Looks good. Thanks, @adaybujeda! I did make some small tweaks to the docs. Off to QA.

[abujeda]

Thanks @pdurbin!! 🚀

[pdurbin]

@kcondon as of 017137d the "Maven Unit Tests" job is marked as failed but it looks like it's because the Coveralls API was down.

The Jenkins tests ran fine (which included unit tests), including the API tests: https://jenkins.dataverse.org/job/IQSS-Dataverse-Develop-PR/job/PR-8744/12/testReport/