-
Notifications
You must be signed in to change notification settings - Fork 170
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create RandomText #790
Comments
@llorllale/z please, pay attention to this issue |
@krzyk good idea, but why have ASCII-only versions? |
@llorllale Well, yeah, I'm just not sure how to limit the number of chars, to make it usefull, there are very limited cases when one would want control characters, so I would limit it to printable ones ( |
@krzyk aren't all utf chars printable? |
@llorllale No, Unicode includes ASCII, and in ASCII you have multiple control characters so by that definition not all are printable (a sample at http://www.utf8-chartable.de/). |
@llorllale so what's your decision? |
@krzyk I'm thinking just
|
@llorllale Yes it makes sense, I'm only wondering if we want |
@krzyk you're right, let's have a |
@krzyk can you modify the description to fit the new requirement? |
@llorllale done, I also added info about the length of the generated text, we didn't discuss that. It is quite common to limit the length of the text. |
@0crat in |
There is an unrecoverable failure on my side. Please, submit it here:
|
@llorllale Job #790 is now in scope, role is |
@proshin-roman thanks |
@elenavolokhova/z please review this job completed by @proshin-roman/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed |
The job #790 is now out of scope |
@0crat quality good |
There is an unrecoverable failure on my side. Please, submit it here:
1.0-SNAPSHOT: Issue: #790, Comment: 387669592 |
@elenavolokhova Thanks, but quality review is not required in this job |
Order was finished, quality is "good": +35 point(s) just awarded to @proshin-roman/z |
@0crat status |
Random strings are quite common in unit tests where one has to produce some kind of a string into
TextOf
, that is not important in itself.E.g.:
Could be replaced with:
RandomText
should have two consructors:Each of the above should optionally accept an integer that will be the length of the text generated, without this number
RandomText
should generate the text with random number of characters.The text was updated successfully, but these errors were encountered: