[HDFS-7025] HDFS Credential Provider related Unit Test Failure

• Type: Test
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.4.1
• Fix Version/s: 2.6.0
• Component/s: encryption
• Target Version/s: 2.6.0
• Hadoop Flags: Reviewed
• Assignee: Xiaoyu Yao
• Reporter: Xiaoyu Yao
• Created: 06/Sep/14 22:43
• Updated: 01/Dec/14 03:09
• Resolved: 07/Sep/14 15:43

Description

Reported by: Xiaomara and investigated by cnauroth.

The credential provider related unit tests failed on Windows. The tests try to set up a URI by taking the build test directory and concatenating it with other strings containing the rest of the URI format, i.e.:

  public void testFactory() throws Exception {
    Configuration conf = new Configuration();
    conf.set(CredentialProviderFactory.CREDENTIAL_PROVIDER_PATH,
        UserProvider.SCHEME_NAME + ":///," +
            JavaKeyStoreProvider.SCHEME_NAME + "://file" + tmpDir + "/test.jks");

This logic is incorrect on Windows, because the file path separator will be '\', which violates URI syntax. Forward slash is not permitted.

The proper fix is to always do path/URI construction through the org.apache.hadoop.fs.Path class, specifically using the constructors that take explicit parent and child arguments.

The affected unit tests are:

* TestCryptoAdminCLI
* TestDFSUtil
* TestEncryptionZones
* TestReservedRawPaths