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

completed assignment #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Yousef-AN
Copy link

@Yousef-AN Yousef-AN commented Oct 4, 2023

(needs fixing)

  • expect(introductionWithLanguageOptionall("Gracie")).toEqual("Hi, my name is Gracie and I am learning to program in JavaScript.");

  • expect(introductionWithLanguageOptional("Gracie", "Python")).toEqual("Hi, my name is Gracie and I am learning to program in Python.");

  • - Error :
    Error: Expected 'Hi, my name is Gracie and I am learning to program in undefined.' to equal 'Hi, my name is Gracie and I am learning to program in JavaScript.'

    • expected - actual

    -Hi, my name is Gracie and I am learning to program in undefined.
    +Hi, my name is Gracie and I am learning to program in JavaScript.

they got the same name, so when adding args it will make an error saying "Javascript" is undefined
_As you can see, I have added 1 more "L" at the end of the first function "Optionall" but it was the same for both when forked. _

…tionall("Gracie")).toEqual("Hi, my name is Gracie and I am learning to program in JavaScript."); expect(introductionWithLanguageOptional("Gracie", "Python")).toEqual("Hi, my name is Gracie and I am learning to program in Python."); they got the same name, so when adding args it will make an error saying "Javascript" is undefined
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant