Skip to content
This repository has been archived by the owner on Apr 13, 2023. It is now read-only.

Clean up assertion messages (particularly copy/paste errors) #3

Open
scottkurz opened this issue Sep 3, 2015 · 1 comment
Open

Comments

@scottkurz
Copy link
Member

From
https://java.net/bugzilla/show_bug.cgi?id=5401

mminella@java.net 2013-09-19 17:38:45 UTC

The messaging through out the TCK is not consistent (many copy paste errors). Just a couple examples:

  • ChunkTests#testChunkRetryOnError() - All of the messages say it's for execution Eliminate the "zip" form factor of the TCK  #1 even though the job is executed 3 times.
  • ChunkTests#testChunkNoProcessorDefined() - the METHOD variable is defined as testChunkDefaultItemCount (incorrect method). The same error occurs in a number of tests.

There are other areas with these types of issues. A quick read through the tests should highlight most of them.

@jvfullam
Copy link

ChunkTests#testChunkRetryOnError() all say execution#1 because they are, in fact, all first executions. However, they are different job instances.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants