Thanks for the report. In this case, though the converted string of queryConfigUrl is non-deterministic due to hash map characteristic, I think this test will always pass as the actual logic to handle notificationMessages in assembleQueryConfigUrl is the same as testAssembleQueryConfigUrl.

assembleQueryConfigUrl:

    if (remoteMessages != null) {
      queryParams.put("messages", queryParamEscaper.escape(GSON.toJson(remoteMessages)));
    }

testAssembleQueryConfigUrl:

    assertTrue(queryConfigUrl
        .contains("messages=" + UrlEscapers.urlFormParameterEscaper()
            .escape(gson.toJson(notificationMessages))));

Thanks for your response! Yes, both assembleQueryConfigUrl and testAssembleQueryConfigUrl use the same method toJson to convert the Object to JSON. However, the returned JSON may not always be the same every time toJson is called. If you check the implementation of toJson, you will find in the end that it needs to iterate the HashMap to convert it to JSON, and the iteration order is non-deterministic. In other words, it's possible that entryA comes before entryB the first time toJson is called, and entryB comes before entryA the second time toJson is called.

Codecov Report

Merging #4083 (8f4486a) into master (b00f238) will not change coverage.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##             master    #4083   +/-   ##
=========================================
  Coverage     52.45%   52.45%           
  Complexity     2612     2612           
=========================================
  Files           484      484           
  Lines         15203    15203           
  Branches       1572     1572           
=========================================
  Hits           7974     7974           
- Misses         6674     6675    +1     
+ Partials        555      554    -1     

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b00f238...8f4486a. Read the comment docs.