C# Poker Game: Unit testing different poker hands without violating DRY principlesImprove testability of constructorAre these the right type of unit tests to write?Unit testing IValueConverterUnit testing a service to return items from a databaseBowling scoring kataTaxi Meter App Business Logic Using TDDGeneric Null/Empty check for each property of a classJasmine tests for TypeScript class's pure functionsOsherove TDD Kata 1: String CalculatorExercise to refactor a class with various credit limit rules
Valid Badminton Score?
Greatest common substring
What is the intuitive meaning of having a linear relationship between the logs of two variables?
The baby cries all morning
Generic lambda vs generic function give different behaviour
What is difference between behavior and behaviour
Is there any reason not to eat food that's been dropped on the surface of the moon?
Why does John Bercow say “unlock” after reading out the results of a vote?
What's the purpose of "true" in bash "if sudo true; then"
Why "be dealt cards" rather than "be dealing cards"?
Opposite of a diet
What is the oldest known work of fiction?
Efficiently merge handle parallel feature branches in SFDX
What is the opposite of 'gravitas'?
Dot above capital letter not centred
How do I define a right arrow with bar in LaTeX?
Trouble understanding overseas colleagues
Is it correct to write "is not focus on"?
voltage of sounds of mp3files
Why are on-board computers allowed to change controls without notifying the pilots?
Can I use my Chinese passport to enter China after I acquired another citizenship?
Why Were Madagascar and New Zealand Discovered So Late?
At which point does a character regain all their Hit Dice?
Time travel short story where a man arrives in the late 19th century in a time machine and then sends the machine back into the past
C# Poker Game: Unit testing different poker hands without violating DRY principles
Improve testability of constructorAre these the right type of unit tests to write?Unit testing IValueConverterUnit testing a service to return items from a databaseBowling scoring kataTaxi Meter App Business Logic Using TDDGeneric Null/Empty check for each property of a classJasmine tests for TypeScript class's pure functionsOsherove TDD Kata 1: String CalculatorExercise to refactor a class with various credit limit rules
$begingroup$
I'm currently in the process of refactoring an old poker game which violates many of the SOLID principles.
I was trying to use the TDD approach to refactoring and I found myself having to hard code a player's hand every time I wrote a unit test for a specific poker hand check.
For example, here are the unit tests for a one pair poker hand check:
[Test]
public void OnePair_IsLowPair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Ace),
new CardSpade(Rank.Ace),
new CardDiamond(Rank.Five),
new CardHeart(Rank.Jack),
new CardClub(Rank.Three)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
[Test]
public void OnePair_IsLowerMiddlePair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Eight),
new CardSpade(Rank.Ace),
new CardDiamond(Rank.Ace),
new CardHeart(Rank.Jack),
new CardClub(Rank.Three)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
[Test]
public void OnePair_IsHigherMiddlePair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Jack),
new CardSpade(Rank.Queen),
new CardDiamond(Rank.Ace),
new CardHeart(Rank.Ace),
new CardClub(Rank.Three)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
[Test]
public void OnePair_IsHighPair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Jack),
new CardSpade(Rank.Deuce),
new CardDiamond(Rank.Six),
new CardHeart(Rank.Ace),
new CardClub(Rank.Ace)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
This is what the class diagram (at the project's current stage) looks like:
My question is how I should go about extracting a method (or methods) out of the testHand initialization.
c# unit-testing nunit
New contributor
$endgroup$
add a comment |
$begingroup$
I'm currently in the process of refactoring an old poker game which violates many of the SOLID principles.
I was trying to use the TDD approach to refactoring and I found myself having to hard code a player's hand every time I wrote a unit test for a specific poker hand check.
For example, here are the unit tests for a one pair poker hand check:
[Test]
public void OnePair_IsLowPair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Ace),
new CardSpade(Rank.Ace),
new CardDiamond(Rank.Five),
new CardHeart(Rank.Jack),
new CardClub(Rank.Three)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
[Test]
public void OnePair_IsLowerMiddlePair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Eight),
new CardSpade(Rank.Ace),
new CardDiamond(Rank.Ace),
new CardHeart(Rank.Jack),
new CardClub(Rank.Three)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
[Test]
public void OnePair_IsHigherMiddlePair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Jack),
new CardSpade(Rank.Queen),
new CardDiamond(Rank.Ace),
new CardHeart(Rank.Ace),
new CardClub(Rank.Three)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
[Test]
public void OnePair_IsHighPair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Jack),
new CardSpade(Rank.Deuce),
new CardDiamond(Rank.Six),
new CardHeart(Rank.Ace),
new CardClub(Rank.Ace)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
This is what the class diagram (at the project's current stage) looks like:
My question is how I should go about extracting a method (or methods) out of the testHand initialization.
c# unit-testing nunit
New contributor
$endgroup$
$begingroup$
Maybe make a "CreateHand()" function that takes a string like "AcAs5dJh3c"?
$endgroup$
– AShelly
43 secs ago
add a comment |
$begingroup$
I'm currently in the process of refactoring an old poker game which violates many of the SOLID principles.
I was trying to use the TDD approach to refactoring and I found myself having to hard code a player's hand every time I wrote a unit test for a specific poker hand check.
For example, here are the unit tests for a one pair poker hand check:
[Test]
public void OnePair_IsLowPair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Ace),
new CardSpade(Rank.Ace),
new CardDiamond(Rank.Five),
new CardHeart(Rank.Jack),
new CardClub(Rank.Three)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
[Test]
public void OnePair_IsLowerMiddlePair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Eight),
new CardSpade(Rank.Ace),
new CardDiamond(Rank.Ace),
new CardHeart(Rank.Jack),
new CardClub(Rank.Three)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
[Test]
public void OnePair_IsHigherMiddlePair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Jack),
new CardSpade(Rank.Queen),
new CardDiamond(Rank.Ace),
new CardHeart(Rank.Ace),
new CardClub(Rank.Three)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
[Test]
public void OnePair_IsHighPair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Jack),
new CardSpade(Rank.Deuce),
new CardDiamond(Rank.Six),
new CardHeart(Rank.Ace),
new CardClub(Rank.Ace)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
This is what the class diagram (at the project's current stage) looks like:
My question is how I should go about extracting a method (or methods) out of the testHand initialization.
c# unit-testing nunit
New contributor
$endgroup$
I'm currently in the process of refactoring an old poker game which violates many of the SOLID principles.
I was trying to use the TDD approach to refactoring and I found myself having to hard code a player's hand every time I wrote a unit test for a specific poker hand check.
For example, here are the unit tests for a one pair poker hand check:
[Test]
public void OnePair_IsLowPair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Ace),
new CardSpade(Rank.Ace),
new CardDiamond(Rank.Five),
new CardHeart(Rank.Jack),
new CardClub(Rank.Three)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
[Test]
public void OnePair_IsLowerMiddlePair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Eight),
new CardSpade(Rank.Ace),
new CardDiamond(Rank.Ace),
new CardHeart(Rank.Jack),
new CardClub(Rank.Three)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
[Test]
public void OnePair_IsHigherMiddlePair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Jack),
new CardSpade(Rank.Queen),
new CardDiamond(Rank.Ace),
new CardHeart(Rank.Ace),
new CardClub(Rank.Three)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
[Test]
public void OnePair_IsHighPair_ReturnsTrue()
SuperCard[] testHand =
new CardClub(Rank.Jack),
new CardSpade(Rank.Deuce),
new CardDiamond(Rank.Six),
new CardHeart(Rank.Ace),
new CardClub(Rank.Ace)
;
bool result = PokerHandEvaluator.OnePair(testHand);
Assert.That(result == true);
This is what the class diagram (at the project's current stage) looks like:
My question is how I should go about extracting a method (or methods) out of the testHand initialization.
c# unit-testing nunit
c# unit-testing nunit
New contributor
New contributor
edited 6 mins ago
Asel S
New contributor
asked 12 mins ago
Asel SAsel S
185
185
New contributor
New contributor
$begingroup$
Maybe make a "CreateHand()" function that takes a string like "AcAs5dJh3c"?
$endgroup$
– AShelly
43 secs ago
add a comment |
$begingroup$
Maybe make a "CreateHand()" function that takes a string like "AcAs5dJh3c"?
$endgroup$
– AShelly
43 secs ago
$begingroup$
Maybe make a "CreateHand()" function that takes a string like "AcAs5dJh3c"?
$endgroup$
– AShelly
43 secs ago
$begingroup$
Maybe make a "CreateHand()" function that takes a string like "AcAs5dJh3c"?
$endgroup$
– AShelly
43 secs ago
add a comment |
0
active
oldest
votes
Your Answer
StackExchange.ifUsing("editor", function ()
return StackExchange.using("mathjaxEditing", function ()
StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix)
StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
);
);
, "mathjax-editing");
StackExchange.ifUsing("editor", function ()
StackExchange.using("externalEditor", function ()
StackExchange.using("snippets", function ()
StackExchange.snippets.init();
);
);
, "code-snippets");
StackExchange.ready(function()
var channelOptions =
tags: "".split(" "),
id: "196"
;
initTagRenderer("".split(" "), "".split(" "), channelOptions);
StackExchange.using("externalEditor", function()
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled)
StackExchange.using("snippets", function()
createEditor();
);
else
createEditor();
);
function createEditor()
StackExchange.prepareEditor(
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: false,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: null,
bindNavPrevention: true,
postfix: "",
imageUploader:
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
,
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
);
);
Asel S is a new contributor. Be nice, and check out our Code of Conduct.
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
var $window = $(window),
onScroll = function(e)
var $elem = $('.new-login-left'),
docViewTop = $window.scrollTop(),
docViewBottom = docViewTop + $window.height(),
elemTop = $elem.offset().top,
elemBottom = elemTop + $elem.height();
if ((docViewTop elemBottom))
StackExchange.using('gps', function() StackExchange.gps.track('embedded_signup_form.view', location: 'question_page' ); );
$window.unbind('scroll', onScroll);
;
$window.on('scroll', onScroll);
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f216313%2fc-poker-game-unit-testing-different-poker-hands-without-violating-dry-principl%23new-answer', 'question_page');
);
Post as a guest
Required, but never shown
0
active
oldest
votes
0
active
oldest
votes
active
oldest
votes
active
oldest
votes
Asel S is a new contributor. Be nice, and check out our Code of Conduct.
Asel S is a new contributor. Be nice, and check out our Code of Conduct.
Asel S is a new contributor. Be nice, and check out our Code of Conduct.
Asel S is a new contributor. Be nice, and check out our Code of Conduct.
Thanks for contributing an answer to Code Review Stack Exchange!
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
Use MathJax to format equations. MathJax reference.
To learn more, see our tips on writing great answers.
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
var $window = $(window),
onScroll = function(e)
var $elem = $('.new-login-left'),
docViewTop = $window.scrollTop(),
docViewBottom = docViewTop + $window.height(),
elemTop = $elem.offset().top,
elemBottom = elemTop + $elem.height();
if ((docViewTop elemBottom))
StackExchange.using('gps', function() StackExchange.gps.track('embedded_signup_form.view', location: 'question_page' ); );
$window.unbind('scroll', onScroll);
;
$window.on('scroll', onScroll);
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f216313%2fc-poker-game-unit-testing-different-poker-hands-without-violating-dry-principl%23new-answer', 'question_page');
);
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
var $window = $(window),
onScroll = function(e)
var $elem = $('.new-login-left'),
docViewTop = $window.scrollTop(),
docViewBottom = docViewTop + $window.height(),
elemTop = $elem.offset().top,
elemBottom = elemTop + $elem.height();
if ((docViewTop elemBottom))
StackExchange.using('gps', function() StackExchange.gps.track('embedded_signup_form.view', location: 'question_page' ); );
$window.unbind('scroll', onScroll);
;
$window.on('scroll', onScroll);
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
var $window = $(window),
onScroll = function(e)
var $elem = $('.new-login-left'),
docViewTop = $window.scrollTop(),
docViewBottom = docViewTop + $window.height(),
elemTop = $elem.offset().top,
elemBottom = elemTop + $elem.height();
if ((docViewTop elemBottom))
StackExchange.using('gps', function() StackExchange.gps.track('embedded_signup_form.view', location: 'question_page' ); );
$window.unbind('scroll', onScroll);
;
$window.on('scroll', onScroll);
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
var $window = $(window),
onScroll = function(e)
var $elem = $('.new-login-left'),
docViewTop = $window.scrollTop(),
docViewBottom = docViewTop + $window.height(),
elemTop = $elem.offset().top,
elemBottom = elemTop + $elem.height();
if ((docViewTop elemBottom))
StackExchange.using('gps', function() StackExchange.gps.track('embedded_signup_form.view', location: 'question_page' ); );
$window.unbind('scroll', onScroll);
;
$window.on('scroll', onScroll);
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
$begingroup$
Maybe make a "CreateHand()" function that takes a string like "AcAs5dJh3c"?
$endgroup$
– AShelly
43 secs ago