Skip to content
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

unittest fouten in GregorianDate #7

Open
VinzzB opened this issue Oct 26, 2014 · 3 comments
Open

unittest fouten in GregorianDate #7

VinzzB opened this issue Oct 26, 2014 · 3 comments

Comments

@VinzzB
Copy link
Owner

VinzzB commented Oct 26, 2014

Volgens de nieuwe unit test falen er enkele methods in de gregorianDate class.
Iemand zin om dit even na te kijken? Je kan dit issue toewijzen in het rechtermenu van git.

@NataIia
Copy link
Collaborator

NataIia commented Oct 26, 2014

failed:

public void test_getMonth_Geeft_Maand()
in gregorian calendar maanden lopen van 0 tot 11. we moeten onderling beslissen welke datum klasse moet veranderd worden gregorian (maanden 0-11) of fromScrarch (maanden 0-12)

public void test_Ctor_Als_String_parameter_Verkeerde_separator_Geef_fout
volgens Patrik '-' is geen verkeerde separator en constructor met string als parameter moet '-' aanvarden

public void test_verschilInMaanden_Als_parameter_DatumPlus365dagen_inSchrikkeljaar_Geeft_11()
ikk snap niet goed waarom tussen 1,1,1600 en 1,1,1601 11 maanden zitten. het moet 12 als aantwoord geven, ne?

public void test_getAmericanFormat_Als_string()
dezelfde probleem als in detMaand() met 0-11 of 1-12 numeratie. moeten bespreken

void test_Ctor_Als_String_parameter_lengte_jaarFormatIs2_Geef_fout()
jaar mag in principe 11 zijn, in gregorian datum throws constructor geen exception.

void test_Ctor_Als_String_parameter_lengte_maandFormatIs1_Geef_fout()
("01/1/2011") is goede datum, waarom moten we exception verwachten?

@VinzzB
Copy link
Owner Author

VinzzB commented Oct 26, 2014

public void test_getMonth_Geeft_Maand():
public void test_getAmericanFormat_Als_string():

  • From Scratch is van 1-12. Zoals in de realiteit de maanden van 1 tot 12 gaan.

public void test_Ctor_Als_String_parameter_Verkeerde_separator_Geef_fout

  • Tijdens de les en het uitvoeren van de Analysis class gaf Fox aan dat dat ook een fout moet werpen. Af te spreken...

public void test_verschilInMaanden_Als_parameter_DatumPlus365dagen_inSchrikkeljaar_Geeft_11():

  • dateNewLeapYear= 1 januari 1600
  • dateNewVerschil366Leap = 31 december 1600 (-1 dag in test method!)

void test_Ctor_Als_String_parameter_lengte_maandFormatIs1_Geef_fout():
void test_Ctor_Als_String_parameter_lengte_jaarFormatIs2_Geef_fout():

  • In Document bij Opmerking: maand moet ingegeven worden met 2 cijfers (Vb 05), jaartal met 4 cijfers (Vb 0567), dagnr met 1 of 2 cijfers

@NataIia
Copy link
Collaborator

NataIia commented Oct 27, 2014

methoden om verschil te berekenen zijn aangepast en geven geen fouten meer.

jaar 0 bestaat niet https://nl.wikipedia.org/wiki/0_(jaar). darom zet gregorian calendar voor de datum 18/10/0000 jaar 1. af te spreken?

public void test_Ctor_Als_String_parameter_Verkeerde_separator_Geef_fout

Tijdens de les en het uitvoeren van de Analysis class gaf Fox aan dat dat ook een fout moet werpen. Af te spreken...

tijdens andere les heeft hij iets anders gezegd. of heb ik niet goed begrepen. nog een keer vragen

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

No branches or pull requests

2 participants