-
-
Notifications
You must be signed in to change notification settings - Fork 371
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
Challenge 43: new challenge for secret shared on social media. #1144
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @djvinnie ! Awesome to see your first challenge!
Sorry to make it a liiitle bit harder to finish it as, looking at the challenge in its first complete version, the link to the microsoft golden key blunder feels a bit weak.. @bendehaan what do you think of the alternative text?
private final String secretKey = new String(Hex.decode("4348524953544D4153324B3233")); | ||
|
||
@Test | ||
void spoilerShouldRevealAnswer() { | ||
var challenge = new Challenge43(); | ||
|
||
assertThat(challenge.spoiler()).isEqualTo(new Spoiler(secretKey)); | ||
} | ||
|
||
@Test | ||
void rightAnswerShouldSolveChallenge() { | ||
var challenge = new Challenge43(); | ||
|
||
assertThat(challenge.answerCorrect(secretKey)).isTrue(); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we, instead of giving a hint about the secret, just say the following please?:
private final String secretKey = new String(Hex.decode("4348524953544D4153324B3233")); | |
@Test | |
void spoilerShouldRevealAnswer() { | |
var challenge = new Challenge43(); | |
assertThat(challenge.spoiler()).isEqualTo(new Spoiler(secretKey)); | |
} | |
@Test | |
void rightAnswerShouldSolveChallenge() { | |
var challenge = new Challenge43(); | |
assertThat(challenge.answerCorrect(secretKey)).isTrue(); | |
} | |
@Test | |
void rightAnswerShouldSolveChallenge() { | |
var challenge = new Challenge43(); | |
assertThat(challenge.answerCorrect(challenge.spoiler())).isTrue(); | |
} |
Thanks for your comments @commjoen ! I will wait to hear Ben's comments before making the changes you have proposed :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @djvinnie ! LGTM!
What kind of changes does this PR include?
Description
Added new challenge according to documentation.
Relations
Relates #616
References
Checklist: