tests: simplify createcopy test

[bradh]

What does this PR do?

Simplifies one test, for multiple CreateCopy format checks.

What are related issues/pull requests?

At #11093 (comment) @rouault identified that the JPEG 2000 test case suppression was obsolete. This PR removes that obsolete code.

The change looks bigger than that, because of the required re-indent and slightly confusing diff. The only part that is actually removed is this:

            # print ('drv = %s, nBands = %d, datatype = %s' % (drv.ShortName, nBands, gdal.GetDataTypeName(datatype)))

            skip = False
            # FIXME: A few cases that crashes and should be investigated
            if drv.ShortName == "JPEG2000":
                if (nBands == 2 or nBands >= 5) or not (
                    datatype == gdal.GDT_Byte
                    or datatype == gdal.GDT_Int16
                    or datatype == gdal.GDT_UInt16
                ):
                    skip = True
            if skip is False:

Tasklist

• Make sure code is correctly formatted (cf pre-commit configuration)
• Add test case(s)
• Add documentation
• Updated Python API documentation (swig/include/python/docs/)
• Review
• Adjust for comments
• All CI builds and checks have passed

Environment

Provide environment details, if relevant:

• OS:
• Compiler:

[coveralls]

coverage: 69.461% (-0.002%) from 69.463%
when pulling 14bf346 on bradh:tests_simplify_createcopy_2024-10-27
into f05ae5c on OSGeo:master.